[PATCH 06/18] tests/test_event_dump: Relax match rule for output
Simon Glass
sjg at chromium.org
Fri Jul 19 17:05:16 CEST 2024
Hi Jiaxun,
On Wed, 17 Jul 2024 at 15:29, Jiaxun Yang <jiaxun.yang at 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 at flygoat.com>
> ---
> test/py/tests/test_event_dump.py | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
Reviewed-by: Simon Glass <sjg at 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
More information about the U-Boot
mailing list