[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250306145607.0aa91e22@gandalf.local.home>
Date: Thu, 6 Mar 2025 14:56:07 -0500
From: Steven Rostedt <rostedt@...dmis.org>
To: Vincent Donnefort <vdonnefort@...gle.com>
Cc: mhiramat@...nel.org, mathieu.desnoyers@...icios.com,
linux-trace-kernel@...r.kernel.org, maz@...nel.org, oliver.upton@...ux.dev,
joey.gouly@....com, suzuki.poulose@....com, yuzenghui@...wei.com,
kvmarm@...ts.linux.dev, linux-arm-kernel@...ts.infradead.org,
jstultz@...gle.com, qperret@...gle.com, will@...nel.org,
kernel-team@...roid.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 02/11] ring-buffer: Expose buffer_data_page material
On Mon, 24 Feb 2025 12:13:44 +0000
Vincent Donnefort <vdonnefort@...gle.com> wrote:
> @@ -61,11 +65,50 @@ enum ring_buffer_type {
> RINGBUF_TYPE_TIME_STAMP,
> };
>
> +#define TS_SHIFT 27
> +#define TS_MASK ((1ULL << TS_SHIFT) - 1)
> +#define TS_DELTA_TEST (~TS_MASK)
By moving this to include/linux you need to add a prefix to prevent name
space conflicts.
RB_TS_SHIFT
RB_TS_MASK
RB_TS_DELTA_TEST
-- Steve
> +
> +/*
> + * We need to fit the time_stamp delta into 27 bits.
> + */
> +static inline bool test_time_stamp(u64 delta)
> +{
> + return !!(delta & TS_DELTA_TEST);
> +}
> +
Powered by blists - more mailing lists