Re: [PATCH v5 5/5] perf test: Test lazy symbol loading

[sashiko-bot@kernel.org]

> From: Alireza Haghdoost <haghdoost@uber.com>
> 
> Add a perf script shell test for --lazy-load-symbols.
> 
> Record a small callchain fixture, require evidence that the controlled
> perf DSO built an on-demand index, and compare only extracted occurrences
> of the controlled test_loop symbol. This avoids coupling the test to
> addresses, diagnostics, or architecture-specific symbols for which the
> eager and lazy loaders have documented differences. When run as root,
> also profile perf in a private mount namespace whose build-id cache
> directory is hidden by a tmpfs, so that names must be read from the
> build-id cache in perf's own namespace.
> 
> Report unsupported recording, missing controlled output, and unavailable
> libelf as skips without replacing a prior failure, and keep helper
> [ ... ]
>   - on x86_64, a binary with an IRELATIVE PLT slot whose IFUNC loses
>     duplicate selection to an alias that loses to another one in turn,
>     so the PLT symbol name depends on how the IFUNC mark is passed on.
> 
> Signed-off-by: Alireza Haghdoost <haghdoost@uber.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261005-perf-symbol-memory-send-v5-0-165dceb2b049@uber.com?part=5