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