[igt-dev] [PATCH i-g-t] i915/gem_busy: Prevent context ban with right ctx id

Dixit, Ashutosh ashutosh.dixit at intel.com
Mon Jan 24 16:29:45 UTC 2022


On Sun, 23 Jan 2022 18:38:33 -0800, Chuansheng Liu wrote:
>
> This patch corrects the ctx id accordindly. Furthermore,
> there are other code places with similar mistakes, will
> try to send out the fixes further.

Yes we know about this issue. Maybe a single patch which fixes these
globally is preferred (to avoid having separate commit messages
etc.). Anyway for this:

Reviewed-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

> Signed-off-by: Chuansheng Liu <chuansheng.liu at intel.com>
> ---
>  tests/i915/gem_busy.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_busy.c b/tests/i915/gem_busy.c
> index b4757736..b8a61084 100644
> --- a/tests/i915/gem_busy.c
> +++ b/tests/i915/gem_busy.c
> @@ -517,7 +517,7 @@ igt_main
>		igt_hang_t hang;
>
>		igt_fixture {
> -			hang = igt_allow_hang(fd, 0, 0);
> +			hang = igt_allow_hang(fd, ctx->id, 0);
>		}
>
>		test_each_engine("hang", fd, ctx, e) {
> --
> 2.25.0.rc2


More information about the igt-dev mailing list