[PATCH v2] cpupower: Fix no-rounding MHz frequency output

[PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Prarit Bhargava]

'cpupower frequency-info -ln' returns kHz values on systems with MHz range
minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
the command returns

hardware limits: 800000 MHz - 4.200000 GHz

The code that causes this error can be removed.  The next else if clause
will handle the output correctly such that

hardware limits: 800.000 MHz - 4.200000 GHz

is displayed correctly.

[v2]: Remove two lines instead of fixing broken code.

Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Cc: Thomas Renninger <trenn@suse.com>
Cc: Stafford Horne <shorne@gmail.com>
Cc: Shuah Khan <shuah@kernel.org>
---
 tools/power/cpupower/utils/cpufreq-info.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
index 3e701f0e9c14..df43cd45d810 100644
--- a/tools/power/cpupower/utils/cpufreq-info.c
+++ b/tools/power/cpupower/utils/cpufreq-info.c
@@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
 		if (speed > 1000000)
 			printf("%u.%06u GHz", ((unsigned int) speed/1000000),
 				((unsigned int) speed%1000000));
-		else if (speed > 100000)
-			printf("%u MHz", (unsigned int) speed);
 		else if (speed > 1000)
 			printf("%u.%03u MHz", ((unsigned int) speed/1000),
 				(unsigned int) (speed%1000));
-- 
2.15.0.rc0.39.g2f0e14e64

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Stafford Horne]

On Wed, Oct 25, 2017 at 09:51:32AM -0400, Prarit Bhargava wrote:
> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
> the command returns
> 
> hardware limits: 800000 MHz - 4.200000 GHz
> 
> The code that causes this error can be removed.  The next else if clause
> will handle the output correctly such that
> 
> hardware limits: 800.000 MHz - 4.200000 GHz
> 
> is displayed correctly.
> 
> [v2]: Remove two lines instead of fixing broken code.
> 
> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> Cc: Thomas Renninger <trenn@suse.com>
> Cc: Stafford Horne <shorne@gmail.com>
> Cc: Shuah Khan <shuah@kernel.org>

Reviewed-by: Stafford Horne <shorne@gmail.com>

> ---
>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
> index 3e701f0e9c14..df43cd45d810 100644
> --- a/tools/power/cpupower/utils/cpufreq-info.c
> +++ b/tools/power/cpupower/utils/cpufreq-info.c
> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>  		if (speed > 1000000)
>  			printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>  				((unsigned int) speed%1000000));
> -		else if (speed > 100000)
> -			printf("%u MHz", (unsigned int) speed);
>  		else if (speed > 1000)
>  			printf("%u.%03u MHz", ((unsigned int) speed/1000),
>  				(unsigned int) (speed%1000));
> -- 
> 2.15.0.rc0.39.g2f0e14e64
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
> the command returns
> 
> hardware limits: 800000 MHz - 4.200000 GHz
> 
> The code that causes this error can be removed.  The next else if clause
> will handle the output correctly such that
> 
> hardware limits: 800.000 MHz - 4.200000 GHz
> 
> is displayed correctly.
> 
> [v2]: Remove two lines instead of fixing broken code.
> 
> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
> Cc: Thomas Renninger <trenn@suse.com>
> Cc: Stafford Horne <shorne@gmail.com>
> Cc: Shuah Khan <shuah@kernel.org>
> ---
>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
> index 3e701f0e9c14..df43cd45d810 100644
> --- a/tools/power/cpupower/utils/cpufreq-info.c
> +++ b/tools/power/cpupower/utils/cpufreq-info.c
> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>  		if (speed > 1000000)
>  			printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>  				((unsigned int) speed%1000000));
> -		else if (speed > 100000)
> -			printf("%u MHz", (unsigned int) speed);
>  		else if (speed > 1000)
>  			printf("%u.%03u MHz", ((unsigned int) speed/1000),
>  				(unsigned int) (speed%1000));
> 

Thanks.  I will queue this up for 4.15-rc1.

-- Shuah

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 10/25/2017 04:01 PM, Stafford Horne wrote:
> On Wed, Oct 25, 2017 at 09:51:32AM -0400, Prarit Bhargava wrote:
>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>> the command returns
>>
>> hardware limits: 800000 MHz - 4.200000 GHz
>>
>> The code that causes this error can be removed.  The next else if clause
>> will handle the output correctly such that
>>
>> hardware limits: 800.000 MHz - 4.200000 GHz
>>
>> is displayed correctly.
>>
>> [v2]: Remove two lines instead of fixing broken code.
>>
>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>> Cc: Thomas Renninger <trenn@suse.com>
>> Cc: Stafford Horne <shorne@gmail.com>
>> Cc: Shuah Khan <shuah@kernel.org>
> 
> Reviewed-by: Stafford Horne <shorne@gmail.com>

Thanks for the review.

-- Shuah

> 
>> ---
>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>> index 3e701f0e9c14..df43cd45d810 100644
>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>  		if (speed > 1000000)
>>  			printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>  				((unsigned int) speed%1000000));
>> -		else if (speed > 100000)
>> -			printf("%u MHz", (unsigned int) speed);
>>  		else if (speed > 1000)
>>  			printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>  				(unsigned int) (speed%1000));
>> -- 
>> 2.15.0.rc0.39.g2f0e14e64
>>
> 
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Rafael J. Wysocki]

On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>> the command returns
>>
>> hardware limits: 800000 MHz - 4.200000 GHz
>>
>> The code that causes this error can be removed.  The next else if clause
>> will handle the output correctly such that
>>
>> hardware limits: 800.000 MHz - 4.200000 GHz
>>
>> is displayed correctly.
>>
>> [v2]: Remove two lines instead of fixing broken code.
>>
>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>> Cc: Thomas Renninger <trenn@suse.com>
>> Cc: Stafford Horne <shorne@gmail.com>
>> Cc: Shuah Khan <shuah@kernel.org>
>> ---
>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>> index 3e701f0e9c14..df43cd45d810 100644
>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>               if (speed > 1000000)
>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>                               ((unsigned int) speed%1000000));
>> -             else if (speed > 100000)
>> -                     printf("%u MHz", (unsigned int) speed);
>>               else if (speed > 1000)
>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>                               (unsigned int) (speed%1000));
>>
>
> Thanks.  I will queue this up for 4.15-rc1.

OK

So are you going to maintain this utility going forward?

Thanks,
Rafael

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>> the command returns
>>>
>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>
>>> The code that causes this error can be removed.  The next else if clause
>>> will handle the output correctly such that
>>>
>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>
>>> is displayed correctly.
>>>
>>> [v2]: Remove two lines instead of fixing broken code.
>>>
>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>> Cc: Thomas Renninger <trenn@suse.com>
>>> Cc: Stafford Horne <shorne@gmail.com>
>>> Cc: Shuah Khan <shuah@kernel.org>
>>> ---
>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>  1 file changed, 2 deletions(-)
>>>
>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>> index 3e701f0e9c14..df43cd45d810 100644
>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>               if (speed > 1000000)
>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>                               ((unsigned int) speed%1000000));
>>> -             else if (speed > 100000)
>>> -                     printf("%u MHz", (unsigned int) speed);
>>>               else if (speed > 1000)
>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>                               (unsigned int) (speed%1000));
>>>
>>
>> Thanks.  I will queue this up for 4.15-rc1.
> 
> OK
> 
> So are you going to maintain this utility going forward?
> 

oops. I was on auto-pilot responding to patches sitting in my Inbox.
Wrong email response. Sorry about that.

Please ignore. Mu bad.

thanks,
-- Shuah

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Rafael J. Wysocki]

On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>> the command returns
>>>>
>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>
>>>> The code that causes this error can be removed.  The next else if clause
>>>> will handle the output correctly such that
>>>>
>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>
>>>> is displayed correctly.
>>>>
>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>
>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>> ---
>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>  1 file changed, 2 deletions(-)
>>>>
>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>               if (speed > 1000000)
>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>                               ((unsigned int) speed%1000000));
>>>> -             else if (speed > 100000)
>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>               else if (speed > 1000)
>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>                               (unsigned int) (speed%1000));
>>>>
>>>
>>> Thanks.  I will queue this up for 4.15-rc1.
>>
>> OK
>>
>> So are you going to maintain this utility going forward?
>>
>
> oops. I was on auto-pilot responding to patches sitting in my Inbox.
> Wrong email response. Sorry about that.
>
> Please ignore. Mu bad.

OK :-)

But that said, from my perspective, cpupower is basically not maintained.

