From: Mathieu Desnoyers Date: Thu, 8 Nov 2018 17:42:29 +0000 (-0500) Subject: Clarify lib_ring_buffer_switch_slow() requirements X-Git-Tag: v2.12.0-rc1~67 X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=ef1a13e0de71e01d19a9d1b2d4abdac5e34eb9bc;p=lttng-ust.git Clarify lib_ring_buffer_switch_slow() requirements Signed-off-by: Mathieu Desnoyers --- diff --git a/libringbuffer/ring_buffer_frontend.c b/libringbuffer/ring_buffer_frontend.c index 0d0f2686..9721df16 100644 --- a/libringbuffer/ring_buffer_frontend.c +++ b/libringbuffer/ring_buffer_frontend.c @@ -1996,9 +1996,11 @@ int lib_ring_buffer_try_switch_slow(enum switch_mode mode, * Force a sub-buffer switch. This operation is completely reentrant : can be * called while tracing is active with absolutely no lock held. * - * Note, however, that as a v_cmpxchg is used for some atomic - * operations, this function must be called from the CPU which owns the buffer - * for a ACTIVE flush. + * For RING_BUFFER_SYNC_PER_CPU ring buffers, as a v_cmpxchg is used for + * some atomic operations, this function must be called from the CPU + * which owns the buffer for a ACTIVE flush. However, for + * RING_BUFFER_SYNC_GLOBAL ring buffers, this function can be called + * from any CPU. */ void lib_ring_buffer_switch_slow(struct lttng_ust_lib_ring_buffer *buf, enum switch_mode mode, struct lttng_ust_shm_handle *handle)