Message ID | 20190605194845.926-1-sean.j.christopherson@intel.com (mailing list archive) |
---|---|
Headers | show |
Series | x86/sgx: Clean up and enhance add pages ioctl | expand |
On Wed, Jun 05, 2019 at 12:48:38PM -0700, Sean Christopherson wrote: > This series is intended to be included in v21 of Jarkko's SGX series and > applies on Jarkko's current master: > > dfc89a83b5bc ("docs: x86/sgx: Document the enclave API") > > The primary goal of the series is to tweak the ioctl for adding pages to > an enclave so that it is somewhat extensible, e.g. add a flags field that > can be reused for access control integration and SGX2/EAUG, and add a size > field so that multiple pages can be added in a single call (batching EADD > has been mentioned at various times in the past). > > The secondary goal is to improve the performance of building enclaves. > Handling multiple pages in a single call helps somewhat, but the real win > (for some enclaves) is using the kernel's zero page as the source for EADD > when possible. For v21 I will add 1, 2 and 6 from these. Thanks. I'll change the size or mrmask field to 32 bits. /Jarkko
On Wed, Jun 12, 2019 at 06:16:05PM +0300, Jarkko Sakkinen wrote: > On Wed, Jun 05, 2019 at 12:48:38PM -0700, Sean Christopherson wrote: > > This series is intended to be included in v21 of Jarkko's SGX series and > > applies on Jarkko's current master: > > > > dfc89a83b5bc ("docs: x86/sgx: Document the enclave API") > > > > The primary goal of the series is to tweak the ioctl for adding pages to > > an enclave so that it is somewhat extensible, e.g. add a flags field that > > can be reused for access control integration and SGX2/EAUG, and add a size > > field so that multiple pages can be added in a single call (batching EADD > > has been mentioned at various times in the past). > > > > The secondary goal is to improve the performance of building enclaves. > > Handling multiple pages in a single call helps somewhat, but the real win > > (for some enclaves) is using the kernel's zero page as the source for EADD > > when possible. > > For v21 I will add 1, 2 and 6 from these. Thanks. I meant to say 1, 2 and 3 (typo). /Jarkko