Fwd: [RFC PATCH 11/16] hppa: locks: remove unused arch_*_relax operations

Fwd: [RFC PATCH 11/16] hppa: locks: remove unused arch_*_relax operations

[Helge Deller]

[-- Attachment #1: Type: text/plain, Size: 1333 bytes --]

Thanks Will,

forwarding to linux-parisc@vger.kernel.org so that it gets into parisc patchwork.

I'll queue this patch up for 3.11 if it's OK for you.

Helge

-------- Original Message --------
Subject: [RFC PATCH 11/16] hppa: locks: remove unused arch_*_relax operations
Date: Fri, 21 Jun 2013 18:17:32 +0100
From: Will Deacon <will.deacon@arm.com>
To: linux-arch@vger.kernel.org
CC: Will Deacon <will.deacon@arm.com>, Helge Deller <deller@gmx.de>

The arch_{spin,read,write}_relax macros are not used anywhere in the
kernel and are typically just aliases for cpu_relax().

This patch removes the unused definitions for parisc.

Cc: Helge Deller <deller@gmx.de>
Signed-off-by: Will Deacon <will.deacon@arm.com>
---
 arch/parisc/include/asm/spinlock.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/arch/parisc/include/asm/spinlock.h b/arch/parisc/include/asm/spinlock.h
index 3516e0b..64f2992 100644
--- a/arch/parisc/include/asm/spinlock.h
+++ b/arch/parisc/include/asm/spinlock.h
@@ -191,8 +191,4 @@ static __inline__ int arch_write_can_lock(arch_rwlock_t *rw)
 #define arch_read_lock_flags(lock, flags) arch_read_lock(lock)
 #define arch_write_lock_flags(lock, flags) arch_write_lock(lock)
 
-#define arch_spin_relax(lock)	cpu_relax()
-#define arch_read_relax(lock)	cpu_relax()
-#define arch_write_relax(lock)	cpu_relax()

Re: Fwd: [RFC PATCH 11/16] hppa: locks: remove unused arch_*_relax operations

[Will Deacon]

On Fri, Jun 21, 2013 at 07:32:37PM +0100, Helge Deller wrote:
> Thanks Will,

Hi Helge,

> forwarding to linux-parisc@vger.kernel.org so that it gets into parisc patchwork.
> 
> I'll queue this patch up for 3.11 if it's OK for you.

Fine by me. Each patch in the series is self-contained, so they can get
merged via the arch trees without any problem.

Thanks!

Will

Re: Fwd: [RFC PATCH 11/16] hppa: locks: remove unused arch_*_relax operations

[Will Deacon]

On Mon, Jun 24, 2013 at 09:55:22AM +0100, Will Deacon wrote:
> On Fri, Jun 21, 2013 at 07:32:37PM +0100, Helge Deller wrote:
> > Thanks Will,
> 
> Hi Helge,
> 
> > forwarding to linux-parisc@vger.kernel.org so that it gets into parisc patchwork.
> > 
> > I'll queue this patch up for 3.11 if it's OK for you.
> 
> Fine by me. Each patch in the series is self-contained, so they can get
> merged via the arch trees without any problem.

Wait a second; turns out the GENERIC_LOCKBREAK code *can* emit calls to
these functions, so I'll need to sort that out before you can take the
arch-side of things.

Better hold off on this until I have that fixed.

Cheers,

Will