Added few more stubs so that control reaches to DestroyDevice().
[mesa.git] / src / intel / vulkan / anv_wsi_x11.c
index a63cb6e7c5bba5bd93735e1ab8529d1604ce4da5..9a50b17f99937b48fc93ac85f8952af142b5c642 100644 (file)
  * IN THE SOFTWARE.
  */
 
+#include <X11/Xlib-xcb.h>
+#include <X11/xshmfence.h>
 #include <xcb/xcb.h>
 #include <xcb/dri3.h>
 #include <xcb/present.h>
 
-#include "anv_wsi.h"
-
-#include "util/hash_table.h"
-
-struct wsi_x11_connection {
-   bool has_dri3;
-   bool has_present;
-};
-
-struct wsi_x11 {
-   struct anv_wsi_interface base;
-
-   pthread_mutex_t                              mutex;
-   /* Hash table of xcb_connection -> wsi_x11_connection mappings */
-   struct hash_table *connections;
-};
-
-static struct wsi_x11_connection *
-wsi_x11_connection_create(struct anv_instance *instance, xcb_connection_t *conn)
-{
-   xcb_query_extension_cookie_t dri3_cookie, pres_cookie;
-   xcb_query_extension_reply_t *dri3_reply, *pres_reply;
-
-   struct wsi_x11_connection *wsi_conn =
-      anv_alloc(&instance->alloc, sizeof(*wsi_conn), 8,
-                VK_SYSTEM_ALLOCATION_SCOPE_INSTANCE);
-   if (!wsi_conn)
-      return NULL;
-
-   dri3_cookie = xcb_query_extension(conn, 4, "DRI3");
-   pres_cookie = xcb_query_extension(conn, 7, "PRESENT");
-
-   dri3_reply = xcb_query_extension_reply(conn, dri3_cookie, NULL);
-   pres_reply = xcb_query_extension_reply(conn, pres_cookie, NULL);
-   if (dri3_reply == NULL || pres_reply == NULL) {
-      free(dri3_reply);
-      free(pres_reply);
-      anv_free(&instance->alloc, wsi_conn);
-      return NULL;
-   }
-
-   wsi_conn->has_dri3 = dri3_reply->present != 0;
-   wsi_conn->has_present = pres_reply->present != 0;
-
-   free(dri3_reply);
-   free(pres_reply);
-
-   return wsi_conn;
-}
-
-static void
-wsi_x11_connection_destroy(struct anv_instance *instance,
-                           struct wsi_x11_connection *conn)
-{
-   anv_free(&instance->alloc, conn);
-}
-
-static struct wsi_x11_connection *
-wsi_x11_get_connection(struct anv_instance *instance, xcb_connection_t *conn)
-{
-   struct wsi_x11 *wsi =
-      (struct wsi_x11 *)instance->wsi[VK_ICD_WSI_PLATFORM_XCB];
-
-   pthread_mutex_lock(&wsi->mutex);
-
-   struct hash_entry *entry = _mesa_hash_table_search(wsi->connections, conn);
-   if (!entry) {
-      /* We're about to make a bunch of blocking calls.  Let's drop the
-       * mutex for now so we don't block up too badly.
-       */
-      pthread_mutex_unlock(&wsi->mutex);
-
-      struct wsi_x11_connection *wsi_conn =
-         wsi_x11_connection_create(instance, conn);
-
-      pthread_mutex_lock(&wsi->mutex);
-
-      entry = _mesa_hash_table_search(wsi->connections, conn);
-      if (entry) {
-         /* Oops, someone raced us to it */
-         wsi_x11_connection_destroy(instance, wsi_conn);
-      } else {
-         entry = _mesa_hash_table_insert(wsi->connections, conn, wsi_conn);
-      }
-   }
-
-   pthread_mutex_unlock(&wsi->mutex);
-
-   return entry->data;
-}
-
-static const VkSurfaceFormatKHR formats[] = {
-   { .format = VK_FORMAT_B8G8R8A8_SRGB, },
-};
-
-static const VkPresentModeKHR present_modes[] = {
-   VK_PRESENT_MODE_MAILBOX_KHR,
-};
-
-static xcb_screen_t *
-get_screen_for_root(xcb_connection_t *conn, xcb_window_t root)
-{
-   xcb_screen_iterator_t screen_iter =
-      xcb_setup_roots_iterator(xcb_get_setup(conn));
-
-   for (; screen_iter.rem; xcb_screen_next (&screen_iter)) {
-      if (screen_iter.data->root == root)
-         return screen_iter.data;
-   }
-
-   return NULL;
-}
-
-static xcb_visualtype_t *
-screen_get_visualtype(xcb_screen_t *screen, xcb_visualid_t visual_id,
-                      unsigned *depth)
-{
-   xcb_depth_iterator_t depth_iter =
-      xcb_screen_allowed_depths_iterator(screen);
-
-   for (; depth_iter.rem; xcb_depth_next (&depth_iter)) {
-      xcb_visualtype_iterator_t visual_iter =
-         xcb_depth_visuals_iterator (depth_iter.data);
-
-      for (; visual_iter.rem; xcb_visualtype_next (&visual_iter)) {
-         if (visual_iter.data->visual_id == visual_id) {
-            if (depth)
-               *depth = depth_iter.data->depth;
-            return visual_iter.data;
-         }
-      }
-   }
-
-   return NULL;
-}
-
-static xcb_visualtype_t *
-connection_get_visualtype(xcb_connection_t *conn, xcb_visualid_t visual_id,
-                          unsigned *depth)
-{
-   xcb_screen_iterator_t screen_iter =
-      xcb_setup_roots_iterator(xcb_get_setup(conn));
-
-   /* For this we have to iterate over all of the screens which is rather
-    * annoying.  Fortunately, there is probably only 1.
-    */
-   for (; screen_iter.rem; xcb_screen_next (&screen_iter)) {
-      xcb_visualtype_t *visual = screen_get_visualtype(screen_iter.data,
-                                                       visual_id, depth);
-      if (visual)
-         return visual;
-   }
-
-   return NULL;
-}
-
-static xcb_visualtype_t *
-get_visualtype_for_window(xcb_connection_t *conn, xcb_window_t window,
-                          unsigned *depth)
-{
-   xcb_query_tree_cookie_t tree_cookie;
-   xcb_get_window_attributes_cookie_t attrib_cookie;
-   xcb_query_tree_reply_t *tree;
-   xcb_get_window_attributes_reply_t *attrib;
-
-   tree_cookie = xcb_query_tree(conn, window);
-   attrib_cookie = xcb_get_window_attributes(conn, window);
-
-   tree = xcb_query_tree_reply(conn, tree_cookie, NULL);
-   attrib = xcb_get_window_attributes_reply(conn, attrib_cookie, NULL);
-   if (attrib == NULL || tree == NULL) {
-      free(attrib);
-      free(tree);
-      return NULL;
-   }
-
-   xcb_window_t root = tree->root;
-   xcb_visualid_t visual_id = attrib->visual;
-   free(attrib);
-   free(tree);
-
-   xcb_screen_t *screen = get_screen_for_root(conn, root);
-   if (screen == NULL)
-      return NULL;
-
-   return screen_get_visualtype(screen, visual_id, depth);
-}
-
-static bool
-visual_has_alpha(xcb_visualtype_t *visual, unsigned depth)
-{
-   uint32_t rgb_mask = visual->red_mask |
-                       visual->green_mask |
-                       visual->blue_mask;
-
-   uint32_t all_mask = 0xffffffff >> (32 - depth);
-
-   /* Do we have bits left over after RGB? */
-   return (all_mask & ~rgb_mask) != 0;
-}
+#include "wsi_common_x11.h"
+#include "anv_private.h"
 
 VkBool32 anv_GetPhysicalDeviceXcbPresentationSupportKHR(
     VkPhysicalDevice                            physicalDevice,
@@ -234,162 +38,26 @@ VkBool32 anv_GetPhysicalDeviceXcbPresentationSupportKHR(
 {
    ANV_FROM_HANDLE(anv_physical_device, device, physicalDevice);
 
-   struct wsi_x11_connection *wsi_conn =
-      wsi_x11_get_connection(device->instance, connection);
-
-   if (!wsi_conn->has_dri3) {
-      fprintf(stderr, "vulkan: No DRI3 support\n");
-      return false;
-   }
-
-   unsigned visual_depth;
-   if (!connection_get_visualtype(connection, visual_id, &visual_depth))
-      return false;
-
-   if (visual_depth != 24 && visual_depth != 32)
-      return false;
-
-   return true;
-}
-
-static VkResult
-x11_surface_get_support(VkIcdSurfaceBase *icd_surface,
-                        struct anv_physical_device *device,
-                        uint32_t queueFamilyIndex,
-                        VkBool32* pSupported)
-{
-   VkIcdSurfaceXcb *surface = (VkIcdSurfaceXcb *)icd_surface;
-
-   struct wsi_x11_connection *wsi_conn =
-      wsi_x11_get_connection(device->instance, surface->connection);
-   if (!wsi_conn)
-      return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-
-   if (!wsi_conn->has_dri3) {
-      fprintf(stderr, "vulkan: No DRI3 support\n");
-      *pSupported = false;
-      return VK_SUCCESS;
-   }
-
-   unsigned visual_depth;
-   if (!get_visualtype_for_window(surface->connection, surface->window,
-                                  &visual_depth)) {
-      *pSupported = false;
-      return VK_SUCCESS;
-   }
-
-   if (visual_depth != 24 && visual_depth != 32) {
-      *pSupported = false;
-      return VK_SUCCESS;
-   }
-
-   *pSupported = true;
-   return VK_SUCCESS;
+   return wsi_get_physical_device_xcb_presentation_support(
+      &device->wsi_device,
+      queueFamilyIndex,
+      connection, visual_id);
 }
 
-static VkResult
-x11_surface_get_capabilities(VkIcdSurfaceBase *icd_surface,
-                             struct anv_physical_device *device,
-                             VkSurfaceCapabilitiesKHR *caps)
-{
-   VkIcdSurfaceXcb *surface = (VkIcdSurfaceXcb *)icd_surface;
-   xcb_get_geometry_cookie_t geom_cookie;
-   xcb_generic_error_t *err;
-   xcb_get_geometry_reply_t *geom;
-   unsigned visual_depth;
-
-   geom_cookie = xcb_get_geometry(surface->connection, surface->window);
-
-   /* This does a round-trip.  This is why we do get_geometry first and
-    * wait to read the reply until after we have a visual.
-    */
-   xcb_visualtype_t *visual =
-      get_visualtype_for_window(surface->connection, surface->window,
-                                &visual_depth);
-
-   geom = xcb_get_geometry_reply(surface->connection, geom_cookie, &err);
-   if (geom) {
-      VkExtent2D extent = { geom->width, geom->height };
-      caps->currentExtent = extent;
-      caps->minImageExtent = extent;
-      caps->maxImageExtent = extent;
-   } else {
-      /* This can happen if the client didn't wait for the configure event
-       * to come back from the compositor.  In that case, we don't know the
-       * size of the window so we just return valid "I don't know" stuff.
-       */
-      caps->currentExtent = (VkExtent2D) { -1, -1 };
-      caps->minImageExtent = (VkExtent2D) { 1, 1 };
-      caps->maxImageExtent = (VkExtent2D) { INT16_MAX, INT16_MAX };
-   }
-   free(err);
-   free(geom);
-
-   if (visual_has_alpha(visual, visual_depth)) {
-      caps->supportedCompositeAlpha = VK_COMPOSITE_ALPHA_INHERIT_BIT_KHR |
-                                      VK_COMPOSITE_ALPHA_PRE_MULTIPLIED_BIT_KHR;
-   } else {
-      caps->supportedCompositeAlpha = VK_COMPOSITE_ALPHA_INHERIT_BIT_KHR |
-                                      VK_COMPOSITE_ALPHA_OPAQUE_BIT_KHR;
-   }
-
-   caps->minImageCount = 2;
-   caps->maxImageCount = 4;
-   caps->supportedTransforms = VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR;
-   caps->currentTransform = VK_SURFACE_TRANSFORM_IDENTITY_BIT_KHR;
-   caps->maxImageArrayLayers = 1;
-   caps->supportedUsageFlags =
-      VK_IMAGE_USAGE_TRANSFER_SRC_BIT |
-      VK_IMAGE_USAGE_SAMPLED_BIT |
-      VK_IMAGE_USAGE_TRANSFER_DST_BIT |
-      VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT;
-
-   return VK_SUCCESS;
-}
-
-static VkResult
-x11_surface_get_formats(VkIcdSurfaceBase *surface,
-                        struct anv_physical_device *device,
-                        uint32_t *pSurfaceFormatCount,
-                        VkSurfaceFormatKHR *pSurfaceFormats)
-{
-   if (pSurfaceFormats == NULL) {
-      *pSurfaceFormatCount = ARRAY_SIZE(formats);
-      return VK_SUCCESS;
-   }
-
-   assert(*pSurfaceFormatCount >= ARRAY_SIZE(formats));
-   typed_memcpy(pSurfaceFormats, formats, *pSurfaceFormatCount);
-   *pSurfaceFormatCount = ARRAY_SIZE(formats);
-
-   return VK_SUCCESS;
-}
-
-static VkResult
-x11_surface_get_present_modes(VkIcdSurfaceBase *surface,
-                              struct anv_physical_device *device,
-                              uint32_t *pPresentModeCount,
-                              VkPresentModeKHR *pPresentModes)
+VkBool32 anv_GetPhysicalDeviceXlibPresentationSupportKHR(
+    VkPhysicalDevice                            physicalDevice,
+    uint32_t                                    queueFamilyIndex,
+    Display*                                    dpy,
+    VisualID                                    visualID)
 {
-   if (pPresentModes == NULL) {
-      *pPresentModeCount = ARRAY_SIZE(present_modes);
-      return VK_SUCCESS;
-   }
-
-   assert(*pPresentModeCount >= ARRAY_SIZE(present_modes));
-   typed_memcpy(pPresentModes, present_modes, *pPresentModeCount);
-   *pPresentModeCount = ARRAY_SIZE(present_modes);
+   ANV_FROM_HANDLE(anv_physical_device, device, physicalDevice);
 
-   return VK_SUCCESS;
+   return wsi_get_physical_device_xcb_presentation_support(
+      &device->wsi_device,
+      queueFamilyIndex,
+      XGetXCBConnection(dpy), visualID);
 }
 
-static VkResult
-x11_surface_create_swapchain(VkIcdSurfaceBase *surface,
-                             struct anv_device *device,
-                             const VkSwapchainCreateInfoKHR* pCreateInfo,
-                             const VkAllocationCallbacks* pAllocator,
-                             struct anv_swapchain **swapchain);
-
 VkResult anv_CreateXcbSurfaceKHR(
     VkInstance                                  _instance,
     const VkXcbSurfaceCreateInfoKHR*            pCreateInfo,
@@ -397,366 +65,32 @@ VkResult anv_CreateXcbSurfaceKHR(
     VkSurfaceKHR*                               pSurface)
 {
    ANV_FROM_HANDLE(anv_instance, instance, _instance);
-
+   const VkAllocationCallbacks *alloc;
    assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_XCB_SURFACE_CREATE_INFO_KHR);
 
-   VkIcdSurfaceXcb *surface;
-
-   surface = anv_alloc2(&instance->alloc, pAllocator, sizeof *surface, 8,
-                        VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
-   if (surface == NULL)
-      return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-
-   surface->base.platform = VK_ICD_WSI_PLATFORM_XCB;
-   surface->connection = pCreateInfo->connection;
-   surface->window = pCreateInfo->window;
-
-   *pSurface = _VkIcdSurfaceBase_to_handle(&surface->base);
-
-   return VK_SUCCESS;
-}
-
-struct x11_image {
-   struct anv_image *                        image;
-   struct anv_device_memory *                memory;
-   xcb_pixmap_t                              pixmap;
-   xcb_get_geometry_cookie_t                 geom_cookie;
-   bool                                      busy;
-};
-
-struct x11_swapchain {
-   struct anv_swapchain                        base;
-
-   xcb_connection_t *                           conn;
-   xcb_window_t                                 window;
-   xcb_gc_t                                     gc;
-   VkExtent2D                                   extent;
-   uint32_t                                     image_count;
-   uint32_t                                     next_image;
-   struct x11_image                             images[0];
-};
-
-static VkResult
-x11_get_images(struct anv_swapchain *anv_chain,
-               uint32_t* pCount, VkImage *pSwapchainImages)
-{
-   struct x11_swapchain *chain = (struct x11_swapchain *)anv_chain;
-
-   if (pSwapchainImages == NULL) {
-      *pCount = chain->image_count;
-      return VK_SUCCESS;
-   }
-
-   assert(chain->image_count <= *pCount);
-   for (uint32_t i = 0; i < chain->image_count; i++)
-      pSwapchainImages[i] = anv_image_to_handle(chain->images[i].image);
-
-   *pCount = chain->image_count;
-
-   return VK_SUCCESS;
-}
-
-static VkResult
-x11_acquire_next_image(struct anv_swapchain *anv_chain,
-                       uint64_t timeout,
-                       VkSemaphore semaphore,
-                       uint32_t *image_index)
-{
-   struct x11_swapchain *chain = (struct x11_swapchain *)anv_chain;
-   struct x11_image *image = &chain->images[chain->next_image];
-
-   if (image->busy) {
-      xcb_generic_error_t *err;
-      xcb_get_geometry_reply_t *geom =
-         xcb_get_geometry_reply(chain->conn, image->geom_cookie, &err);
-      if (!geom) {
-         free(err);
-         return vk_error(VK_ERROR_OUT_OF_DATE_KHR);
-      }
-
-      if (geom->width != chain->extent.width ||
-          geom->height != chain->extent.height) {
-         free(geom);
-         return vk_error(VK_ERROR_OUT_OF_DATE_KHR);
-      }
-      free(geom);
-
-      image->busy = false;
-   }
-
-   *image_index = chain->next_image;
-   chain->next_image = (chain->next_image + 1) % chain->image_count;
-   return VK_SUCCESS;
-}
-
-static VkResult
-x11_queue_present(struct anv_swapchain *anv_chain,
-                  struct anv_queue *queue,
-                  uint32_t image_index)
-{
-   struct x11_swapchain *chain = (struct x11_swapchain *)anv_chain;
-   struct x11_image *image = &chain->images[image_index];
-
-   assert(image_index < chain->image_count);
-
-   xcb_void_cookie_t cookie;
-
-   cookie = xcb_copy_area(chain->conn,
-                          image->pixmap,
-                          chain->window,
-                          chain->gc,
-                          0, 0,
-                          0, 0,
-                          chain->extent.width,
-                          chain->extent.height);
-   xcb_discard_reply(chain->conn, cookie.sequence);
-
-   image->geom_cookie = xcb_get_geometry(chain->conn, chain->window);
-   image->busy = true;
-
-   xcb_flush(chain->conn);
-
-   return VK_SUCCESS;
-}
-
-static VkResult
-x11_swapchain_destroy(struct anv_swapchain *anv_chain,
-                      const VkAllocationCallbacks *pAllocator)
-{
-   struct x11_swapchain *chain = (struct x11_swapchain *)anv_chain;
-   xcb_void_cookie_t cookie;
-
-   for (uint32_t i = 0; i < chain->image_count; i++) {
-      struct x11_image *image = &chain->images[i];
-
-      if (image->busy)
-         xcb_discard_reply(chain->conn, image->geom_cookie.sequence);
-
-      cookie = xcb_free_pixmap(chain->conn, image->pixmap);
-      xcb_discard_reply(chain->conn, cookie.sequence);
-
-      anv_DestroyImage(anv_device_to_handle(chain->base.device),
-                       anv_image_to_handle(image->image), pAllocator);
-
-      anv_FreeMemory(anv_device_to_handle(chain->base.device),
-                     anv_device_memory_to_handle(image->memory), pAllocator);
-   }
-
-   anv_free2(&chain->base.device->alloc, pAllocator, chain);
+   if (pAllocator)
+     alloc = pAllocator;
+   else
+     alloc = &instance->alloc;
 
-   return VK_SUCCESS;
+   return wsi_create_xcb_surface(alloc, pCreateInfo, pSurface);
 }
 
-static VkResult
-x11_surface_create_swapchain(VkIcdSurfaceBase *icd_surface,
-                             struct anv_device *device,
-                             const VkSwapchainCreateInfoKHR *pCreateInfo,
-                             const VkAllocationCallbacks* pAllocator,
-                             struct anv_swapchain **swapchain_out)
-{
-   VkIcdSurfaceXcb *surface = (VkIcdSurfaceXcb *)icd_surface;
-   struct x11_swapchain *chain;
-   xcb_void_cookie_t cookie;
-   VkResult result;
-
-   int num_images = pCreateInfo->minImageCount;
-
-   assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_SWAPCHAIN_CREATE_INFO_KHR);
-
-   size_t size = sizeof(*chain) + num_images * sizeof(chain->images[0]);
-   chain = anv_alloc2(&device->alloc, pAllocator, size, 8,
-                      VK_SYSTEM_ALLOCATION_SCOPE_OBJECT);
-   if (chain == NULL)
-      return vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-
-   chain->base.device = device;
-   chain->base.destroy = x11_swapchain_destroy;
-   chain->base.get_images = x11_get_images;
-   chain->base.acquire_next_image = x11_acquire_next_image;
-   chain->base.queue_present = x11_queue_present;
-
-   chain->conn = surface->connection;
-   chain->window = surface->window;
-   chain->extent = pCreateInfo->imageExtent;
-   chain->image_count = num_images;
-   chain->next_image = 0;
-
-   for (uint32_t i = 0; i < chain->image_count; i++) {
-      VkDeviceMemory memory_h;
-      VkImage image_h;
-      struct anv_image *image;
-      struct anv_surface *surface;
-      struct anv_device_memory *memory;
-
-      anv_image_create(anv_device_to_handle(device),
-         &(struct anv_image_create_info) {
-            .isl_tiling_flags = ISL_TILING_X_BIT,
-            .stride = 0,
-            .vk_info =
-         &(VkImageCreateInfo) {
-            .sType = VK_STRUCTURE_TYPE_IMAGE_CREATE_INFO,
-            .imageType = VK_IMAGE_TYPE_2D,
-            .format = pCreateInfo->imageFormat,
-            .extent = {
-               .width = pCreateInfo->imageExtent.width,
-               .height = pCreateInfo->imageExtent.height,
-               .depth = 1
-            },
-            .mipLevels = 1,
-            .arrayLayers = 1,
-            .samples = 1,
-            /* FIXME: Need a way to use X tiling to allow scanout */
-            .tiling = VK_IMAGE_TILING_OPTIMAL,
-            .usage = VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT,
-            .flags = 0,
-         }},
-         NULL,
-         &image_h);
-
-      image = anv_image_from_handle(image_h);
-      assert(anv_format_is_color(image->format));
-
-      surface = &image->color_surface;
-
-      anv_AllocateMemory(anv_device_to_handle(device),
-         &(VkMemoryAllocateInfo) {
-            .sType = VK_STRUCTURE_TYPE_MEMORY_ALLOCATE_INFO,
-            .allocationSize = image->size,
-            .memoryTypeIndex = 0,
-         },
-         NULL /* XXX: pAllocator */,
-         &memory_h);
-
-      memory = anv_device_memory_from_handle(memory_h);
-      memory->bo.is_winsys_bo = true;
-
-      anv_BindImageMemory(VK_NULL_HANDLE, anv_image_to_handle(image),
-                          memory_h, 0);
-
-      int ret = anv_gem_set_tiling(device, memory->bo.gem_handle,
-                                   surface->isl.row_pitch, I915_TILING_X);
-      if (ret) {
-         /* FINISHME: Choose a better error. */
-         result = vk_errorf(VK_ERROR_OUT_OF_DEVICE_MEMORY,
-                            "set_tiling failed: %m");
-         goto fail;
-      }
-
-      int fd = anv_gem_handle_to_fd(device, memory->bo.gem_handle);
-      if (fd == -1) {
-         /* FINISHME: Choose a better error. */
-         result = vk_errorf(VK_ERROR_OUT_OF_DEVICE_MEMORY,
-                            "handle_to_fd failed: %m");
-         goto fail;
-      }
-
-      uint32_t bpp = 32;
-      uint32_t depth = 24;
-      xcb_pixmap_t pixmap = xcb_generate_id(chain->conn);
-
-      cookie =
-         xcb_dri3_pixmap_from_buffer_checked(chain->conn,
-                                             pixmap,
-                                             chain->window,
-                                             image->size,
-                                             pCreateInfo->imageExtent.width,
-                                             pCreateInfo->imageExtent.height,
-                                             surface->isl.row_pitch,
-                                             depth, bpp, fd);
-
-      chain->images[i].image = image;
-      chain->images[i].memory = memory;
-      chain->images[i].pixmap = pixmap;
-      chain->images[i].busy = false;
-
-      xcb_discard_reply(chain->conn, cookie.sequence);
-   }
-
-   chain->gc = xcb_generate_id(chain->conn);
-   if (!chain->gc) {
-      /* FINISHME: Choose a better error. */
-      result = vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-      goto fail;
-   }
-
-   cookie = xcb_create_gc(chain->conn,
-                          chain->gc,
-                          chain->window,
-                          XCB_GC_GRAPHICS_EXPOSURES,
-                          (uint32_t []) { 0 });
-   xcb_discard_reply(chain->conn, cookie.sequence);
-
-   *swapchain_out = &chain->base;
-
-   return VK_SUCCESS;
-
- fail:
-   return result;
-}
-
-VkResult
-anv_x11_init_wsi(struct anv_instance *instance)
-{
-   struct wsi_x11 *wsi;
-   VkResult result;
-
-   wsi = anv_alloc(&instance->alloc, sizeof(*wsi), 8,
-                   VK_SYSTEM_ALLOCATION_SCOPE_INSTANCE);
-   if (!wsi) {
-      result = vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-      goto fail;
-   }
-
-   int ret = pthread_mutex_init(&wsi->mutex, NULL);
-   if (ret != 0) {
-      if (ret == ENOMEM) {
-         result = vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-      } else {
-         /* FINISHME: Choose a better error. */
-         result = vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-      }
-
-      goto fail_alloc;
-   }
-
-   wsi->connections = _mesa_hash_table_create(NULL, _mesa_hash_pointer,
-                                              _mesa_key_pointer_equal);
-   if (!wsi->connections) {
-      result = vk_error(VK_ERROR_OUT_OF_HOST_MEMORY);
-      goto fail_mutex;
-   }
-
-   wsi->base.get_support = x11_surface_get_support;
-   wsi->base.get_capabilities = x11_surface_get_capabilities;
-   wsi->base.get_formats = x11_surface_get_formats;
-   wsi->base.get_present_modes = x11_surface_get_present_modes;
-   wsi->base.create_swapchain = x11_surface_create_swapchain;
-
-   instance->wsi[VK_ICD_WSI_PLATFORM_XCB] = &wsi->base;
-
-   return VK_SUCCESS;
-
-fail_mutex:
-   pthread_mutex_destroy(&wsi->mutex);
-fail_alloc:
-   anv_free(&instance->alloc, wsi);
-fail:
-   instance->wsi[VK_ICD_WSI_PLATFORM_XCB] = NULL;
-
-   return result;
-}
-
-void
-anv_x11_finish_wsi(struct anv_instance *instance)
+VkResult anv_CreateXlibSurfaceKHR(
+    VkInstance                                  _instance,
+    const VkXlibSurfaceCreateInfoKHR*           pCreateInfo,
+    const VkAllocationCallbacks*                pAllocator,
+    VkSurfaceKHR*                               pSurface)
 {
-   struct wsi_x11 *wsi =
-      (struct wsi_x11 *)instance->wsi[VK_ICD_WSI_PLATFORM_XCB];
+   ANV_FROM_HANDLE(anv_instance, instance, _instance);
+   const VkAllocationCallbacks *alloc;
 
-   if (wsi) {
-      _mesa_hash_table_destroy(wsi->connections, NULL);
+   assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_XLIB_SURFACE_CREATE_INFO_KHR);
 
-      pthread_mutex_destroy(&wsi->mutex);
+   if (pAllocator)
+     alloc = pAllocator;
+   else
+     alloc = &instance->alloc;
 
-      anv_free(&instance->alloc, wsi);
-   }
+   return wsi_create_xlib_surface(alloc, pCreateInfo, pSurface);
 }