lkml.org 
[lkml]   [2020]   [Jun]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v32 19/21] selftests/x86: Add a selftest for SGX
    On Wed, Jun 03, 2020 at 03:06:34PM -0700, Sean Christopherson wrote:
    > On Mon, Jun 01, 2020 at 10:52:16AM +0300, Jarkko Sakkinen wrote:
    > > Add a selftest for SGX. It is a trivial test where a simple enclave
    > > copies one 64-bit word of memory between two memory locations.
    >
    > ...
    >
    > > --- /dev/null
    > > +++ b/tools/testing/selftests/sgx/Makefile
    > > @@ -0,0 +1,53 @@
    > > +top_srcdir = ../../../..
    > > +
    > > +include ../lib.mk
    > > +
    > > +.PHONY: all clean
    > > +
    > > +CAN_BUILD_X86_64 := $(shell ../x86/check_cc.sh $(CC) \
    > > + ../x86/trivial_64bit_program.c)
    > > +
    > > +ifndef OBJCOPY
    > > +OBJCOPY := $(CROSS_COMPILE)objcopy
    > > +endif
    > > +
    > > +INCLUDES := -I$(top_srcdir)/tools/include
    > > +HOST_CFLAGS := -Wall -Werror -g $(INCLUDES) -fPIC -z noexecstack
    >
    > There's a pending pull request for 5.8[1] that includes a patch that turns
    > off automatic READ_IMPLIES_EXEC for x86_64[2]. Assuming that gets pulled
    > for 5.8, we should drop noexecstack to verify/ensure that the workaround is
    > no longer needed to avoid conflicts with the EPCM permissions.
    >
    > [1] https://lkml.kernel.org/r/20200603213452.GG19667@zn.tnic
    > [2] https://lkml.kernel.org/r/20200327064820.12602-4-keescook@chromium.org

    If the check is removed from encl.c, then this needs to be taken into
    account in backporting.

    /Jarkko

    \
     
     \ /
      Last update: 2020-06-15 22:18    [W:4.048 / U:0.364 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site