On Fri, 14 Feb 2025 23:14:26 +0100
Jiri Olsa <[email protected]> wrote:

> > +   while (fscanf(fp, "%16s %16s %c %*s\n", addr_str, size_str, &type) == 
> > 3) {
> > +           uint64_t addr;
> > +           uint64_t size;
> > +
> > +           /* Only care about functions */
> > +           if (type != 't' && type != 'T')
> > +                   continue;  
> 
> hi,
> I think we need the 'W' check in here [1]

Good catch!

> 
> jirka
> 
> 
> [1] https://lore.kernel.org/bpf/[email protected]/

Bah, I just pulled from patchwork and forgot about that fix.

Will send a v4.

-- Steve

Reply via email to