Re: [Xen-ia64-devel] [8/9] add mca.c for supporting INIT handler

[Alex Williamson <alex.williamson@xxxxxx>]

On Fri, 2006-06-23 at 01:50 +0900, Akio Takebe wrote:
> This is main patch for supporting INIT handler.
> I update this patch.
> - into xen/arch/ia64/xen
> - make smaller mca.c.
>   (non-ifdef code)

Hi Akio,

   Maybe the question I should have asked the first time around was why
is init handling so different between Xen/ia64 and Linux/ia64?  I know I
mentioned splitting this out into a separate file as a possibility, but
I still have concerns about how we maintain this long term, especially
as we add more components of MCA handling in the (near) future.  I'm
certainly willing to live with large chunks of CPE/CMC/MCA specific code
#ifdef'd out.  Would it make things any better if we pulled in updates
to the files from newer Linux/ia64 source?  MCA handling has undergone
some overhauling in upstream kernels that may help.  Is this separate
file approach really the right answer?  Thanks,

        Alex

-- 
Alex Williamson                             HP Open Source & Linux Org.


_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel