Arch-specific arch_p2m_set_access_required() is reponsible for
XEN_DOMCTL_set_access_required domctl-op, and shall be wrapped.

Signed-off-by: Penny Zheng <penny.zh...@amd.com>
---
 xen/arch/arm/mem_access.c    | 2 ++
 xen/arch/x86/mm/mem_access.c | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/xen/arch/arm/mem_access.c b/xen/arch/arm/mem_access.c
index 2af92bb402..e205b6be8f 100644
--- a/xen/arch/arm/mem_access.c
+++ b/xen/arch/arm/mem_access.c
@@ -450,11 +450,13 @@ int p2m_get_mem_access(struct domain *d, gfn_t gfn,
     return ret;
 }
 
+#ifdef CONFIG_DOMCTL
 void arch_p2m_set_access_required(struct domain *d, bool access_required)
 {
     ASSERT(atomic_read(&d->pause_count));
     p2m_get_hostp2m(d)->access_required = access_required;
 }
+#endif /* CONFIG_DOMCTL */
 
 /*
  * Local variables:
diff --git a/xen/arch/x86/mm/mem_access.c b/xen/arch/x86/mm/mem_access.c
index 21b5b7ecda..de694c16e9 100644
--- a/xen/arch/x86/mm/mem_access.c
+++ b/xen/arch/x86/mm/mem_access.c
@@ -480,6 +480,7 @@ int p2m_get_mem_access(struct domain *d, gfn_t gfn, 
xenmem_access_t *access,
     return _p2m_get_mem_access(p2m, gfn, access);
 }
 
+#ifdef CONFIG_DOMCTL
 void arch_p2m_set_access_required(struct domain *d, bool access_required)
 {
     ASSERT(atomic_read(&d->pause_count));
@@ -498,6 +499,7 @@ void arch_p2m_set_access_required(struct domain *d, bool 
access_required)
         }
     }
 }
+#endif /* CONFIG_DOMCTL */
 
 bool p2m_mem_access_sanity_check(const struct domain *d)
 {
-- 
2.34.1


Reply via email to