On Thu, Aug 13, 2015 at 02:06:04PM +0300, Michael S. Tsirkin wrote:
> On Fri, Aug 07, 2015 at 04:55:56PM -0300, Eduardo Habkost wrote:
[...]
> > @@ -1159,7 +1158,7 @@ static void xen_wakeup_notifier(Notifier *notifier, 
> > void *data)
> >  }
> >  
> >  /* return 0 means OK, or -1 means critical issue -- will exit(1) */
> > -int xen_hvm_init(ram_addr_t *below_4g_mem_size, ram_addr_t 
> > *above_4g_mem_size,
> > +int xen_hvm_init(PCMachineState *pcms,
> >                   MemoryRegion **ram_memory)
> >  {
> >      int i, rc;
> 
> This breaks the build: you didn't update the version in xen-hvm-stub.c

Oops, sorry.

> 
> Please test on config with all options enabled.

I incorrectly assumed the changes wouldn't affect anything except PC, so
I was building it with --target-list=x86_64-softmmu --enable-xen. I
should remember to build it at least once with all targets enabled, next
time.

-- 
Eduardo

Reply via email to