Re: [PATCH v2] ring-buffer: Simplify reservation with try_cmpxchg() loop

2024-02-19 Thread Steven Rostedt
On Mon, 19 Feb 2024 17:30:03 -0500 Steven Rostedt wrote: > - /*C*/ write = local_add_return(info->length, &tail_page->write); > + /*C*/ if (!local_try_cmpxchg(&tail_page->write, &w, w + > info->length)) { > + if (info.add_timestamp & (RB_ADD_STAMP_FORCE | > RB_ADD_STAMP_

[PATCH v2] ring-buffer: Simplify reservation with try_cmpxchg() loop

2024-02-19 Thread Steven Rostedt
From: "Steven Rostedt (Google)" Instead of using local_add_return() to reserve the ring buffer data, Mathieu Desnoyers suggested using local_cmpxchg(). This would simplify the reservation with the time keeping code. Although, it does not get rid of the double time stamps (before_stamp and write_