public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Zachary Amsden <zach@vmware.com>
To: Jeremy Fitzhardinge <jeremy@goop.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Andrew Morton <akpm@osdl.org>, Chris Wright <chrisw@sous-sol.org>,
	Christian Limpach <Christian.Limpach@cl.cam.ac.uk>,
	jeremy@xensource.com
Subject: Re: [PATCH 2.6.17] Clean up and refactor i386 sub-architecture setup
Date: Tue, 20 Jun 2006 18:24:35 -0700	[thread overview]
Message-ID: <44989FD3.1040805@vmware.com> (raw)
In-Reply-To: <44989E25.3090402@goop.org>

Jeremy Fitzhardinge wrote:
> Zachary Amsden wrote:
>> This is cleaner than the patches I sent in March, although we want to 
>> re-use parts of the mach-default code, not replace it entirely.  
>> Hence my interest in the multi-subarch generic kernel.  I'd be glad 
>> to look into it.
> In my current Xen patch, I split the mach-default/setup.c into setup.c 
> and setup-memory.c; Xen uses setup.c as-is, and then provides its own 
> setup-xen.c.  That solves my immediate problem, but I don't know if it 
> generalizes enough; certainly factoring default/setup.c into a cluster 
> of reusable setup-*.c pieces is a pretty lightweight way of reusing 
> those pieces.

I was thinking more of having mach-xen/built-in.o, 
mach-default/built-in.o, mach-es7000/built-in.o, mach-voyager/built-in.o 
all be linked specially so they can be compiled into the same kernel 
either as one giant batch, with weak linkage and a function table to 
indirect calls to them (thus the generic kernel can jettison the modules 
outside of the subarch it has chosen at boot time, potentially keeping 
the default kernel as well to allow subarches to fallback on the 
traditional indirections).  And if compiled as a specific kernel, those 
weak linkages get promoted to direct instead of indirect calls.

You may have to separate the namespaces at the identifier level as well, 
use some elven magic, but I haven't worked out all the details yet.

Zach

  reply	other threads:[~2006-06-21  1:24 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-20 23:42 [PATCH 2.6.17] Clean up and refactor i386 sub-architecture setup Jeremy Fitzhardinge
2006-06-21  0:08 ` Zachary Amsden
2006-06-21  0:22   ` Chris Wright
2006-06-21  0:24   ` Jeremy Fitzhardinge
2006-06-21  0:40     ` Zachary Amsden
2006-06-21  1:17       ` Jeremy Fitzhardinge
2006-06-21  1:24         ` Zachary Amsden [this message]
2006-06-21  1:32           ` Jeremy Fitzhardinge

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=44989FD3.1040805@vmware.com \
    --to=zach@vmware.com \
    --cc=Christian.Limpach@cl.cam.ac.uk \
    --cc=akpm@osdl.org \
    --cc=chrisw@sous-sol.org \
    --cc=jeremy@goop.org \
    --cc=jeremy@xensource.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