On Wed, Jan 6, 2016 at 4:48 PM, Ian Campbell <ian.campb...@citrix.com> wrote:
> On Wed, 2015-12-23 at 15:53 +0100, Tamas K Lengyel wrote: > > Introduce new vm_event domctl option which allows an event subscriber > > to request all vCPUs not currently pending a vm_event request to be > > paused, > > thus allowing the subscriber to sync up on the state of the domain. This > > is especially useful when the subscribed wants to disable certain events > > from being delivered and wants to ensure no more requests are pending on > > the > > ring before doing so. > > > > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > > Cc: Stefano Stabellini <stefano.stabell...@eu.citrix.com> > > Cc: Ian Campbell <ian.campb...@citrix.com> > > Cc: Wei Liu <wei.l...@citrix.com> > > Cc: Razvan Cojocaru <rcojoc...@bitdefender.com> > > Signed-off-by: Tamas K Lengyel <ta...@tklengyel.com> > > --- > > tools/libxc/include/xenctrl.h | 11 +++++++++++ > > tools/libxc/xc_vm_event.c | 16 ++++++++++++++++ > > Tools side is pretty trivial, assuming there is agreement on the underlying > hypercall interface: > > Acked-by: Ian Campbell <ian.campb...@citrix.com> > Thanks, we've decided that this patch is actually not needed as the pause reference count is already good enough. > > > +/*** > > * Memory sharing operations. > > Do you also maintain this? If so do you fancy sending a patch to fix: > > > * > > * Unles otherwise noted, these calls return 0 on succes, -1 and errno > on > > "Unless" and "success" ? > Sure, that could be done in a separate patch. IMHO the whole sharing subsystem could use a cleanup series of its own to fix things like this, style issues and whatnot. Tamas
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
