From: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
To: Sean Christopherson <sean.j.christopherson@intel.com>
Cc: linux-kernel@vger.kernel.org, x86@kernel.org,
linux-sgx@vger.kernel.org, akpm@linux-foundation.org,
andriy.shevchenko@linux.intel.com, asapek@google.com,
bp@alien8.de, cedric.xing@intel.com, chenalexchen@google.com,
conradparker@google.com, cyhanish@google.com,
dave.hansen@intel.com, haitao.huang@intel.com,
josh@joshtriplett.org, kai.huang@intel.com, kai.svahn@intel.com,
kmoy@google.com, ludloff@google.com, luto@kernel.org,
nhorman@redhat.com, npmccallum@redhat.com, puiterwijk@redhat.com,
rientjes@google.com, tglx@linutronix.de, yaozhangx@google.com
Subject: Re: [PATCH v32 00/21] Intel SGX foundations
Date: Thu, 18 Jun 2020 00:53:16 +0300 [thread overview]
Message-ID: <20200617215316.GB53442@linux.intel.com> (raw)
In-Reply-To: <20200617000655.GB19300@linux.intel.com>
On Tue, Jun 16, 2020 at 05:06:55PM -0700, Sean Christopherson wrote:
> On Tue, Jun 16, 2020 at 11:09:58PM +0300, Jarkko Sakkinen wrote:
> > On Wed, Jun 10, 2020 at 01:59:03PM -0700, Sean Christopherson wrote:
> > > On Mon, Jun 01, 2020 at 10:51:57AM +0300, Jarkko Sakkinen wrote:
> > > > v29:
> > > > * The selftest has been moved to selftests/sgx. Because SGX is an execution
> > > > environment of its own, it really isn't a great fit with more "standard"
> > > > x86 tests.
> > > >
> > > > The RSA key is now generated on fly and the whole signing process has
> > > > been made as part of the enclave loader instead of signing the enclave
> > > > during the compilation time.
> > > >
> > > > Finally, the enclave loader loads now the test enclave directly from its
> > > > ELF file, which means that ELF file does not need to be coverted as raw
> > > > binary during the build process.
> > >
> > > Something in the above rework broke the selftest. I'm getting intermittent
> > > EINIT failures with SGX_INVALID_SIGNATURE. I'm guessing it's related to
> > > the dynamic RSA key generation, e.g. only ~15% of runs fail. Verified that
> > > v29 selftest fails and v28 passes. My internal tests also pass, i.e. it's
> > > all but guaranteed to be a selftest issue, not a kernel issue.
> > >
> > > Jarkko, I don't have bandwidth to dig into this right now, hopefully this
> > > reproduces in your environment. Let me know if that's not the case.
> >
> > I haven't experienced but I'll try to stress test it.
> >
> > Just to know how complex test should reproduce your issue, can you
> > reproduce the issue by running the selftest sequentially in a loop or
> > do I need to do something more complex than that?
>
> I didn't even get that complex, just running the selftest manually will
> eventually fail for me, e.g. the first failure I saw was a one-off run of
> the selftest.
OK, I'll see how it behaves in my Geminilake NUC.
/Jarkko
next prev parent reply other threads:[~2020-06-17 21:53 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-01 7:51 [PATCH v32 00/21] Intel SGX foundations Jarkko Sakkinen
2020-06-01 7:51 ` [PATCH v32 01/21] x86/cpufeatures: x86/msr: Add Intel SGX hardware bits Jarkko Sakkinen
2020-06-01 7:51 ` [PATCH v32 02/21] x86/cpufeatures: x86/msr: Add Intel SGX Launch Control " Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 03/21] x86/mm: x86/sgx: Signal SIGSEGV with PF_SGX Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 04/21] x86/sgx: Add SGX microarchitectural data structures Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 05/21] x86/sgx: Add wrappers for ENCLS leaf functions Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 06/21] x86/cpu/intel: Detect SGX support Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 07/21] x86/cpu/intel: Add nosgx kernel parameter Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 08/21] x86/sgx: Initialize metadata for Enclave Page Cache (EPC) sections Jarkko Sakkinen
2020-06-01 15:15 ` Randy Dunlap
2020-06-15 20:07 ` Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 09/21] x86/sgx: Add __sgx_alloc_epc_page() and sgx_free_epc_page() Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 10/21] mm: Introduce vm_ops->may_mprotect() Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 11/21] x86/sgx: Linux Enclave Driver Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 12/21] x86/sgx: Add provisioning Jarkko Sakkinen
2020-06-05 20:09 ` Darren Kenny
2020-06-16 19:55 ` Jarkko Sakkinen
2020-06-16 20:03 ` Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 13/21] x86/sgx: Add a page reclaimer Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 14/21] x86/sgx: ptrace() support for the SGX driver Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 15/21] x86/vdso: Add support for exception fixup in vDSO functions Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 16/21] x86/fault: Add helper function to sanitize error code Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 17/21] x86/traps: Attempt to fixup exceptions in vDSO before signaling Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 18/21] x86/vdso: Implement a vDSO for Intel SGX enclave call Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 19/21] selftests/x86: Add a selftest for SGX Jarkko Sakkinen
2020-06-03 22:06 ` Sean Christopherson
2020-06-15 20:17 ` Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 20/21] docs: x86/sgx: Document SGX micro architecture and kernel internals Jarkko Sakkinen
2020-06-01 7:52 ` [PATCH v32 21/21] x86/sgx: Update MAINTAINERS Jarkko Sakkinen
2020-06-10 20:59 ` [PATCH v32 00/21] Intel SGX foundations Sean Christopherson
2020-06-16 20:09 ` Jarkko Sakkinen
2020-06-17 0:06 ` Sean Christopherson
2020-06-17 21:53 ` Jarkko Sakkinen [this message]
2020-06-15 12:13 ` Jethro Beekman
2020-06-16 20:11 ` Jarkko Sakkinen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200617215316.GB53442@linux.intel.com \
--to=jarkko.sakkinen@linux.intel.com \
--cc=akpm@linux-foundation.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=asapek@google.com \
--cc=bp@alien8.de \
--cc=cedric.xing@intel.com \
--cc=chenalexchen@google.com \
--cc=conradparker@google.com \
--cc=cyhanish@google.com \
--cc=dave.hansen@intel.com \
--cc=haitao.huang@intel.com \
--cc=josh@joshtriplett.org \
--cc=kai.huang@intel.com \
--cc=kai.svahn@intel.com \
--cc=kmoy@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sgx@vger.kernel.org \
--cc=ludloff@google.com \
--cc=luto@kernel.org \
--cc=nhorman@redhat.com \
--cc=npmccallum@redhat.com \
--cc=puiterwijk@redhat.com \
--cc=rientjes@google.com \
--cc=sean.j.christopherson@intel.com \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
--cc=yaozhangx@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.