[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4A7A3A78.7080200@redhat.com>
Date: Thu, 06 Aug 2009 10:05:44 +0800
From: Amerigo Wang <amwang@...hat.com>
To: "Eric W. Biederman" <ebiederm@...ssion.com>
CC: Neil Horman <nhorman@...hat.com>, linux-kernel@...r.kernel.org,
tony.luck@...el.com, linux-ia64@...r.kernel.org,
akpm@...ux-foundation.org, Ingo Molnar <mingo@...e.hu>,
Anton Vorontsov <avorontsov@...mvista.com>
Subject: Re: [Patch 0/7] Implement crashkernel=auto
Eric W. Biederman wrote:
> Neil Horman <nhorman@...hat.com> writes:
>
>> You could of
>> course boot the installer kernel with a crashkernel line pre-selected suppose,
>> but then you have to go to the trouble of figuring that allocation size out each
>> time. This gives you a nice convienent way to get a reasonable block of memory
>> without the need to do all that extra work.
>>
>
> My big concern is that you are moving policy into the kernel, when it isn't at
> all clear that policy is the right thing to do, and the existing mechanisms give
> you enough rope to do this all in userspace.
>
How? This doesn't remove the existing mechanism, just provides a new
choice for user like me who doesn't know how much memory should be
reserved, or who simply doesn't want to concern this since he/she has
very enough memory.
> You also have to build (or at least load) the whole kdump image after
> the system boots, and configure someplace for this to be saved.
>
> What class of problems do you expect to catch with this?
>
Again, try to save the user from choosing numbers for "crashkernel=".
> What has me puzzled is that the mkdumprd that ships with fedora isn't
> usable without patching, and it seems to be steadily getting worse.
Please explain why it is not usable? The patch won't break the
userspace, since it modifies the "crashkernel=" command line dynamically.
Thanks.
--
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