[xen staging] xl: free bitmaps on exit

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[xen staging] xl: free bitmaps on exit

patchbot
commit 89bfc1effdf1aaaaa10b7c9f3d584c8a431dcd65
Author:     Olaf Hering <[hidden email]>
AuthorDate: Wed Nov 28 13:24:34 2018 +0100
Commit:     Wei Liu <[hidden email]>
CommitDate: Fri Nov 30 12:37:52 2018 +0000

    xl: free bitmaps on exit
   
    Every invocation of xl via valgrind will show three leaks.
    Since libxl_bitmap_alloc uses NOGC, the caller has to free the memory
    after use. And since xl_ctx_free might be called before
    parse_global_config, also move the libxl_bitmap_init calls into
    xl_ctx_alloc.
   
    Also move the call to atexit() after xl_ctx_alloc, because the latter is
    also called again in postfork.
   
    Signed-off-by: Olaf Hering <[hidden email]>
    Acked-by: Wei Liu <[hidden email]>
---
 tools/xl/xl.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/tools/xl/xl.c b/tools/xl/xl.c
index 7d2142f16f..ddd29b3f1b 100644
--- a/tools/xl/xl.c
+++ b/tools/xl/xl.c
@@ -209,11 +209,8 @@ static void parse_global_config(const char *configfile,
     if (!xlu_cfg_get_long (config, "max_maptrack_frames", &l, 0))
         max_maptrack_frames = l;
 
-    libxl_bitmap_init(&global_vm_affinity_mask);
     libxl_cpu_bitmap_alloc(ctx, &global_vm_affinity_mask, 0);
-    libxl_bitmap_init(&global_hvm_affinity_mask);
     libxl_cpu_bitmap_alloc(ctx, &global_hvm_affinity_mask, 0);
-    libxl_bitmap_init(&global_pv_affinity_mask);
     libxl_cpu_bitmap_alloc(ctx, &global_pv_affinity_mask, 0);
 
     if (!xlu_cfg_get_string (config, "vm.cpumask", &buf, 0))
@@ -323,11 +320,17 @@ void xl_ctx_alloc(void) {
         exit(1);
     }
 
+    libxl_bitmap_init(&global_vm_affinity_mask);
+    libxl_bitmap_init(&global_hvm_affinity_mask);
+    libxl_bitmap_init(&global_pv_affinity_mask);
     libxl_childproc_setmode(ctx, &childproc_hooks, 0);
 }
 
 static void xl_ctx_free(void)
 {
+    libxl_bitmap_dispose(&global_pv_affinity_mask);
+    libxl_bitmap_dispose(&global_hvm_affinity_mask);
+    libxl_bitmap_dispose(&global_vm_affinity_mask);
     if (ctx) {
         libxl_ctx_free(ctx);
         ctx = NULL;
@@ -383,10 +386,10 @@ int main(int argc, char **argv)
         (progress_use_cr ? XTL_STDIOSTREAM_PROGRESS_USE_CR : 0));
     if (!logger) exit(EXIT_FAILURE);
 
-    atexit(xl_ctx_free);
-
     xl_ctx_alloc();
 
+    atexit(xl_ctx_free);
+
     ret = libxl_read_file_contents(ctx, XL_GLOBAL_CONFIG,
             &config_data, &config_len);
     if (ret)
--
generated by git-patchbot for /home/xen/git/xen.git#staging

_______________________________________________
Xen-changelog mailing list
[hidden email]
https://lists.xenproject.org/xen-changelog