On Sun, Apr 13, 2025 at 08:43:50AM -0700, Eric Biggers wrote: > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > All of the CRC library's CPU feature static_keys are initialized by > initcalls and never change afterwards, so there's no need for them to be > in the regular .data section. Put them in .data..ro_after_init instead. > > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> > --- > > I'm planning to take this via the crc tree. > > arch/arm/lib/crc-t10dif-glue.c | 4 ++-- > arch/arm/lib/crc32-glue.c | 4 ++-- > arch/arm64/lib/crc-t10dif-glue.c | 4 ++-- > arch/loongarch/lib/crc32-loongarch.c | 2 +- > arch/mips/lib/crc32-mips.c | 2 +- > arch/powerpc/lib/crc-t10dif-glue.c | 2 +- > arch/powerpc/lib/crc32-glue.c | 2 +- > arch/s390/lib/crc32-glue.c | 2 +- Acked-by: Heiko Carstens <hca@xxxxxxxxxxxxx> # s390 I just realized that we can get rid of the static key in the s390 piece and end up with slightly better code. Could you add the patch below to your tree, please? If this would go via the s390 tree this would result in a merge conflict, which is unnecessary.
