On Fri, May 29, 2015 at 2:37 AM, Tamas K Lengyel <tkleng...@sec.in.tum.de>
wrote:

> The sharing vm_event ring being enabled is not necessary for mem_sharing
> memops.
>
> Signed-off-by: Tamas K Lengyel <tkleng...@sec.in.tum.de>
> ---
>  xen/arch/x86/mm/mem_sharing.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c
> index 0700f00..16e329e 100644
> --- a/xen/arch/x86/mm/mem_sharing.c
> +++ b/xen/arch/x86/mm/mem_sharing.c
> @@ -1320,10 +1320,6 @@ int
> mem_sharing_memop(XEN_GUEST_HANDLE_PARAM(xen_mem_sharing_op_t) arg)
>      if ( !hap_enabled(d) || !d->arch.hvm_domain.mem_sharing_enabled )
>          goto out;
>
> -    rc = -ENODEV;
> -    if ( unlikely(!d->vm_event->share.ring_page) )
> -        goto out;
> -
>
Reviewed-by: Andres Lagar-Cavilla <and...@lagarcavilla.org>

>      switch ( mso.op )
>      {
>          case XENMEM_sharing_op_nominate_gfn:
> --
> 2.1.4
>
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to