[Top] [All Lists]

Re: [PATCH 2/3] mips:powertv: Make kernel command line size configurable

To: Paul Gortmaker <>
Subject: Re: [PATCH 2/3] mips:powertv: Make kernel command line size configurable (resend)
From: David VomLehn <>
Date: Fri, 8 May 2009 09:34:00 -0700
Authentication-results: sj-dkim-4;; dkim=pass ( sig from verified; );
Dkim-signature: v=1; a=rsa-sha256; q=dns/txt; l=1429; t=1241800440; x=1242664440; c=relaxed/simple; s=sjdkim4002; h=Content-Type:From:Subject:Content-Transfer-Encoding:MIME-Version;;; z=From:=20David=20VomLehn=20<> |Subject:=20Re=3A=20[PATCH=202/3]=20mips=3Apowertv=3A=20Mak e=20kernel=20command=20line=20size=0A=09configurable=20(rese nd) |Sender:=20; bh=deS/nAUJmBtlUNEKOGKSAEjDeBbbp8QeLmth4BUN69E=; b=n/eDT9enyxUq98Z3/ck/BSenxSGAZYf+WkQJiuEbYfVjkJq8EzmfyDqCTy rcexybjPnXyR1Kr/s02o/zbwJGPr5tG7cjVcNMlLoBIvCPKg6hJM6OO+rVqf kAAcPTAgPL;
In-reply-to: <>
Original-recipient: rfc822;
References: <> <>
User-agent: Mutt/1.5.18 (2008-05-17)
On Fri, May 08, 2009 at 11:25:35AM -0400, Paul Gortmaker wrote:
> On Mon, May 4, 2009 at 6:57 PM, David VomLehn <> wrote:
> > Most platforms can get by perfectly well with the default command line size,
> > but some platforms need more. This patch allows the command line size to
> > be configured for those platforms that need it. The default remains 256
> > characters.
> The one thing I see when I look at this patch, is that it lands in the
> arch/mips/Kconfig -- but is there really anything fundamentally
> architecture specific about the allowed length of the kernel command
> line?.  It probably belongs somewhere alongside the setting for
> LOG_BUF_LEN or similar (and then add the other respective changes
> to make all arch actually respect the setting.)
> Paul.
> >
> > Signed-off-by: David VomLehn <>
> > ---
> >  arch/mips/Kconfig             |    7 +++++++
> >  arch/mips/include/asm/setup.h |    2 +-

The reason I put this configuration configuration in the architecture-
specific Kconfig is because COMMAND_LINE_SIZE is defined in the
architecture-specific file arch/mips/include/asm/setup.h. I strongly
agree that this really should not be an architecture-specific definition,
but it's much more complex to get a patch to change COMMAND_LINE_SIZE
in every architecture. Fixing in the MIPS tree seems like a good

David VomLehn

<Prev in Thread] Current Thread [Next in Thread>