<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Feb 21, 2018 at 1:32 PM, 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,<br>
<span class=""><br>
On 21 February 2018 at 20:50, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Wed, Feb 21, 2018 at 6:05 AM, Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a>> wrote:<br>
</span><span class="">>> @@ -832,7 +832,7 @@ x11_acquire_next_image_from_<wbr>queue(struct x11_swapchain<br>
>> *chain,<br>
>>     VkResult result = wsi_queue_pull(&chain-><wbr>acquire_queue,<br>
>>                                      &image_index, timeout);<br>
>>     /* On error, the thread has shut down, so safe to update chain->status<br>
>> */<br>
>> -   if (result < 0)<br>
>> +   if (result < 0 || image_index == UINT32_MAX)<br>
><br>
> This only fixes an issue created by the previous patch.<br>
><br>
>><br>
>>        return x11_swapchain_result(chain, result);<br>
>><br>
>>     assert(image_index < chain->base.image_count);<br>
>> @@ -938,7 +938,7 @@ x11_manage_fifo_queues(void *state)<br>
>>        result = wsi_queue_pull(&chain-><wbr>present_queue, &image_index,<br>
>> INT64_MAX);<br>
>>        if (result < 0) {<br>
>>           goto fail;<br>
>> -      } else if (chain->status < 0) {<br>
>> +      } else if (chain->status < 0 || image_index == UINT32_MAX) {<br>
><br>
> The one time UINT32_MAX is pushed onto the queue, chain->status is set to<br>
> VK_ERROR_OUT_OF_DATE_KHR.<br>
<br>
</span>Right you are; oops. Should it just be squashed in?<br>
</blockquote></div><br></div><div class="gmail_extra">Just a second.  Let me send patches.<br></div></div>