[xen master] arm/bitops: encapsulate violation of MISRA C:2012 Rule 10.1

[patchbot@xxxxxxx]

commit 2f848d3cee46685ca0a9bc77dbb23417b4b35652
Author:     Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
AuthorDate: Wed Nov 29 10:32:39 2023 +0100
Commit:     Julien Grall <julien@xxxxxxx>
CommitDate: Wed Nov 29 18:21:58 2023 +0100

    arm/bitops: encapsulate violation of MISRA C:2012 Rule 10.1
    
    The definitions of ffs{l}? violate Rule 10.1, by using the well-known
    pattern (x & -x); its usage is wrapped by the ISOLATE_LSB macro.
    
    No functional change.
    
    Signed-off-by: Nicola Vetrini <nicola.vetrini@xxxxxxxxxxx>
    Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>
---
 xen/arch/arm/include/asm/bitops.h | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/xen/arch/arm/include/asm/bitops.h 
b/xen/arch/arm/include/asm/bitops.h
index 71ae14cab3..ab030b6cb0 100644
--- a/xen/arch/arm/include/asm/bitops.h
+++ b/xen/arch/arm/include/asm/bitops.h
@@ -9,6 +9,8 @@
 #ifndef _ARM_BITOPS_H
 #define _ARM_BITOPS_H
 
+#include <xen/macros.h>
+
 #include <asm/asm_defns.h>
 
 /*
@@ -155,8 +157,8 @@ static inline int fls(unsigned int x)
 }
 
 
-#define ffs(x) ({ unsigned int __t = (x); fls(__t & -__t); })
-#define ffsl(x) ({ unsigned long __t = (x); flsl(__t & -__t); })
+#define ffs(x) ({ unsigned int __t = (x); fls(ISOLATE_LSB(__t)); })
+#define ffsl(x) ({ unsigned long __t = (x); flsl(ISOLATE_LSB(__t)); })
 
 /**
  * find_first_set_bit - find the first set bit in @word
--
generated by git-patchbot for /home/xen/git/xen.git#master