[Mesa-dev] [PATCH v4] android: add vulkan build for intel

[Tapani Pälli]

On 01/30/2017 04:00 PM, Emil Velikov wrote:
> On 30 January 2017 at 11:37, Tapani Pälli <tapani.palli at intel.com> wrote:
>> fixes to issues spotted by Emil Velikov:
>>
>>    - set ANV_TIMESTAMP corretly
>>    - fix typo with VULKAN_GEM_FILES
>>
>> v2: update to use Makefile.sources under vulkan
>>     instead of having own
>>
>> v3: update to changes to generate from vk.xml
>>     (commit c7fc310)
>>
>> v4: remove 'hw' relative path
>>     cleanups, remove unnecessary cruft
>>
>>     review from Emil Velikov:
>>
>>     - move to vulkan folder
>>     - remove timestamp gen, no longer necessary
>>     - more cleanups
>>
> Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
>
> Does one need the VK_USE_PLATFORM_ANDROID_KHR
> define/VK_KHR_ANDROID_SURFACE_EXTENSION_NAME/winsys implementation or
> the vulkan HAL is enough ?

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:

dEQP-VK.wsi.android.surface.create
dEQP-VK.wsi.android.surface.create_custom_allocator
dEQP-VK.wsi.android.surface.query_support
dEQP-VK.wsi.android.surface.query_capabilities
dEQP-VK.wsi.android.surface.query_formats
dEQP-VK.wsi.android.surface.query_present_modes
dEQP-VK.wsi.android.surface.initial_size

but we get 'Duplicate extensions' failure from 
dEQP-VK.api.info.instance.extensions 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.

Thanks;

// Tapani