Hello
I test this patch and it fixes my problem.
Thanks
On Thu, 2017-10-05 at 15:36 +0100, Daniel Stone wrote:
> Multi-plane sub-sampled textures have partial width/height, e.g.
> YUV420/I420 has a full-size Y plane, followed by a half-width/height U
> plane, and a half-width/height V plane.
>
> zwp_linux_dmabuf_v1 allows clients to pass an explicit pitch for each
> plane, but for wl_shm this must be inferred. gl-renderer was correctly
> accounting for the width and height when subsampling, but the pitch was
> being taken as the pitch for the first plane.
>
> This does not match the requirements for GStreamer's waylandsink, in
> particular, as well as other clients. Fix the SHM upload path to
> correctly set the pitch for each plane, according to subsampling.
>
> Tested with:
> $ gst-launch-1.0 videotestsrc ! waylandsink
>
Tested-by: Fabien Lahoudere <fabien.lahoud...@collabora.co.uk>
> Signed-off-by: Daniel Stone <dani...@collabora.com>
> Fixes: fdeefe42418 ("gl-renderer: add support of WL_SHM_FORMAT_YUV420")
> Reported-by: Fabien Lahoudere <fabien.lahoud...@collabora.co.uk>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103063
> Cc: Vincent Abriou <vincent.abr...@st.com>
> ---
> libweston/gl-renderer.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libweston/gl-renderer.c b/libweston/gl-renderer.c
> index 244ce309..40bf0bb6 100644
> --- a/libweston/gl-renderer.c
> +++ b/libweston/gl-renderer.c
> @@ -1445,14 +1445,13 @@ gl_renderer_flush_damage(struct weston_surface
> *surface)
> goto done;
> }
>
> - glPixelStorei(GL_UNPACK_ROW_LENGTH_EXT, gs->pitch);
> -
> if (gs->needs_full_upload) {
> glPixelStorei(GL_UNPACK_SKIP_PIXELS_EXT, 0);
> glPixelStorei(GL_UNPACK_SKIP_ROWS_EXT, 0);
> wl_shm_buffer_begin_access(buffer->shm_buffer);
> for (j = 0; j < gs->num_textures; j++) {
> glBindTexture(GL_TEXTURE_2D, gs->textures[j]);
> + glPixelStorei(GL_UNPACK_ROW_LENGTH_EXT, gs->pitch /
> gs->hsub[j]);
> glTexImage2D(GL_TEXTURE_2D, 0,
> gs->gl_format[j],
> gs->pitch / gs->hsub[j],
> @@ -1477,6 +1476,7 @@ gl_renderer_flush_damage(struct weston_surface *surface)
> glPixelStorei(GL_UNPACK_SKIP_ROWS_EXT, r.y1);
> for (j = 0; j < gs->num_textures; j++) {
> glBindTexture(GL_TEXTURE_2D, gs->textures[j]);
> + glPixelStorei(GL_UNPACK_ROW_LENGTH_EXT, gs->pitch /
> gs->hsub[j]);
> glTexSubImage2D(GL_TEXTURE_2D, 0,
> r.x1 / gs->hsub[j],
> r.y1 / gs->vsub[j],
--
Fabien Lahoudere,
Embedded Linux developer at Collabora.
This message is intended for the use of only the person(s) ("intended
recipient") to whom it is
addressed. It may contain information that is privileged and confidential.
Accordingly any
dissemination, distribution, copying or other use of this message or any of its
content by any
person other than the intended recipient may constitute a breach of civil or
criminal law and is
strictly prohibited. If you are not the intended recipient, please contact the
sender as soon as
possible.
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel
