mirror/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

selftests/sgx: Ensure test enclave buffer is entirely preserved

Browse Source 
Attach the "used" attribute to instruct the compiler to preserve the static
encl_buffer, even if it appears it is not entirely referenced in the enclave
code, as expected by the external tests manipulating page permissions.

Signed-off-by: Jo Van Bulck <jo.vanbulck@cs.kuleuven.be>
Signed-off-by: Dave Hansen <dave.hansen@linux.intel.com>
Reviewed-by: Jarkko Sakkinen <jarkko@kernel.org>
Acked-by: Kai Huang <kai.huang@intel.com>
Link: https://lore.kernel.org/all/a2732938-f3db-a0af-3d68-a18060f66e79@cs.kuleuven.be/
Link: https://lore.kernel.org/all/20231005153854.25566-11-jo.vanbulck%40cs.kuleuven.be
This commit is contained in:
Jo Van Bulck
2023-10-05 17:38:51 +02:00
committed by Dave Hansen
parent 9fd552ee32
commit a4c39ef4ed
2 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
tools/testing/selftests/sgx/defines.h
View File

@@ -13,6 +13,7 @@
#define __aligned(x) __attribute__((__aligned__(x)))
#define __packed __attribute__((packed))
#define __used __attribute__((used))
#include "../../../../arch/x86/include/asm/sgx.h"
#include "../../../../arch/x86/include/asm/enclu.h"

9
tools/testing/selftests/sgx/test_encl.c
View File

@@ -5,11 +5,12 @@
#include "defines.h"
/*
* Data buffer spanning two pages that will be placed first in .data
* segment. Even if not used internally the second page is needed by
* external test manipulating page permissions.
* Data buffer spanning two pages that will be placed first in the .data
* segment. Even if not used internally the second page is needed by external
* test manipulating page permissions, so mark encl_buffer as "used" to make
* sure it is entirely preserved by the compiler.
*/
static uint8_t encl_buffer[8192] = { 1 };
static uint8_t __used encl_buffer[8192] = { 1 };
enum sgx_enclu_function {
EACCEPT = 0x5,