Re: [PATCH] perf symbol: correction while adjusting symbol

[Ajay Kaher]

> On 24/11/22, 3:20 PM, "Leo Yan" <leo.yan@xxxxxxxxxx> wrote:
>>
>> Fixes: 2d86612aacb7 ("perf symbol: Correct address for bss symbols")
>> Signed-off-by: Ajay Kaher <akaher@xxxxxxxxxx>
>
> Just curious how you can produce this issue?  IIUC, the runtime symbol
> files are copied into .debug folder and they can be found by perf tool
> by matching build ID.  Seems to me, you manully use
> "objcopy --only-keep-debug" command to strip runtime info from elf files
> under .debug folder.  Do I understand correctly?

While creating debuginfo rpm we can have extra stripping.

> Though I have above question, this patch itself looks good to me,
> thanks for the fixing!
>
> Reviewed-by: Leo Yan <leo.yan@xxxxxxxxxx>

Thanks.
- Ajay