[PATCH] compositor-wayland: Expose zlinux_dmabuf when it is available in the parent compositor

[Pekka Paalanen]

Hi Emmanuel,

all looks good except the summary has a minor flaw: you don't need
zlinux_dmabuf to be available in the parent compositor, and this patch
also does not check for it. :-)

On Fri, 25 Sep 2015 10:14:19 +0200
Emmanuel Gil Peyrot <emmanuel.peyrot at collabora.com> wrote:

> This mirrors what 5ffb440c3fe19669de73e51b90ea337fa2d5ae78 did for
> compositor-x11, and allows testing without having to restart the
> compositor-drm everytime.
> ---
>  src/compositor-wayland.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
> index f6c84d4..900c331 100644
> --- a/src/compositor-wayland.c
> +++ b/src/compositor-wayland.c
> @@ -48,6 +48,7 @@
>  #include "shared/cairo-util.h"
>  #include "fullscreen-shell-client-protocol.h"
>  #include "presentation_timing-server-protocol.h"
> +#include "linux-dmabuf.h"
>  
>  #define WINDOW_TITLE "Weston Compositor"
>  
> @@ -2022,6 +2023,12 @@ wayland_backend_create(struct weston_compositor *compositor, int use_pixman,
>  
>  	wl_event_source_check(b->parent.wl_source);
>  
> +	if (compositor->renderer->import_dmabuf) {
> +		if (linux_dmabuf_setup(compositor) < 0)
> +			weston_log("Error: initializing dmabuf "
> +			           "support failed.\n");
> +	}
> +
>  	compositor->backend = &b->base;
>  	return b;
>  err_display:

With the summary fixed:
Reviewed-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150925/d3318980/attachment.sig>