linux-kernel - Re: [PATCH v10 17/30] tracing: load/unload page callbacks for simple_ring

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20260129122136.69873587@gandalf.local.home>
Date: Thu, 29 Jan 2026 12:21:36 -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,
 aneesh.kumar@...nel.org, kernel-team@...roid.com,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v10 17/30] tracing: load/unload page callbacks for
 simple_ring_buffer

On Mon, 26 Jan 2026 10:44:06 +0000
Vincent Donnefort <vdonnefort@...gle.com> wrote:

> Add load/unload callback used for each admitted page in the ring-buffer.
> This will be later useful for the pKVM hypervisor which uses a different
> VA space and need to dynamically map/unmap the ring-buffer pages.
> 
> Signed-off-by: Vincent Donnefort <vdonnefort@...gle.com>
> 
> diff --git a/include/linux/simple_ring_buffer.h b/include/linux/simple_ring_buffer.h
> index f324df2f875b..ecd0e988c699 100644
> --- a/include/linux/simple_ring_buffer.h
> +++ b/include/linux/simple_ring_buffer.h
> @@ -110,4 +110,11 @@ int simple_ring_buffer_reset(struct simple_rb_per_cpu *cpu_buffer);
>   */
>  int simple_ring_buffer_swap_reader_page(struct simple_rb_per_cpu *cpu_buffer);
>  
> +int __simple_ring_buffer_init(struct simple_rb_per_cpu *cpu_buffer,
> +			      struct simple_buffer_page *bpages,
> +			      const struct ring_buffer_desc *desc,
> +			      void *(*load_page)(unsigned long va),
> +			      void (*unload_page)(void *va));
> +void __simple_ring_buffer_unload(struct simple_rb_per_cpu *cpu_buffer,
> +				 void (*unload_page)(void *));

Underscored functions are for internally used functions and I rather not
have them for generic use (in include/linux).

Perhaps rename them as:

 int simple_ring_buffer_init_va()
 void simple_ring_buffer_init_va()

Or maybe "_mm()" would be better?

-- Steve