Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752336AbaFBRnE (ORCPT ); Mon, 2 Jun 2014 13:43:04 -0400 Received: from g4t3427.houston.hp.com ([15.201.208.55]:18455 "EHLO g4t3427.houston.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752075AbaFBRm4 (ORCPT ); Mon, 2 Jun 2014 13:42:56 -0400 Message-ID: <538CB78A.8090500@hp.com> Date: Mon, 02 Jun 2014 13:42:34 -0400 From: Waiman Long User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.12) Gecko/20130109 Thunderbird/10.0.12 MIME-Version: 1.0 To: Davidlohr Bueso CC: Linus Torvalds , Peter Zijlstra , Mikulas Patocka , "James E.J. Bottomley" , Helge Deller , John David Anglin , Parisc List , Linux Kernel Mailing List , Paul McKenney , "Vinod, Chegu" , Thomas Gleixner , Rik van Riel , Andrew Morton , Peter Anvin , Andi Kleen , "Chandramouleeswaran, Aswin" , "Norton, Scott J" , Jason Low Subject: Re: [PATCH v2] introduce atomic_pointer to fix a race condition in cancelable mcs spinlocks References: <20140602162525.GH16155@laptop.programming.kicks-ass.net> <1401729145.5185.5.camel@buesod1.americas.hpqcorp.net> In-Reply-To: <1401729145.5185.5.camel@buesod1.americas.hpqcorp.net> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/02/2014 01:12 PM, Davidlohr Bueso wrote: > On Mon, 2014-06-02 at 10:09 -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. > I agree, this is the best way out of this mess. Furthermore, it would > also be nice to consolidate opt spinning in a common CONFIG option -- > right now mutexes and rwsems create their own dependencies. > I would suggest adding a RWSEM_SPIN_ON_OWNER to control opt spinning in rwsem. Currently MUTEX_SPIN_ON_OWNER is doing that for mutex, and it is disabled when mutex debugging is turned on. So I think it is better to allow them to be disabled separately. -Longman -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/