Re: [PATCH v3 0/6] perf script: Bounded and lazy symbol loading

From: Alireza Haghdoost

Date: Wed Sep 30 2026 - 18:16:30 EST


On Fri, Sep 25, 2026 at 2:56 PM Ian Rogers <irogers@xxxxxxxxxx> wrote:
>
> So, I'm pretty against the complexity of dealing with symbols that may
> fail due to a memory pressure issue.


Hi Ian,

That was aimed at --max-symbol-bytes. I'll drop it in v4.

Following the other thread [1], it looks like we agree that lazy loading is
worth having on its own. Whether or not the reference-counting series
lands, loading a name only when a sample needs it is a good change. As
in the numbers I posted there, it cuts memory use substantially and
makes perf script faster. I'll prepare v4 of this series without the
cap:

1. the ELF_C_READ_MMAP fix
2. reading DSO data from the debug file the symbol table came from
3. the shared duplicate-symbol selection
4. --lazy-load-symbols
5. the lazy-loading tests

Patches 2 and 3 stay because lazy loading needs them, and Namhyung
asked for them.

Namhyung, on that thread you suggested a limit that prints an offset
when the name is not loaded. I said I could do that for the byte cap
in v4. I'd rather land lazy loading first and add that limit later if
there is a need or interest.

[1] https://lore.kernel.org/all/20260928075237.3055101-1-irogers@xxxxxxxxxx/

Thanks,
Alireza