Thomas, who sort of maintained it, but then basically became a patch
reviewer for it, does not respond to patches any more and I am not
sufficiently familiar with the code to be able to effectively review
the patches myself, nor I have the time to get more familiar with it.

For this reason, I'm inclined to drop this code from the kernel source
tree unless somebody steps in to fill the gap.

Thanks,
Rafael

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>> the command returns
>>>>>
>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>
>>>>> The code that causes this error can be removed.  The next else if clause
>>>>> will handle the output correctly such that
>>>>>
>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>
>>>>> is displayed correctly.
>>>>>
>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>
>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>> ---
>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>  1 file changed, 2 deletions(-)
>>>>>
>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>               if (speed > 1000000)
>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>                               ((unsigned int) speed%1000000));
>>>>> -             else if (speed > 100000)
>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>               else if (speed > 1000)
>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>                               (unsigned int) (speed%1000));
>>>>>
>>>>
>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>
>>> OK
>>>
>>> So are you going to maintain this utility going forward?
>>>
>>
>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>> Wrong email response. Sorry about that.
>>
>> Please ignore. Mu bad.
> 
> OK :-)
> 
> But that said, from my perspective, cpupower is basically not maintained.
> 
> Thomas, who sort of maintained it, but then basically became a patch
> reviewer for it, does not respond to patches any more and I am not
> sufficiently familiar with the code to be able to effectively review
> the patches myself, nor I have the time to get more familiar with it.
> 
> For this reason, I'm inclined to drop this code from the kernel source
> tree unless somebody steps in to fill the gap.
> 

Please don't drop this from kernel sources.. I think this is useful. If you
are looking for a maintainer, I will be happy to step up to maintain it.


thanks,
-- Shuah

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Rafael J. Wysocki]

On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>> the command returns
>>>>>>
>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>
>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>> will handle the output correctly such that
>>>>>>
>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>
>>>>>> is displayed correctly.
>>>>>>
>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>
>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>> ---
>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>  1 file changed, 2 deletions(-)
>>>>>>
>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>               if (speed > 1000000)
>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>                               ((unsigned int) speed%1000000));
>>>>>> -             else if (speed > 100000)
>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>               else if (speed > 1000)
>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>                               (unsigned int) (speed%1000));
>>>>>>
>>>>>
>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>
>>>> OK
>>>>
>>>> So are you going to maintain this utility going forward?
>>>>
>>>
>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>> Wrong email response. Sorry about that.
>>>
>>> Please ignore. Mu bad.
>>
>> OK :-)
>>
>> But that said, from my perspective, cpupower is basically not maintained.
>>
>> Thomas, who sort of maintained it, but then basically became a patch
>> reviewer for it, does not respond to patches any more and I am not
>> sufficiently familiar with the code to be able to effectively review
>> the patches myself, nor I have the time to get more familiar with it.
>>
>> For this reason, I'm inclined to drop this code from the kernel source
>> tree unless somebody steps in to fill the gap.
>>
>
> Please don't drop this from kernel sources.. I think this is useful. If you
> are looking for a maintainer, I will be happy to step up to maintain it.

Cool, please do that then. :-)

I will be happy to take pull requests with cpupower changes so that
they go in along with the other PM material.

Thanks,
Rafael

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>> the command returns
>>>>>>>
>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>
>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>> will handle the output correctly such that
>>>>>>>
>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>
>>>>>>> is displayed correctly.
>>>>>>>
>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>
>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>> ---
>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>
>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>               if (speed > 1000000)
>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>> -             else if (speed > 100000)
>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>               else if (speed > 1000)
>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>
>>>>>>
>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>
>>>>> OK
>>>>>
>>>>> So are you going to maintain this utility going forward?
>>>>>
>>>>
>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>> Wrong email response. Sorry about that.
>>>>
>>>> Please ignore. Mu bad.
>>>
>>> OK :-)
>>>
>>> But that said, from my perspective, cpupower is basically not maintained.
>>>
>>> Thomas, who sort of maintained it, but then basically became a patch
>>> reviewer for it, does not respond to patches any more and I am not
>>> sufficiently familiar with the code to be able to effectively review
>>> the patches myself, nor I have the time to get more familiar with it.
>>>
>>> For this reason, I'm inclined to drop this code from the kernel source
>>> tree unless somebody steps in to fill the gap.
>>>
>>
>> Please don't drop this from kernel sources.. I think this is useful. If you
>> are looking for a maintainer, I will be happy to step up to maintain it.
> 
> Cool, please do that then. :-)
> 
> I will be happy to take pull requests with cpupower changes so that
> they go in along with the other PM material.
> 

