diff mbox

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

Message ID 51C49C45.1060207@gmx.de (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Helge Deller June 21, 2013, 6:32 p.m. UTC 
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(-)

Comments

Will Deacon June 24, 2013, 8:55 a.m. UTC | #1 
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
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Will Deacon June 24, 2013, 11:37 a.m. UTC | #2 
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
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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()
-
 #endif /* __ASM_SPINLOCK_H */