On 9/19/22 23:39, Zhiquan Li wrote: > --- a/arch/x86/kernel/cpu/sgx/virt.c > +++ b/arch/x86/kernel/cpu/sgx/virt.c > @@ -46,10 +46,12 @@ static int __sgx_vepc_fault(struct sgx_vepc *vepc, > if (epc_page) > return 0; > > - epc_page = sgx_alloc_epc_page(vepc, false); > + epc_page = sgx_alloc_epc_page((void *)addr, false); > if (IS_ERR(epc_page)) > return PTR_ERR(epc_page); One thing not clear from the changelog: This actually changes the value getting passed into sgx_alloc_epc_page() and set in the page->owner field. What effect does this have? If I apply these and run the tree at this commit, what happens? What behavior changes? Was this 'vepc' value simply not used before?
