Re: [patch] setup_boot_APIC_clock() irq-enable fix

From: Thomas Gleixner
Date: Thu Mar 22 2007 - 06:49:43 EST


On Thu, 2007-03-22 at 10:31 +0100, Ingo Molnar wrote:
> Subject: [patch] setup_boot_APIC_clock() irq-enable fix
> From: Ingo Molnar <mingo@xxxxxxx>
>
> latest -git triggers an irqtrace/lockdep warning of a leaked
> irqs-off condition:
>
> BUG: at kernel/fork.c:1033 copy_process()
>
> after some debugging it turns out that commit ca1b940c accidentally left
> interrupts disabled - which trickled down all the way to the first time
> we fork a kernel thread and triggered the warning.
>
> the fix is to re-enable interrupts in the 'else' branch of
> setup_boot_APIC_clock()'s pmtimers calibration path.
>
> Reported-by: Michal Piotrowski <michal.k.k.piotrowski@xxxxxxxxx>
> Signed-off-by: Ingo Molnar <mingo@xxxxxxx>

Acked-by: Thomas Gleixner <tglx@xxxxxxxxxxxxxxxxxxxxxxxxxxxx>



-
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/