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

[Micledore] backport patch: linux-raspberrypi: stop setting powersave as the default CPU governor #1243

Merged
merged 1 commit into from
Nov 16, 2023

Conversation

clopez
Copy link
Contributor

@clopez clopez commented Nov 9, 2023

This cherry-picks 0a4a68d from master into Mickledore branch

* The defconfigs from the RPi Kernel set `powersave` as the default
  CPU governor, which is a bad idea as that reduces performance by
  setting the CPU frequency to the minimum one.

* In case of a Yocto build user-space is not configured by default
  to change the CPU governor, so `powersave` will remain the CPU
  governor and it will slow down everything.

* I submitted a fix for those defconfigs to upstream here:
  raspberrypi/linux#5666
  However, we shouldn't wait until that is accepted and backported
  to all the stable branches (if ever is).

* This patch unsets the selection of `powersave` as the default CPU
  governor.

* Then the CPU governor that will be selected as default after this
  patch will be the Linux's default one that is either `ondemand` or
  `schedutil` depending on the Kernel version.

Signed-off-by: Carlos Alberto Lopez Perez <[email protected]>
Copy link
Owner

@agherzan agherzan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm happy with this backport. Approved and thanks for preparing the PR.

@clopez
Copy link
Contributor Author

clopez commented Nov 15, 2023

Thanks! But I don't have commit/push rights, need someone else to merge it for me

@agherzan
Copy link
Owner

Yes. I will deal with that - I'm just waiting for the CI to finish.

@agherzan agherzan enabled auto-merge (rebase) November 16, 2023 16:39
@agherzan agherzan merged commit f9af665 into agherzan:mickledore Nov 16, 2023
18 checks passed
webkit-commit-queue pushed a commit to clopez/WebKit that referenced this pull request Dec 14, 2023
…change CPU frequency governor for RPis

https://bugs.webkit.org/show_bug.cgi?id=266353

Reviewed by Philippe Normand.

This updates the Yocto layers used for the RPi build with the following major changes:

 - meta-webkit: include libbacktrace and libportal into the base build.
   libbacktrace is required by the default build of WPE now and libportal is required by Cog since bug 266351

 - meta-raspberrypi: stop setting "powersave" as the default CPU frequency governor and use
   "schedutil" instead. This will likely mean a big change in the performance results of all tests.
   See agherzan/meta-raspberrypi#1243 for context.

 - poky, meta-openembedded: just update to the last version of the Mickledore branch

 - backport a change landed in meta-openembedded master to the Mickledore branch that
   updates libbacktrace to allow building it on riscv64.

* Tools/yocto/meta-openembedded_backport-libbacktrace-patch-to-mickledore-branch.patch: Added.
* Tools/yocto/riscv/manifest.xml:
* Tools/yocto/rpi/manifest.xml:
* Tools/yocto/targets.conf:

Canonical link: https://commits.webkit.org/272030@main
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