Skype bi-directional video call crashes X server (xserver, mesa, drm, kernel from git, r600g+glamor)

Keith Packard keithp at keithp.com
Sat Dec 27 13:00:31 PST 2014


Eric Anholt <eric at anholt.net> writes:

>> --- 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:
>
> If the point is to upload only from the src_[xywh] recctangle, shouldn't
> the glamor_upload_sub_pixmap_to_texture() calls be using src_w instead
> of width, too?

It doesn't need to, but it could as an optimization. Skipping lines at
the top and bottom is also just an optimization as the source rectangle
defines a subset of the provided buffer, after all. I just fixed that
optimization.

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 810 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20141227/8a1e8b40/attachment.sig>


More information about the dri-devel mailing list