On Thu, Aug 22, 2019 at 06:55:01PM +0300, Jarkko Sakkinen wrote: > Add @secs_page to sgx_encl_eldu() and @is_secs_child to sgx_encl_ewb() > (replacing earlier @do_free) in order to state explicitly in the call > site when we are using SECS. These replace the use of > SGX_ENCL_PAGE_IS_SECS() macro. > > In order to fully remove SGX_ENCL_PAGE_IS_SECS(), replace > sgx_encl_get_index() with a macro SGX_ENCL_PAGE_INEX() as the > conditional logic is no longer required. > > Cc: Sean Christopherson <sean.j.christopherson@xxxxxxxxx> > Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx> > --- > v3: Forgot to flip the conditions in __sgx_encl_eldu(). > v2: Use @secs_page in the ELDU flow. Rename @secs_child as > @is_secs_child in EWB flow. > arch/x86/kernel/cpu/sgx/encl.c | 51 ++++++++++++++----------------- > arch/x86/kernel/cpu/sgx/encl.h | 4 +-- > arch/x86/kernel/cpu/sgx/reclaim.c | 34 +++++++++++++-------- > 3 files changed, 47 insertions(+), 42 deletions(-) This has been squashed. /Jarkko
