On Mon, Sep 03, 2018 at 05:26:43PM +0300, Andy Shevchenko wrote: > 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. Sure! > > +#include <asm/sgx.h> > > +#include <asm/sgx_pr.h> > > Don't we put linux/* followed by asm/* ? Yes. > > > +#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 /Jarkko
