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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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-efi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux