On Tue, Feb 17, 2015 at 3:31 PM, Jan Beulich <jbeul...@suse.com> wrote:
>>>> On 13.02.15 at 17:33, <tamas.leng...@zentific.com> wrote:
>> @@ -611,13 +611,22 @@ int vm_event_domctl(struct domain *d,
>> xen_domctl_vm_event_op_t *vec,
>> }
>> break;
>>
>> - case XEN_VM_EVENT_PAGING_DISABLE:
>> + case XEN_VM_EVENT_DISABLE:
>> {
>> if ( ved->ring_page )
>> rc = vm_event_disable(d, ved);
>> }
>> break;
>>
>> + case XEN_VM_EVENT_RESUME:
>> + {
>> + if ( ved->ring_page )
>> + vm_event_resume(d, ved);
>> + else
>> + rc = -ENODEV;
>> + }
>> + break;
>
> Stray braces again.
Ack.
>
> I also find it confusing that the same set of changes repeats three
> times here - is that an indication of a problem with an earlier patch?
No it's not. There are three rings vm_event can use, thus three rings
that can be resumed.
>
> Jan
Thanks,
Tamas
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
