On Thu, 2021-08-26 at 11:58 +0200, Borislav Petkov wrote: > On Thu, Aug 26, 2021 at 02:52:32AM +0300, Jarkko Sakkinen wrote: > > Similarly as sgx_virt_*, decorate sgx_set_attribute() with ifdef, so that > > calling it without appropraite config flags, will cause a compilation > > error, and not a linking error. > > Please explain what exactly is this fixing. IOW, how can I reproduce the > failure? You're right, fixes tag is not necessary. I made this change because I'm including the header to set_memory.c, and IMHO it is better to make sure when possible that we get compilation errors than linker errors, if for some reason kernel did not have SGX support. It's also incoherent that KVM specific functions are compilation flagged but sgx_set_attribute() is not. /Jarkko