Sounds like a plan. I can do that. I will work on getting git setup and
send pull requests. We can get that going for 4.15 unless you think it is
late for you to get pull requests.

thanks,
-- Shuah

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Rafael J. Wysocki]

On Wed, Nov 1, 2017 at 11:30 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
> On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
>> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>>> the command returns
>>>>>>>>
>>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>>
>>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>>> will handle the output correctly such that
>>>>>>>>
>>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>>
>>>>>>>> is displayed correctly.
>>>>>>>>
>>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>>
>>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>>> ---
>>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>>               if (speed > 1000000)
>>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>>> -             else if (speed > 100000)
>>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>>               else if (speed > 1000)
>>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>>
>>>>>>>
>>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>>
>>>>>> OK
>>>>>>
>>>>>> So are you going to maintain this utility going forward?
>>>>>>
>>>>>
>>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>>> Wrong email response. Sorry about that.
>>>>>
>>>>> Please ignore. Mu bad.
>>>>
>>>> OK :-)
>>>>
>>>> But that said, from my perspective, cpupower is basically not maintained.
>>>>
>>>> Thomas, who sort of maintained it, but then basically became a patch
>>>> reviewer for it, does not respond to patches any more and I am not
>>>> sufficiently familiar with the code to be able to effectively review
>>>> the patches myself, nor I have the time to get more familiar with it.
>>>>
>>>> For this reason, I'm inclined to drop this code from the kernel source
>>>> tree unless somebody steps in to fill the gap.
>>>>
>>>
>>> Please don't drop this from kernel sources.. I think this is useful. If you
>>> are looking for a maintainer, I will be happy to step up to maintain it.
>>
>> Cool, please do that then. :-)
>>
>> I will be happy to take pull requests with cpupower changes so that
>> they go in along with the other PM material.
>>
>
> Sounds like a plan. I can do that. I will work on getting git setup and
> send pull requests. We can get that going for 4.15 unless you think it is
> late for you to get pull requests.

That should be fine.  There still is a couple of days at least before
the merge window. :-)

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Prarit Bhargava]

On 11/01/2017 06:30 PM, Shuah Khan wrote:
> On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
>> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>>> the command returns
>>>>>>>>
>>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>>
>>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>>> will handle the output correctly such that
>>>>>>>>
>>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>>
>>>>>>>> is displayed correctly.
>>>>>>>>
>>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>>
>>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>>> ---
>>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>>               if (speed > 1000000)
>>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>>> -             else if (speed > 100000)
>>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>>               else if (speed > 1000)
>>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>>
>>>>>>>
>>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>>
>>>>>> OK
>>>>>>
>>>>>> So are you going to maintain this utility going forward?
>>>>>>
>>>>>
>>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>>> Wrong email response. Sorry about that.
>>>>>
>>>>> Please ignore. Mu bad.
>>>>
>>>> OK :-)
>>>>
>>>> But that said, from my perspective, cpupower is basically not maintained.
>>>>
>>>> Thomas, who sort of maintained it, but then basically became a patch
>>>> reviewer for it, does not respond to patches any more and I am not
>>>> sufficiently familiar with the code to be able to effectively review
>>>> the patches myself, nor I have the time to get more familiar with it.
>>>>
>>>> For this reason, I'm inclined to drop this code from the kernel source
>>>> tree unless somebody steps in to fill the gap.
>>>>
>>>
>>> Please don't drop this from kernel sources.. I think this is useful. If you
>>> are looking for a maintainer, I will be happy to step up to maintain it.
>>
>> Cool, please do that then. :-)
>>
>> I will be happy to take pull requests with cpupower changes so that
>> they go in along with the other PM material.
>>
> 
> Sounds like a plan. I can do that. I will work on getting git setup and
> send pull requests. We can get that going for 4.15 unless you think it is
> late for you to get pull requests.

Shuah, I have two other cleanup patches that should be applied to cpupower.  I
will post them shortly.

P.

