From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhang Rui Date: Mon, 04 Feb 2013 01:44:36 +0000 Subject: Re: [patch] PM: intel_powerclamp: off by one in start_power_clamp() Message-Id: <1359942276.2250.32.camel@rzhang1-mobl4> List-Id: References: <20130124085122.GA3547@elgon.mountain> In-Reply-To: <20130124085122.GA3547@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: linux-pm@vger.kernel.org, Jacob Pan , Arjan van de Ven , kernel-janitors@vger.kernel.org On Thu, 2013-01-24 at 11:51 +0300, Dan Carpenter wrote: > This value has already been clamped correctly to 0 through 49 in > powerclamp_set_cur_state() so this patch doesn't actually change > anything. But we should fix it anyway for consistency. > > set_target_ratio is used as an offset into an array with > MAX_TARGET_RATIO (50) elements. > > Signed-off-by: Dan Carpenter applied to thermal-next. BTW, Jacob, it would be good to get your comments on this as you're the author of this driver. thanks, rui > > diff --git a/drivers/thermal/intel_powerclamp.c b/drivers/thermal/intel_powerclamp.c > index a85ff38..ab3ed90 100644 > --- a/drivers/thermal/intel_powerclamp.c > +++ b/drivers/thermal/intel_powerclamp.c > @@ -504,7 +504,7 @@ static int start_power_clamp(void) > return -EINVAL; > } > > - set_target_ratio = clamp(set_target_ratio, 0U, MAX_TARGET_RATIO); > + set_target_ratio = clamp(set_target_ratio, 0U, MAX_TARGET_RATIO - 1); > /* prevent cpu hotplug */ > get_online_cpus(); >