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

Cherry pick upstream bgfx commits needed to get wayland working #11539

Merged
merged 15 commits into from
Sep 28, 2023

Conversation

belegdol
Copy link
Contributor

As full upstream sync has resulted in numerous issues, particularly with macOS (see #11493), cherry-pick the commits needed for wayland support (#11451) instead.
I have tested this on AMD opensource driver for now. I will test a few more configs and remove the draft status when done.

issam3105 and others added 15 commits September 10, 2023 15:14
* allow headless context creation using EGL

* Fixed dynamic lib import

---------

Co-authored-by: Dahmen issam <issam.dahmen@fittingbox.com>
Co-authored-by: Dahmen issam <issam.dahmen@fittingbox.com>
* Fixed EGL library name on Linux

* Cleanup.

---------

Co-authored-by: Dahmen issam <issam.dahmen@fittingbox.com>
Co-authored-by: Бранимир Караџић <branimirkaradzic@gmail.com>
* Initial proof-of-concept of vulkan renderer working on wayland

* Get wayland surface from the window handle
* Allow simultaneous X11 and Wayland support

* Add NativeWindowHandleType

* Set default value for g_platformData.type

* Use g_platformData.type to check for a native Wayland window

* Stub getNativeWindowHandleType on platform where Wayland is not an option

* Implement getNativeWindowHandleType for GLFW

* Add getNativeWindowHandleType to the remaining C++ examples

* Add getNativeWindowHandleType to the C example
@belegdol
Copy link
Contributor Author

belegdol commented Sep 10, 2023

No immediate crashes on Fedora 38 running 535.xx nvidia driver or on windows 11.
xbr-lv3-multipass, xbr-lv4-dilation and xbr-lv4-multipass are acting up but this appears to be expected.

@belegdol
Copy link
Contributor Author

belegdol commented Sep 10, 2023

No crashes with normal shaders on Fedora 37 running 470.xx driver on a system with GeForce 680M either.

@belegdol belegdol marked this pull request as ready for review September 10, 2023 17:45
@cuavas
Copy link
Member

cuavas commented Sep 11, 2023

I’ve read the diff, and I think this is relatively safe. I’m still going to hold off until after freeze to merge it as we’re no longer early in a release cycle.

@cuavas cuavas merged commit 80155df into mamedev:master Sep 28, 2023
5 checks passed
@belegdol belegdol deleted the bgfx_wayland_cherry_pick branch September 28, 2023 15:23
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.

4 participants