Messages in this thread |  | | Date | Sat, 24 May 2025 02:45:35 +0300 | | From | Jarkko Sakkinen <> | | Subject | Re: [PATCH v6 1/5] x86/sgx: Introduce a counter to count the sgx_(vepc_)open() |
| |
On Fri, May 23, 2025 at 08:58:54AM -0700, Dave Hansen wrote: > On 5/23/25 08:54, Jarkko Sakkinen wrote: > >> +void sgx_dec_usage_count(void) > >> +{ > >> + atomic64_dec(&sgx_usage_count); > >> +} > > I think these both should be static inlines in arch/x86/kernel/cpu/sgx.h. > > Global symbols is over the top. Even if I think disassembly (when doing > > debugging, bug hunting or similar tasks), it'd nicer that way. > > If they're just used in a single file, make them 'static' and let the > compiler decide whether to inline them or not.
This would be totally fine.
BR, Jarkko
|  |