Re: [PATCH i-g-t 3/3] tests/xe_sriov_scheduling: fix ticks_delta check for non-inverted logic

Zbigniew Kempczyński <[email protected]> Fri, 31 Jul 2026 17:54:11 +0200
Newsgroups org.freedesktop.lists.igt-dev
Message-ID <dslty6wmwqeohnirbfmiu7ejpxbmpqvqmjqx3uj254fly2fytm@x44g6pdsjjod>
On Fri, Jul 31, 2026 at 04:18:06PM +0200, Marcin Bernatowicz wrote:
> xe_spin now stores elapsed ticks directly in ticks_delta (not inverted).
> Update subm_is_work_complete() accordingly.
>

Reviewed-by: Zbigniew Kempczyński <[email protected]>

--
Zbigniew

> Signed-off-by: Marcin Bernatowicz <[email protected]>
> Cc: Adam Miszczak <[email protected]>
> Cc: Kamil Konieczny <[email protected]>
> Cc: Lukasz Laguna <[email protected]>
> Cc: Zbigniew Kempczyński <[email protected]>
> ---
>  tests/intel/xe_sriov_scheduling.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_sriov_scheduling.c b/tests/intel/xe_sriov_scheduling.c
> index 47cc5d9b0..5d30b961d 100644
> --- a/tests/intel/xe_sriov_scheduling.c
> +++ b/tests/intel/xe_sriov_scheduling.c
> @@ -189,7 +189,7 @@ static void subm_exec_slot(struct subm *s, unsigned int slot)
>  
>  static bool subm_is_work_complete(struct subm *s, unsigned int slot)
>  {
> -	return s->expected_ticks <= ~s->spin[slot]->ticks_delta;
> +	return s->expected_ticks <= s->spin[slot]->ticks_delta;
>  }
>  
>  static bool subm_is_exec_queue_banned(struct subm *s)
> -- 
> 2.43.0
>