> 
> thanks,
> -- Shuah
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 11/01/2017 06:42 PM, Prarit Bhargava wrote:
> 
> 
> On 11/01/2017 06:30 PM, Shuah Khan wrote:
>> On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
>>> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>>>> the command returns
>>>>>>>>>
>>>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>>>
>>>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>>>> will handle the output correctly such that
>>>>>>>>>
>>>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>>>
>>>>>>>>> is displayed correctly.
>>>>>>>>>
>>>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>>>> ---
>>>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>>>
>>>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>>>               if (speed > 1000000)
>>>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>>>> -             else if (speed > 100000)
>>>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>>>               else if (speed > 1000)
>>>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>>>
>>>>>>>>
>>>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>>>
>>>>>>> OK
>>>>>>>
>>>>>>> So are you going to maintain this utility going forward?
>>>>>>>
>>>>>>
>>>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>>>> Wrong email response. Sorry about that.
>>>>>>
>>>>>> Please ignore. Mu bad.
>>>>>
>>>>> OK :-)
>>>>>
>>>>> But that said, from my perspective, cpupower is basically not maintained.
>>>>>
>>>>> Thomas, who sort of maintained it, but then basically became a patch
>>>>> reviewer for it, does not respond to patches any more and I am not
>>>>> sufficiently familiar with the code to be able to effectively review
>>>>> the patches myself, nor I have the time to get more familiar with it.
>>>>>
>>>>> For this reason, I'm inclined to drop this code from the kernel source
>>>>> tree unless somebody steps in to fill the gap.
>>>>>
>>>>
>>>> Please don't drop this from kernel sources.. I think this is useful. If you
>>>> are looking for a maintainer, I will be happy to step up to maintain it.
>>>
>>> Cool, please do that then. :-)
>>>
>>> I will be happy to take pull requests with cpupower changes so that
>>> they go in along with the other PM material.
>>>
>>
>> Sounds like a plan. I can do that. I will work on getting git setup and
>> send pull requests. We can get that going for 4.15 unless you think it is
>> late for you to get pull requests.
> 
> Shuah, I have two other cleanup patches that should be applied to cpupower.  I
> will post them shortly.
> 

Hi Prarit,

Does this tool build for you? I am seeing:

utils/helpers/amd.c:7:21: fatal error: pci/pci.h: No such file or directory
 #include <pci/pci.h>
                     ^
compilation terminated.
Makefile:221: recipe for target 'utils/helpers/amd.o' failed
make: *** [utils/helpers/amd.o] Error 1

thanks,
-- Shuah

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Prarit Bhargava]

On 11/02/2017 02:59 PM, Shuah Khan wrote:
> On 11/01/2017 06:42 PM, Prarit Bhargava wrote:
>>
>>
>> On 11/01/2017 06:30 PM, Shuah Khan wrote:
>>> On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
>>>> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>>>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>>>>> the command returns
>>>>>>>>>>
>>>>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>>>>
>>>>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>>>>> will handle the output correctly such that
>>>>>>>>>>
>>>>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>>>>
>>>>>>>>>> is displayed correctly.
>>>>>>>>>>
>>>>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>>>>
>>>>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>>>>> ---
>>>>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>>>>
>>>>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>>>>               if (speed > 1000000)
>>>>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>>>>> -             else if (speed > 100000)
>>>>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>>>>               else if (speed > 1000)
>>>>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>>>>
>>>>>>>> OK
>>>>>>>>
>>>>>>>> So are you going to maintain this utility going forward?
>>>>>>>>
>>>>>>>
>>>>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>>>>> Wrong email response. Sorry about that.
>>>>>>>
>>>>>>> Please ignore. Mu bad.
>>>>>>
>>>>>> OK :-)
>>>>>>
>>>>>> But that said, from my perspective, cpupower is basically not maintained.
>>>>>>
>>>>>> Thomas, who sort of maintained it, but then basically became a patch
>>>>>> reviewer for it, does not respond to patches any more and I am not
>>>>>> sufficiently familiar with the code to be able to effectively review
>>>>>> the patches myself, nor I have the time to get more familiar with it.
>>>>>>
>>>>>> For this reason, I'm inclined to drop this code from the kernel source
>>>>>> tree unless somebody steps in to fill the gap.
>>>>>>
>>>>>
>>>>> Please don't drop this from kernel sources.. I think this is useful. If you
>>>>> are looking for a maintainer, I will be happy to step up to maintain it.
>>>>
>>>> Cool, please do that then. :-)
>>>>
>>>> I will be happy to take pull requests with cpupower changes so that
>>>> they go in along with the other PM material.
>>>>
>>>
>>> Sounds like a plan. I can do that. I will work on getting git setup and
>>> send pull requests. We can get that going for 4.15 unless you think it is
>>> late for you to get pull requests.
>>
>> Shuah, I have two other cleanup patches that should be applied to cpupower.  I
>> will post them shortly.
>>
> 
> Hi Prarit,
> 
> Does this tool build for you? I am seeing:
> 
> utils/helpers/amd.c:7:21: fatal error: pci/pci.h: No such file or directory
>  #include <pci/pci.h>
>                      ^
> compilation terminated.
> Makefile:221: recipe for target 'utils/helpers/amd.o' failed
> make: *** [utils/helpers/amd.o] Error 1
> 

