armbian / linux-rockchip

Improved Rockchip Linux
Other
99 stars 142 forks source link

Add dynamic-power-coefficient properties to all cores #175

Closed ThomasKaiser closed 2 months ago

ThomasKaiser commented 2 months ago

Correct dynamic-power-coefficient properties for the cores are a prerequisit for schedutil/EAS to work correctly.

RK's 100/300 values are BS of course, just compare with RK3399: https://patchwork.kernel.org/project/linux-arm-kernel/patch/1500974575-2244-1-git-send-email-wxt@rock-chips.com/#20763985

ThomasKaiser commented 2 months ago

The capacity-dmips-mhz values are BS too. If this DMIPS nonsense should be followed then according to my own measurements dhrystone attests the A76 being not 1.9 times but 2.32 times faster than the A55 in RK3588[s], as such it should read for the A55:

capacity-dmips-mhz = <441>;

But ofc this is stupid since everybody (should) know that DMIPS from last century are something that doesn't represent anything we do today on these machines. And the whole idea of EAS on systems mixing in-order and out-of-order cores is flawed by design :)

But for schedutil to have at least a chance to somehow work correctly with EAS in mind some properties have to be set as best as possible (and RK's BSP defaults are just weird)