From ff50e805746d8f614bac8b00e2889028b47f481e Mon Sep 17 00:00:00 2001 From: Dave Airlie Date: Thu, 1 Feb 2024 10:56:05 +1000 Subject: [PATCH] zink: use sparse residency for buffers. GL ARB_sparse_buffer allows unbound regions in buffers. VK sparseBinding insists all regions must be bound before first use. This means we need to use sparseResidencyBuffer to back GL sparse buffers to get the same semantics. Fixes GL and piglit sparse buffer tests on zink/nvk. Fixes: c90246b68268 ("zink: implement sparse buffer creation/mapping") Cc: mesa-stable Part-of: --- src/gallium/drivers/zink/zink_resource.c | 2 +- src/gallium/drivers/zink/zink_screen.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/zink/zink_resource.c b/src/gallium/drivers/zink/zink_resource.c index 6e55d81cddc..3198effe03f 100644 --- a/src/gallium/drivers/zink/zink_resource.c +++ b/src/gallium/drivers/zink/zink_resource.c @@ -309,7 +309,7 @@ create_bci(struct zink_screen *screen, const struct pipe_resource *templ, unsign bci.usage |= VK_BUFFER_USAGE_CONDITIONAL_RENDERING_BIT_EXT; if (templ->flags & PIPE_RESOURCE_FLAG_SPARSE) - bci.flags |= VK_BUFFER_CREATE_SPARSE_BINDING_BIT; + bci.flags |= VK_BUFFER_CREATE_SPARSE_BINDING_BIT | VK_BUFFER_CREATE_SPARSE_RESIDENCY_BIT; return bci; } diff --git a/src/gallium/drivers/zink/zink_screen.c b/src/gallium/drivers/zink/zink_screen.c index fb9ab486b04..26e61d381c4 100644 --- a/src/gallium/drivers/zink/zink_screen.c +++ b/src/gallium/drivers/zink/zink_screen.c @@ -940,7 +940,7 @@ zink_get_param(struct pipe_screen *pscreen, enum pipe_cap param) return screen->info.feats.features.shaderCullDistance; case PIPE_CAP_SPARSE_BUFFER_PAGE_SIZE: - return screen->info.feats.features.sparseBinding ? ZINK_SPARSE_BUFFER_PAGE_SIZE : 0; + return screen->info.feats.features.sparseResidencyBuffer ? ZINK_SPARSE_BUFFER_PAGE_SIZE : 0; /* Sparse texture */ case PIPE_CAP_MAX_SPARSE_TEXTURE_SIZE: