[Piglit] [PATCH] require VK_KHR_external_semaphore_fd in func.sync.semaphore-fd.sync-fd

[Tapani Pälli]

On 25.01.2018 07:27, Jason Ekstrand wrote:
> On Wed, Jan 24, 2018 at 6:53 AM, Tapani Pälli <tapani.palli at intel.com 
> <mailto:tapani.palli at intel.com>> wrote:
> 
>     This fixes test for me after Mesa commit 1f79d986af which will
>     make device expose entrypoints only for enabled extensions.
> 
>     Signed-off-by: Tapani Pälli <tapani.palli at intel.com
>     <mailto:tapani.palli at intel.com>>
>     ---
>       src/tests/func/sync/semaphore-fd.c | 6 ++++++
>       1 file changed, 6 insertions(+)
> 
>     diff --git a/src/tests/func/sync/semaphore-fd.c
>     b/src/tests/func/sync/semaphore-fd.c
>     index 69e56c2..a977a05 100644
>     --- a/src/tests/func/sync/semaphore-fd.c
>     +++ b/src/tests/func/sync/semaphore-fd.c
>     @@ -45,9 +45,15 @@ struct buffer_layout {
>       static void
>       init_context(struct test_context *ctx, float priority)
>       {
>     +   const char *extension_names[] = {
>     +      "VK_KHR_external_semaphore_fd",
> 
> 
> To be technically correct, we should enable all three external memory 
> extensions and all three external fence extensions.  With that,
> 
> Reviewed-by: Jason Ekstrand <jason at jlekstrand.net 
> <mailto:jason at jlekstrand.net>>

OK I'll add them all, thanks Jason!


>     +   };
>     +
>           VkResult result = vkCreateDevice(t_physical_dev,
>               &(VkDeviceCreateInfo) {
>                   .sType = VK_STRUCTURE_TYPE_DEVICE_CREATE_INFO,
>     +            .enabledExtensionCount = 1,
>     +            .ppEnabledExtensionNames = extension_names,
>                   .queueCreateInfoCount = 1,
>                   .pQueueCreateInfos = &(VkDeviceQueueCreateInfo) {
>                       .sType = VK_STRUCTURE_TYPE_DEVICE_QUEUE_CREATE_INFO,
>     --
>     2.13.6
> 
>