Messages in this thread | | | Date | Fri, 07 Feb 2014 15:39:32 -0500 | From | Prarit Bhargava <> | Subject | Re: [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; > } >
| |