linux-kernel - [PATCH] h8300/bitops: Correct smp_mb__{before,after} wrt clear

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-Id: <1461047421-26977-1-git-send-email-dave@stgolabs.net>
Date:	Mon, 18 Apr 2016 23:30:21 -0700
From:	Davidlohr Bueso <dave@...olabs.net>
To:	ysato@...rs.sourceforge.jp
Cc:	uclinux-h8-devel@...ts.sourceforge.jp, dave@...olabs.net,
	linux-kernel@...r.kernel.org, Davidlohr Bueso <dbueso@...e.de>
Subject: [PATCH] h8300/bitops: Correct smp_mb__{before,after} wrt clear_bit

(1) We no longer have smp_mb_{before,after}_clear_bit(), and should
be using the _atomic suffix.

(2) By not defining smp_mb__{before,after} calls, h8300 will default
to smp_mb, but since the arch is !SMP will be a compiler barrier
anyway.

Based on the above, rename the calls.

Signed-off-by: Davidlohr Bueso <dbueso@...e.de>
---
 arch/h8300/include/asm/bitops.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/h8300/include/asm/bitops.h b/arch/h8300/include/asm/bitops.h
index 05999aba1d6a..842f785977e4 100644
--- a/arch/h8300/include/asm/bitops.h
+++ b/arch/h8300/include/asm/bitops.h
@@ -53,8 +53,8 @@ static inline void FNAME(int nr, volatile unsigned long *addr)	\
 /*
  * clear_bit() doesn't provide any barrier for the compiler.
  */
-#define smp_mb__before_clear_bit()	barrier()
-#define smp_mb__after_clear_bit()	barrier()
+#define smp_mb__before_atomic()	barrier()
+#define smp_mb__after_atomic()	barrier()
 
 H8300_GEN_BITOP(set_bit,    "bset")
 H8300_GEN_BITOP(clear_bit,  "bclr")
-- 
2.8.1