Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Tamas K Lengyel
On Tue, Feb 16, 2016 at 10:48 AM, Corneliu ZUZU wrote: > On 2/16/2016 6:02 PM, Tamas K Lengyel wrote: > > >> @@ -143,77 +72,75 @@ int monitor_domctl(struct domain *d, struct >> xen_domctl_monitor_op *mop) >> >> case XEN_DOMCTL_MONITOR_EVENT_MOV_TO_MSR: >> { >> >

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Corneliu ZUZU
On 2/16/2016 6:02 PM, Tamas K Lengyel wrote: @@ -143,77 +72,75 @@ int monitor_domctl(struct domain *d, struct xen_domctl_monitor_op *mop) case XEN_DOMCTL_MONITOR_EVENT_MOV_TO_MSR: { So since we will now have two separate booleans, requested_status and old_status

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Tamas K Lengyel
> > > @@ -143,77 +72,75 @@ int monitor_domctl(struct domain *d, struct > xen_domctl_monitor_op *mop) > > case XEN_DOMCTL_MONITOR_EVENT_MOV_TO_MSR: > { > So since we will now have two separate booleans, requested_status and old_status and then manually verify they are opposite.. > -

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Razvan Cojocaru
On 02/16/2016 09:08 AM, Corneliu ZUZU wrote: > This patch moves monitor_domctl to common-side. > Purpose: move what's common to common, prepare for implementation > of such vm-events on ARM. > > * move get_capabilities to arch-side => arch_monitor_get_capabilities. > * add arch-side monitor op

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Corneliu ZUZU
On 2/16/2016 2:34 PM, Jan Beulich wrote: On 16.02.16 at 12:20, wrote: On 2/16/2016 12:45 PM, Jan Beulich wrote: On 16.02.16 at 09:13, wrote: On 2/16/2016 9:08 AM, Corneliu ZUZU wrote: This patch moves monitor_domctl to common-side. Purpose:

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Jan Beulich
>>> On 16.02.16 at 12:20, wrote: > On 2/16/2016 12:45 PM, Jan Beulich wrote: > On 16.02.16 at 09:13, wrote: >>> On 2/16/2016 9:08 AM, Corneliu ZUZU wrote: This patch moves monitor_domctl to common-side. Purpose: move what's common to

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Corneliu ZUZU
On 2/16/2016 12:45 PM, Jan Beulich wrote: On 16.02.16 at 09:13, wrote: On 2/16/2016 9:08 AM, Corneliu ZUZU wrote: This patch moves monitor_domctl to common-side. Purpose: move what's common to common, prepare for implementation of such vm-events on ARM. * move

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Stefano Stabellini
On Tue, 16 Feb 2016, Corneliu ZUZU wrote: > This patch moves monitor_domctl to common-side. > Purpose: move what's common to common, prepare for implementation > of such vm-events on ARM. > > * move get_capabilities to arch-side => arch_monitor_get_capabilities. > * add arch-side monitor op

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Jan Beulich
>>> On 16.02.16 at 09:13, wrote: > On 2/16/2016 9:08 AM, Corneliu ZUZU wrote: >> This patch moves monitor_domctl to common-side. >> Purpose: move what's common to common, prepare for implementation >> of such vm-events on ARM. >> >> * move get_capabilities to arch-side =>

Re: [Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-16 Thread Corneliu ZUZU
On 2/16/2016 9:08 AM, Corneliu ZUZU wrote: This patch moves monitor_domctl to common-side. Purpose: move what's common to common, prepare for implementation of such vm-events on ARM. * move get_capabilities to arch-side => arch_monitor_get_capabilities. * add arch-side monitor op handling

[Xen-devel] [PATCH v4 2/2] xen/vm-events: Move parts of monitor_domctl code to common-side.

2016-02-15 Thread Corneliu ZUZU
This patch moves monitor_domctl to common-side. Purpose: move what's common to common, prepare for implementation of such vm-events on ARM. * move get_capabilities to arch-side => arch_monitor_get_capabilities. * add arch-side monitor op handling function => arch_monitor_domctl_op. e.g.