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

cobalt/ force Starboard platform and EGL+GLES2 #4601

Closed

Conversation

yell0wd0g
Copy link
Contributor

@yell0wd0g yell0wd0g commented Dec 18, 2024

Extracted from @TyHolc 's #4538 , this CL just hardcodes two command line flags, namely to use the starboard Ozone platform and to force EGL for windowing API (which in turn forces GLES2 see here); using eGL is meant to avoid using or falling back to Angle.

Extracted from @TyHolc 's  #4538 , this CL just hardcodes two command line flags, namely to use the starboard Ozone platform and to force EGL+GLES2 for rendering graphics (to avoid using or falling back to Angle).
@yell0wd0g yell0wd0g requested a review from TyHolc December 18, 2024 00:43
@datadog-cobalt-youtube
Copy link

Datadog Report

Branch report: feature/yell0wd0g-ozone-starboard-and-egl-flags
Commit report: 25df1c4
Test service: cobalt

✅ 0 Failed, 11678 Passed, 0 Skipped, 2m 18.3s Total Time

@TyHolc
Copy link
Contributor

TyHolc commented Dec 18, 2024

Extracted from @TyHolc 's #4538 , this CL just hardcodes two command line flags, namely to use the starboard Ozone platform and to force EGL for windowing API (which in turn forces GLES2 see here); using eGL is meant to avoid using or falling back to Angle.

Yeah, should be fine to add those here. Note that until the other crashes are addressed that this will crash due to --ozone-platform=starboard (when normally it would default to x11)

@yell0wd0g
Copy link
Contributor Author

Had a better way in #4607 , let me close this CL in favour of that one.

@yell0wd0g yell0wd0g closed this Dec 18, 2024
yell0wd0g added a commit that referenced this pull request Dec 18, 2024
This CL adds the command line option --use-gl=egl if not present, or enforces that this is the value passed to the program. This is to avoid falling back to Angle, which is undesired at the moment.

This CL replaces #4601 which had a different approach.

Bug: b/371272304
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