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

Re: [Xen-devel] [PATCH] qemu-xen-trad: fix sys-queue.h usage on BSD systems



On Thu, 31 May 2012, Roger Pau Monne wrote:
> Stefano Stabellini wrote:
> > On Wed, 30 May 2012, Roger Pau Monne wrote:
> >> BSD systems already have a sys/queue.h file, which has more macros
> >> than the one Qemu uses, and some header files depend on having that
> >> macros defined (sys/disk.h for example). Disable sys-queue.h on BSD
> >> systems and include the native one.
> >>
> >> This is not a backport because the original patch is too dificult to
> >> backport, it's commit 72cf2d4f0e181d0d3a3122e04129c58a95da713e.
> >
> > The upstream commit message states:
> >
> > "Problem: Our file sys-queue.h is a copy of the BSD file, but there are
> > some additions and it's not entirely compatible. Because of that, there
> > have been conflicts with system headers on BSD systems."
> >
> > Wouldn't this be a problem if we apply the simple patch below?
> 
> Doing a diff -bB shows that the Qemu version is just a stripped version 
> of the original NetBSD header, with many macros removed, but no new ones 
> added, so I think the patch is safe.

OK. Please add says this in the commit message.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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