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

Re: [Xen-devel] [PATCH] qemu build: disable C++ optional code



On Mon, 7 Jul 2014, Ian Campbell wrote:
> On Mon, 2014-07-07 at 12:47 +0100, Stefano Stabellini wrote:
> > Disable C++ optimizations for the intruction emulators: we don't need
> > them and they can cause build issues with Xen.
> 
> This doesn't help distros who want to use their existing qemu packages
> with Xen though, or in general people who prefer their own qemu to ours.

That is true, however if you are going to compile QEMU to use it as Xen
device model and nothing else, you might as well disable things you
don't need, such KVM support and C++ optimization.
That's why I sent the patch.


> What sort of build issues do we see? Is this to do with
> xen/include/public not being C++ safe or is it code issues within the
> Xen parts or qemu?

The problem is that we use --extra-cflags to pass

-I$(XEN_ROOT)/tools/include
-I$(XEN_ROOT)/tools/libxc
-I$(XEN_ROOT)/tools/xenstore

tools/utils contains an header file named utils.h, same name as
disas/libvixl/utils.h in QEMU. Files under disas/libvixl #include
"utils.h" picking up the wrong one. For some reason, I have only seen
this build problem on ARM so far.



> > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > 
> > 
> > diff --git a/tools/Makefile b/tools/Makefile
> > index f4aa200..da20f5a 100644
> > --- a/tools/Makefile
> > +++ b/tools/Makefile
> > @@ -201,6 +201,7 @@ subdir-all-qemu-xen-dir: qemu-xen-dir-find
> >             --disable-kvm \
> >             --disable-docs \
> >             --disable-guest-agent \
> > +           --cxx=none \
> >             --python=$(PYTHON) \
> >             $(CONFIG_QEMUU_EXTRA_ARGS) \
> >             $(IOEMU_CONFIGURE_CROSS); \
> 
> 

_______________________________________________
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®.