Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757450AbZKKMBo (ORCPT ); Wed, 11 Nov 2009 07:01:44 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755345AbZKKMBn (ORCPT ); Wed, 11 Nov 2009 07:01:43 -0500 Received: from e23smtp03.au.ibm.com ([202.81.31.145]:58165 "EHLO e23smtp03.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757384AbZKKMBm (ORCPT ); Wed, 11 Nov 2009 07:01:42 -0500 Date: Wed, 11 Nov 2009 17:31:40 +0530 From: Gautham R Shenoy To: Peter Zijlstra , Nathan Fontenot , Benjamin Herrenschmidt Cc: Balbir Singh , linux-kernel@vger.kernel.org, Dipankar Sarma , Arun R Bharadwaj , Andrew Morton , Vaidyanathan Srinivasan , linuxppc-dev@lists.ozlabs.org, Ingo Molnar Subject: Re: [PATCH v5 0/4] pseries: Add cede support for cpu-offline Message-ID: <20091111120140.GA11317@in.ibm.com> Reply-To: ego@in.ibm.com References: <20091030052106.25493.42109.stgit@sofia.in.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20091030052106.25493.42109.stgit@sofia.in.ibm.com> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3405 Lines: 87 On Fri, Oct 30, 2009 at 10:52:43AM +0530, Gautham R Shenoy wrote: > Hi, Hi Peter, Did you get a chance to look at this new design ? > > This is version 5 of patch series that provides a framework to choose the > state a pseries CPU must be put to when it is offlined. > > Previous versions can be found here: > Version 4: http://lkml.org/lkml/2009/10/9/59 > Version 3: http://lkml.org/lkml/2009/9/15/164 > Version 2: http://lkml.org/lkml/2009/8/28/102 > Version 1: http://lkml.org/lkml/2009/8/6/236 > > Changes from the previous version include: > - Rebased against Nathan Fontenot's latest "pseries kernel handling of dynamic > logical paritioning v4" patches found here: > http://lkml.org/lkml/2009/10/21/98 > > - Added boot-time option to disable putting the offlined vcpus into an > extended H_CEDE state. > > - Addressed Ben's comments regarding the if-else sequencing in > pseries_mach_cpu_die(). > > - Addition of comments for pseries_cpu_die() to distinguish it from > pseries_mach_cpu_die() > > Also, > > - This approach addresses Peter Z's objections regarding layering > violations. The user simply offlines the cpu and doesn't worry about what > state the CPU should be put into. That part is automatically handled by the > kernel. > > - It does not add any additional sysfs interface instead uses the existing > sysfs interface to offline CPUs. > > - On platforms which do not have support for ceding the vcpu with a > latency specifier value, the offlining mechanism defaults to the current > method of calling rtas_stop_self(). > > The patchset has been tested on the available pseries platforms and it works > as per the expectations. I believe that the patch set is ready for inclusion. > --- > > Gautham R Shenoy (4): > pseries: Serialize cpu hotplug operations during deactivate Vs deallocate > pseries: Add code to online/offline CPUs of a DLPAR node. > pSeries: Add hooks to put the CPU into an appropriate offline state > pSeries: extended_cede_processor() helper function. > > > Documentation/cpu-hotplug.txt | 6 + > arch/powerpc/include/asm/lppaca.h | 9 + > arch/powerpc/platforms/pseries/dlpar.c | 129 ++++++++++++++++ > arch/powerpc/platforms/pseries/hotplug-cpu.c | 182 ++++++++++++++++++++++- > arch/powerpc/platforms/pseries/offline_states.h | 18 ++ > arch/powerpc/platforms/pseries/plpar_wrappers.h | 22 +++ > arch/powerpc/platforms/pseries/smp.c | 19 ++ > arch/powerpc/xmon/xmon.c | 3 > drivers/base/cpu.c | 2 > include/linux/cpu.h | 13 ++ > 10 files changed, 387 insertions(+), 16 deletions(-) > create mode 100644 arch/powerpc/platforms/pseries/offline_states.h > > -- > Thanks and Regards > gautham. > -- > 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/ > -- Thanks and Regards gautham -- 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/