Messages in this thread | | | Date | Wed, 31 Aug 2022 21:21:41 +0300 | From | Jarkko Sakkinen <> | Subject | Re: [PATCH v2 3/6] x86/sgx: Handle VA page allocation failure for EAUG on PF. |
| |
On Wed, Aug 31, 2022 at 11:08:33AM -0700, Reinette Chatre wrote: > Hi Jarkko, > > On 8/31/2022 10:38 AM, Jarkko Sakkinen wrote: > > From: Haitao Huang <haitao.huang@linux.intel.com> > > > > VM_FAULT_NOPAGE is expected behaviour for -EBUSY failure path, when > > augmenting a page, as this means that the reclaimer thread has been > > triggered, and the intention is just to round-trip in ring-3, and > > retry with a new page fault. > > > > Fixes: 5a90d2c3f5ef ("x86/sgx: Support adding of pages to an initialized enclave") > > Signed-off-by: Haitao Huang <haitao.huang@linux.intel.com> > > Tested-by: Vijay Dhanraj <vijay.dhanraj@intel.com> > > Reviewed-by: Reinette Chatre <reinette.chatre@intel.com> > > Signed-off-by: Jarkko Sakkinen <jarkko@kernel.org> > > --- > > v3: > > * Added Reinette's ack. > > The ack came with a caveat related to a white space issue that > you did not fix in this version.
Reinette, I'm sorry but I might possibly have some sort of blind spot here.
I looked at the diff and I could not see any issues, and did not run into issues with checkpath.
Can you point out what exactly is the whitespace issue?
BR, Jarkko
| |