Re: [PATCH 0/3] Early use of boot service memory

From: jerry . hoemann
Date: Fri Nov 15 2013 - 14:13:26 EST


On Fri, Nov 15, 2013 at 07:24:17AM +0100, Ingo Molnar wrote:
>
> * jerry.hoemann@xxxxxx <jerry.hoemann@xxxxxx> wrote:
>
> > On Thu, Nov 14, 2013 at 08:44:04PM +0200, Pekka Enberg wrote:
> > > On Thu, Nov 14, 2013 at 8:04 PM, <jerry.hoemann@xxxxxx> wrote:
> > > > Making this issue a quirk will be a lot more practical. Its a small, focused
> > > > change whose implications are limited and more easily understood.
> > >
> > > There's nothing practical with requiring users to pass a kernel option
> > > to make kdump work. It's a workaround, sure, but it's not a proper
> > > fix.
> >
> > One already has to specify command line arguments to enable kdump.
> > See "crashkernel=" in Documentation/kernel-parameters.txt.
>
> That option is already a usability barrier. Adding yet another
> usability barrier improves things how?

Because of the bug in the way efi_reserve_boot_services is used:

On pre 3.9 kernel crash dump doesn't work at all w/ fragmented memory < 896M.

On post 3.9 it doesn't work by default and even when changing
invocation, has some issue (see vivek's email.)

i am looking for a solution for all these cases.

>
> > As i said in an earlier mail we are working w/ distros. [...]
>
> The point being?

Earlier reviewer asserted:

"There's nothing practical with requiring users to pass a kernel option"

I pointed out that these type of arguments can and do get added by distros
and that users need not be actively involved. the tools abstract
these details out.

we are working w/ distros to get kdump working on large servers which
it currently does not do. as such, this change would be seamless to
users.

but as i've said in earlier replies, i'm willing to do a white list.
its not as flexible, but its much better than the current situation
that we face (especially on pre 3.9 kernels.)


>
> > [...] distros can and do specify lots of interesting command line
> > arguments for their systems. Distros have tools for configuring
> > kdump. User must already use these tools or manually edit multiple
> > config files, to get kdump to work. I would work with distros to
> > help integrate this change into their tools.
>
> Here you describe a method that has already successfully cut the kdump
> user base to a fraction of its potential size. Why should we assist to
> that effort of engineered obscurity?
>
> > As i said in earlier mail, i am willing to change implementation to
> > some type of black/white listing.
>
> Is it possible to fix it the way hpa suggested?

I think the changes to enable ,high is a step in the
right direction. its an improvement But it is still green.

We are having lots more problems w/ upstream kdump than we are having
w/ the kdump in distros.

So, to answer your question with a slight twist:

Is it possible to back ports lots of green code across multiple
versions and distros and get a bug free user experiences? I guess so.

is it the right way to go? i personally don't think so.

but hey, others may have a different view.


Jerry


>
> Thanks,
>
> Ingo

--

----------------------------------------------------------------------------
Jerry Hoemann Software Engineer Hewlett-Packard

3404 E Harmony Rd. MS 57 phone: (970) 898-1022
Ft. Collins, CO 80528 FAX: (970) 898-XXXX
email: jerry.hoemann@xxxxxx
----------------------------------------------------------------------------

--
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/