[lkml]   [2002]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] i386 arch subdivision into machine types for 2.5.8
James Bottomley <> 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. :)

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:22    [W:0.117 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site