Re: [Xen-devel] [PATCH] x86/mem_sharing: Relax sanity check for memops

[Andres Lagar Cavilla <andres@xxxxxxxxxxxxxxxx>]

On Fri, May 29, 2015 at 2:37 AM, Tamas K Lengyel <tklengyel@xxxxxxxxxxxxx> wrote:

The sharing vm_event ring being enabled is not necessary for mem_sharing
memops.

Signed-off-by: Tamas K Lengyel <tklengyel@xxxxxxxxxxxxx>
---
Â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 <andres@xxxxxxxxxxxxxxxx>Â

  Âswitch ( mso.op )
  Â{
    Âcase XENMEM_sharing_op_nominate_gfn:
--
2.1.4

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