[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20120209144857.GC5650@redhat.com>
Date: Thu, 9 Feb 2012 09:48:57 -0500
From: Don Zickus <dzickus@...hat.com>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
Cc: Vivek Goyal <vgoyal@...hat.com>, x86@...nel.org,
LKML <linux-kernel@...r.kernel.org>,
kexec-list <kexec@...ts.infradead.org>
Subject: Re: [PATCH] x86, kdump: No need to disable ioapic in crash path
On Wed, Feb 08, 2012 at 02:55:14PM -0800, Eric W. Biederman wrote:
> > I ran lkdtm by panic'ing in the interrupt handle thus leaving device
> > interrupt un-ack'd and the apic might have been un-ack'd too (jprobes
> > hooked in at do_IRQ). 3 out 3 times the second kernel came up on my core2
> > quad.
>
> That sounds like more than enough basic testing for me. Document your
> testing in a patch description and let's get the unnecessary local apic
> and ioapic stomping removed from the kexec on panic path.
>
> There were bugs. We deleted the code that had them. The bugs are gone
> and there are no new problems goes over very well in my book.
Great. Thanks. I'll put together the patch.
Cheers,
Don
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists