diff options
author | Jo Van Bulck <jo.vanbulck@cs.kuleuven.be> | 2023-10-05 17:38:44 +0200 |
---|---|---|
committer | Dave Hansen <dave.hansen@linux.intel.com> | 2023-12-08 10:05:26 -0800 |
commit | 853a57a43ebdb8c024160c1a0990bae85f4bcc2f (patch) | |
tree | 45199e7dec950e2e11139ee1913b73053e26fc37 /tools/testing/selftests/sgx/test_encl.c | |
parent | b84fc2e0139ba4b23b8039bd7cfd242894fe8f8b (diff) | |
download | linux-stable-853a57a43ebdb8c024160c1a0990bae85f4bcc2f.tar.gz linux-stable-853a57a43ebdb8c024160c1a0990bae85f4bcc2f.tar.bz2 linux-stable-853a57a43ebdb8c024160c1a0990bae85f4bcc2f.zip |
selftests/sgx: Include memory clobber for inline asm in test enclave
Add the "memory" clobber to the EMODPE and EACCEPT asm blocks to tell the
compiler the assembly code accesses to the secinfo struct. This ensures
the compiler treats the asm block as a memory barrier and the write to
secinfo will be visible to ENCLU.
Fixes: 20404a808593 ("selftests/sgx: Add test for EPCM permission changes")
Signed-off-by: Jo Van Bulck <jo.vanbulck@cs.kuleuven.be>
Signed-off-by: Dave Hansen <dave.hansen@linux.intel.com>
Reviewed-by: Kai Huang <kai.huang@intel.com>
Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org>
Link: https://lore.kernel.org/all/20231005153854.25566-4-jo.vanbulck%40cs.kuleuven.be
Diffstat (limited to 'tools/testing/selftests/sgx/test_encl.c')
-rw-r--r-- | tools/testing/selftests/sgx/test_encl.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/tools/testing/selftests/sgx/test_encl.c b/tools/testing/selftests/sgx/test_encl.c index c0d6397295e3..ae791df3e5a5 100644 --- a/tools/testing/selftests/sgx/test_encl.c +++ b/tools/testing/selftests/sgx/test_encl.c @@ -24,10 +24,11 @@ static void do_encl_emodpe(void *_op) secinfo.flags = op->flags; asm volatile(".byte 0x0f, 0x01, 0xd7" - : + : /* no outputs */ : "a" (EMODPE), "b" (&secinfo), - "c" (op->epc_addr)); + "c" (op->epc_addr) + : "memory" /* read from secinfo pointer */); } static void do_encl_eaccept(void *_op) @@ -42,7 +43,8 @@ static void do_encl_eaccept(void *_op) : "=a" (rax) : "a" (EACCEPT), "b" (&secinfo), - "c" (op->epc_addr)); + "c" (op->epc_addr) + : "memory" /* read from secinfo pointer */); op->ret = rax; } |