Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752231Ab2BTGPA (ORCPT ); Mon, 20 Feb 2012 01:15:00 -0500 Received: from e28smtp02.in.ibm.com ([122.248.162.2]:45666 "EHLO e28smtp02.in.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751675Ab2BTGO7 (ORCPT ); Mon, 20 Feb 2012 01:14:59 -0500 Message-ID: <4F41E4C1.9010901@linux.vnet.ibm.com> Date: Mon, 20 Feb 2012 11:44:25 +0530 From: Raghavendra K T User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.24) Gecko/20111108 Fedora/3.1.16-1.fc14 Thunderbird/3.1.16 MIME-Version: 1.0 To: Jeremy Fitzhardinge , Gleb Natapov CC: Tejun Heo , John Stultz , LKML , Thomas Gleixner , Srivatsa Vaddagiri , Peter Zijlstra , David Howells , Andrew Jones Subject: Re: [PATCH 1/1] linux headers: header file(s) changes to enable spinlock use jumplabel References: <20120217082527.15283.72980.sendpatchset@oc5400248562.ibm.com> <4F403268.7040804@goop.org> <20120219092456.GB10038@redhat.com> <4F41D72E.7000605@goop.org> In-Reply-To: <4F41D72E.7000605@goop.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit x-cbid: 12022006-5816-0000-0000-0000015DBBE2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2131 Lines: 44 On 02/20/2012 10:46 AM, Jeremy Fitzhardinge wrote: [...] >>>> So we get following error when we try to include jump_label.h from >>>> arch/x86/include/asm/spinlock.h because of cyclic dependency >>>> -> -> -> ... -> >>> What about splitting the jump_label_key_deferred stuff into a separate >>> jump_label_deferred.h, and just include that where it's needed? >>> >> Andrew Jones did exactly that (CCed). Sorry, did not get it. Tried to search the patch. Is it similar work or same work?. Could you please point. shall try both the way (current way and jump_label_deferred way). So whichever makes maintainer happy, let that go :) But does pvlock have to use jump >> label? I looked at the code and it is used like paravirt patching. Meaning >> it is patched only once on a boot up when XEN is detected. May be use >> paravirt patching instead of jump label? What if jump label will want >> to use spinlock for some reason in the future (it uses mutex currently)? > > The point of the pv ticketlocks is to avoid any pvop calls on the > lock/unlock fastpath, relegating them to only the slow path. > Unfortunately, the pv unlock case can't be identical with the non-pv > unlock, and jump_labels are lighter weight and more efficient than pvops. > > It doesn't matter if jump_labels start using spinlocks; all we need the > jump_label machinery to do is patch the jump sites in the code so that > one of two execution paths can be selected. Since all the ticketlock > jump_label patching happens before SMP is enabled, there's no problem > with changing a lock while a cpu is executing the code. > I also felt agreeing with Jeremy. seemed to me that latter is more performance friendly. no?. (Hmm. Thinking.. By the way is it not that Jeremy's earlier version had implementation similar to what Gleb asked ?) -- 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/