Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from
Apr 26, 2024

Conversation

ThomasKaiser
Copy link

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/[email protected]/#20763985

@ThomasKaiser
Copy link
Author

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)

@amazingfate amazingfate merged commit c744a54 into armbian:rk-5.10-rkr6 Apr 26, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants