Kertesz Laszlo <[email protected]> writes:

> Ok, rebuilt the xserver package with debugging symbols (seems that
> checkinstall strips stuff by default). I got a bigger gdb.txt. See if it
> helps.

I found a bug -- glamor_xv_put_image was mis-computing the number of
lines of changed video when the client drew only a subset of the
image. I think the client is drawing at src_y=1, src_h=239 for some
weird reason (I suspect a bug in the client).

Try this patch:

From eaa4225413b31314070f9a52d9290649e79a3b0f Mon Sep 17 00:00:00 2001
From: Keith Packard <[email protected]>
Date: Sat, 27 Dec 2014 09:11:33 -0800
Subject: [PATCH] glamor: Fix nlines in glamor_xv_put_image when src_y is odd

The number of lines of video to update in the texture needs to be
computed from the height of the updated source, not the full height of
the source.

Signed-off-by: Keith Packard <[email protected]>
---
 glamor/glamor_xv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/glamor/glamor_xv.c b/glamor/glamor_xv.c
index 1c877da..83e24ad 100644
--- a/glamor/glamor_xv.c
+++ b/glamor/glamor_xv.c
@@ -435,7 +435,7 @@ glamor_xv_put_image(glamor_port_private *port_priv,
     }
 
     top = (src_y) & ~1;
-    nlines = (src_y + height) - top;
+    nlines = (src_y + src_h) - top;
 
     switch (id) {
     case FOURCC_YV12:
-- 
2.1.4

-- 
[email protected]

Attachment: signature.asc
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to