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

Hw cursors fix for nvidia #6847

Merged
merged 3 commits into from
Jul 13, 2024
Merged

Hw cursors fix for nvidia #6847

merged 3 commits into from
Jul 13, 2024

Conversation

UjinT34
Copy link
Contributor

@UjinT34 UjinT34 commented Jul 11, 2024

Describe your PR, what does it fix/add?

Adds an option cursor:allow_dumb_copy to allow dumb copy from cursor resource buffer to gbm buffer if rendering to gbm fails.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

Works for me for the most part. Lacks some features (scaling at least). Only meant as a workaround to get hw cursors which is required to test direct scanout, vrr with hw cursors and explicit sync. Probably needs a better way of copying.

Is it ready for merging, or does it need work?

Requires hyprwm/aquamarine#12

@vaxerski
Copy link
Member

as I've said in the AQ MR, I'd rather use drm_dumb buffers for this if possible. Should be faster than a dataptr.

@UjinT34
Copy link
Contributor Author

UjinT34 commented Jul 11, 2024

I don't know how to use them and you don't seem to have enough time to deal with this issue.
Better to have somewhat slow solution than a missing feature which blocks other features.

@vaxerski
Copy link
Member

fair enough

@vaxerski vaxerski merged commit c612b3e into hyprwm:aquamarine Jul 13, 2024
4 of 11 checks passed
vaxerski pushed a commit that referenced this pull request Jul 13, 2024
* logging

* fallback to dump cursor copy

* unmap gbm buffer when done

Signed-off-by: Vaxry <[email protected]>
vaxerski pushed a commit that referenced this pull request Jul 13, 2024
* logging

* fallback to dump cursor copy

* unmap gbm buffer when done

Signed-off-by: Vaxry <[email protected]>
DawfukFR pushed a commit to DawfukFR/Hyprland that referenced this pull request Jul 14, 2024
* logging

* fallback to dump cursor copy

* unmap gbm buffer when done

Signed-off-by: Vaxry <[email protected]>
vaxerski pushed a commit that referenced this pull request Jul 20, 2024
* logging

* fallback to dump cursor copy

* unmap gbm buffer when done

Signed-off-by: Vaxry <[email protected]>
vaxerski pushed a commit that referenced this pull request Jul 20, 2024
* logging

* fallback to dump cursor copy

* unmap gbm buffer when done

Signed-off-by: Vaxry <[email protected]>
@gulafaran
Copy link
Contributor

@UjinT34 any chance we could get scaling into this?

@UjinT34
Copy link
Contributor Author

UjinT34 commented Aug 18, 2024

@gulafaran worked on #7040

I don't have access to nvidia hardware atm. And I am not really interested in HW cursors without VRR #7386
These workarounds are mostly meant for testing direct scanout and VRR. Don't really see a point in a proper implementation without working VRR.

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.

3 participants