[Mesa-dev] [PATCH 0/7] DRI3 support for EGL (v3)

[Boyan Ding]

2015-10-31 0:03 GMT+08:00 Martin Peres <martin.peres at linux.intel.com>:
> First of all, I would like to thank Boyan for his work here. I rebased his patch
> series, fixed minor issues here and there and reviewed it. You can check the
> changes in every patch but the biggest changes are related to the build system.
>
> Speaking about the build system it seems like scons is not building the EGL X11
> platform, so I did not bother adding support for the X11 DRI3 there.
>
> I tested the code using GLES apps and ran a full piglit/cts run on all the
> platforms. Nothing display-related failed.
>
> Finally, I added a patch that improves the reporting of which DRI version is
> being used. I tested that the reporting was accurate by checking which node
> was being used by running the application with ezbench's env_dump
> (/utils/env_dump).
>
> I intend to push that series at the end of next week unless someone opposes.
>
> Boyan Ding (6):
>   loader: Add dri3 helper
>   glx/dri3: Convert to use dri3 helper in loader library
>   egl_dri2: Add a function to let platform code return dri drawable from
>     _EGLSurface
>   egl/x11: Implement dri3 support with loader's dri3 helper
>   loader/dri3: Expose function to create __DRIimage from pixmap
>   egl/x11_dri3: Implement EGL_KHR_image_pixmap
>
> Martin Peres (1):
>   egl: make it clear which platform x11 backend is being used (dri2 or
>     3)

Hi Martin,

Thanks for bringing this up. I was always thinking about rebasing and
refining them a little bit but recent work at school ate up all my
spare time. The code now is mostly in shape, the only technical concern
I have is EGL_BUFFER_PRESERVED, which I forgot to handle. But it didn't
seem to cause anything wrong, even with Plasma 5, which is saying that
it uses buffer preservation for the lack of EGL_NV_post_sub_buffer.

There are still some misc styling issues pointed out by Emil [1][2],
and EGL_WL_bind_wayland_display support when libdrm is not new enough.
I don't think these are blocking issues though and there are still time
to stablize.

Regards,
Boyan Ding

[1] http://lists.freedesktop.org/archives/mesa-dev/2015-July/089971.html
[2] http://lists.freedesktop.org/archives/mesa-dev/2015-July/089972.html