<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Jan 30, 2017 10:03 PM, "Tapani Pälli" <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text"><br>
<br>
On 01/30/2017 04:00 PM, Emil Velikov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 30 January 2017 at 11:37, Tapani Pälli <<a href="mailto:tapani.palli@intel.com" target="_blank">tapani.palli@intel.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
fixes to issues spotted by Emil Velikov:<br>
<br>
   - set ANV_TIMESTAMP corretly<br>
   - fix typo with VULKAN_GEM_FILES<br>
<br>
v2: update to use Makefile.sources under vulkan<br>
    instead of having own<br>
<br>
v3: update to changes to generate from vk.xml<br>
    (commit c7fc310)<br>
<br>
v4: remove 'hw' relative path<br>
    cleanups, remove unnecessary cruft<br>
<br>
    review from Emil Velikov:<br>
<br>
    - move to vulkan folder<br>
    - remove timestamp gen, no longer necessary<br>
    - more cleanups<br>
<br>
</blockquote>
Reviewed-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com" target="_blank">emil.velikov@collabora.com</a>><br>
<br>
Does one need the VK_USE_PLATFORM_ANDROID_KHR<br>
define/VK_KHR_ANDROID_SURFACE_<wbr>EXTENSION_NAME/winsys implementation or<br>
the vulkan HAL is enough ?<br>
</blockquote>
<br></div>
We need some more implementation and probably the define but I wanted to leave this enabling later as this needs some more investigation. We do currently pass following tests:<br>
<br>
<a href="http://dEQP-VK.wsi.android.surface.cr">dEQP-VK.wsi.android.surface.cr</a><wbr>eate<br>
<a href="http://dEQP-VK.wsi.android.surface.cr">dEQP-VK.wsi.android.surface.cr</a><wbr>eate_custom_allocator<br>
dEQP-VK.wsi.android.surface.qu<wbr>ery_support<br>
dEQP-VK.wsi.android.surface.qu<wbr>ery_capabilities<br>
dEQP-VK.wsi.android.surface.qu<wbr>ery_formats<br>
dEQP-VK.wsi.android.surface.qu<wbr>ery_present_modes<br>
<a href="http://dEQP-VK.wsi.android.surface.in">dEQP-VK.wsi.android.surface.in</a><wbr>itial_size<br>
<br>
but we get 'Duplicate extensions' failure from dEQP-VK.api.info.instance.exte<wbr>nsions and I can see that 'VK_KHR_surface' and 'VK_KHR_android_surface' are 2 times in the extensions list, I don't understand why but will try to figure this out.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">No idea if this helps but there were some older versions of the loader that had a bug where they would search some paths twice and accidentally have duplicate extension names show up.  I'm not sure how android expects those extensions to be implemented but it could be that both the driver and the android loader are implementing them.  Just a drive-by thought.</div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div></div></div>