Messages in this thread | | | Date | Thu, 19 May 2016 23:18:24 -0700 | From | Davidlohr Bueso <> | Subject | Re: [RFC][PATCH 0/7] locking/rwsem: Convert rwsem count to atomic_long_t |
| |
On Tue, 17 May 2016, Peter Zijlstra wrote:
>Subject: locking,rwsem: Remove rwsem_atomic_add() and rwsem_atomic_update() >From: Jason Low <jason.low2@hpe.com> >Date: Mon, 16 May 2016 17:38:02 -0700 > >The rwsem-xadd count has been converted to an atomic variable and the >rwsem code now directly uses atomic_long_add() and >atomic_long_add_return(), so we can remove the arch implementations of >rwsem_atomic_add() and rwsem_atomic_update(). > >Cc: Waiman Long <Waiman.Long@hpe.com> >Cc: Heiko Carstens <heiko.carstens@de.ibm.com> >Cc: Jason Low <jason.low2@hp.com> >Cc: Richard Henderson <rth@twiddle.net> >Cc: Ingo Molnar <mingo@redhat.com> >Cc: Fenghua Yu <fenghua.yu@intel.com> >Cc: Andrew Morton <akpm@linux-foundation.org> >Cc: Martin Schwidefsky <schwidefsky@de.ibm.com> >Cc: Linus Torvalds <torvalds@linux-foundation.org> >Cc: Tim Chen <tim.c.chen@linux.intel.com> >Cc: Christoph Lameter <cl@linux.com> >Cc: Thomas Gleixner <tglx@linutronix.de> >Cc: Arnd Bergmann <arnd@arndb.de> >Cc: Terry Rudd <terry.rudd@hpe.com> >Cc: Peter Hurley <peter@hurleysoftware.com> >Cc: Davidlohr Bueso <dave@stgolabs.net>
Acked-by: Davidlohr Bueso <dave@stgolabs.net>
>Cc: Ivan Kokshaysky <ink@jurassic.park.msu.ru> >Cc: Tony Luck <tony.luck@intel.com> >Cc: Matt Turner <mattst88@gmail.com> >Signed-off-by: Jason Low <jason.low2@hpe.com> >Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org> >--- > arch/alpha/include/asm/rwsem.h | 42 ----------------------------------------- > arch/ia64/include/asm/rwsem.h | 7 ------ > arch/s390/include/asm/rwsem.h | 37 ------------------------------------ > arch/x86/include/asm/rwsem.h | 18 ----------------- > include/asm-generic/rwsem.h | 16 --------------- > 5 files changed, 120 deletions(-)
Nice. This, along with Michal's work getting rid of a lot of superfluous implementations, have gotten rid of plenty of rwsem code in arch/*
Thanks, Davidlohr
| |