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

Jason Ekstrand jason at jlekstrand.net
Thu Jan 25 05:27:16 UTC 2018


On Wed, Jan 24, 2018 at 6:53 AM, Tapani Pälli <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>
> ---
>  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>


> +   };
> +
>      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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20180124/55d40bb9/attachment.html>


More information about the Piglit mailing list