[PATCH V2 0/2] selftests: sgx: Fix build of test_sgx

* [PATCH V2 0/2] selftests: sgx: Fix build of test_sgx
@ 2022-02-14 18:41 Muhammad Usama Anjum
  2022-02-14 18:41 ` [PATCH V2 1/2] selftests: x86: add validity check and allow field splitting Muhammad Usama Anjum
  2022-02-14 18:41 ` [PATCH V2 2/2] selftests: sgx: Treat CC as one argument Muhammad Usama Anjum
  0 siblings, 2 replies; 7+ messages in thread
From: Muhammad Usama Anjum @ 2022-02-14 18:41 UTC (permalink / raw)
  To: Jarkko Sakkinen, Dave Hansen, Shuah Khan, Andy Lutomirski,
	Jethro Beekman, Borislav Petkov, open list:INTEL SGX,
	open list:KERNEL SELFTEST FRAMEWORK, open list, David.Laight
  Cc: Muhammad Usama Anjum, kernel, kernelci

CC can have multiple sub-strings like "ccache gcc". Build system of sgx
selftest is unable to support this use case and test_sgx isn't built.
Then rsync gives an erorr:

rsync: [sender] link_stat "//linux/build/kselftest/sgx/test_sgx" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]

This can be fixed if CC is considered to have one string while passing to
check_cc.sh script and inside this script, CC is expanded before execution.

Following build tests have been performed to confirm that these patches
don't break anything else. The verification has also been done through
KernelCI scripts locally as KernelCI has caught this bug:
1) cd tools/testing/selftests && make
2) make -C tools/testing/selftests
3) make -C tools/testing/selftests O=build
4) make -C tools/testing/selftests O=build CC="ccache gcc"
5) ./kci_build make_kselftest

Muhammad Usama Anjum (2):
  selftests: x86: add validity check and allow field splitting
  selftests: sgx: Treat CC as one argument

 tools/testing/selftests/sgx/Makefile    | 2 +-
 tools/testing/selftests/x86/check_cc.sh | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.30.2

^ permalink raw reply	[flat|nested] 7+ messages in thread