> -----Original Message----- > From: Huang, Kai <kai.hu...@intel.com> > Sent: Thursday, August 7, 2025 2:50 AM > To: Reshetova, Elena <elena.reshet...@intel.com>; Hansen, Dave > <dave.han...@intel.com> > Cc: sea...@google.com; mi...@kernel.org; Scarlata, Vincent R > <vincent.r.scarl...@intel.com>; x...@kernel.org; jar...@kernel.org; > Annapurve, Vishal <vannapu...@google.com>; linux-kernel@vger.kernel.org; > Mallick, Asit K <asit.k.mall...@intel.com>; Aktas, Erdem > <erdemak...@google.com>; Cai, Chong <cho...@google.com>; Bondarevska, > Nataliia <bond...@google.com>; linux-...@vger.kernel.org; Raynor, Scott > <scott.ray...@intel.com> > Subject: Re: [PATCH v11 3/5] x86/sgx: Define error codes for use by > ENCLS[EUPDATESVN] > > On Wed, 2025-08-06 at 11:11 +0300, Elena Reshetova wrote: > > Add error codes for ENCLS[EUPDATESVN], then SGX CPUSVN update > > process can know the execution state of EUPDATESVN and notify > > userspace. > > I would also add some text to explain why other error codes are not > defined: > > EUPDATESVN will be called when no active SGX user is guaranteed. Only add > the error codes that can legally happen. E.g., it could also fail due to > "SGX not ready" when there's SGX users but it wouldn't happen in this > implementation.
Sure, will add in the next iteration, thanks!