Re: [PATCH bpf-next v3 3/3] veristat: Report max stack depth

[Eduard Zingerman <eddyz87@gmail.com>]

On Wed, 2026-05-13 at 21:35 +0200, Paul Chaignon wrote:

[...]

> @@ -834,6 +835,7 @@ static struct stat_def {
>  	[SIZE] = { "Program size", {"prog_size"}, },
>  	[JITED_SIZE] = { "Jited size", {"prog_size_jited"}, },
>  	[STACK] = {"Stack depth", {"stack_depth", "stack"}, },
> +	[MAX_STACK] = {"Max stack depth", {"max_stack_depth"}, },
>  	[PROG_TYPE] = { "Program type", {"prog_type"}, },
>  	[ATTACH_TYPE] = { "Attach type", {"attach_type", }, },
>  	[MEMORY_PEAK] = { "Peak memory (MiB)", {"mem_peak", }, },
> @@ -1023,7 +1025,7 @@ static int parse_verif_log(char * const buf, size_t buf_sz, struct verif_stats *
>  				&s->stats[MARK_READ_MAX_LEN]))
>  			continue;
>  
> -		if (1 == sscanf(cur, "stack depth %511s", stack))
> +		if (2 == sscanf(cur, "stack depth %511s max %ld", stack, &s->stats[MAX_STACK]))
>  			continue;
>  	}
>  	while ((token = strtok_r(cnt++ ? NULL : stack, "+", &state))) {

Thinking about it some more I think that the meaning of the
STACK/"Stack depth" metric should swapped for max stack depth.

[...]