It does not build on latest due to some other unrelated change.  I'm looking at
it now.

P.

> thanks,
> -- Shuah
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Prarit Bhargava]

On 11/02/2017 02:59 PM, Shuah Khan wrote:
> On 11/01/2017 06:42 PM, Prarit Bhargava wrote:
>>
>>
>> On 11/01/2017 06:30 PM, Shuah Khan wrote:
>>> On 11/01/2017 04:07 PM, Rafael J. Wysocki wrote:
>>>> On Wed, Nov 1, 2017 at 10:49 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>> On 11/01/2017 03:46 PM, Rafael J. Wysocki wrote:
>>>>>> On Wed, Nov 1, 2017 at 10:38 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>>>> On 11/01/2017 03:33 PM, Rafael J. Wysocki wrote:
>>>>>>>> On Wed, Nov 1, 2017 at 10:00 PM, Shuah Khan <shuah@kernel.org> wrote:
>>>>>>>>> On 10/25/2017 07:51 AM, Prarit Bhargava wrote:
>>>>>>>>>> 'cpupower frequency-info -ln' returns kHz values on systems with MHz range
>>>>>>>>>> minimum CPU frequency range.  For example, on a 800MHz to 4.20GHz system
>>>>>>>>>> the command returns
>>>>>>>>>>
>>>>>>>>>> hardware limits: 800000 MHz - 4.200000 GHz
>>>>>>>>>>
>>>>>>>>>> The code that causes this error can be removed.  The next else if clause
>>>>>>>>>> will handle the output correctly such that
>>>>>>>>>>
>>>>>>>>>> hardware limits: 800.000 MHz - 4.200000 GHz
>>>>>>>>>>
>>>>>>>>>> is displayed correctly.
>>>>>>>>>>
>>>>>>>>>> [v2]: Remove two lines instead of fixing broken code.
>>>>>>>>>>
>>>>>>>>>> Signed-off-by: Prarit Bhargava <prarit@redhat.com>
>>>>>>>>>> Cc: Thomas Renninger <trenn@suse.com>
>>>>>>>>>> Cc: Stafford Horne <shorne@gmail.com>
>>>>>>>>>> Cc: Shuah Khan <shuah@kernel.org>
>>>>>>>>>> ---
>>>>>>>>>>  tools/power/cpupower/utils/cpufreq-info.c | 2 --
>>>>>>>>>>  1 file changed, 2 deletions(-)
>>>>>>>>>>
>>>>>>>>>> diff --git a/tools/power/cpupower/utils/cpufreq-info.c b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> index 3e701f0e9c14..df43cd45d810 100644
>>>>>>>>>> --- a/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> +++ b/tools/power/cpupower/utils/cpufreq-info.c
>>>>>>>>>> @@ -93,8 +93,6 @@ static void print_speed(unsigned long speed)
>>>>>>>>>>               if (speed > 1000000)
>>>>>>>>>>                       printf("%u.%06u GHz", ((unsigned int) speed/1000000),
>>>>>>>>>>                               ((unsigned int) speed%1000000));
>>>>>>>>>> -             else if (speed > 100000)
>>>>>>>>>> -                     printf("%u MHz", (unsigned int) speed);
>>>>>>>>>>               else if (speed > 1000)
>>>>>>>>>>                       printf("%u.%03u MHz", ((unsigned int) speed/1000),
>>>>>>>>>>                               (unsigned int) (speed%1000));
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Thanks.  I will queue this up for 4.15-rc1.
>>>>>>>>
>>>>>>>> OK
>>>>>>>>
>>>>>>>> So are you going to maintain this utility going forward?
>>>>>>>>
>>>>>>>
>>>>>>> oops. I was on auto-pilot responding to patches sitting in my Inbox.
>>>>>>> Wrong email response. Sorry about that.
>>>>>>>
>>>>>>> Please ignore. Mu bad.
>>>>>>
>>>>>> OK :-)
>>>>>>
>>>>>> But that said, from my perspective, cpupower is basically not maintained.
>>>>>>
>>>>>> Thomas, who sort of maintained it, but then basically became a patch
>>>>>> reviewer for it, does not respond to patches any more and I am not
>>>>>> sufficiently familiar with the code to be able to effectively review
>>>>>> the patches myself, nor I have the time to get more familiar with it.
>>>>>>
>>>>>> For this reason, I'm inclined to drop this code from the kernel source
>>>>>> tree unless somebody steps in to fill the gap.
>>>>>>
>>>>>
>>>>> Please don't drop this from kernel sources.. I think this is useful. If you
>>>>> are looking for a maintainer, I will be happy to step up to maintain it.
>>>>
>>>> Cool, please do that then. :-)
>>>>
>>>> I will be happy to take pull requests with cpupower changes so that
>>>> they go in along with the other PM material.
>>>>
>>>
>>> Sounds like a plan. I can do that. I will work on getting git setup and
>>> send pull requests. We can get that going for 4.15 unless you think it is
>>> late for you to get pull requests.
>>
>> Shuah, I have two other cleanup patches that should be applied to cpupower.  I
>> will post them shortly.
>>
> 
> Hi Prarit,
> 
> Does this tool build for you? I am seeing:
> 
> utils/helpers/amd.c:7:21: fatal error: pci/pci.h: No such file or directory
>  #include <pci/pci.h>
>                      ^
> compilation terminated.
> Makefile:221: recipe for target 'utils/helpers/amd.o' failed
> make: *** [utils/helpers/amd.o] Error 1

