[Mesa-dev] [PATCH 18/19] radv: generate same device UUID as radeonsi

[Andres Rodriguez]

This is required for interop use cases. The same device must report
identical UUIDs through the GL and Vulkan APIs so that users can
identify when it is safe to perform a memory object import.

Signed-off-by: Andres Rodriguez <andre...@gmail.com>
---
 src/amd/vulkan/Makefile.am   |  1 +
 src/amd/vulkan/radv_device.c | 14 ++++++++------
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/src/amd/vulkan/Makefile.am b/src/amd/vulkan/Makefile.am
index a645432..4834059 100644
--- a/src/amd/vulkan/Makefile.am
+++ b/src/amd/vulkan/Makefile.am
@@ -67,6 +67,7 @@ VULKAN_LIB_DEPS = \
        $(top_builddir)/src/amd/addrlib/libamdgpu_addrlib.la \
        $(top_builddir)/src/compiler/nir/libnir.la \
        $(top_builddir)/src/util/libmesautil.la \
+       $(top_builddir)/src/loader/libloader.la \
        $(LLVM_LIBS) \
        $(LIBELF_LIBS) \
        $(PTHREAD_LIBS) \
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 427d357..ec726e4 100644
--- a/src/amd/vulkan/radv_device.c
+++ b/src/amd/vulkan/radv_device.c
@@ -44,6 +44,7 @@
 #include "sid.h"
 #include "gfx9d.h"
 #include "util/debug.h"
+#include "loader/loader.h"
 
 static int
 radv_device_get_cache_uuid(enum radeon_family family, void *uuid)
@@ -63,12 +64,13 @@ radv_device_get_cache_uuid(enum radeon_family family, void 
*uuid)
 }
 
 static void
-radv_get_device_uuid(drmDevicePtr device, void *uuid) {
+radv_get_device_uuid(int fd, void *uuid) {
+       char *tag = loader_get_id_path_tag_for_fd(fd);
        memset(uuid, 0, VK_UUID_SIZE);
-       memcpy((char*)uuid + 0, &device->businfo.pci->domain, 2);
-       memcpy((char*)uuid + 2, &device->businfo.pci->bus, 1);
-       memcpy((char*)uuid + 3, &device->businfo.pci->dev, 1);
-       memcpy((char*)uuid + 4, &device->businfo.pci->func, 1);
+       if (tag) {
+               strncpy(uuid, tag, VK_UUID_SIZE);
+               free(tag);
+       }
 }
 
 static const VkExtensionProperties instance_extensions[] = {
@@ -303,7 +305,7 @@ radv_physical_device_init(struct radv_physical_device 
*device,
        fprintf(stderr, "WARNING: radv is not a conformant vulkan 
implementation, testing use only.\n");
        device->name = get_chip_name(device->rad_info.family);
 
-       radv_get_device_uuid(drm_device, device->device_uuid);
+       radv_get_device_uuid(fd, device->device_uuid);
 
        if (device->rad_info.family == CHIP_STONEY ||
            device->rad_info.chip_class >= GFX9) {
-- 
2.9.3

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev