diff mbox series

x86/mem_sharing: gate enabling on cpu_has_vmx

Message ID 20200525144606.126767-1-tamas.lengyel@intel.com (mailing list archive)
State New, archived
Headers show
Series x86/mem_sharing: gate enabling on cpu_has_vmx | expand

Commit Message

Tamas K Lengyel May 25, 2020, 2:46 p.m. UTC 
From: Tamas K Lengyel <tamas@tklengyel.com>

It is unclear whether mem_sharing was ever made to work on other architectures
but at this time the only verified platform for it is vmx. No plans to support
or maintain it on other architectures. Make this explicit by checking during
initialization.

Signed-off-by: Tamas K Lengyel <tamas@tklengyel.com>
---
 xen/arch/x86/mm/mem_sharing.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Wei Liu May 25, 2020, 6:47 p.m. UTC | #1 
On Mon, May 25, 2020 at 08:46:06AM -0600, Tamas K Lengyel wrote:
> From: Tamas K Lengyel <tamas@tklengyel.com>
> 
> It is unclear whether mem_sharing was ever made to work on other architectures
> but at this time the only verified platform for it is vmx. No plans to support
> or maintain it on other architectures. Make this explicit by checking during
> initialization.
> 
> Signed-off-by: Tamas K Lengyel <tamas@tklengyel.com>

Reviewed-by: Wei Liu <wl@xen.org>
diff mbox series

Patch

diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c
index 7271e5c90b..19922ab5d1 100644
--- a/xen/arch/x86/mm/mem_sharing.c
+++ b/xen/arch/x86/mm/mem_sharing.c
@@ -1444,7 +1444,7 @@  static inline int mem_sharing_control(struct domain *d, bool enable,
 {
     if ( enable )
     {
-        if ( unlikely(!is_hvm_domain(d)) )
+        if ( unlikely(!is_hvm_domain(d) || !cpu_has_vmx) )
             return -EOPNOTSUPP;
 
         if ( unlikely(!hap_enabled(d)) )