[Bug 109332] xf86-video-amdgpu - commit 0d60233d introduced stuttering with mouse cursor operations

[Bug 109332] xf86-video-amdgpu - commit 0d60233d introduced stuttering with mouse cursor operations

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 1020 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=109332

Michel Dänzer <michel@daenzer.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         QA Contact|xorg-team@lists.x.org       |
          Component|Driver/AMDgpu               |DRM/AMDgpu
           Assignee|xorg-driver-ati@lists.x.org |dri-devel@lists.freedesktop
                   |                            |.org
            Version|git                         |unspecified
                 CC|                            |harry.wentland@amd.com,
                   |                            |nicholas.kazlauskas@amd.com
            Product|xorg                        |DRI

--- Comment #1 from Michel Dänzer <michel@daenzer.net> ---
Still a DC issue. This is perfectly normal usage of the cursor UAPI, Wayland
compositors such as gnome-shell use it similarly.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 3009 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Bug 109332] xf86-video-amdgpu - commit 0d60233d introduced stuttering with mouse cursor operations

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 1114 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=109332

--- Comment #2 from Nicholas Kazlauskas <nicholas.kazlauskas@amd.com> ---
In this case it's not actually the mouse operations themselves causing the
stuttering but the framebuffer change. This is a known limitation with the
async plane updates API in DRM and not something we can really fix in DC until
there's a better API for async updates.

The page-flip IOCTL advertises itself as non-blocking but in reality it's not -
if there's any commit work touching the plane it's going to be blocked until
that finishes and that'll be any non asynchronous cursor update, gamma update,
property update, or anything else that occurs. This is what the stuttering
comes from.

So while it's a valid use of the cursor UAPI to double buffer cursors it's not
something I'd really recommend doing for an atomic driver with how DRM is
currently setup. Maybe this can be a configurable option in the meantime? I'm
not sure how much work that would be to add and maintain, though.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 1986 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Bug 109332] xf86-video-amdgpu - commit 0d60233d introduced stuttering with mouse cursor operations

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 510 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=109332

--- Comment #3 from Michel Dänzer <michel@daenzer.net> ---
(In reply to Nicholas Kazlauskas from comment #2)
> Maybe this can be a configurable option in the meantime?
> I'm not sure how much work that would be to add and maintain, though.

Yeah, I'm afraid I'm not really hot on adding an option for this. We can't ask
Wayland compositors to work around this, either.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 1442 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Bug 109332] xf86-video-amdgpu - commit 0d60233d introduced stuttering with mouse cursor operations

[bugzilla-daemon]

[-- Attachment #1.1: Type: text/plain, Size: 553 bytes --]

https://bugs.freedesktop.org/show_bug.cgi?id=109332

tempel.julian@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from tempel.julian@gmail.com ---
This was fixed in kernel with "Allow cursor async updates for framebuffer
swaps".

-- 
You are receiving this mail because:
You are the assignee for the bug.

[-- Attachment #1.2: Type: text/html, Size: 2150 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel