Re: [RESEND PATCH 0/3 v4] powercap, intel_rapl, implement time window checks

From: Pandruvada, Srinivas
Date: Mon Mar 28 2016 - 14:07:58 EST


+Jacob

On Mon, 2016-03-21 at 08:27 -0400, Prarit Bhargava wrote:
> This was submitted back in December and not picked up after review.
>
> P.
>
> -----8<----
> The Intel ASDM provides a maximum time window that can be specified
> when
> setting a time window in the RAPL driver.ÂÂWhile the ASDM doesn't
> explicitly
> provide a minimum time window value, it does provide a minimum time
> window
> unit that also can be used as a minimum value.
>
> This patchset implements barrier checking for the time windows, and
> adds
> reporting of a known bug in which the maxmimum time window value may
> be
> erroneously set to 0, as well as a module parameter to avoid the
> maximum
> window checks on broken BIOSes.
>
> Cc: "Rafael J. Wysocki" <rafael.j.wysocki@xxxxxxxxx>
> Cc: Prarit Bhargava <prarit@xxxxxxxxxx>
> Cc: Radivoje Jovanovic <radivoje.jovanovic@xxxxxxxxx>
> Cc: Seiichi Ikarashi <s.ikarashi@xxxxxxxxxxxxxx>
> Cc: Mathias Krause <minipli@xxxxxxxxxxxxxx>
> Cc: Ajay Thomas <ajay.thomas.david.rajamanickam@xxxxxxxxx>
> Signed-off-by: Prarit Bhargava <prarit@xxxxxxxxxx>
>
> Prarit Bhargava (3):
> Â powercap, intel_rapl, implement get_max_time_window
> Â powercap, intel_rapl, implement check for minimum time window
> Â powercap, intel_rapl, Add ignore_max_time_window_check module
> ÂÂÂÂparameter for broken BIOSes
>
> Âdrivers/powercap/intel_rapl.cÂÂÂ|ÂÂÂ50
> +++++++++++++++++++++++++++++++++++++++
> Âdrivers/powercap/powercap_sys.c |ÂÂÂÂ6 +++--
> Â2 files changed, 54 insertions(+), 2 deletions(-)
>