> -----Original Message-----
> From: Jarkko Sakkinen <jar...@kernel.org>
> Sent: Friday, May 23, 2025 6:54 PM
> To: Reshetova, Elena <elena.reshet...@intel.com>
> Cc: Hansen, Dave <dave.han...@intel.com>; sea...@google.com; Huang, Kai
> <kai.hu...@intel.com>; mi...@kernel.org; linux-...@vger.kernel.org; linux-
> ker...@vger.kernel.org; x...@kernel.org; Mallick, Asit K
> <asit.k.mall...@intel.com>; Scarlata, Vincent R
> <vincent.r.scarl...@intel.com>; Cai, Chong <cho...@google.com>; Aktas,
> Erdem <erdemak...@google.com>; Annapurve, Vishal
> <vannapu...@google.com>; dionnagl...@google.com;
> bond...@google.com; Raynor, Scott <scott.ray...@intel.com>
> Subject: Re: [PATCH v6 1/5] x86/sgx: Introduce a counter to count the
> sgx_(vepc_)open()
> > /*
> > diff --git a/arch/x86/kernel/cpu/sgx/main.c
> b/arch/x86/kernel/cpu/sgx/main.c
> > index 2de01b379aa3..a018b01b8736 100644
> > --- a/arch/x86/kernel/cpu/sgx/main.c
> > +++ b/arch/x86/kernel/cpu/sgx/main.c
> > @@ -917,6 +917,20 @@ int sgx_set_attribute(unsigned long
> *allowed_attributes,
> > }
> > EXPORT_SYMBOL_GPL(sgx_set_attribute);
> >
> > +/* Counter to count the active SGX users */
> > +static atomic64_t sgx_usage_count;
> > +
> > +int sgx_inc_usage_count(void)
> > +{
> > + atomic64_inc(&sgx_usage_count);
> > + return 0;
> > +}
>
> Maybe this was discussed but why this is not just a void-function?
The last patch is cleaner if the prototype is already
returning int here. Also error unwinding takes this into account
right in this patch. Do you have objections to leave it as it is?
Best Regards,
Elena.
