From 7ce83699852262c7723e05de25ba21bf341d9933 Mon Sep 17 00:00:00 2001 From: Karol Herbst Date: Sun, 20 Jul 2025 21:27:29 +0200 Subject: [PATCH] rusticl/queue: do not return event status errors on flush/finish Fixes random fails in the test_events userevents test as it sets an event to -1 and clFinish returned that error code making the test fail. Fixes: 3129fd8dcfa ("rusticl/queue: check device error status") Part-of: --- src/gallium/frontends/rusticl/core/queue.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/frontends/rusticl/core/queue.rs b/src/gallium/frontends/rusticl/core/queue.rs index a73db4f2280..3ec83bcf998 100644 --- a/src/gallium/frontends/rusticl/core/queue.rs +++ b/src/gallium/frontends/rusticl/core/queue.rs @@ -476,10 +476,10 @@ impl Queue { // Waiting on the last event is good enough here as the queue will process it in order // It's not a problem if the weak ref is invalid as that means the work is already done // and waiting isn't necessary anymore. - let err = last.upgrade().map(|e| e.wait()).unwrap_or_default(); - if err < 0 { - return Err(err); - } + // + // We also ignore any error state of events as it's the callers responsibility to check + // for it if it cares. + last.upgrade().map(|e| e.wait()); } Ok(()) }