On 4/28/2023 8:31 PM, Muhammad Usama Anjum wrote: > On 4/21/23 8:31 AM, Yi Lai wrote: >> The "test_encl.elf" file used by test_sgx is not installed in >> INSTALL_PATH. Attempting to execute test_sgx causes false negative: > I've ran the following on next-20230427: > make -C tools/testing/selftests O=build2 -j `nproc` > > find | grep test_encl.elf > ./build2/kselftest/sgx/test_encl.elf > > test_encl.elf is being copied already. Am i missing something? >> >> " >> enclave executable open(): No such file or directory >> main.c:188:unclobbered_vdso:Failed to load the test enclave. >> " >> >> Add "test_encl.elf" to TEST_FILES so that it will be installed. >> >> Fixes: 2adcba79e69d ("selftests/x86: Add a selftest for SGX") >> Signed-off-by: Yi Lai <yi1.lai@xxxxxxxxx> >> --- >> tools/testing/selftests/sgx/Makefile | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/tools/testing/selftests/sgx/Makefile b/tools/testing/selftests/sgx/Makefile >> index 75af864e07b6..50aab6b57da3 100644 >> --- a/tools/testing/selftests/sgx/Makefile >> +++ b/tools/testing/selftests/sgx/Makefile >> @@ -17,6 +17,7 @@ ENCL_CFLAGS := -Wall -Werror -static -nostdlib -nostartfiles -fPIC \ >> -fno-stack-protector -mrdrnd $(INCLUDES) >> >> TEST_CUSTOM_PROGS := $(OUTPUT)/test_sgx >> +TEST_FILES := $(OUTPUT)/test_encl.elf > I think this isn't needed just like sign_key.o is being copied already. > test_encl.elf is being copied. >> Hi, When using tools/testing/selftests/kselftest_install.sh test_encl.elf can be built but not copied to tools/testing/selftests/kselftest_install/sgx/ >> ifeq ($(CAN_BUILD_X86_64), 1) >> all: $(TEST_CUSTOM_PROGS) $(OUTPUT)/test_encl.elf > Regards, Yi Lai