Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752792Ab0HZLtd (ORCPT ); Thu, 26 Aug 2010 07:49:33 -0400 Received: from casper.infradead.org ([85.118.1.10]:49283 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751188Ab0HZLta convert rfc822-to-8bit (ORCPT ); Thu, 26 Aug 2010 07:49:30 -0400 Subject: Re: [PATCH] lglock: make lg_lock_global() actually lock globally From: Peter Zijlstra To: Nick Piggin Cc: Tejun Heo , Linus Torvalds , Jonathan Corbet , Rusty Russell , Al Viro , LKML In-Reply-To: <1282823139.1975.689.camel@laptop> References: <20100825132815.108d8217@bike.lwn.net> <4C762BF9.5010305@kernel.org> <20100826094613.GA6411@amd> <4C7638B7.9060306@kernel.org> <1282817329.1975.486.camel@laptop> <20100826113858.GA6856@amd> <1282823139.1975.689.camel@laptop> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT Date: Thu, 26 Aug 2010 13:49:00 +0200 Message-ID: <1282823340.1975.697.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 877 Lines: 19 On Thu, 2010-08-26 at 13:45 +0200, Peter Zijlstra wrote: > On Thu, 2010-08-26 at 21:38 +1000, Nick Piggin wrote: > > I think for CPU plug, stop_machine is reasonable (especially > > considering it is required in unload, which means any frequent > > amount of cpu plug activity already will require stop_machine to > > run anyway). > > How is it required? > > Its currently implemented as such, and its sure a lot easier to do that > way, but I could imagine that unplugging a CPU could be done without it. Not that reworking unplug to not use stop_machine is anywhere near my todo list, but it would be really nice to have ;-) -- 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/