]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
s390/bitops: Slightly optimize ffs() and fls64()
authorHeiko Carstens <hca@linux.ibm.com>
Mon, 4 Aug 2025 15:08:54 +0000 (17:08 +0200)
committerAlexander Gordeev <agordeev@linux.ibm.com>
Wed, 20 Aug 2025 14:38:23 +0000 (16:38 +0200)
Use a simpler algorithm to calculate the result of ffs() and fls64().
This generates slightly better code and increases readability.

Kernel image size is reduced by ~3kb (gcc 15.1.0 + defconfig).

Suggested-by: Nina Schoetterl-Glausch <nsg@linux.ibm.com>
Signed-off-by: Heiko Carstens <hca@linux.ibm.com>
Signed-off-by: Alexander Gordeev <agordeev@linux.ibm.com>
arch/s390/include/asm/bitops.h

index a5ca0a9476916c9f5327972e55ebb1faff52a056..10d7573d1582c4eb606061dd572152b8999593a3 100644 (file)
@@ -193,10 +193,9 @@ static inline unsigned long __ffs(unsigned long word)
  */
 static inline int ffs(int word)
 {
-       unsigned long mask = 2 * BITS_PER_LONG - 1;
        unsigned int val = (unsigned int)word;
 
-       return (1 + (__flogr(-val & val) ^ (BITS_PER_LONG - 1))) & mask;
+       return BITS_PER_LONG - __flogr(-val & val);
 }
 
 /**
@@ -223,9 +222,7 @@ static inline unsigned long __fls(unsigned long word)
  */
 static inline int fls64(unsigned long word)
 {
-       unsigned long mask = 2 * BITS_PER_LONG - 1;
-
-       return (1 + (__flogr(word) ^ (BITS_PER_LONG - 1))) & mask;
+       return BITS_PER_LONG - __flogr(word);
 }
 
 /**