From d2ddd2beff0feda877cc6d8a0e9ca97e848bcb78 Mon Sep 17 00:00:00 2001 From: Andrew Cooper Date: Fri, 5 Jun 2026 20:28:35 +0100 Subject: [PATCH] x86/hvm: Partially revert ("xen/mem_access: wrap memory access when VM_EVENT=n") It is erroneous to check current like this; most commonly, introspection is dom0 issuing hypercalls against domU. The use of vm_event_is_enabled() is only for the IS_ENABLED(CONFIG_VM_EVENT) short circut, so just use that directly. Reported-by: Hady Azzam Fixes: b18e38e42da6 ("xen/mem_access: wrap memory access when VM_EVENT=n") Signed-off-by: Andrew Cooper Reviewed-by: Jason Andryuk Tested-by: Hady Azzam Release-Acked-by: Oleksii Kurochko --- xen/arch/x86/hvm/hvm.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c index f759a397c5..cbcef18449 100644 --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -4789,7 +4789,7 @@ static int do_altp2m_op( break; case HVMOP_altp2m_set_mem_access: - if ( !vm_event_is_enabled(current) ) + if ( !IS_ENABLED(CONFIG_VM_EVENT) ) { rc = -EOPNOTSUPP; break; @@ -4804,7 +4804,7 @@ static int do_altp2m_op( break; case HVMOP_altp2m_set_mem_access_multi: - if ( !vm_event_is_enabled(current) ) + if ( !IS_ENABLED(CONFIG_VM_EVENT) ) { rc = -EOPNOTSUPP; break; @@ -4841,7 +4841,7 @@ static int do_altp2m_op( break; case HVMOP_altp2m_get_mem_access: - if ( !vm_event_is_enabled(current) ) + if ( !IS_ENABLED(CONFIG_VM_EVENT) ) { rc = -EOPNOTSUPP; break;