Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933289AbcDYRzj (ORCPT ); Mon, 25 Apr 2016 13:55:39 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:52527 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932791AbcDYRzi (ORCPT ); Mon, 25 Apr 2016 13:55:38 -0400 Date: Mon, 25 Apr 2016 10:55:36 -0700 From: Greg KH To: Jarkko Sakkinen Cc: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" , "open list:X86 ARCHITECTURE (32-BIT AND 64-BIT)" , "open list:STAGING SUBSYSTEM" Subject: Re: [PATCH 3/6] intel_sgx: driver for Intel Secure Guard eXtensions Message-ID: <20160425175536.GC16786@kroah.com> References: <1461605698-12385-1-git-send-email-jarkko.sakkinen@linux.intel.com> <1461605698-12385-4-git-send-email-jarkko.sakkinen@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1461605698-12385-4-git-send-email-jarkko.sakkinen@linux.intel.com> User-Agent: Mutt/1.6.0 (2016-04-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1099 Lines: 26 On Mon, Apr 25, 2016 at 08:34:10PM +0300, Jarkko Sakkinen wrote: > 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. > > Intel SGX driver provides a ioctl interface for loading and initializing > enclaves and a pager in order to support oversubscription. > > Signed-off-by: Jarkko Sakkinen > --- > arch/x86/include/asm/sgx.h | 4 +- > drivers/staging/Kconfig | 2 + > drivers/staging/Makefile | 1 + > drivers/staging/intel_sgx/Kconfig | 13 + > drivers/staging/intel_sgx/Makefile | 12 + > drivers/staging/intel_sgx/isgx.h | 238 +++++++ > drivers/staging/intel_sgx/isgx_compat_ioctl.c | 179 +++++ No "new" kernel code should ever need compat_ioctl support, please just create your structures to not require this at all, it isn't that difficult. thanks, greg k-h