Re: [PATCH 7/7] x86/sgx: Add a reserved field to sgx_enclave_add_region to drop 'packed'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2019-06-12 08:23, Sean Christopherson wrote:
On Wed, Jun 12, 2019 at 06:14:39PM +0300, Jarkko Sakkinen wrote:
On Wed, Jun 05, 2019 at 12:48:45PM -0700, Sean Christopherson wrote:
A reserved field could prove useful in the future, and packed structs
can result in inefficiencies due to its impact on alignment.

Signed-off-by: Sean Christopherson <sean.j.christopherson@xxxxxxxxx>

Rather should extend mrmask just to 32 bits.

I considered that option, but extending mrmask would probably confusing
to userspace, e.g. we'd have to document that bits 31:15 are ignored.


Yeah this doesn't make a whole lot of sense to me.

--
Jethro Beekman | Fortanix

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature


[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux