<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Feb 13, 2018 at 7:29 AM, Daniel Stone <span dir="ltr"><<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jason,<br>
This should be titled '... wsi_image', not '... wsi_image_base'.<span class=""><br></span></blockquote><div><br></div><div>Fixed.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
On 9 February 2018 at 23:43, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> diff --git a/src/vulkan/wsi/wsi_common_<wbr>private.h b/src/vulkan/wsi/wsi_common_<wbr>private.h<br>
> index 503b2a0..c5002ec 100644<br>
> --- a/src/vulkan/wsi/wsi_common_<wbr>private.h<br>
> +++ b/src/vulkan/wsi/wsi_common_<wbr>private.h<br>
> @@ -35,10 +35,11 @@ struct wsi_image {<br>
>        VkCommandBuffer *blit_cmd_buffers;<br>
>     } prime;<br>
><br>
> -   uint32_t size;<br>
> -   uint32_t offset;<br>
> -   uint32_t row_pitch;<br>
> -   int fd;<br>
> +   int num_planes;<br>
> +   uint32_t sizes[4];<br>
> +   uint32_t offsets[4];<br>
> +   uint32_t row_pitches[4];<br>
> +   int fds[4];<br>
>  };<br>
<br>
</span>I'd also like to squash this in:<br>
@@ -201,6 +201,8 @@ wsi_create_native_image(const struct wsi_swapchain *chain,<br>
    VkResult result;<br>
<br>
    memset(image, 0, sizeof(*image));<br>
+   for (int i = 0; i < ARRAY_SIZE(image->fds); i++)<br>
+      image->fds[i] = -1;<br></blockquote><div><br></div><div>Done.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Current users are safe, but it's allocated with zalloc(), and<br>
debugging something with stdout having been closed can be pretty<br>
frustrating. :)<br>
<br>
Cheers,<br>
Daniel<br>
</blockquote></div><br></div></div>