Re: [PATCH v2] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-10-13 Thread Sergei Trofimovich via Elfutils-devel
On Thu, 13 Oct 2022 16:40:36 +0200 Mark Wielaard wrote: > Hi, > > On Mon, 2022-08-08 at 01:17 +0200, Mark Wielaard wrote: > > On Sun, Aug 07, 2022 at 07:31:38PM +0100, Sergei Trofimovich via > > Elfutils-devel wrote: > > > binutils started producing 0-sized

[PATCH v2] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-08-07 Thread Sergei Trofimovich via Elfutils-devel
binutils started producing 0-sized DIEs on functions interspersed by nested sections (".section ...; .previous). This led to run-low_high_pc.sh failure in form of: FAIL: run-low_high_pc.sh [b] main.c [2d] main [b] ../sysdeps/i386/start.S [26]

[PATCH] tests: run-low_high_pc.sh: drop redundant 'lx' suffix

2022-08-06 Thread Sergei Trofimovich via Elfutils-devel
Noticed when debugged test failure: lowpc: 8049000, highpc: 8049000lx ../sysdeps/i386/crti.S: [2def] '_init' highpc <= lowpc Signed-off-by: Sergei Trofimovich --- tests/ChangeLog | 4 tests/low_high_pc.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git

[PATCH] tests: do not fail on zero sized DIEs (binutils-2.39 compatible)

2022-08-06 Thread Sergei Trofimovich via Elfutils-devel
binutils started producing 0-sized DIEs on functions interspersed by nested sections (".section ...; .previous). This led to run-low_high_pc.sh failure in form of: FAIL: run-low_high_pc.sh [b] main.c [2d] main [b] ../sysdeps/i386/start.S [26]

[PATCH v2] elflint: fix undefined 'buffer_left' reference

2021-06-07 Thread Sergei Trofimovich via Elfutils-devel
Link failure is reproducible on gcc-11.1.0 target: ``` $ autoreconf -i -f $ ./configure --enable-maintainer-mode --disable-debuginfod \ --host=x86_64-pc-linux-gnu \ CFLAGS=-march=znver3 \ CXXFLAGS=-march=znver3 \ LDFLAGS=" " $ make CCLD elflint ld: elflint.o: in function

[PATCH] elflint: fix undefined 'buffer_left' reference

2021-06-06 Thread Sergei Trofimovich via Elfutils-devel
Link failure is reproducible on gcc-11.1.0 target: ``` $ autoreconf -i -f $ ./configure --enable-maintainer-mode --disable-debuginfod \ --host=x86_64-pc-linux-gnu \ CFLAGS=-march=znver3 \ CXXFLAGS=-march=znver3 \ LDFLAGS=" " $ make CCLD elflint ld: elflint.o: in function

[PATCH] tests: use ${CC} instead of 'gcc' in tests

2021-01-31 Thread Sergei Trofimovich via Elfutils-devel
To better support cross-compilation Gentoo provides a way to configure system without 'gcc' binary and only provide tool-prefixed tools, like 'x86_64-pc-linux-gnu-gcc'. The packages are built as ./configure --host=x86_64-pc-linux-gnu. In https://bugs.gentoo.org/718872 Agostino Sarubbo found a few

Re: [PATCH] src/elflint.c: add debug print for GNU_RETAIN and others

2021-01-12 Thread Sergei Trofimovich via Elfutils-devel
On Tue, 12 Jan 2021 10:14:55 +0100 Mark Wielaard wrote: > Hi Sergei, > > On Thu, 2020-12-31 at 12:11 +0000, Sergei Trofimovich via Elfutils-devel > wrote: > > --- > > src/elflint.c | 5 - > > 1 file changed, 4 insertions(+), 1 deletion(-) > >

[PATCH] src/elflint.c: fix printing of unknown flags

2020-12-31 Thread Sergei Trofimovich via Elfutils-devel
before the change section_flags_string() ignored unknown section flags: snprintf() did write numeric value int obuffer, but "*cp = '\0'" hegated the effect. The change advances the 'cp' pointer'. While at it add a '|' separator between known and unknown flags. Signed-off-by: Sergei Trofimovich

[PATCH] src/elflint.c: add debug print for GNU_RETAIN and others

2020-12-31 Thread Sergei Trofimovich via Elfutils-devel
--- src/elflint.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/elflint.c b/src/elflint.c index b3cbaade..7b7f7937 100644 --- a/src/elflint.c +++ b/src/elflint.c @@ -2789,7 +2789,10 @@ section_flags_string (GElf_Word flags, char *buf, size_t len) NEWFLAG

[PATCH] elfutils/configure.ac: use $READELF, not readelf

2020-06-14 Thread Sergei Trofimovich via Elfutils-devel
Allow user to specify own readelf. Use detected readelf, not 'readelf'. Noticed when was building elfutils on tuple-prefixed toolchain: ``` checking whether the compiler generates build-ids... ./configure: line 5197: readelf: command not found no ``` Signed-off-by: Sergei Trofimovich ---