Re: [PATCH v6 0/3] PM: Intel PowerClamp driver

From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Jacob Pan <jacob.jun.pan@linux.intel.com>,
	Zhang Rui <rui.zhang@intel.com>
Cc: Linux PM <linux-pm@vger.kernel.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Rafael Wysocki <rafael.j.wysocki@intel.com>,
	Len Brown <len.brown@intel.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@elte.hu>,
	Joe Perches <joe@perches.com>, Rob Landley <rob@landley.net>,
	Arjan van de Ven <arjan@linux.intel.com>,
	Paul McKenney <paulmck@linux.vnet.ibm.com>
Subject: Re: [PATCH v6 0/3] PM: Intel PowerClamp driver
Date: Fri, 11 Jan 2013 23:37:27 +0100	[thread overview]
Message-ID: <11047380.k1dbzbnuXT@vostro.rjw.lan> (raw)
In-Reply-To: <1357297965-17839-1-git-send-email-jacob.jun.pan@linux.intel.com>

On Friday, January 04, 2013 03:12:42 AM Jacob Pan wrote:
> v6 changes:
> - clamp module parameters duration and window size, reword warning
>   messages when input parameters are out of range.
> 
> We have done some experiment with idle injection on Intel platforms.
> The idea is to use the increasingly power efficient package level
> C-states for power capping and passive thermal control.
> 
> Documentation is included in the patch to explain the theory of
> operation, performance implication, calibration, scalability, and user
> interface. Please refer to the following file for more details.
> 
> Documentation/thermal/intel_powerclamp.txt
> 
> Arjan van de Ven created the original idea and driver, I have been
> refining driver in hope that they can be to be useful beyond a proof
> of concept.
> 
> Jacob Pan (3):
>   tick: export nohz tick idle symbols for module use
>   x86/nmi: export local_touch_nmi() symbol for modules
>   PM: Introduce Intel PowerClamp Driver
> 
>  Documentation/thermal/intel_powerclamp.txt |  307 +++++++++++
>  arch/x86/kernel/nmi.c                      |    1 +
>  drivers/thermal/Kconfig                    |   10 +
>  drivers/thermal/Makefile                   |    2 +
>  drivers/thermal/intel_powerclamp.c         |  788 ++++++++++++++++++++++++++++
>  kernel/time/tick-sched.c                   |    2 +
>  6 files changed, 1110 insertions(+)
>  create mode 100644 Documentation/thermal/intel_powerclamp.txt
>  create mode 100644 drivers/thermal/intel_powerclamp.c

I'm not seeing any more comments here, so I think this is suitable for v3.9.

Rui, are you going to take it or do you want me to?

Rafael

-- 
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.