Re: [PATCH] i386 arch subdivision into machine types for 2.5.8

[Eric W. Biederman]

James Bottomley <[e-mail]> writes:

> This patch tries to split arch/i386 up into machine specific directories 
> (similar to the way arch/arm is done).  The idea is to separate out those 
> machines which don't look like standard PCs (particularly from an SMP 
> standpoint).  For the current kernel, all it really does is to get the visws 
> stuff into a separate directory (arch/i386/visws).  I've also taken some files 
> which aren't going to be used by non-pc SMP machines (mainly related to mpbios 
> and ioapic) and placed them into arch/i386/generic.

A couple of comments.
- There is no way to build a generic kernel, that just needs
  a command line to select the architecture.  Something that is important
  for installers.  Even better would auto detection of the platform from
  firmware information, but you can't always do that.

- By just allowing redirecting setup_memory_region you don't allow for
  architectures that don't have the 384K memory hole.

- The hooks you add aren't used and are so generic it isn't obvious what
  they are supposed do from their names.

- setup_arch.h is nasty.  What code it has depends on what it is defined
  when it is included.  Couldn't 2 headers to this job better?  Or better yet
  can't you just use function calls?

And of course you don't look at allowing different firmware implementations,
but I'm doing that, so it is covered. :)

Eric