Re: [RFC] IA64: Use early_parm to handle mvec_name and nomca

From: Horms <horms_at_verge.net.au>
Date: 2006-03-13 19:12:41
On Sun, Mar 12, 2006 at 10:33:27PM -0800, Chen, Kenneth W wrote:
> Horms wrote on Sunday, March 12, 2006 8:08 PM
> > IA64: Use early_parm to handle mvec_name and nomca
> > 
> > I'm not sure of the worthiness of this idea, so please consider it an RFC. 
> > Its key merits are:
> > 
> > * Reuse existing infrastructure
> > * Greatly tightens up the parsing of nomca
> > * Greatly simplifies the parsing of machvec
> 
> It is kind of odd though that parse_early_param() is called twice,
> once from init/main.c:start_kernel and once from arch/ia64/kernel/
> setup.c:setup_arch.  Though you are not the one introduces that
> Oddity.

Yes, that is odd. I guess the arch/ia64/kernel/setup.c one could be
removed. That is, unless the init/main.c is called too late. I'll
investigate that.

> The other thing is that the code you changed are going to be thrown
> away after initialization, so I take your motivation of the patch
> is in some form of uncluttered the source code?

Yes. It seems to me that the parsing is greatly simplified with this
approach.

> If I were you, I would put machvec code into machvec.c, like this:

That is an excellent idea.

-- 
Horms
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Mon Mar 13 19:13:45 2006

This archive was generated by hypermail 2.1.8 : 2006-03-13 19:13:52 EST