The following commit has been merged into the x86/entry branch of tip:

Commit-ID:     e82587336695f14283987c9aa0bfd775b520856d
Gitweb:        
https://git.kernel.org/tip/e82587336695f14283987c9aa0bfd775b520856d
Author:        Peter Zijlstra <[email protected]>
AuthorDate:    Tue, 02 Jun 2020 14:24:47 +02:00
Committer:     Peter Zijlstra <[email protected]>
CommitterDate: Mon, 15 Jun 2020 14:10:08 +02:00

x86, kcsan: Remove __no_kcsan_or_inline usage

Now that KCSAN relies on -tsan-distinguish-volatile we no longer need
the annotation for constant_test_bit(). Remove it.

Signed-off-by: Peter Zijlstra (Intel) <[email protected]>
---
 arch/x86/include/asm/bitops.h | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/arch/x86/include/asm/bitops.h b/arch/x86/include/asm/bitops.h
index 35460fe..0367efd 100644
--- a/arch/x86/include/asm/bitops.h
+++ b/arch/x86/include/asm/bitops.h
@@ -201,12 +201,8 @@ arch_test_and_change_bit(long nr, volatile unsigned long 
*addr)
        return GEN_BINARY_RMWcc(LOCK_PREFIX __ASM_SIZE(btc), *addr, c, "Ir", 
nr);
 }
 
-static __no_kcsan_or_inline bool constant_test_bit(long nr, const volatile 
unsigned long *addr)
+static __always_inline bool constant_test_bit(long nr, const volatile unsigned 
long *addr)
 {
-       /*
-        * Because this is a plain access, we need to disable KCSAN here to
-        * avoid double instrumentation via instrumented bitops.
-        */
        return ((1UL << (nr & (BITS_PER_LONG-1))) &
                (addr[nr >> _BITOPS_LONG_SHIFT])) != 0;
 }

Reply via email to