public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Matt Mackall <mpm@selenic.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@osdl.org>,
	ak@suse.de
Subject: Re: [PATCH] x86 built-in command line (resend)
Date: Mon, 31 Jul 2006 15:57:25 -0500	[thread overview]
Message-ID: <20060731205725.GL6908@waste.org> (raw)
In-Reply-To: <44CE5B74.4060409@zytor.com>

On Mon, Jul 31, 2006 at 12:35:16PM -0700, H. Peter Anvin wrote:
> Matt Mackall wrote:
> >On Mon, Jul 31, 2006 at 12:07:02PM -0700, H. Peter Anvin wrote:
> >>Matt Mackall wrote:
> >>>I'm resending this as-is because the earlier thread petered out
> >>>without any strong arguments against this approach. x86_64 patch to
> >>>follow.
> >>"No strong arguments?"
> >>
> >>I still maintain that this patch has the wrong priority in case more 
> >>than one set of arguments are provided.
> >
> >But you still haven't answered how that lets you work around firmware
> >that passes parameters you don't like.
> >
> 
> That a fairly unique problem, and is most likely in a minority 
> application.  For that case a CONFIG option to ignore the 
> firmware-provided command line would make sense.  I do not believe it 
> should be the only option or even the default.

It's not the default. The default is all args come from the
bootloader.

At the risk of repeating myself, here are all possible features and
behaviors carefully enumerated again:

Possible features:
a) allow dealing with bootloaders that don't pass arguments
b) allow dealing with bootloaders that pass bogus arguments
c) allow dealing with bootloaders that run up against length limits
d) allow dealing with bootloaders where changing arguments dynamically
   is difficult
e) provide friendly defaults

Possible behaviors:
1) command line overrides built-in (won't work with b, works with the
rest)
2) built-in overrides command line (not so great for e, works with the
rest)
3) command line appends to built-in (generally broken as our command
parser can't arbitrarily override earlier arguments in most cases)
4) built-in appends to command line (same story)

Now I basically think behavior (e) is worthless. Embedded folks don't
care if the kernel's friendly and it's a solved problem for distros
too. Anyone else is building a kernel for themselves and don't need
defaults.

By comparison, the value of (b) is that you can control things you
otherwise can't. 

> It would be particularly good if this could be standardized across 
> architectures, which is another reason to do it right.

Yes. They should all clearly do (2).

-- 
Mathematics is the supreme nostalgia of our time.

      reply	other threads:[~2006-07-31 20:58 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-31 17:13 [PATCH] x86 built-in command line (resend) Matt Mackall
2006-07-31 18:48 ` Roman Zippel
2006-07-31 19:07 ` H. Peter Anvin
2006-07-31 19:28   ` Matt Mackall
2006-07-31 19:35     ` H. Peter Anvin
2006-07-31 20:57       ` Matt Mackall [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060731205725.GL6908@waste.org \
    --to=mpm@selenic.com \
    --cc=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox