<div dir="ltr"><div><div><div>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<br><br></div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div>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.<br><br></div>--Jason<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Oct 16, 2016 at 9:24 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This series builds on top of the previous sharing patches I sent.<br>
<br>
The aim here is to share the X11 and wayland WSI code between<br>
the two vulkan drivers so we have a consistent implementation and<br>
one place to fix bugs.<br>
<br>
The series modifies the anv code in place until it's suitable<br>
for sharing, then it moves it to shared directory, and ports<br>
radv to use it.<br>
<br>
The final code leaves the WSI APIs in the drivers, but they<br>
call directly into the shared code once they shed their driver<br>
specific structs, and pick a pAllocator.<br>
<br>
Dave.<br>
<br>
______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div>