Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1588094
| From | Ingo Molnar <mingo@kernel.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c |
| Date | 2017-02-25 10:10 +0100 |
| Message-ID | <teGwy-5C9-5@gated-at.bofh.it> (permalink) |
| References | <tesMV-41Q-1@gated-at.bofh.it> <tetzj-4CB-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
* Linus Torvalds <torvalds@linux-foundation.org> wrote:
> I'm pretty sure that the thing that triggered this is once more commit
> a9b4f08770b4 ("x86/ioapic: Restore IO-APIC irq_chip retrigger
> callback") which seems to retrigger stale irqs that simply should not
> be retriggered.
>
> They aren't actually active any more, if they ever were.
>
> So that commit seems to act like a random CONFIG_DEBUG_SHIRQ. It's
> good for testing, but not good for actual users.
Yeah, so some distros like Fedora already have CONFIG_DEBUG_SHIRQ=y enabled, but
part of the problem is that CONFIG_DEBUG_SHIRQ=y has this:
#ifdef CONFIG_DEBUG_SHIRQ_FIXME
if (!retval && (irqflags & IRQF_SHARED)) {
/*
* It's a shared IRQ -- the driver ought to be prepared for it
* to happen immediately, so let's make sure....
* We disable the irq to make sure that a 'real' IRQ doesn't
* run in parallel with our fake.
*/
unsigned long flags;
disable_irq(irq);
local_irq_save(flags);
handler(irq, dev_id);
local_irq_restore(flags);
enable_irq(irq);
}
#endif
Note that the '_FIXME' postfix effectively turns off this particular debug check
...
Thomas and me realized this risk a week ago ago, and tried to resurrect full
CONFIG_DEBUG_SHIRQ=y functionality to more reliably trigger these problems:
https://lkml.org/lkml/2017/2/15/341
... but were forced to revert that fix because it's not working on x86 yet (it's
crashing). We also thought we fixed the problems exposed in drivers, as the
retrigger changes have been in -tip and -next for some time, but were clearly too
optimistic about that.
So, should we revert the hw-retrigger change:
a9b4f08770b4 x86/ioapic: Restore IO-APIC irq_chip retrigger callback
... until we managed to fix CONFIG_DEBUG_SHIRQ=y? If you'd like to revert it
upstream straight away:
Acked-by: Ingo Molnar <mingo@kernel.org>
Thanks,
Ingo
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Linus Torvalds <torvalds@linux-foundation.org> - 2017-02-24 20:20 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Ingo Molnar <mingo@kernel.org> - 2017-02-25 10:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Linus Torvalds <torvalds@linux-foundation.org> - 2017-02-25 19:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Thomas Gleixner <tglx@linutronix.de> - 2017-02-27 11:20 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Thomas Gleixner <tglx@linutronix.de> - 2017-02-27 14:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Ingo Molnar <mingo@kernel.org> - 2017-02-27 16:50 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Tony Lindgren <tony@atomide.com> - 2017-02-27 17:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Tony Lindgren <tony@atomide.com> - 2017-02-27 17:30 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Thomas Gleixner <tglx@linutronix.de> - 2017-02-27 18:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Thomas Gleixner <tglx@linutronix.de> - 2017-02-27 18:10 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Linus Torvalds <torvalds@linux-foundation.org> - 2017-02-27 23:30 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Ingo Molnar <mingo@kernel.org> - 2017-02-28 11:30 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Thomas Gleixner <tglx@linutronix.de> - 2017-02-28 15:00 +0100
Re: [WARNING: A/V UNSCANNABLE][Merge tag 'media/v4.11-1' of git] ff58d005cd: BUG: unable to handle kernel NULL pointer dereference at 0000039c Sean Young <sean@mess.org> - 2017-02-25 12:20 +0100
[PATCH] [media] serial_ir: ensure we're ready to receive interrupts Sean Young <sean@mess.org> - 2017-02-25 12:30 +0100
Re: [PATCH] [media] serial_ir: ensure we're ready to receive interrupts Mauro Carvalho Chehab <mchehab@infradead.org> - 2017-02-25 14:40 +0100
Re: [PATCH] [media] serial_ir: ensure we're ready to receive interrupts Mauro Carvalho Chehab <mchehab@infradead.org> - 2017-02-25 15:00 +0100
csiph-web