Re: [PATCH dwarves] btf_encoder: filter var names before btf name check

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Oct 22, 2024 at 03:17:09PM -0700, Stephen Brennan wrote:
> Alan Maguire <alan.maguire@xxxxxxxxxx> writes:
> > ...as this means we avoid seeing multiple messages of the form

> > PAHOLE: Warning: Found invalid variable name when encoding btf, ignored (sym: '__UNIQUE_ID___addressable__RNvXsa_NtCs2vSfyxKMEZc_4core4charNtB5_11EscapeDebugNtNtNtNtB7_4iter6traits10exact_size17ExactSizeIterator3len57').
> 
> a beautiful variable name :P

A unique one :-)
 
> > Such symbols are filtered anyway due to the __UNIQUE_ID prefix, so
> > warning about them is not necessary; simply reordering the checks
> > should ensure we will not see these warnings.

> > Reported-by: Arnaldo Carvalho de Melo <acme@xxxxxxxxxx>
> > Signed-off-by: Alan Maguire <alan.maguire@xxxxxxxxxx>

> Sorry I'm late to review. This is a good catch and is clearly correct
> to me.

Thanks, since it hasn't moved to master, I'm adding it.
 
> Reviewed-by: Stephen Brennan <stephen.s.brennan@xxxxxxxxxx>

- Arnaldo




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux