[Mesa-dev] anv/radv: WSI sharing code

Jason Ekstrand jason at jlekstrand.net
Tue Oct 18 20:39:08 UTC 2016

I've dug through the whole thing now.  I'm not a fan of patch 21 (re-order
UNORM and sRGB) and gave detailed comments on it.  The rest are

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

My only other real comment is that I think I'd rather we put a bit more
stuff in wsi_device so we're not passing so much around.  In particular,
the allocation functions and format functions could go there and maybe an
alloc.  If you wanted to clean that up as a follow-on patch, that's fine,
but I would like it cleaned up if you don't mind.


On Sun, Oct 16, 2016 at 9:24 PM, Dave Airlie <airlied at gmail.com> wrote:

> This series builds on top of the previous sharing patches I sent.
> The aim here is to share the X11 and wayland WSI code between
> the two vulkan drivers so we have a consistent implementation and
> one place to fix bugs.
> The series modifies the anv code in place until it's suitable
> for sharing, then it moves it to shared directory, and ports
> radv to use it.
> The final code leaves the WSI APIs in the drivers, but they
> call directly into the shared code once they shed their driver
> specific structs, and pick a pAllocator.
> Dave.
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161018/c1f68fac/attachment.html>

More information about the mesa-dev mailing list