This is strange.  I saw it, tried a git-bisect and now I can't reproduce it.
I'll check out a new tree and see if I can make this reproduce.

P.

> 
> thanks,
> -- Shuah
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Prarit Bhargava]

On 11/02/2017 02:59 PM, Shuah Khan wrote:
> On 11/01/2017 06:42 PM, Prarit Bhargava wrote:
> 
> Hi Prarit,
> 
> Does this tool build for you? I am seeing:
> 
> utils/helpers/amd.c:7:21: fatal error: pci/pci.h: No such file or directory
>  #include <pci/pci.h>
>                      ^
> compilation terminated.
> Makefile:221: recipe for target 'utils/helpers/amd.o' failed
> make: *** [utils/helpers/amd.o] Error 1
> 

Got it ... you have to have the pciutils-devel package installed which
installs /usr/include/pci/pci.h.

That file contains:

struct pci_access {
  /* Options you can change: */
  unsigned int method;                  /* Access method */
  int writeable;                        /* Open in read/write mode */
  int buscentric;                       /* Bus-centric view of the world */
<snip>

P.

> thanks,
> -- Shuah
> 

Re: [PATCH v2] cpupower: Fix no-rounding MHz frequency output

[Shuah Khan]

On 11/02/2017 02:18 PM, Prarit Bhargava wrote:
> 
> 
> On 11/02/2017 02:59 PM, Shuah Khan wrote:
>> On 11/01/2017 06:42 PM, Prarit Bhargava wrote:
>>
>> Hi Prarit,
>>
>> Does this tool build for you? I am seeing:
>>
>> utils/helpers/amd.c:7:21: fatal error: pci/pci.h: No such file or directory
>>  #include <pci/pci.h>
>>                      ^
>> compilation terminated.
>> Makefile:221: recipe for target 'utils/helpers/amd.o' failed
>> make: *** [utils/helpers/amd.o] Error 1
>>
> 
> Got it ... you have to have the pciutils-devel package installed which
> installs /usr/include/pci/pci.h.
> 
> That file contains:
> 
> struct pci_access {
>   /* Options you can change: */
>   unsigned int method;                  /* Access method */
>   int writeable;                        /* Open in read/write mode */
>   int buscentric;                       /* Bus-centric view of the world */
> <snip>
> 

Thanks for looking into this.

-- Shuah