From: Francis Giraldeau <[email protected]>

Extract the constant within the code as #define. The define is added to
frontend.h in order to be included in other source files.

Signed-off-by: Francis Giraldeau <[email protected]>
Signed-off-by: Mathieu Desnoyers <[email protected]>
---
 lib/ringbuffer/frontend.h     | 3 +++
 lib/ringbuffer/frontend_api.h | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/ringbuffer/frontend.h b/lib/ringbuffer/frontend.h
index 6ff1545..4b9b011 100644
--- a/lib/ringbuffer/frontend.h
+++ b/lib/ringbuffer/frontend.h
@@ -51,6 +51,9 @@
 /* Internal helpers */
 #include <wrapper/ringbuffer/frontend_internal.h>
 
+/* Max ring buffer nesting count, see lib_ring_buffer_get_cpu(). */
+#define RING_BUFFER_MAX_NESTING 4
+
 /* Buffer creation/removal and setup operations */
 
 /*
diff --git a/lib/ringbuffer/frontend_api.h b/lib/ringbuffer/frontend_api.h
index a6f2c6f..85943b9 100644
--- a/lib/ringbuffer/frontend_api.h
+++ b/lib/ringbuffer/frontend_api.h
@@ -58,7 +58,7 @@ int lib_ring_buffer_get_cpu(const struct 
lib_ring_buffer_config *config)
        nesting = ++per_cpu(lib_ring_buffer_nesting, cpu);
        barrier();
 
-       if (unlikely(nesting > 4)) {
+       if (unlikely(nesting > RING_BUFFER_MAX_NESTING)) {
                WARN_ON_ONCE(1);
                per_cpu(lib_ring_buffer_nesting, cpu)--;
                rcu_read_unlock_sched_notrace();
-- 
2.1.4

_______________________________________________
lttng-dev mailing list
[email protected]
https://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev

Reply via email to