Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable

From: Ingo Molnar
Date: Fri Jun 20 2008 - 09:47:53 EST



* Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:

> Subject: clean up and comment local_bh_enable code
>
> There's no need to use local_irq_save() over local_irq_disable() in
> the local_bh_enable code since it is a bug to call it with irqs
> disabled and do_softirq will enable irqs if there is any pending work.
> Consolidate the code from local_bh_enable and ..._ip to avoid having a
> disconnect between them in the warnings they trigger that is currently
> there. Also always trigger the warning on in_irq(), not just in the
> trace-irqflags case.

applied to tip/core/softirq for testing, thanks Johannes.

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/