[Xen-devel] [PATCH v2] x86/mem-sharing: statically initialize audit list head and lock

["Jan Beulich" <JBeulich@xxxxxxxx>]

There's no need to execute any instructions for doing so. Drop the then
effectively empty mem_sharing_init() altogether.

Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
---
v2: Drop mem_sharing_init().

--- a/xen/arch/x86/mm/mem_sharing.c
+++ b/xen/arch/x86/mm/mem_sharing.c
@@ -65,8 +65,8 @@ static DEFINE_PER_CPU(pg_lock_data_t, __
 
 #if MEM_SHARING_AUDIT
 
-static struct list_head shr_audit_list;
-static spinlock_t shr_audit_lock;
+static LIST_HEAD(shr_audit_list);
+static DEFINE_SPINLOCK(shr_audit_lock);
 static DEFINE_RCU_READ_LOCK(shr_audit_read_lock);
 
 /* RCU delayed free of audit list entry */
@@ -1650,13 +1650,3 @@ int mem_sharing_domctl(struct domain *d,
 
     return rc;
 }
-
-void __init mem_sharing_init(void)
-{
-    printk("Initing memory sharing.\n");
-#if MEM_SHARING_AUDIT
-    spin_lock_init(&shr_audit_lock);
-    INIT_LIST_HEAD(&shr_audit_list);
-#endif
-}
-
--- a/xen/arch/x86/mm.c
+++ b/xen/arch/x86/mm.c
@@ -368,8 +368,6 @@ void __init arch_init_memory(void)
 
     efi_init_memory();
 
-    mem_sharing_init();
-
 #ifndef NDEBUG
     if ( highmem_start )
     {
--- a/xen/include/asm-x86/mem_sharing.h
+++ b/xen/include/asm-x86/mem_sharing.h
@@ -88,7 +88,6 @@ int mem_sharing_notify_enomem(struct dom
 int mem_sharing_memop(XEN_GUEST_HANDLE_PARAM(xen_mem_sharing_op_t) arg);
 int mem_sharing_domctl(struct domain *d, 
                        struct xen_domctl_mem_sharing_op *mec);
-void mem_sharing_init(void);
 
 /* Scans the p2m and relinquishes any shared pages, destroying 
  * those for which this domain holds the final reference.





_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel