This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-ia64-devel] [patch] [4/4] Support INIT handler of xen

On Thu, 2006-06-15 at 06:36 +0900, Akio Takebe wrote:
> Hi, Alex
> I forgot README.origin update, I'm sorry.
> The first I made a small patch with cut&paste without ifdef.
> But I remade a big patch with ifdef, because I think we may leverage the 
> original mac code. I don't know this leverage, because another fujitsu 
> member will make other mca code.
> I think my ifdef is ugly, so I want to remake non-ifdef patch.
> But I refer linux code, can I make xen-mca code wiht cut&paste 
> without ifdef in arch/ia64/linux-xen?

Hi Akio,

   The ifdefs are kind of ugly, and I wasn't sure how many lines of code
were actually left enabled in those files.  How many lines of code was
the original cut&paste version?  If only a few hundred lines, maybe it
does make sense to separate it out.  I think it should probably go in
arch/ia64/xen though, not linux-xen.  Thanks,

Alex Williamson                             HP Open Source & Linux Org.

Xen-ia64-devel mailing list