Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- From: Ingo Molnar <mingo@xxxxxxx>
- Date: Fri, 20 Jun 2008 15:46:37 +0200
* 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/
- Follow-Ups:
- Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- From: Ingo Molnar
- Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- References:
- [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- From: Johannes Berg
- Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- From: Linus Torvalds
- Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- From: Johannes Berg
- [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- Prev by Date: [PATCH] kbuild: sparse needs CF not CHECKFLAGS
- Next by Date: [PATCH] ACPI: don't walk tables if ACPI was disabled
- Previous by thread: Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- Next by thread: Re: [PATCH/RFC] remove irqs_disabled warning from local_bh_enable
- Index(es):