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 Wed, Jun 12, 2019 at 08:23:42AM -0700, 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.

I meant u64.

Not a big deal. With the reserved field we would have to document the
same in different terms.

/Jarkko




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

  Powered by Linux