[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170413163757.wwhttkpm3v7emz33@hirez.programming.kicks-ass.net>
Date: Thu, 13 Apr 2017 18:37:57 +0200
From: Peter Zijlstra <peterz@...radead.org>
To: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Cc: linux-kernel@...r.kernel.org, mingo@...nel.org,
jiangshanlai@...il.com, dipankar@...ibm.com,
akpm@...ux-foundation.org, mathieu.desnoyers@...icios.com,
josh@...htriplett.org, tglx@...utronix.de, rostedt@...dmis.org,
dhowells@...hat.com, edumazet@...gle.com, fweisbec@...il.com,
oleg@...hat.com, bobby.prani@...il.com,
Will Deacon <will.deacon@....com>,
Boqun Feng <boqun.feng@...ux.vnet.ibm.com>,
Benjamin Herrenschmidt <benh@...nel.crashing.org>,
Paul Mackerras <paulus@...ba.org>,
linuxppc-dev@...ts.ozlabs.org
Subject: Re: [PATCH tip/core/rcu 02/40] rcu: Make arch select
smp_mb__after_unlock_lock() strength
On Thu, Apr 13, 2017 at 09:26:51AM -0700, Paul E. McKenney wrote:
> ARCH_WEAK_RELEASE_ACQUIRE actually works both ways.
>
> To see this, imagine some strange alternate universe in which the Power
> hardware guys actually did decide to switch PPC to doing RCsc as you
> suggest. There would still be a lot of Power hardware out there that
> still does RCpc. Therefore, powerpc builds that needed to run on old
> Power hardware would select ARCH_WEAK_RELEASE_ACQUIRE, while kernels
> built to run only on the shiny new (but mythical) alternate-universe
> Power hardware would avoid selecting this Kconfig option.
Ah, but Power software guys could do it today by replacing an LWSYNC
with a SYNC in say arch_spin_unlock().
And yes, I know this isn't a popular suggestion, but it would do the
trick.
Its just that since there's one (PPC) we can sort of pressure them with
the pain of being the only ones to hit all the bugs. But the moment more
appear (and I'm afraid it'll be MIPS, with the excuse that PPC already
does this) it will be ever so much harder to get rid of it.
Then again, maybe I should just give up and accept the Linux kernel has
RCpc locks..
Powered by blists - more mailing lists