Re: [PATCH] sched: move CONFIG_IRQ_TIME_ACCOUNTING to common code

From: Ingo Molnar
Date: Tue Feb 07 2012 - 03:29:13 EST



* Dmitry Antipov <dmitry.antipov@xxxxxxxxxx> wrote:

> Since CONFIG_IRQ_TIME_ACCOUNTING is architecture-agnostic,
> move it from x86 area to common code.
>
> Signed-off-by: Dmitry Antipov <dmitry.antipov@xxxxxxxxxx>
> ---
> arch/x86/Kconfig | 11 -----------
> lib/Kconfig.debug | 11 +++++++++++
> 2 files changed, 11 insertions(+), 11 deletions(-)

That's not enough, right now on x86 this code needs special
architecture glue, see how the noirqtime flag is used to call
into enable_sched_clock_irqtime() after TSC init.

So if you apply your patch and boot CONFIG_IRQ_TIME_ACCOUNTING=y
on say an ARM system, nothing will be done, because
sched_clock_irqtime is always zero.

Thanks,

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/