From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org (bombadil.infradead.org [18.85.46.34]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by bilbo.ozlabs.org (Postfix) with ESMTPS id 76ACEB6F34 for ; Fri, 7 Aug 2009 01:14:14 +1000 (EST) Subject: Re: [PATCH 0/3] cpu: idle state framework for offline CPUs. From: Peter Zijlstra To: svaidy@linux.vnet.ibm.com In-Reply-To: <1249570998.32113.499.camel@twins> References: <20090805142311.553.78286.stgit@sofia.in.ibm.com> <20090806015855.GA20596@sli10-desk.sh.intel.com> <20090806043351.GC14333@dirshya.in.ibm.com> <1249570998.32113.499.camel@twins> Content-Type: text/plain Date: Thu, 06 Aug 2009 17:13:42 +0200 Message-Id: <1249571622.32113.510.camel@twins> Mime-Version: 1.0 Cc: "Brown, Len" , Gautham R Shenoy , "linux-kernel@vger.kernel.org" , "Pallipadi, Venkatesh" , Shaohua Li , Ingo Molnar , "linuxppc-dev@lists.ozlabs.org" , "Darrick J. Wong" List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 2009-08-06 at 17:03 +0200, Peter Zijlstra wrote: > On Thu, 2009-08-06 at 10:03 +0530, Vaidyanathan Srinivasan wrote: > > This was the > > main objection to Venki's deepest sleep state for offline cpus patch. > > Well, my main objection was that is was a single raw function pointer > without any management layer around it. > > We have the exact same mess with the idle routine - and that has bitten > us in the past. That said, I still think it makes sense to share this with the idle routine, we can make some callbacks unsuitable for hot-unplug just fine.