Re: [PATCH v22 12/24] x86/sgx: Linux Enclave Driver

[Sean Christopherson]

On Tue, Oct 08, 2019 at 07:59:24PM +0200, Borislav Petkov wrote:
> > diff --git a/arch/x86/include/uapi/asm/sgx.h b/arch/x86/include/uapi/asm/sgx.h
> > new file mode 100644
> > index 000000000000..c45eeed68144
> > --- /dev/null
> > +++ b/arch/x86/include/uapi/asm/sgx.h
> > @@ -0,0 +1,55 @@
> > +/* SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) WITH Linux-syscall-note */
> 
> checkpatch is bitching for some reason, I guess it doesn't like the
> "WITH" thing or so:
> 
> WARNING: 'SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) WITH Linux-syscall-note */' is not supported in LICENSES/...
> #98: FILE: arch/x86/include/uapi/asm/sgx.h:1:
> +/* SPDX-License-Identifier: (GPL-2.0 OR BSD-3-Clause) WITH Linux-syscall-note */
> 
> WARNING: 'SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause WITH Linux-syscall-note */' is not supported in LICENSES/...
> #159: FILE: arch/x86/include/uapi/asm/sgx_errno.h:1:
> +/* SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause WITH Linux-syscall-note */
> 
> And building this fails:
> 
> arch/x86/kernel/cpu/sgx/encl.c: In function âsgx_mmu_notifier_releaseâ:
> arch/x86/kernel/cpu/sgx/encl.c:77:3: error: implicit declaration of function âmmu_notifier_unregister_no_releaseâ; did you mean âmmu_notifier_unregisterâ? [-Werror=implicit-function-declaration]
>    mmu_notifier_unregister_no_release(mn, mm);
>    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>    mmu_notifier_unregister
> arch/x86/kernel/cpu/sgx/encl.c:78:3: error: implicit declaration of function âmmu_notifier_call_srcuâ; did you mean âmmu_notifier_releaseâ? [-Werror=implicit-function-declaration]
>    mmu_notifier_call_srcu(&encl_mm->rcu,
>    ^~~~~~~~~~~~~~~~~~~~~~
>    mmu_notifier_release
> cc1: some warnings being treated as errors
> make[4]: *** [scripts/Makefile.build:265: arch/x86/kernel/cpu/sgx/encl.o] Error 1
> make[4]: *** Waiting for unfinished jobs....
> make[3]: *** [scripts/Makefile.build:509: arch/x86/kernel/cpu/sgx] Error 2
> make[2]: *** [scripts/Makefile.build:509: arch/x86/kernel/cpu] Error 2
> make[1]: *** [scripts/Makefile.build:509: arch/x86/kernel] Error 2
> make: *** [Makefile:1670: arch/x86] Error 2
> make: *** Waiting for unfinished jobs....
> 
> Got a fixed version which I can review instead?

The build error is due to mmu notifier changes that are going into 5.4,
whereas I believe this is based on 5.3.

Jarkko has rebased to 5.4-rc1 and addressed the mmu notifier issue, but I
don't know exact status of his branch.  I'd prefer not to send you a borked
patch.

There are already a handful of driver changes on our todo list for v23, as
well as the vDSO and selftest updates.  What if you stop here for now and
restart when Jarkko sends v23?  In theory that'll happen later this week.