Re: [Xen-devel] [PATCH v5] common/vm_event: Initialize vm_event lists on domain creation

[Tamas K Lengyel <tamas@xxxxxxxxxxxxx>]

On Tue, Aug 29, 2017 at 8:17 AM, Alexandru Isaila
<aisaila@xxxxxxxxxxxxxxx> wrote:
> The patch splits the vm_event into three structures:vm_event_share,
> vm_event_paging, vm_event_monitor. The allocation for the
> structure is moved to vm_event_enable so that it can be
> allocated/init when needed and freed in vm_event_disable.
>
> Signed-off-by: Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>
>
> ---
> Changes since V4:
>         - Replaced all NULL checks with vm_event_check_ring
>
> Note: Did not test on arm, compliled on arm and x86.

This looks good to me as is but could you at least check with the
xen-access test tool on x86 that it still works as expected?

Thanks,
Tamas

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel