From 0c638d08e5ed3f90ab7bd1c3ae4b6a798eb0aa83 Mon Sep 17 00:00:00 2001 From: Paulo Zanoni Date: Fri, 26 Sep 2025 16:30:58 -0700 Subject: [PATCH] anv: we never set I915_EXEC_FENCE_OUT MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Don't check something that will never be there, just assert() it. Reviewed-by: José Roberto de Souza Signed-off-by: Paulo Zanoni Part-of: --- src/intel/vulkan/i915/anv_batch_chain.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/intel/vulkan/i915/anv_batch_chain.c b/src/intel/vulkan/i915/anv_batch_chain.c index 752ea5dbc28..94ad5a5e50a 100644 --- a/src/intel/vulkan/i915/anv_batch_chain.c +++ b/src/intel/vulkan/i915/anv_batch_chain.c @@ -677,17 +677,15 @@ anv_gem_execbuffer_impl(struct anv_queue *queue, const char *func, int line) { struct anv_device *device = queue->device; - int ret; - const unsigned long request = (execbuf->flags & I915_EXEC_FENCE_OUT) ? - DRM_IOCTL_I915_GEM_EXECBUFFER2_WR : - DRM_IOCTL_I915_GEM_EXECBUFFER2; + + assert((execbuf->flags & I915_EXEC_FENCE_OUT) == 0); if (unlikely(device->info->no_hw)) return VK_SUCCESS; do { - ret = intel_ioctl(device->fd, request, execbuf); + ret = intel_ioctl(device->fd, DRM_IOCTL_I915_GEM_EXECBUFFER2, execbuf); } while (ret && errno == ENOMEM); if (ret)