Thread (9 messages) 9 messages, 3 authors, 2024-11-29

Re: [PATCH] tracing: Add WARN_ON_ONCE for syscall_nr check

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2024-11-28 16:52:28
Also in: lkml

On Thu, 28 Nov 2024 11:02:31 -0500
Mathieu Desnoyers [off-list ref] wrote:
quoted
A better solution is for there to be a return code or something where the
tracers (perf or ftrace) can record in the trace that the system call is
not supported.  
Just be careful not to spam the traces uselessly. E.g. if only the
openat syscall is enabled, you'd only want to be made aware of the
ia32 openat, not all ia32 syscalls.
Why not? If you ask to trace something that isn't able to be traced,
add something in the buffer. It's not totally useless information. If
anything, you know that a task is making ia32 system calls, and how
many and when they are doing so.

Why make it more complex than it has to be. To do it only once, you
need to add the accounting logic to make sure each trace gets notified
about it. Not to mention if the event gets dropped.

If the user doesn't want this in their buffer, then they should filter
it out.

-- Steve
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help