Claude review: Re: [PATCH 01/15] tracepoint: Add trace_invoke_##name() API

[Claude Code Review Bot <claude-review@example.com>]

Patch Review

The core API patch. Three changes to `include/linux/tracepoint.h`:

1. **`__DECLARE_TRACE`** — adds `trace_invoke_##name()` after `trace_##name()`:
```c
static inline void trace_invoke_##name(proto)
{
    __do_trace_##name(args);
}
```
This correctly skips the `static_branch_unlikely()` but as noted above, it also **omits the LOCKDEP assertion**. The commit message claims it "retains the LOCKDEP RCU-watching assertion" which is inaccurate.

2. **`__DECLARE_TRACE_SYSCALL`** — same pattern but includes `might_fault()`:
```c
static inline void trace_invoke_##name(proto)
{
    might_fault();
    __do_trace_##name(args);
}
```
This correctly preserves the `might_fault()` check. Same LOCKDEP concern applies here too.

3. **`!TRACEPOINTS_ENABLED` stub** — adds an empty inline:
```c
static inline void trace_invoke_##name(proto)
{ }
```
Correct — compiles out cleanly when tracepoints are disabled.

**Minor nit**: The `trace_invoke_` naming prefix is somewhat verbose. An alternative like `trace_do_` was presumably considered and rejected, but the chosen name is fine — it's explicit about what it does.

---
Generated by Claude Code Patch Reviewer