Re: [PATCH] thermal: power_allocator: using round the division when re-divvying up power

2021-03-17 Thread gao yunxiao
Lukasz,Daniel, Thank you for your review! On 16/03/2021, Lukasz Luba wrote: > > > On 3/16/21 1:15 PM, Daniel Lezcano wrote: >> On 15/03/2021 10:51, Lukasz Luba wrote: >>> >>> >>> On 3/15/21 8:25 AM, gao.yunxi...@gmail.com wrote: From: "jeson.gao" The division is used directly in r

Re: [PATCH] thermal: power_allocator: using round the division when re-divvying up power

2021-03-16 Thread Lukasz Luba
On 3/16/21 1:15 PM, Daniel Lezcano wrote: On 15/03/2021 10:51, Lukasz Luba wrote: On 3/15/21 8:25 AM, gao.yunxi...@gmail.com wrote: From: "jeson.gao" The division is used directly in re-divvying up power, the decimal part will be discarded, devices will get less than the extra_actor_powe

Re: [PATCH] thermal: power_allocator: using round the division when re-divvying up power

2021-03-16 Thread Daniel Lezcano
On 15/03/2021 10:51, Lukasz Luba wrote: > > > On 3/15/21 8:25 AM, gao.yunxi...@gmail.com wrote: >> From: "jeson.gao" >> >> The division is used directly in re-divvying up power, the decimal >> part will >> be discarded, devices will get less than the extra_actor_power - 1. >> if using round the

Re: [PATCH] thermal: power_allocator: using round the division when re-divvying up power

2021-03-15 Thread Lukasz Luba
On 3/15/21 8:25 AM, gao.yunxi...@gmail.com wrote: From: "jeson.gao" The division is used directly in re-divvying up power, the decimal part will be discarded, devices will get less than the extra_actor_power - 1. if using round the division to make the calculation more accurate. For example

[PATCH] thermal: power_allocator: using round the division when re-divvying up power

2021-03-15 Thread gao . yunxiao6
From: "jeson.gao" The division is used directly in re-divvying up power, the decimal part will be discarded, devices will get less than the extra_actor_power - 1. if using round the division to make the calculation more accurate. For example: actor0 received more than it's max_power, it has the