Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: WxA5p+uoDmJ4VbcbauXeav2YNxWknxvvaDV3hwkoVig2EQ7jrnddJ/iNA/+K3jMwtDscNtbOfF
 3+WWzun7nn4A==
IronPort-SDR: /gWmaGFNqAeq/bzL2INUDgjRonw+XaBkidyl7xPn2/YlxAIu2XcIOhW96YY0r+CzZeyK0a0yhH
 shT4W2sQryag==
Date:   Wed, 10 Jun 2020 13:59:03 -0700
From:   Sean Christopherson <sean.j.christopherson@intel.com>
To:     Jarkko Sakkinen <jarkko.sakkinen@linux.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
Message-ID: <20200610205903.GF18790@linux.intel.com>
References: <20200601075218.65618-1-jarkko.sakkinen@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200601075218.65618-1-jarkko.sakkinen@linux.intel.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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.