Re: [PATCH i-g-t 1/2] benchmarks/gem_wsim: Fail on thread assertions

[Adam Miszczak <adam.miszczak@linux.intel.com>]

On 9/24/2025 1:06 PM, Marcin Bernatowicz wrote:
> Call igt_thread_assert_no_failures() before returning so thread errors
> affect the exit status.
>
> Signed-off-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>
> Cc: Adam Miszczak <adam.miszczak@linux.intel.com>
> Cc: Kamil Konieczny <kamil.konieczny@linux.intel.com>
> Cc: Lukasz Laguna <lukasz.laguna@intel.com>
> ---
>   benchmarks/gem_wsim.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c
> index 7a6b10028..098415b53 100644
> --- a/benchmarks/gem_wsim.c
> +++ b/benchmarks/gem_wsim.c
> @@ -63,6 +63,7 @@
>   #include "i915/gem_mman.h"
>   
>   #include "igt_syncobj.h"
> +#include "igt_thread.h"
>   #include "intel_allocator.h"
>   #include "xe_drm.h"
>   #include "xe/xe_ioctl.h"
> @@ -3362,5 +3363,7 @@ err:
>   	if (is_xe)
>   		xe_device_put(fd);
>   
> +	igt_thread_assert_no_failures();
> +
>   	return exitcode;
>   }

LGTM,
Reviewed-by: Adam Miszczak <adam.miszczak@linux.intel.com>