On Mon, Aug 27, 2018 at 9:57 PM Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx> wrote: > > From: Sean Christopherson <sean.j.christopherson@xxxxxxxxx> > > Intel(R) SGX is a set of CPU instructions that can be used by applications > to set aside private regions of code and data. The code outside the enclave > is disallowed to access the memory inside the enclave by the CPU access > control. > > Add a check for SGX to arch/x86 and a new config option, INTEL_SGX_CORE. > Expose a boolean variable 'sgx_enabled' to query whether or not the SGX > support is available. > + prompt "Intel SGX core functionality" > + def_bool n Default 'default' is 'n'. Perhaps changing prompt to bool will make it one line less. > +#include <asm/sgx.h> > +#include <asm/sgx_pr.h> Don't we put linux/* followed by asm/* ? > +#include <linux/freezer.h> > +#include <linux/highmem.h> > +#include <linux/kthread.h> > +#include <linux/ratelimit.h> > +#include <linux/sched/signal.h> > +#include <linux/slab.h> -- With Best Regards, Andy Shevchenko