luajit2 icon indicating copy to clipboard operation
luajit2 copied to clipboard

some test cases can't pass in ARM, but in X86_64 is OK.

Open NagamineLee opened this issue 4 years ago • 4 comments

OS: centos 7.6 ARCH: aarch64(ARM)

LuaJIT compile command:

make TARGET_STRIP=@: CCDEBUG=-g CC=cc XCFLAGS="-DLUA_USE_APICHECK -DLUA_USE_ASSERT -DLUAJIT_NUMMODE=2 -g -O1 -std=gnu99"

only two cases occurs exception:

=== test/ffi/ffi_jit_struct.lua
/DATA/yfli1/test/upngx-test/luajit-test-suite/luajit/bin/luajit-2.1.0-beta3: bad light userdata pointer
stack traceback:
	[C]: in function 'alloc'
	catch_cpp.lua:22: in main chunk
	[C]: at 0x00405004
Failed test when running /DATA/yfli1/test/upngx-test/luajit-test-suite/luajit/bin/luajit-2.1.0-beta3 catch_cpp.lua 1: 256

=== test/misc/catch_wrap.lua
/DATA/yfli1/test/upngx-test/luajit-test-suite/luajit/bin/luajit-2.1.0-beta3: bad light userdata pointer
stack traceback:
	[C]: in function 'wrapon'
	catch_wrap.lua:3: in main chunk
	[C]: at 0x00405004
Failed test when running /DATA/yfli1/test/upngx-test/luajit-test-suite/luajit/bin/luajit-2.1.0-beta3 catch_wrap.lua 1: 256

NagamineLee avatar Mar 17 '20 08:03 NagamineLee

@NagamineLee Which luajit test suite are you using? OpenResty's luajit2-test-suite repo passes completely on aarch64. I just tried (I make sure that after every commit myself):

All tests successful.

Both in the valgrind test mode and the normal test mode.

agentzh avatar Mar 17 '20 22:03 agentzh

Yeah, I used OpenResty's luajit2-test-suite repo and the new release of luajit, but some cases could not pass in ARM. compile configurations has some effects?

NagamineLee avatar Mar 18 '20 02:03 NagamineLee

@NagamineLee There are known issues which prevent the test suite from passing fully on ARMv7. But I don't see any failures on Aarch64. Also, the test suite requires Lua 5.2 compatibility support in LuaJIT. I use the following make arguments

XCFLAGS="-DLUAJIT_ENABLE_LUA52COMPAT -DLUA_USE_APICHECK -DLUA_USE_ASSERT"

agentzh avatar Mar 18 '20 02:03 agentzh

I have tried it, but also failed. A full test process required, like.travis.yml, and I will try follow your test process again.

I will also test in a new env(CentOS7.6 on ARM).

NagamineLee avatar Mar 18 '20 03:03 NagamineLee