On Fri, Aug 23, 2019 at 06:02:48PM +0300, Jarkko Sakkinen wrote: > 3. Why don't just store SECINFO flags on EADD? Doing > calc_vma_prot_bits() on fly is not expensive. Now > data in SECINFO flags is scattered to two different > fields (desc and vm_max_prot_bits). I'd just have a field > called secinfo_flags. Makes sense to have that field as a cache so that we don't have to do recalc per page in the sgx_encl_may_map() and thus makes sense to keep TCS tagging in desc. /Jarkko
