[PATCH wayland v2 1/8] wayland-egl: import libwayland-egl.so frontend library from Mesa
Miguel Angel Vico
mvicomoya at nvidia.com
Tue Dec 12 21:10:04 UTC 2017
Hi,
On Tue, 12 Dec 2017 20:58:30 +0100
Arnaud Vrac <rawoul at gmail.com> wrote:
> Hi Emil,
>
> On Tue, Oct 10, 2017 at 3:43 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> > From: Emil Velikov <emil.velikov at collabora.com>
> >
> > Currently the client-facing libwayland-egl API is defined by a header
> > file shipped by Wayland, but the implementation is left to each vendor.
> >
> > This can cause collisions when multiple implementations are installed on
> > the same system. Importing the implementation into Wayland with a stable
> > and versioned driver-facing ABI allows multiple drivers to coexist on
> > the same system.
> >
> > Pull the sample implementation from Mesa commit 677edff5cfd
> > ("wayland-egl: rework and simplify wl_egl_window initialization")
> > It has been used by the Mesa open source drivers, NVIDIA and others[1].
> >
> > v2: Reword commit message, rebase on top of newer Mesa.
> >
> > [1] https://github.com/thayama/wayland-egl
> >
>
> This looks very good, I only have one concern. The
> wayland-egl-backend.h header cannot be used in C++ code because of the
> "private" field in the wl_egl_window structure. Including the header
> results in the following compilation error:
>
> .../wayland-egl-backend.h:56:8: expected unqualified-id before ‘private’
>
> Would it be ok to alter the API and rename this field ?
A simple rename is fine, as it won't break the ABI. You can go ahead
and propose a patch :-)
Thanks for catching this,
Miguel.
>
> Regards,
> -Arnaud
>
> > Cc: Miguel A. Vico <mvicomoya at nvidia.com>
> > Cc: James Jones <jajones at nvidia.com>
> > Cc: Daniel Stone <daniels at collabora.com>
> > Cc: duncan-roe <duncan_roe at optusnet.com.au>
> > Cc: Takanari Hayama <taki at igel.co.jp>
> > Suggested-by: Daniel Stone <daniels at collabora.com>
> > Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> > ---
> > egl/wayland-egl-abi-check.c | 235 ++++++++++++++++++++++++++++++++++++++++++
> > egl/wayland-egl-backend.h | 63 +++++++++++
> > egl/wayland-egl-symbols-check | 16 +++
> > egl/wayland-egl.c | 109 ++++++++++++++++++++
> > egl/wayland-egl.pc.in | 11 ++
> > 5 files changed, 434 insertions(+)
> > create mode 100644 egl/wayland-egl-abi-check.c
> > create mode 100644 egl/wayland-egl-backend.h
> > create mode 100755 egl/wayland-egl-symbols-check
> > create mode 100644 egl/wayland-egl.c
> > create mode 100644 egl/wayland-egl.pc.in
> >
> > diff --git a/egl/wayland-egl-abi-check.c b/egl/wayland-egl-abi-check.c
> > new file mode 100644
> > index 0000000..62c51a2
> > --- /dev/null
> > +++ b/egl/wayland-egl-abi-check.c
> > @@ -0,0 +1,235 @@
> > +/*
> > + * Copyright (c) 2017, NVIDIA CORPORATION. All rights reserved.
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining a
> > + * copy of this software and associated documentation files (the "Software"),
> > + * to deal in the Software without restriction, including without limitation
> > + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice shall be included in
> > + * all copies or substantial portions of the Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
> > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
> > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> > + * DEALINGS IN THE SOFTWARE.
> > + */
> > +
> > +#include <stddef.h> /* offsetof */
> > +#include <stdio.h> /* printf */
> > +
> > +#include "wayland-egl-backend.h" /* Current struct wl_egl_window implementation */
> > +
> > +/*
> > + * Following are previous implementations of wl_egl_window.
> > + *
> > + * DO NOT EVER CHANGE!
> > + */
> > +
> > +/* From: 214fc6e850 - Benjamin Franzke : egl: Implement libwayland-egl */
> > +struct wl_egl_window_v0 {
> > + struct wl_surface *surface;
> > +
> > + int width;
> > + int height;
> > + int dx;
> > + int dy;
> > +
> > + int attached_width;
> > + int attached_height;
> > +};
> > +
> > +/* From: ca3ed3e024 - Ander Conselvan de Oliveira : egl/wayland: Don't invalidate drawable on swap buffers */
> > +struct wl_egl_window_v1 {
> > + struct wl_surface *surface;
> > +
> > + int width;
> > + int height;
> > + int dx;
> > + int dy;
> > +
> > + int attached_width;
> > + int attached_height;
> > +
> > + void *private;
> > + void (*resize_callback)(struct wl_egl_window *, void *);
> > +};
> > +
> > +/* From: 690ead4a13 - Stencel, Joanna : egl/wayland-egl: Fix for segfault in dri2_wl_destroy_surface. */
> > +#define WL_EGL_WINDOW_VERSION_v2 2
> > +struct wl_egl_window_v2 {
> > + struct wl_surface *surface;
> > +
> > + int width;
> > + int height;
> > + int dx;
> > + int dy;
> > +
> > + int attached_width;
> > + int attached_height;
> > +
> > + void *private;
> > + void (*resize_callback)(struct wl_egl_window *, void *);
> > + void (*destroy_window_callback)(void *);
> > +};
> > +
> > +/* From: 2d5d61bc49 - Miguel A. Vico : wayland-egl: Make wl_egl_window a versioned struct */
> > +#define WL_EGL_WINDOW_VERSION_v3 3
> > +struct wl_egl_window_v3 {
> > + const intptr_t version;
> > +
> > + int width;
> > + int height;
> > + int dx;
> > + int dy;
> > +
> > + int attached_width;
> > + int attached_height;
> > +
> > + void *private;
> > + void (*resize_callback)(struct wl_egl_window *, void *);
> > + void (*destroy_window_callback)(void *);
> > +
> > + struct wl_surface *surface;
> > +};
> > +
> > +
> > +/* This program checks we keep a backwards-compatible struct wl_egl_window
> > + * definition whenever it is modified in wayland-egl-backend.h.
> > + *
> > + * The previous definition should be added above as a new struct
> > + * wl_egl_window_vN, and the appropriate checks should be added below
> > + */
> > +
> > +#define MEMBER_SIZE(type, member) sizeof(((type *)0)->member)
> > +
> > +#define CHECK_RENAMED_MEMBER(a_ver, b_ver, a_member, b_member) \
> > + do { \
> > + if (offsetof(struct wl_egl_window ## a_ver, a_member) != \
> > + offsetof(struct wl_egl_window ## b_ver, b_member)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "offsetof(struct wl_egl_window" #a_ver "::" #a_member ") != " \
> > + "offsetof(struct wl_egl_window" #b_ver "::" #b_member ")\n"); \
> > + return 1; \
> > + } \
> > + \
> > + if (MEMBER_SIZE(struct wl_egl_window ## a_ver, a_member) != \
> > + MEMBER_SIZE(struct wl_egl_window ## b_ver, b_member)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "MEMBER_SIZE(struct wl_egl_window" #a_ver "::" #a_member ") != " \
> > + "MEMBER_SIZE(struct wl_egl_window" #b_ver "::" #b_member ")\n"); \
> > + return 1; \
> > + } \
> > + } while (0)
> > +
> > +#define CHECK_MEMBER(a_ver, b_ver, member) CHECK_RENAMED_MEMBER(a_ver, b_ver, member, member)
> > +#define CHECK_MEMBER_CURRENT(a_ver, member) CHECK_MEMBER(a_ver,, member)
> > +
> > +#define CHECK_SIZE(a_ver, b_ver) \
> > + do { \
> > + if (sizeof(struct wl_egl_window ## a_ver) > \
> > + sizeof(struct wl_egl_window ## b_ver)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "sizeof(struct wl_egl_window" #a_ver ") > " \
> > + "sizeof(struct wl_egl_window" #b_ver ")\n"); \
> > + return 1; \
> > + } \
> > + } while (0)
> > +
> > +#define CHECK_SIZE_CURRENT(a_ver) \
> > + do { \
> > + if (sizeof(struct wl_egl_window ## a_ver) != \
> > + sizeof(struct wl_egl_window)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "sizeof(struct wl_egl_window" #a_ver ") != " \
> > + "sizeof(struct wl_egl_window)\n"); \
> > + return 1; \
> > + } \
> > + } while (0)
> > +
> > +#define CHECK_VERSION(a_ver, b_ver) \
> > + do { \
> > + if ((WL_EGL_WINDOW_VERSION ## a_ver) >= \
> > + (WL_EGL_WINDOW_VERSION ## b_ver)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "WL_EGL_WINDOW_VERSION" #a_ver " >= " \
> > + "WL_EGL_WINDOW_VERSION" #b_ver "\n"); \
> > + return 1; \
> > + } \
> > + } while (0)
> > +
> > +#define CHECK_VERSION_CURRENT(a_ver) \
> > + do { \
> > + if ((WL_EGL_WINDOW_VERSION ## a_ver) != \
> > + (WL_EGL_WINDOW_VERSION)) { \
> > + printf("Backards incompatible change detected!\n " \
> > + "WL_EGL_WINDOW_VERSION" #a_ver " != " \
> > + "WL_EGL_WINDOW_VERSION\n"); \
> > + return 1; \
> > + } \
> > + } while (0)
> > +
> > +int main(int argc, char **argv)
> > +{
> > + /* Check wl_egl_window_v1 ABI against wl_egl_window_v0 */
> > + CHECK_MEMBER(_v0, _v1, surface);
> > + CHECK_MEMBER(_v0, _v1, width);
> > + CHECK_MEMBER(_v0, _v1, height);
> > + CHECK_MEMBER(_v0, _v1, dx);
> > + CHECK_MEMBER(_v0, _v1, dy);
> > + CHECK_MEMBER(_v0, _v1, attached_width);
> > + CHECK_MEMBER(_v0, _v1, attached_height);
> > +
> > + CHECK_SIZE(_v0, _v1);
> > +
> > + /* Check wl_egl_window_v2 ABI against wl_egl_window_v1 */
> > + CHECK_MEMBER(_v1, _v2, surface);
> > + CHECK_MEMBER(_v1, _v2, width);
> > + CHECK_MEMBER(_v1, _v2, height);
> > + CHECK_MEMBER(_v1, _v2, dx);
> > + CHECK_MEMBER(_v1, _v2, dy);
> > + CHECK_MEMBER(_v1, _v2, attached_width);
> > + CHECK_MEMBER(_v1, _v2, attached_height);
> > + CHECK_MEMBER(_v1, _v2, private);
> > + CHECK_MEMBER(_v1, _v2, resize_callback);
> > +
> > + CHECK_SIZE(_v1, _v2);
> > +
> > + /* Check wl_egl_window_v3 ABI against wl_egl_window_v2 */
> > + CHECK_RENAMED_MEMBER(_v2, _v3, surface, version);
> > + CHECK_MEMBER (_v2, _v3, width);
> > + CHECK_MEMBER (_v2, _v3, height);
> > + CHECK_MEMBER (_v2, _v3, dx);
> > + CHECK_MEMBER (_v2, _v3, dy);
> > + CHECK_MEMBER (_v2, _v3, attached_width);
> > + CHECK_MEMBER (_v2, _v3, attached_height);
> > + CHECK_MEMBER (_v2, _v3, private);
> > + CHECK_MEMBER (_v2, _v3, resize_callback);
> > + CHECK_MEMBER (_v2, _v3, destroy_window_callback);
> > +
> > + CHECK_SIZE (_v2, _v3);
> > + CHECK_VERSION(_v2, _v3);
> > +
> > + /* Check current wl_egl_window ABI against wl_egl_window_v3 */
> > + CHECK_MEMBER_CURRENT(_v3, version);
> > + CHECK_MEMBER_CURRENT(_v3, width);
> > + CHECK_MEMBER_CURRENT(_v3, height);
> > + CHECK_MEMBER_CURRENT(_v3, dx);
> > + CHECK_MEMBER_CURRENT(_v3, dy);
> > + CHECK_MEMBER_CURRENT(_v3, attached_width);
> > + CHECK_MEMBER_CURRENT(_v3, attached_height);
> > + CHECK_MEMBER_CURRENT(_v3, private);
> > + CHECK_MEMBER_CURRENT(_v3, resize_callback);
> > + CHECK_MEMBER_CURRENT(_v3, destroy_window_callback);
> > + CHECK_MEMBER_CURRENT(_v3, surface);
> > +
> > + CHECK_SIZE_CURRENT (_v3);
> > + CHECK_VERSION_CURRENT(_v3);
> > +
> > + return 0;
> > +}
> > diff --git a/egl/wayland-egl-backend.h b/egl/wayland-egl-backend.h
> > new file mode 100644
> > index 0000000..82f025c
> > --- /dev/null
> > +++ b/egl/wayland-egl-backend.h
> > @@ -0,0 +1,63 @@
> > +/*
> > + * Copyright © 2011 Benjamin Franzke
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining a
> > + * copy of this software and associated documentation files (the "Software"),
> > + * to deal in the Software without restriction, including without limitation
> > + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice (including the next
> > + * paragraph) shall be included in all copies or substantial portions of the
> > + * Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> > + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> > + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> > + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> > + * DEALINGS IN THE SOFTWARE.
> > + *
> > + * Authors:
> > + * Benjamin Franzke <benjaminfranzke at googlemail.com>
> > + */
> > +
> > +#ifndef _WAYLAND_EGL_PRIV_H
> > +#define _WAYLAND_EGL_PRIV_H
> > +
> > +#include <stdint.h>
> > +
> > +#ifdef __cplusplus
> > +extern "C" {
> > +#endif
> > +
> > +#define WL_EGL_WINDOW_VERSION 3
> > +
> > +struct wl_surface;
> > +
> > +struct wl_egl_window {
> > + const intptr_t version;
> > +
> > + int width;
> > + int height;
> > + int dx;
> > + int dy;
> > +
> > + int attached_width;
> > + int attached_height;
> > +
> > + void *private;
> > + void (*resize_callback)(struct wl_egl_window *, void *);
> > + void (*destroy_window_callback)(void *);
> > +
> > + struct wl_surface *surface;
> > +};
> > +
> > +#ifdef __cplusplus
> > +}
> > +#endif
> > +
> > +#endif
> > diff --git a/egl/wayland-egl-symbols-check b/egl/wayland-egl-symbols-check
> > new file mode 100755
> > index 0000000..e7105ea
> > --- /dev/null
> > +++ b/egl/wayland-egl-symbols-check
> > @@ -0,0 +1,16 @@
> > +#!/bin/sh
> > +
> > +FUNCS=$(nm -D --defined-only ${1-.libs/libwayland-egl.so} | grep -o "T .*" | cut -c 3- | while read func; do
> > +( grep -q "^$func$" || echo $func ) <<EOF
> > +wl_egl_window_resize
> > +wl_egl_window_create
> > +wl_egl_window_destroy
> > +wl_egl_window_get_attached_size
> > +_fini
> > +_init
> > +EOF
> > +done)
> > +
> > +test ! -n "$FUNCS" || echo $FUNCS
> > +test ! -n "$FUNCS"
> > +
> > diff --git a/egl/wayland-egl.c b/egl/wayland-egl.c
> > new file mode 100644
> > index 0000000..e7cea89
> > --- /dev/null
> > +++ b/egl/wayland-egl.c
> > @@ -0,0 +1,109 @@
> > +/*
> > + * Copyright © 2011 Kristian Høgsberg
> > + * Copyright © 2011 Benjamin Franzke
> > + *
> > + * Permission is hereby granted, free of charge, to any person obtaining a
> > + * copy of this software and associated documentation files (the "Software"),
> > + * to deal in the Software without restriction, including without limitation
> > + * the rights to use, copy, modify, merge, publish, distribute, sublicense,
> > + * and/or sell copies of the Software, and to permit persons to whom the
> > + * Software is furnished to do so, subject to the following conditions:
> > + *
> > + * The above copyright notice and this permission notice (including the next
> > + * paragraph) shall be included in all copies or substantial portions of the
> > + * Software.
> > + *
> > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> > + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> > + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
> > + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
> > + * HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
> > + * WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
> > + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> > + * DEALINGS IN THE SOFTWARE.
> > + *
> > + * Authors:
> > + * Kristian Høgsberg <krh at bitplanet.net>
> > + * Benjamin Franzke <benjaminfranzke at googlemail.com>
> > + */
> > +
> > +#include <stdlib.h>
> > +#include <string.h>
> > +
> > +#include "wayland-egl.h"
> > +#include "wayland-egl-backend.h"
> > +
> > +/* GCC visibility */
> > +#if defined(__GNUC__)
> > +#define WL_EGL_EXPORT __attribute__ ((visibility("default")))
> > +#else
> > +#define WL_EGL_EXPORT
> > +#endif
> > +
> > +WL_EGL_EXPORT void
> > +wl_egl_window_resize(struct wl_egl_window *egl_window,
> > + int width, int height,
> > + int dx, int dy)
> > +{
> > + if (width <= 0 || height <= 0)
> > + return;
> > +
> > + egl_window->width = width;
> > + egl_window->height = height;
> > + egl_window->dx = dx;
> > + egl_window->dy = dy;
> > +
> > + if (egl_window->resize_callback)
> > + egl_window->resize_callback(egl_window, egl_window->private);
> > +}
> > +
> > +WL_EGL_EXPORT struct wl_egl_window *
> > +wl_egl_window_create(struct wl_surface *surface,
> > + int width, int height)
> > +{
> > + struct wl_egl_window *egl_window;
> > +
> > + if (width <= 0 || height <= 0)
> > + return NULL;
> > +
> > + egl_window = calloc(1, sizeof *egl_window);
> > + if (!egl_window)
> > + return NULL;
> > +
> > + /* Cast away the constness to set the version number.
> > + *
> > + * We want the const notation since it gives an explicit
> > + * feedback to the backend implementation, should it try to
> > + * change it.
> > + *
> > + * The latter in itself is not too surprising as these days APIs
> > + * tend to provide bidirectional version field.
> > + */
> > + intptr_t *version = (intptr_t *)&egl_window->version;
> > + *version = WL_EGL_WINDOW_VERSION;
> > +
> > + egl_window->surface = surface;
> > +
> > + egl_window->width = width;
> > + egl_window->height = height;
> > +
> > + return egl_window;
> > +}
> > +
> > +WL_EGL_EXPORT void
> > +wl_egl_window_destroy(struct wl_egl_window *egl_window)
> > +{
> > + if (egl_window->destroy_window_callback)
> > + egl_window->destroy_window_callback(egl_window->private);
> > + free(egl_window);
> > +}
> > +
> > +WL_EGL_EXPORT void
> > +wl_egl_window_get_attached_size(struct wl_egl_window *egl_window,
> > + int *width, int *height)
> > +{
> > + if (width)
> > + *width = egl_window->attached_width;
> > + if (height)
> > + *height = egl_window->attached_height;
> > +}
> > diff --git a/egl/wayland-egl.pc.in b/egl/wayland-egl.pc.in
> > new file mode 100644
> > index 0000000..8a40cfa
> > --- /dev/null
> > +++ b/egl/wayland-egl.pc.in
> > @@ -0,0 +1,11 @@
> > +prefix=@prefix@
> > +exec_prefix=@exec_prefix@
> > +libdir=@libdir@
> > +includedir=@includedir@
> > +
> > +Name: wayland-egl
> > +Description: Mesa wayland-egl library
> > +Version: @VERSION@
> > +Requires: wayland-client
> > +Libs: -L${libdir} -lwayland-egl
> > +Cflags: -I${includedir}
> > --
> > 2.14.1
--
Miguel
More information about the wayland-devel
mailing list