WARNING - OLD ARCHIVES

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/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH]local apic initialization


On 25 Mar 2005, at 21:23, Ian Pratt wrote:

It didn't apply totally cleanly, but I fixed it up, and it booted OK for
me.

How much testing have you been able to give the patch?

I'm not sure that some of the #if 0's are actually going to help from a
maintenance POV, so it might actually be better to remove any whole
functions etc that aren't used.

I'd make it a pre-condition of acceptance: the patch will be much smaller since a lot of the added code fragments are actually if 0'ed out. It makes it hard to work out what moving parts have actually changed.

 -- Keir



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel

<Prev in Thread] Current Thread [Next in Thread>