Open ColinIanKing opened 1 month ago
I bisected this issue down, the following commit trips the segfault, e.g.:
./igzip_rand_test
Window Size: 32 K
Test Seed : 4660
Randoms : 64
igzip_rand_test stateless: .make: *** [make.inc:169: igzip_rand_test.run] Segmentation fault (core dumped)
The commit in question that causes the issue is:
1fdc5941a37f2d2780339b55ff052f272e7566be is the first bad commit
commit 1fdc5941a37f2d2780339b55ff052f272e7566be
Author: Roy Oursler <roy.j.oursler@intel.com>
Date: Tue Oct 2 14:01:57 2018 -0700
igzip: Modify set_long_icf to handle small end_in
Change-Id: I24c3420df5d9e84d27fe28eff96155e5fcd51760
Signed-off-by: Roy Oursler <roy.j.oursler@intel.com>
igzip/igzip_icf_body.c | 24 +++++++++++++-----------
igzip/igzip_set_long_icf_fg_04.asm | 42 ++++++++++++++++++++++++++----------------
igzip/igzip_set_long_icf_fg_06.asm | 42 ++++++++++++++++++++++++++----------------
3 files changed, 65 insertions(+), 43 deletions(-)
Thanks Colin. We are not validating the 32-bit implementation, as said in the Release Notes. We are planning to remove this 32-bit support in the next release.
Just to confirm, is that all 32 bit support to be removed for all architectures?
Yes, ARM and PowerPC have 64-bit versions only.
I noticed this was failing on hppa and i386 builds when packaging isa-l for Debian.
Built with: make -f Makefile.unx arch=32 host_cpu=base_aliases check -j 8
Debugging on i386 with valgrind I see the stack trace as follows:
Debugging with gdb I get: