* [RFD] Remove the userspace governor and the cooling device set state sysfs entry
@ 2021-09-22 9:59 Daniel Lezcano
2021-09-30 10:10 ` Lukasz Luba
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Lezcano @ 2021-09-22 9:59 UTC (permalink / raw)
To: Zhang Rui, Rafael J. Wysocki, Linux Kernel Mailing List,
Linux PM mailing list
Hi,
the userspace governor is sending temperature when polling is active and
trip point crossed events. Nothing else.
In the other side, the cooling device have their cooling device
set_cur_state read-writable all the time.
The thermal framework is wrongly used by userspace as a power capping
framework by acting on the cooling device opaque state. This one then
competes with the in-kernel governor decision.
As the new netlink thermal notification is able to provide the same
information than the userspace governor.
I propose to remove the userspace governor and the cur_state entry in
the sysfs exported file.
The DTPM framework is the right framework to do power capping and
moreover it deals with the aggregation via the dev pm qos.
Does it make sense ?
-- Daniel
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFD] Remove the userspace governor and the cooling device set state sysfs entry
2021-09-22 9:59 [RFD] Remove the userspace governor and the cooling device set state sysfs entry Daniel Lezcano
@ 2021-09-30 10:10 ` Lukasz Luba
2021-10-01 10:30 ` Daniel Lezcano
0 siblings, 1 reply; 4+ messages in thread
From: Lukasz Luba @ 2021-09-30 10:10 UTC (permalink / raw)
To: Daniel Lezcano
Cc: Zhang Rui, Rafael J. Wysocki, Linux Kernel Mailing List,
Linux PM mailing list
Hi Daniel,
On 9/22/21 10:59 AM, Daniel Lezcano wrote:
>
> Hi,
>
> the userspace governor is sending temperature when polling is active and
> trip point crossed events. Nothing else.
>
> In the other side, the cooling device have their cooling device
> set_cur_state read-writable all the time.
>
> The thermal framework is wrongly used by userspace as a power capping
> framework by acting on the cooling device opaque state. This one then
> competes with the in-kernel governor decision.
>
> As the new netlink thermal notification is able to provide the same
> information than the userspace governor.
>
> I propose to remove the userspace governor and the cur_state entry in
> the sysfs exported file.
>
> The DTPM framework is the right framework to do power capping and
> moreover it deals with the aggregation via the dev pm qos.
>
> Does it make sense ?
It sounds that we should be OK with the information from netlink.
I don't see objections. We can also extend the netlink packet when
needed. I'm fine with removing the user-space governor.
Regards,
Lukasz
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFD] Remove the userspace governor and the cooling device set state sysfs entry
2021-09-30 10:10 ` Lukasz Luba
@ 2021-10-01 10:30 ` Daniel Lezcano
2021-10-01 11:11 ` Lukasz Luba
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Lezcano @ 2021-10-01 10:30 UTC (permalink / raw)
To: Lukasz Luba
Cc: Zhang Rui, Rafael J. Wysocki, Linux Kernel Mailing List,
Linux PM mailing list
Hi Lukasz,
On 30/09/2021 12:10, Lukasz Luba wrote:
> Hi Daniel,
>
> On 9/22/21 10:59 AM, Daniel Lezcano wrote:
>>
>> Hi,
>>
>> the userspace governor is sending temperature when polling is active and
>> trip point crossed events. Nothing else.
>>
>> In the other side, the cooling device have their cooling device
>> set_cur_state read-writable all the time.
>>
>> The thermal framework is wrongly used by userspace as a power capping
>> framework by acting on the cooling device opaque state. This one then
>> competes with the in-kernel governor decision.
>>
>> As the new netlink thermal notification is able to provide the same
>> information than the userspace governor.
>>
>> I propose to remove the userspace governor and the cur_state entry in
>> the sysfs exported file.
>>
>> The DTPM framework is the right framework to do power capping and
>> moreover it deals with the aggregation via the dev pm qos.
>>
>> Does it make sense ?
>
> It sounds that we should be OK with the information from netlink.
> I don't see objections. We can also extend the netlink packet when
> needed. I'm fine with removing the user-space governor.
thank you for your answer. I'll propose a patch with a warn when the
userspace governor is registered in order to let know people the
governor is deprecated.
Probably, it would make sense to emit a warning also when the cooling
device state is set from userspace.
For the next release, we should be able to drop the userspace governor
and make the cooling device state read-only.
Does it make sense ?
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFD] Remove the userspace governor and the cooling device set state sysfs entry
2021-10-01 10:30 ` Daniel Lezcano
@ 2021-10-01 11:11 ` Lukasz Luba
0 siblings, 0 replies; 4+ messages in thread
From: Lukasz Luba @ 2021-10-01 11:11 UTC (permalink / raw)
To: Daniel Lezcano
Cc: Zhang Rui, Rafael J. Wysocki, Linux Kernel Mailing List,
Linux PM mailing list
On 10/1/21 11:30 AM, Daniel Lezcano wrote:
>
> Hi Lukasz,
>
> On 30/09/2021 12:10, Lukasz Luba wrote:
>> Hi Daniel,
>>
>> On 9/22/21 10:59 AM, Daniel Lezcano wrote:
>>>
>>> Hi,
>>>
>>> the userspace governor is sending temperature when polling is active and
>>> trip point crossed events. Nothing else.
>>>
>>> In the other side, the cooling device have their cooling device
>>> set_cur_state read-writable all the time.
>>>
>>> The thermal framework is wrongly used by userspace as a power capping
>>> framework by acting on the cooling device opaque state. This one then
>>> competes with the in-kernel governor decision.
>>>
>>> As the new netlink thermal notification is able to provide the same
>>> information than the userspace governor.
>>>
>>> I propose to remove the userspace governor and the cur_state entry in
>>> the sysfs exported file.
>>>
>>> The DTPM framework is the right framework to do power capping and
>>> moreover it deals with the aggregation via the dev pm qos.
>>>
>>> Does it make sense ?
>>
>> It sounds that we should be OK with the information from netlink.
>> I don't see objections. We can also extend the netlink packet when
>> needed. I'm fine with removing the user-space governor.
>
> thank you for your answer. I'll propose a patch with a warn when the
> userspace governor is registered in order to let know people the
> governor is deprecated.
>
> Probably, it would make sense to emit a warning also when the cooling
> device state is set from userspace.
>
> For the next release, we should be able to drop the userspace governor
> and make the cooling device state read-only.
>
> Does it make sense ?
>
>
Yes, it makes sense.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-01 11:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-22 9:59 [RFD] Remove the userspace governor and the cooling device set state sysfs entry Daniel Lezcano
2021-09-30 10:10 ` Lukasz Luba
2021-10-01 10:30 ` Daniel Lezcano
2021-10-01 11:11 ` Lukasz Luba
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.