Skip to content

drm-lease: Add Multi-GPU Support #10099

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Joel-Valenciano
Copy link

@Joel-Valenciano Joel-Valenciano commented Apr 17, 2025

Adds multi-GPU support for the wp_drm_lease_v1 protocol, and should fix the issue in #10053.

It seems to work fine on my system; I can start and stop monado several times without issues, but I'm not sure if it's fully ready to merge.

Turns PROTO::lease into an array of CDRMLeaseProtocol pointers, adds a backend parameter to the constructor for CDRMLeaseProtocol, and adds WP<CDRMLeaseDeviceResource> parameters to the constructors for a few resource types.

Resource freeing could be an issue, I had a few crashes with the onDestroy callbacks for at least the CDRMLeaseResource.

@vaxerski
Copy link
Member

check how wl_output globals are done. You should do something similar to that for each drm node (instead of a std;:string)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants