zink: reject resource-imports with modifiers
Zink doesn't support forwarding DRM modifiers yet, so whenever those are
used, we end up ignoring them. That's not going to do the right thing in
most cases, so let's reject them instead.
Since d686835171c, the dri2 code tries to create a 0x0 surface without
any format when trying to import. This makes this go from
rendering-issues to asserting in debug builds, making things even worse.
Fixes: d686835171c ("gallium/dri2: Support I915_FORMAT_MOD_Y_TILED_GEN12_MC_CCS import")
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3654
Reviewed-by: Daniel Stone <daniels@collabora.com>
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7214>
diff --git a/src/gallium/drivers/zink/zink_resource.c b/src/gallium/drivers/zink/zink_resource.c
index e53e941..88bc347 100644
--- a/src/gallium/drivers/zink/zink_resource.c
+++ b/src/gallium/drivers/zink/zink_resource.c
@@ -36,6 +36,8 @@
#include "frontend/sw_winsys.h"
+#include "drm-uapi/drm_fourcc.h"
+
static void
zink_resource_destroy(struct pipe_screen *pscreen,
struct pipe_resource *pres)
@@ -339,6 +341,7 @@
if (result != VK_SUCCESS)
return false;
whandle->handle = fd;
+ whandle->modifier = DRM_FORMAT_MOD_INVALID;
}
return true;
}
@@ -349,6 +352,9 @@
struct winsys_handle *whandle,
unsigned usage)
{
+ if (whandle->modifier != DRM_FORMAT_MOD_INVALID)
+ return NULL;
+
return resource_create(pscreen, templ, whandle, usage);
}
