similar to: [PATCH xserver] Make PixmapDirtyUpdateRec::src a DrawablePtr

From: Michel Dänzer <michel.daenzer at amd.com> Signed-off-by: Michel Dänzer <michel.daenzer at amd.com> --- Chris / Ilia / Ben, this should be manageable for the intel/nouveau drivers, right? src/amdgpu_drv.h | 26 ++++++++++++++++++++++++++ src/amdgpu_kms.c | 18 +++++++++--------- src/drmmode_display.c | 8 ++++++-- 3 files changed, 41 insertions(+), 11 deletions(-)

The "pick best" logic takes rotation into account. However flipping a rotated CRTC can't work, so we disable that. Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu> --- At first blush, this seems to make DRI3 function reasonably with rotated CRTCs src/nouveau_present.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/nouveau_present.c

First of all, we were not forcing the pixmap to be moved in before attempting to present it. Secondly, in certain configurations, we avoid ever backing pixmaps with vram (in GPUs with 32MB of VRAM or less). This fixes the observed cases where we ended up assuming that a bo was backing a pixmap without doing the explicit move-in. Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu> ---

It turns out there's a difference in X between a CRTC being "disabled" and simply having it's DPMS turned off. This is problematic though, because if DPMS is turned off you can't really use the CRTC as a normal CRTC anyway since page flipping and vblanks will be non-functional. As a result, we've been considering DPMS-on CRTCs as enabled and attempt to perform pageflips,

Apparently it now wants a drawable. Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu> --- No idea about whether this is correct. Saw the new warnings though. src/drmmode_display.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/src/drmmode_display.c b/src/drmmode_display.c index 2480122..f677e24 100644 --- a/src/drmmode_display.c +++

We don't really expect to have too many events in the queue. If there are, then the algorithm we use isn't appropriate. Add a warning when the queue gets very long, as it's an indication of something having gone wrong. Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu> --- src/drmmode_display.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git

https://bugs.freedesktop.org/show_bug.cgi?id=60369 Priority: medium Bug ID: 60369 CC: airlied at freedesktop.org Assignee: nouveau at lists.freedesktop.org Summary: src/nouveau_exa.c:142:31: error: 'CREATE_PIXMAP_USAGE_SHARED' undeclared (first use in this function) QA Contact: