On Sun, May 31, 2020 at 7:28 PM AliOS system security <alios_sys_security@xxxxxxxxxxxxxxxxx> wrote: > > When System.map was generated, the kernel used mksysmap to filter the > kernel symbols, but all the symbols with the second letter 'L' in the > kernel were filtered out, not just the symbols starting with 'dot + L'. > > For example: > ashimida@ubuntu:~/linux$ cat System.map |grep ' .L' > ashimida@ubuntu:~/linux$ nm -n vmlinux |grep ' .L' > ffff0000088028e0 t bLength_show > ...... > ffff0000092e0408 b PLLP_OUTC_lock > ffff0000092e0410 b PLLP_OUTA_lock > > I see that in the original patch[1], the original intent should be to > filter all local symbols starting with '.L', so I wonder if the code > here may add a '\' before '.L'? You are right, but please avoid the interrogatory sentence. Shall I reword as follows? The original intent should be to filter out all local symbols starting with '.L', so the dot should be escaped. > > [1]. mksysmap: Add h8300 local symbol pattern > > Signed-off-by: AliOS system security <alios_sys_security@xxxxxxxxxxxxxxxxx> > --- > scripts/mksysmap | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/mksysmap b/scripts/mksysmap > index a35acc0..9aa23d1 100755 > --- a/scripts/mksysmap > +++ b/scripts/mksysmap > @@ -41,4 +41,4 @@ > # so we just ignore them to let readprofile continue to work. > # (At least sparc64 has __crc_ in the middle). > > -$NM -n $1 | grep -v '\( [aNUw] \)\|\(__crc_\)\|\( \$[adt]\)\|\( .L\)' > $2 > +$NM -n $1 | grep -v '\( [aNUw] \)\|\(__crc_\)\|\( \$[adt]\)\|\( \.L\)' > $2 > -- > 2.7.4 > -- Best Regards Masahiro Yamada
