[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <m1vdc8iase.fsf@fess.ebiederm.org>
Date: Sat, 03 Apr 2010 12:41:21 -0700
From: ebiederm@...ssion.com (Eric W. Biederman)
To: Joerg Roedel <joro@...tes.org>
Cc: Chris Wright <chrisw@...s-sol.org>,
Joerg Roedel <joerg.roedel@....com>, nhorman@...hat.com,
nhorman@...driver.com, kexec@...ts.infradead.org,
linux-kernel@...r.kernel.org, hbabu@...ibm.com,
iommu@...ts.linux-foundation.org, vgoyal@...hat.com
Subject: Re: [PATCH 3/4] Revert "x86: disable IOMMUs on kernel crash"
Joerg Roedel <joro@...tes.org> writes:
> On Sat, Apr 03, 2010 at 10:49:09AM -0700, Eric W. Biederman wrote:
>> Joerg Roedel <joro@...tes.org> writes:
>
>> > Another problem: This also breaks if the kdump kernel has no
>> > iommu-support.
>>
>> Not a problem. We require a lot of things of the kdump kernel,
>> and it is immediately apparent in a basic sanity test.
>
> Only if the sanity test is done on an iommu machine which I don't want
> to rely on.
That makes no sense.
The requirements on the kdump kernel has always been that it somehow
figure out to recover a machine that is in a random hardware state.
That requires drivers for the hardware, that is critical to the
machines operation.
The easy test for sysadmins is to do:
echo > /proc/sysrq-trigger
Anyone who thinks the result from one piece of hardware applies to
another is deluded.
We have been down the path of doing lots of things in the crashing
kernel with lkcd, in practice it was worthless in the event of real
world crashes.
kexec on panic isn't perfect but it at least is an architecture that
works often enough to be usable. It does require testing to make certain
the basic code paths don't regress, but even so it is a lot easier
to maintain and keep useful than any alternative I know of.
Eric
--
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