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

[android] Enable low end device mode #4625

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

borongc
Copy link
Contributor

@borongc borongc commented Dec 23, 2024

  1. Enable low end device mode to reduce memory usage.
  2. The rendering issue was due to the memory limits in Blink, which is resolved in m126+, where two upstream commits ([cc] Simplify and adjust memory limits on Android #4636, [cc] Use simple compositor memory limits on Android #4637) were cherry picked.

b/380310632

base/system/sys_info.cc Outdated Show resolved Hide resolved
base/system/sys_info.cc Outdated Show resolved Hide resolved
@johnxwork
Copy link
Contributor

Is it possible you could run a quick local test and find out how much memory this switch can save us? It will be very helpful in evaluating the impact and tradeoff in the future.

@borongc
Copy link
Contributor Author

borongc commented Dec 26, 2024

@johnxwork See the comment in the bug. This is about 22.3% reduction on the memory footprint with low-end-device-mode is enabled.

@borongc borongc force-pushed the cobalt_low_end_device branch 3 times, most recently from fcb9a08 to 66693ec Compare January 2, 2025 09:07
@borongc borongc requested a review from andrewsavage1 January 2, 2025 09:09
1. Enable low end device mode to reduce memory usage.
2. Exclude to limit AmountOfPhysicalMemory on low end device mode, which causes the rendering issue.

b/380310632
@borongc borongc force-pushed the cobalt_low_end_device branch from 66693ec to 20b1d41 Compare January 3, 2025 23:58
@borongc borongc changed the title Enable low end device mode [android] Enable low end device mode Jan 3, 2025
@borongc borongc requested a review from andrewsavage1 January 4, 2025 00:01
@borongc borongc merged commit 173ecc3 into youtube:main Jan 6, 2025
63 of 65 checks passed
@borongc borongc deleted the cobalt_low_end_device branch January 6, 2025 04:23
borongc added a commit that referenced this pull request Jan 7, 2025
RGBA_4444 doesn't work with overlay video mode.
- Disable it as it is enabled with low-end-device-mode,
#4625.

b/380310632
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.

3 participants