From: Sudeep Holla <redacted>
linux/processor.h has exactly same defination for spin_until_cond.
Drop the redundant defination in asm/processor.h
Cc: Nicholas Piggin <npiggin@gmail.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Sudeep Holla <redacted>
Signed-off-by: Christophe Leroy <redacted>
---
That's just a rebase
arch/powerpc/include/asm/processor.h | 11 -----------
1 file changed, 11 deletions(-)
@@ -339,17 +339,6 @@ static inline unsigned long __pack_fe01(unsigned int fpmode)#define spin_end() HMT_medium()-#define spin_until_cond(cond) \-do{\-if(unlikely(!(cond))){\-spin_begin();\-do{\-spin_cpu_relax();\-}while(!(cond));\-spin_end();\-}\-}while(0)-#endif/* Check that a certain kernel stack pointer is valid in task_struct p */
From: Sudeep Holla <redacted>
This implementation uses spin_until_cond in wd_smp_lock including
neither linux/processor.h nor asm/processor.h
This patch includes linux/processor.h here for spin_until_cond usage.
Cc: Nicholas Piggin <npiggin@gmail.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Signed-off-by: Sudeep Holla <redacted>
Signed-off-by: Christophe Leroy <redacted>
---
arch/powerpc/kernel/watchdog.c | 1 +
1 file changed, 1 insertion(+)
On Fri, Jun 11, 2021 at 07:10:57PM +0000, Christophe Leroy wrote:
From: Sudeep Holla <redacted>
linux/processor.h has exactly same defination for spin_until_cond.
Drop the redundant defination in asm/processor.h
Wow you must be real good at ML archaeology, this must have been at-least
3+ years old. I found this when I wanted to you spin_until_cond. Thanks
anyways for digging the original patch, nobody would have remembered even
if you posted fresh 😉.
--
Regards,
Sudeep