From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: Sun, 25 Oct 2015 16:02:24 +0000 (-0400)
Subject: Callstack context: bump number of entries to 128
X-Git-Tag: v2.11.0-rc1~25
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=3685cc8094c0fdf561f1ef6379f0e451d14d149e;p=lttng-modules.git

Callstack context: bump number of entries to 128

Use a limit that fits in a 4096 bytes page on a 64-bit system. The only
reason for the prior 25 entries limitation was a bug in the header size
calculation (now fixed).

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
---

diff --git a/lttng-context-callstack.c b/lttng-context-callstack.c
index bdc1761e..fd730bbc 100644
--- a/lttng-context-callstack.c
+++ b/lttng-context-callstack.c
@@ -32,7 +32,7 @@
  *
  *   size = cpus * nest * depth * sizeof(unsigned long)
  *
- * Which is about 800 bytes per CPU on 64-bit host and a depth of 25.
+ * Which is 4096 bytes per CPU on 64-bit host and a depth of 128.
  * The allocation is done at the initialization to avoid memory
  * allocation overhead while tracing, using a shallow stack.
  *
@@ -62,7 +62,7 @@
 #include "wrapper/vmalloc.h"
 #include "lttng-tracer.h"
 
-#define MAX_ENTRIES 25
+#define MAX_ENTRIES 128
 
 enum lttng_cs_ctx_modes {
 	CALLSTACK_KERNEL = 0,