Message ID | 20240717-docker-image-v1-6-7c7fc6251e7a@flygoat.com |
---|---|
State | Changes Requested |
Delegated to: | Tom Rini |
Headers | show |
Series | New CI image and fixes | expand |
Hi Jiaxun, On Wed, 17 Jul 2024 at 15:29, Jiaxun Yang <jiaxun.yang@flygoat.com> wrote: > > event_dump.py relies on addr2line to obtain source location > information, however newer addr2line is unable to determine > line numbers for some functions. Is this a bug in the tool? > > With addr2line from binutils 2.34 we got: > > Event type Id Source location > -------------------- ------------------------------ ------------------------------ > EVT_FT_FIXUP bootmeth_vbe_ft_fixup :? > EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup :? > EVT_LAST_STAGE_INIT install_smbios_table :? > EVT_MISC_INIT_F sandbox_early_getopt_check arch/sandbox/cpu/start.c:61 > EVT_TEST h_adder_simple :? > > Which will fail the test. > > Relax the source location regex to .*:.*, this is sufficent to > show that addr2line is being called and returned a possible > line number. > > Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> > --- > test/py/tests/test_event_dump.py | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Simon Glass <sjg@chromium.org> > > diff --git a/test/py/tests/test_event_dump.py b/test/py/tests/test_event_dump.py > index e282c67335cd..e87825abcd1a 100644 > --- a/test/py/tests/test_event_dump.py > +++ b/test/py/tests/test_event_dump.py > @@ -16,9 +16,9 @@ def test_event_dump(u_boot_console): > out = util.run_and_log(cons, ['scripts/event_dump.py', sandbox]) > expect = '''.*Event type Id Source location > -------------------- ------------------------------ ------------------------------ > -EVT_FT_FIXUP bootmeth_vbe_ft_fixup .*boot/vbe_request.c:.* > -EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup .*boot/vbe_simple_os.c:.* > -EVT_LAST_STAGE_INIT install_smbios_table .*lib/efi_loader/efi_smbios.c:.* > -EVT_MISC_INIT_F sandbox_early_getopt_check .*arch/sandbox/cpu/start.c:.* > -EVT_TEST h_adder_simple .*test/common/event.c:''' > +EVT_FT_FIXUP bootmeth_vbe_ft_fixup .*:.* > +EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup .*:.* > +EVT_LAST_STAGE_INIT install_smbios_table .*:.* > +EVT_MISC_INIT_F sandbox_early_getopt_check .*:.* > +EVT_TEST h_adder_simple .*:''' > assert re.match(expect, out, re.MULTILINE) is not None > > -- > 2.45.2 > Regards, Simon
diff --git a/test/py/tests/test_event_dump.py b/test/py/tests/test_event_dump.py index e282c67335cd..e87825abcd1a 100644 --- a/test/py/tests/test_event_dump.py +++ b/test/py/tests/test_event_dump.py @@ -16,9 +16,9 @@ def test_event_dump(u_boot_console): out = util.run_and_log(cons, ['scripts/event_dump.py', sandbox]) expect = '''.*Event type Id Source location -------------------- ------------------------------ ------------------------------ -EVT_FT_FIXUP bootmeth_vbe_ft_fixup .*boot/vbe_request.c:.* -EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup .*boot/vbe_simple_os.c:.* -EVT_LAST_STAGE_INIT install_smbios_table .*lib/efi_loader/efi_smbios.c:.* -EVT_MISC_INIT_F sandbox_early_getopt_check .*arch/sandbox/cpu/start.c:.* -EVT_TEST h_adder_simple .*test/common/event.c:''' +EVT_FT_FIXUP bootmeth_vbe_ft_fixup .*:.* +EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup .*:.* +EVT_LAST_STAGE_INIT install_smbios_table .*:.* +EVT_MISC_INIT_F sandbox_early_getopt_check .*:.* +EVT_TEST h_adder_simple .*:''' assert re.match(expect, out, re.MULTILINE) is not None
event_dump.py relies on addr2line to obtain source location information, however newer addr2line is unable to determine line numbers for some functions. With addr2line from binutils 2.34 we got: Event type Id Source location -------------------- ------------------------------ ------------------------------ EVT_FT_FIXUP bootmeth_vbe_ft_fixup :? EVT_FT_FIXUP bootmeth_vbe_simple_ft_fixup :? EVT_LAST_STAGE_INIT install_smbios_table :? EVT_MISC_INIT_F sandbox_early_getopt_check arch/sandbox/cpu/start.c:61 EVT_TEST h_adder_simple :? Which will fail the test. Relax the source location regex to .*:.*, this is sufficent to show that addr2line is being called and returned a possible line number. Signed-off-by: Jiaxun Yang <jiaxun.yang@flygoat.com> --- test/py/tests/test_event_dump.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-)