[Xen-devel] RE: [PATCH] Re: [Xen-staging] [xen-unstable] [qemu]

On Thu, 2007-05-10 at 13:01 -0700, Christian Limpach wrote:
> >    ia64intrin.h is all but empty in gcc 4.1.2, so there's no point in
> > including it.  dm-exec is still making use of a few of these 
> > too, but I
> > guess it's easy enough to inline them there.  Patch below.  Thanks,
> 
> Would it also work to just leave the include in exec-all.h?  I was
> confused and thought that it was a local include which is why I put
> #ifndef CONFIG_DM around all the testandset definitions.
> 
> So, if I remove the #ifndef CONFIG_DM, anything including exec-all.h
> should still compile, right?  Do I still need the extra #define's in
> that case?

   ia64intrin.h is basically empty, so making it included for the
dm-exec build doesn't really help.  The defines still need to come from
somewhere.  We could replace <ia64intrin.h> with <asm/gcc_intrin.h>, but
for this trivial case, I'd rather not get tied up including kernel
headers in userspace tools.  Thanks,

        Alex

-- 
Alex Williamson                             HP Open Source & Linux Org.


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

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:

[Xen-devel] RE: [PATCH] Re: [Xen-staging] [xen-unstable] [qemu] Removeatomic_set_bit, atomic_clear_bit and ia64_intrinsic.h and its, Christian Limpach

Next by Date:

[Xen-devel] [PATCH] 0/4 Xen Scheduling Groups, Mike D. Day

Previous by Thread:

[Xen-devel] RE: [PATCH] Re: [Xen-staging] [xen-unstable] [qemu] Removeatomic_set_bit, atomic_clear_bit and ia64_intrinsic.h and its, Christian Limpach

Next by Thread:

[Xen-devel] [PATCH] 0/4 Xen Scheduling Groups, Mike D. Day

Indexes:

[Date] [Thread] [Top] [All Lists]