From patchwork Fri Jun 6 15:06:07 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peter Zijlstra X-Patchwork-Id: 4312151 Return-Path: X-Original-To: patchwork-linux-parisc@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 7B8BC9F170 for ; Fri, 6 Jun 2014 15:06:18 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id AB47620171 for ; Fri, 6 Jun 2014 15:06:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C09F220166 for ; Fri, 6 Jun 2014 15:06:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751692AbaFFPGO (ORCPT ); Fri, 6 Jun 2014 11:06:14 -0400 Received: from casper.infradead.org ([85.118.1.10]:57991 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751608AbaFFPGO (ORCPT ); Fri, 6 Jun 2014 11:06:14 -0400 Received: from dhcp-077-248-225-117.chello.nl ([77.248.225.117] helo=laptop) by casper.infradead.org with esmtpsa (Exim 4.80.1 #2 (Red Hat Linux)) id 1Wsvib-0005Ag-69; Fri, 06 Jun 2014 15:06:09 +0000 Received: by laptop (Postfix, from userid 1000) id 7AC2B1035CED7; Fri, 6 Jun 2014 17:06:07 +0200 (CEST) Date: Fri, 6 Jun 2014 17:06:07 +0200 From: Peter Zijlstra To: Linus Torvalds Cc: Mikulas Patocka , "James E.J. Bottomley" , Helge Deller , John David Anglin , Parisc List , Linux Kernel Mailing List , Paul McKenney , "Vinod, Chegu" , Waiman Long , Thomas Gleixner , Rik van Riel , Andrew Morton , Davidlohr Bueso , Peter Anvin , Andi Kleen , "Chandramouleeswaran, Aswin" , "Norton, Scott J" , Jason Low , mingo@kernel.org Subject: Re: [PATCH v2] introduce atomic_pointer to fix a race condition in cancelable mcs spinlocks Message-ID: <20140606150607.GT13930@laptop.programming.kicks-ass.net> References: <20140602162525.GH16155@laptop.programming.kicks-ass.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-parisc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-parisc@vger.kernel.org X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On Mon, Jun 02, 2014 at 10:09:35AM -0700, Linus Torvalds wrote: > On Mon, Jun 2, 2014 at 9:25 AM, Peter Zijlstra wrote: > > > > And I can't say I'm a particular fan of these ops either, as alternative > > I'm almost inclined to just exclude parisc from using opt spinning. > > Please do. Something like so; if the rwsem stuff lands in .15 we need more for that, it doesn't have a convenient CONFIG symbol like this. Linus will you take this from email, or should I get it through tip/locking/urgent or so? Reviewed-by: Paul E. McKenney Reviewed-by: Davidlohr Bueso --- Subject: locking, mutex: Disable optimistic spinning for PA-RISC PA-RISC's cmpxchg is not save against normal stores and the code used for optimistic spinning is known broken because of this. Disable for now. Reported-by: Mikulas Patocka Signed-off-by: Peter Zijlstra --- kernel/Kconfig.locks | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 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 --git a/kernel/Kconfig.locks b/kernel/Kconfig.locks index 35536d9c0964..9c239e080c2d 100644 --- a/kernel/Kconfig.locks +++ b/kernel/Kconfig.locks @@ -222,7 +222,7 @@ endif config MUTEX_SPIN_ON_OWNER def_bool y - depends on SMP && !DEBUG_MUTEXES + depends on SMP && !DEBUG_MUTEXES && !PARISC config ARCH_USE_QUEUE_RWLOCK bool