[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [Xen-devel] Re: [patch] final header fixes


  • To: "Hollis Blanchard" <hollisb@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxxx>
  • From: "Magenheimer, Dan (HP Labs Fort Collins)" <dan.magenheimer@xxxxxx>
  • Date: Thu, 24 Mar 2005 13:17:17 -0800
  • Delivery-date: Thu, 24 Mar 2005 21:20:01 +0000
  • List-id: List for Xen developers <xen-devel.lists.sourceforge.net>
  • Thread-index: AcUwtQdseQFDIB6oQH681NHMMFd5BQAAPJsA
  • Thread-topic: [Xen-devel] Re: [patch] final header fixes

arch-ia64.h doesn't declare/define struct xen_regs so
this wouldn't work very well on ia64.  I would prefer
<asm/regs.h> (or even just leaving it as the forward
struct reference).

> -----Original Message-----
> From: Hollis Blanchard [mailto:hollisb@xxxxxxxxxx] 
> Sent: Thursday, March 24, 2005 2:03 PM
> To: xen-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: Magenheimer, Dan (HP Labs Fort Collins)
> Subject: Re: [Xen-devel] Re: [patch] final header fixes
> 
> On Thursday 24 March 2005 14:40, Dan Magenheimer wrote:
> > This second patch looks good to me, though I would prefer
> > to change to '#include <asm/regs.h>' from 'struct xen_regs;'
> > as xen_regs is #define'd to pt_regs on ia64 and the explicit
> > use of struct xen_regs in a header could cause header ordering
> > problems later.  Will that work for ppc?
> 
> Ah, sure... I was just trying to avoid extra dependency trees 
> where possible. 
> Apparently it is not possible here. :)
> 
> So that should be #include <public/xen.h> (which includes 
> arch-*.h which 
> defines xen_regs). It looks like we may want to move ia64's 
> #define to 
> arch-ia64.h ...
> 
> -- 
> Hollis Blanchard
> IBM Linux Technology Center
> 


-------------------------------------------------------
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_ide95&alloc_id396&op=click
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.