[PATCH 2/3] Revert "locking/local_lock: Make the empty local_lock_*() function a macro."

From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
To: torvalds@linux-foundation.org
Cc: akpm@linux-foundation.org, bigeasy@linutronix.de,
	boqun.feng@gmail.com, bp@alien8.de, linux-kernel@vger.kernel.org,
	longman@redhat.com, mingo@kernel.org, peterz@infradead.org,
	tglx@linutronix.de, will@kernel.org,
	Dennis Zhou <dennis@kernel.org>, Tejun Heo <tj@kernel.org>,
	Christoph Lameter <cl@linux.com>
Subject: [PATCH 2/3] Revert "locking/local_lock: Make the empty local_lock_*() function a macro."
Date: Thu, 24 Mar 2022 18:39:26 +0100	[thread overview]
Message-ID: <20220324173927.2230447-3-bigeasy@linutronix.de> (raw)
In-Reply-To: <20220324173927.2230447-1-bigeasy@linutronix.de>

With volatile removed from arch_raw_cpu_ptr() the compiler no longer
creates the per-CPU reference. The usage of the macro can be reverted
now.

Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
 include/linux/local_lock_internal.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/linux/local_lock_internal.h b/include/linux/local_lock_internal.h
index 6d635e8306d64..975e33b793a77 100644
--- a/include/linux/local_lock_internal.h
+++ b/include/linux/local_lock_internal.h
@@ -44,9 +44,9 @@ static inline void local_lock_debug_init(local_lock_t *l)
 }
 #else /* CONFIG_DEBUG_LOCK_ALLOC */
 # define LOCAL_LOCK_DEBUG_INIT(lockname)
-# define local_lock_acquire(__ll)  do { typecheck(local_lock_t *, __ll); } while (0)
-# define local_lock_release(__ll)  do { typecheck(local_lock_t *, __ll); } while (0)
-# define local_lock_debug_init(__ll)  do { typecheck(local_lock_t *, __ll); } while (0)
+static inline void local_lock_acquire(local_lock_t *l) { }
+static inline void local_lock_release(local_lock_t *l) { }
+static inline void local_lock_debug_init(local_lock_t *l) { }
 #endif /* !CONFIG_DEBUG_LOCK_ALLOC */
 
 #define INIT_LOCAL_LOCK(lockname)	{ LOCAL_LOCK_DEBUG_INIT(lockname) }
-- 
2.35.1

