lkml.org 
[lkml]   [2014]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] tools, cpupower Fix error condition in cmd_freq_set()


On 02/07/2014 01:55 PM, Prarit Bhargava wrote:
> On a system which has only 4.00GHz set as the only available frequency,
>
> [root@amd-pike-05 ~]# cpupower frequency-info
> <snip>
> current policy: frequency should be within 4.00 GHz and 4.00 GHz.
> The governor "userspace" may decide which speed to use
> within this range.
> <snip>
>
> setting the frequency to 2.8GHz does not return an error.
>
> [root@amd-pike-05 ~]# cpupower frequency-set --freq 2800MHz
> Setting cpu: 0
> Setting cpu: 1
> Setting cpu: 2
> Setting cpu: 3
> Setting cpu: 4
> Setting cpu: 5
> Setting cpu: 6
> Setting cpu: 7
> [root@amd-pike-05 ~]#
>
> The frequency set error should stop with an error message after the first
> failure. Fix the error return in cmd_freq_set() to read errors as less
> than zero.
>
> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> Cc: Dominik Brodowski <linux@dominikbrodowski.net>
> Cc: Thomas Renninger <trenn@suse.de>
> Cc: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
> Cc: Alan Cox <alan@linux.intel.com>
> Cc: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>
> ---
> tools/power/cpupower/utils/cpufreq-set.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/power/cpupower/utils/cpufreq-set.c b/tools/power/cpupower/utils/cpufreq-set.c
> index a416de8..e5f78623 100644
> --- a/tools/power/cpupower/utils/cpufreq-set.c
> +++ b/tools/power/cpupower/utils/cpufreq-set.c
> @@ -320,7 +320,7 @@ int cmd_freq_set(int argc, char **argv)
>
> printf(_("Setting cpu: %d\n"), cpu);
> ret = do_one_cpu(cpu, &new_pol, freq, policychange);
> - if (ret)
> + if (ret < 0)

Self-nack ... this won't work.

P.
> break;
> }
>


\
 
 \ /
  Last update: 2014-02-07 22:01    [W:0.033 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site