On Mon, Jun 03, 2019 at 04:48:47PM -0700, Xing, Cedric wrote: > > How about this for the intermediate patch: > > > > struct sgx_enclave_add_region { > > __u64 addr; > > __u64 src; > > __u64 size; > > __u64 secinfo; > > __u16 mrmask; > > __u16 reserved16; > > __u32 reserved; > > } > > > > and with the flags field: > > > > struct sgx_enclave_add_region { > > __u64 addr; > > __u64 src; > > __u64 size; > > __u64 secinfo; > > __u16 mrmask; > > __u16 flags; > > What is "flags" here? In the RFC, @flags holds SGX_ALLOW_{READ,WRITE,EXEC}. > > > __u32 reserved; > > }