qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH] xen: handle inbound migration of VMs without io


From: Paul Durrant
Subject: Re: [Qemu-devel] [PATCH] xen: handle inbound migration of VMs without ioreq server pages
Date: Tue, 9 Aug 2016 15:23:05 +0000

> -----Original Message-----
> From: Anthony PERARD [mailto:address@hidden
> Sent: 09 August 2016 16:19
> To: Paul Durrant
> Cc: address@hidden; address@hidden; Stefano
> Stabellini
> Subject: Re: [PATCH] xen: handle inbound migration of VMs without ioreq
> server pages
> 
> On Mon, Aug 01, 2016 at 10:16:25AM +0100, Paul Durrant wrote:
> > VMs created on older versions on Xen will not have been provisioned with
> > pages to support creation of non-default ioreq servers. In this case
> > the ioreq server API is not supported and QEMU's only option is to fall
> > back to using the default ioreq server pages as it did prior to
> > commit 3996e85c ("Xen: Use the ioreq-server API when available").
> >
> > This patch therefore changes the code in xen_common.h to stop
> considering
> > a failure of xc_hvm_create_ioreq_server() as a hard failure but simply
> > as an indication that the guest is too old to support the ioreq server
> > API. Instead a boolean is set to cause reversion to old behaviour such
> > that the default ioreq server is then used.
> >
> > Signed-off-by: Paul Durrant <address@hidden>
> > Cc: Stefano Stabellini <address@hidden>
> > Cc: Anthony Perard <address@hidden>
> 
> There are just two lines too long:
> 
> WARNING: line over 80 characters
> #31: FILE: include/hw/xen/xen_common.h:110:
> +static inline int xen_get_default_ioreq_server_info(xc_interface *xc,
> domid_t dom,
> 
> WARNING: line over 80 characters
> #34: FILE: include/hw/xen/xen_common.h:113:
> +                                                    evtchn_port_t 
> *bufioreq_evtchn)
> 
> With that fixed: Acked-by: Anthony PERARD <address@hidden>
> 
> Thanks,
> 

Ok, I'll post v2 with those fixes and your ack. Thanks,

  Paul

> --
> Anthony PERARD



reply via email to

[Prev in Thread] Current Thread [Next in Thread]