Fix vpp crashing when attempting to run in kubernetes Pod 38/17838/5
authorArtem Belov <artem.belov@xored.com>
Tue, 26 Feb 2019 01:47:34 +0000 (01:47 +0000)
committerDamjan Marion <dmarion@me.com>
Tue, 26 Feb 2019 11:26:03 +0000 (11:26 +0000)
mmap does not fail but writing to mapped memory is causing sigbus.

Change-Id: I5135f32eede67fccb4aaa07a501cd262d254ed8d
Signed-off-by: Artem Belov <artem.belov@xored.com>
src/vppinfra/pmalloc.c

index dd772f3..f421665 100644 (file)
@@ -18,6 +18,7 @@
 #include <sys/types.h>
 #include <sys/stat.h>
 #include <fcntl.h>
+#include <unistd.h>
 #include <linux/mempolicy.h>
 #include <linux/memfd.h>
 
@@ -263,6 +264,7 @@ pmalloc_map_pages (clib_pmalloc_main_t * pm, clib_pmalloc_arena_t * a,
   int old_mpol = -1;
   long unsigned int mask[16] = { 0 };
   long unsigned int old_mask[16] = { 0 };
+  uword page_size = 1 << a->log2_subpage_sz;
   uword size = (uword) n_pages << pm->def_log2_page_sz;
 
   clib_error_free (pm->error);
@@ -335,6 +337,25 @@ pmalloc_map_pages (clib_pmalloc_main_t * pm, clib_pmalloc_arena_t * a,
       goto error;
     }
 
+  /* Check if huge page is not allocated,
+     wrong allocation will generate the SIGBUS */
+  if (a->log2_subpage_sz != pm->sys_log2_page_sz)
+    {
+      for (int i = 0; i < n_pages; i++)
+       {
+         unsigned char flag;
+         mincore (va + i * page_size, 1, &flag);
+         // flag is 1 if the page was successfully allocated and in memory
+         if (!flag)
+           {
+             pm->error =
+               clib_error_return_unix (0,
+                                       "Unable to fulfill huge page allocation request");
+             goto error;
+           }
+       }
+    }
+
   clib_memset (va, 0, size);
 
   rv = set_mempolicy (old_mpol, old_mask, sizeof (old_mask) * 8 + 1);