Hi Tvrtko,

> +static void all(int i915)
> +{
> +     const struct intel_execution_engine2 *e;
> +
> +     __for_each_physical_engine(i915, e)
> +             basic(i915, e, false);

ops... there is an error in my previous patch.

I restored the meaining of flags because you and Chris didn't
like the boolean true/false meaning, indeed here I still check
with:

        if ((flags & HANG) == 0) {

but then I forgot to update the test cases.

This means that this line needs to be

        basic(i915, e, 0);

as the rest.

I'm sorry about the confusion! In any case,

Reviewed-by: Andi Shyti <[email protected]>

Thanks,
Andi
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to