Re: [PATCH v3 2/2] rust: add tracepoint support

2024-06-28 Thread Alice Ryhl
On Wed, Jun 26, 2024 at 8:43 PM Steven Rostedt wrote: > > On Wed, 26 Jun 2024 10:48:23 +0200 > Alice Ryhl wrote: > > > > > > > Because your hooks/rust_binder.h and events/rust_binder.h use the same > > > TRACE_SYSTEM name? Could you try something like: > > > > > > #define TRACE_SYSTEM rus

Re: [PATCH v3 2/2] rust: add tracepoint support

2024-06-26 Thread Steven Rostedt
On Wed, 26 Jun 2024 10:48:23 +0200 Alice Ryhl wrote: > > > > Because your hooks/rust_binder.h and events/rust_binder.h use the same > > TRACE_SYSTEM name? Could you try something like: > > > > #define TRACE_SYSTEM rust_binder_hook > > > > in your hooks/rust_binder.h? > > I was able to

Re: [PATCH v3 2/2] rust: add tracepoint support

2024-06-26 Thread Alice Ryhl
On Tue, Jun 25, 2024 at 8:11 PM Boqun Feng wrote: > > On Fri, Jun 21, 2024 at 02:52:10PM +0200, Alice Ryhl wrote: > [...] > > > > Hmm, I tried using the support where I have both events and hooks: > > > > #define CREATE_TRACE_POINTS > > #define CREATE_RUST_TRACE_POINTS > > #include > > #include

Re: [PATCH v3 2/2] rust: add tracepoint support

2024-06-25 Thread Boqun Feng
On Fri, Jun 21, 2024 at 02:52:10PM +0200, Alice Ryhl wrote: [...] > > Hmm, I tried using the support where I have both events and hooks: > > #define CREATE_TRACE_POINTS > #define CREATE_RUST_TRACE_POINTS > #include > #include > > But it's not really working. Initially I thought that it's becau

Re: [PATCH v3 2/2] rust: add tracepoint support

2024-06-21 Thread Alice Ryhl
On Fri, Jun 21, 2024 at 12:35 PM Alice Ryhl wrote: > > Make it possible to have Rust code call into tracepoints defined by C > code. It is still required that the tracepoint is declared in a C > header, and that this header is included in the input to bindgen. > > Signed-off-by: Alice Ryhl > ---

[PATCH v3 2/2] rust: add tracepoint support

2024-06-21 Thread Alice Ryhl
Make it possible to have Rust code call into tracepoints defined by C code. It is still required that the tracepoint is declared in a C header, and that this header is included in the input to bindgen. Signed-off-by: Alice Ryhl --- include/linux/tracepoint.h | 18 +++- include/t