On 06/08/2018 10:09 AM, Jarkko Sakkinen wrote: > --- a/arch/x86/crypto/aesni-intel_asm.S > +++ b/arch/x86/crypto/aesni-intel_asm.S > @@ -45,6 +45,8 @@ > #define MOVADQ movaps > #define MOVUDQ movups > > +#ifndef AESNI_INTEL_MINIMAL > + > #ifdef __x86_64__ > > # constants in mergeable sections, linker can reorder and merge > @@ -133,6 +135,8 @@ ALL_F: .octa 0xffffffffffffffffffffffffffffffff > #define keysize 2*15*16(%arg1) > #endif > > +#endif /* AESNI_INTEL_MINIMAL */ > + I'd really prefer that these get moved into a separate file rather than a scattered set of #ifdefs. This just seem fragile to me. Can we have a "aesni-intel_asm-minimal.S"? Or, at least bunch the minimal set of things *together*?
