On Mon, 29 Nov 2010, Alexander Graf wrote:
>
> On 29.11.2010, at 16:10, Anthony PERARD wrote:
>
> > On Mon, 29 Nov 2010, Alexander Graf wrote:
> >
> >>
> >> On 23.11.2010, at 20:51, Anthony.Perard@xxxxxxxxxx wrote:
> >>
> >>> From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> >>>
> >>> Hi all,
> >>>
> >>> Here is the V7 of the patch series that adds Xen device model support in
> >>> QEMU.
> >>>
> >>> The change made on it since the v6:
> >>> - I introduce a patch from Alexander Graf to add a generic layer for the
> >>> Xen calls.
> >>> - So compatibility support have been transformed from macros to function.
> >>> - Now, xenfv machine uses a new field default_machine_opts to specify
> >>> that Xen will be use.
> >>> - in xen_init, xen_mode is override to XEN_ATTACH if it was XEN_EMULATE.
> >>
> >> Please try to compile the code on Xen 4.0 :). I'm just trying to rebase
> >> xenner against your tree and the build already fails because of missing
> >> xen_common.h includes.
> >
> > :(, I don't have any error with both Xen 4.0.{0,1}.
> >
> > # Configured with: '../configure' '--target-list=i386-softmmu'
> > '--enable-xen'
> > '--extra-cflags=-I/tmp/xen/xen-4.0.1/dist/install/usr/include'
> > '--extra-ldflags=-L/tmp/xen/xen-4.0.1/dist/install/usr/lib'
> >
> > Did you use other options with your configure?
> > Where are the missings includes?
> > Also, I only compile on 32bits.
>
> Ugh. Yes, my bad. It broke in my xenner patches :). Sorry. The issue was that
> #include "xen_interfaces.h" without a preceding #include "xen_common.h"
> breaks. Maybe it'd be good to move the "xen_common.h" include into "xen.h"?
> Doesn't really matter though. I'll try to take on a code review today, so we
> can merge the patches asap!
Actually, it's better to put "#include xen_common.h" in xen_interfaces.h
because I use stuff from there.
Thanks,
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
Home