> -----Original Message-----
> From: Hansen, Dave <[email protected]>
> Sent: Friday, August 1, 2025 7:45 PM
> To: Reshetova, Elena <[email protected]>
> Cc: [email protected]; [email protected]; Huang, Kai
> <[email protected]>; [email protected]; [email protected]; linux-
> [email protected]; [email protected]; Mallick, Asit K
> <[email protected]>; Scarlata, Vincent R 
> <[email protected]>;
> Cai, Chong <[email protected]>; Aktas, Erdem <[email protected]>;
> Annapurve, Vishal <[email protected]>; Bondarevska, Nataliia
> <[email protected]>; Raynor, Scott <[email protected]>
> Subject: Re: [PATCH v10 2/6] x86/sgx: Introduce functions to count the
> sgx_(vepc_)open()
> 
> On 8/1/25 04:25, Elena Reshetova wrote:
> > Note: the sgx_inc_usage_count prototype is defined to return
> > int for the cleanliness of the follow-up patches. When the
> > EUPDATESVN SGX instruction will be enabled in the follow-up patch,
> > the sgx_inc_usage_count will start to return int.
> 
> Please use parenthesis for function_names().

Sure, will fix. 

> 
> Second, sgx_inc_usage_count() _already_ returns 'int'. Now. In this patch.
> 
> So I'm confused what this is trying to say.
> 
> Is this trying to say that sgx_inc_usage_count() always returns success
> (0) for now, but the future implementation can fail? *That's* why it
> needs to have an 'int' return type.

Yes, I will rephrase this better. 

Best Regards,
Elena.

Reply via email to