From 7ad7621fc8aca88919ceb625a4750817fd5e9525 Mon Sep 17 00:00:00 2001 From: Peter van Dijk Date: Mon, 1 Feb 2021 22:26:19 +0100 Subject: [PATCH] luawrapper test: improve comment --- pdns/test-luawrapper.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/pdns/test-luawrapper.cc b/pdns/test-luawrapper.cc index 44448ed82c..1004790231 100644 --- a/pdns/test-luawrapper.cc +++ b/pdns/test-luawrapper.cc @@ -10,8 +10,11 @@ BOOST_AUTO_TEST_SUITE(test_lua_lightuserdata) BOOST_AUTO_TEST_CASE(test_registerFunction) { - // this test comes from luawrapper/tests/custom_types.cc, TEST(CustomTypes, MemberFunctions) - // on luajit/arm64, as shipped by Debian Buster and others, this test crashes because lightuserdata can only hold 47 bits of address + // This test comes from luawrapper/tests/custom_types.cc, TEST(CustomTypes, MemberFunctions). + // In some versions of luajit, as shipped by Debian Buster and others, Lua lightuserdata + // objects can only hold 47 bits of the address of a pointer. If the kernel puts our heap + // above that 47 bit limit, this test crashes. Many arm64 Linux kernels are known to put + // the heap in that problematic area. struct Object { void increment() { ++value; } -- 2.47.2