On Wed, 29 Apr 2026 17:51:44 +0900
Masami Hiramatsu (Google) <[email protected]> wrote:

> Hi Steve,
> 
> BTW, to prevent regressions during future expansions, how about adding the 
> following line?

Nothing against this, but I think it should be a separate patch.

Thanks,

-- Steve


> 
> diff --git a/kernel/trace/trace_uprobe.c b/kernel/trace/trace_uprobe.c
> index 2cabf8a23ec5..c5ee7920dec6 100644
> --- a/kernel/trace/trace_uprobe.c
> +++ b/kernel/trace/trace_uprobe.c
> @@ -979,6 +979,7 @@ static struct uprobe_cpu_buffer 
> *prepare_uprobe_buffer(struct trace_uprobe *tu,
>       ucb = uprobe_buffer_get();
>       ucb->dsize = tu->tp.size + dsize;
>  
> +     BUILD_BUG_ON(MAX_UCB_BUFFER_SIZE < MAX_PROBE_EVENT_SIZE);
>       if (WARN_ON_ONCE(ucb->dsize > MAX_UCB_BUFFER_SIZE)) {
>               ucb->dsize = MAX_UCB_BUFFER_SIZE;
>               dsize = MAX_UCB_BUFFER_SIZE - tu->tp.size;
> 

Reply via email to