On Thu, Mar 12, 2015 at 4:56 PM, Tim Deegan <t...@xen.org> wrote: > At 01:11 +0100 on 18 Feb (1424218303), Tamas K Lengyel wrote: > > Thus far mem_access and mem_sharing memops had been able to signal > > to Xen to start pulling responses off the corresponding rings. In this > patch > > we retire these memops and add them to the option to the vm_event_op > domctl. > > > > The vm_event_op domctl suboptions are the same for each ring thus we > > consolidate them into XEN_VM_EVENT_ENABLE/DISABLE/RESUME. > > > > As part of this patch in libxc we also rename the > mem_access_enable/disable > > functions to monitor_enable/disable and move them into xc_monitor.c. > > > > Signed-off-by: Tamas K Lengyel <tamas.leng...@zentific.com> > > Acked-by: Wei Liu <wei.l...@citrix.com> > > I think there's still an outstanding question about the repeated code > in vm_event.c, which could be unified -- last time you suggested maybe > just getting rid of the wakeup operation entirely. >
I'm still trying to figure out if there is a different use-case that may necessitate having the two redundant operations that we have today. For now it doesn't hurt and we can retire one if it is indeed verified there is no need to have two. > > Nevertheless I think this patch is good on its own. So, > > Acked-by: Tim Deegan <t...@xen.org> > > Cheers, > > Tim > Thanks, Tamas
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
