From 05974a7934cd709e3218400ee7e17c96c81cd84d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Zbigniew=20Kempczy=C5=84ski?=
<zbigniew.kempczynski@intel.com>
Date: Thu, 23 Feb 2023 09:11:19 +0100
Subject: [PATCH] i915/gem_softpin: Ensure gpu is idle when we use manual
placement
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
For separate execution there is no risk we may reserve offset which is
still in use by some job. But for running bunch of allocator tests
which may be still on gpu such risk exists. To avoid failures on
grouped execution ensure for 'nopin' subtests gpu is idle first.
Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
---
tests/i915/gem_softpin.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/i915/gem_softpin.c b/tests/i915/gem_softpin.c
index ad6d3f531..c29bfd43d 100644
--- a/tests/i915/gem_softpin.c
+++ b/tests/i915/gem_softpin.c
@@ -1449,13 +1449,17 @@ igt_main
igt_describe("Check that we can combine manual placement with automatic"
" GTT placement.");
- igt_subtest("allocator-nopin")
+ igt_subtest("allocator-nopin") {
+ gem_quiescent_gpu(fd);
test_allocator_nopin(fd, false);
+ }
igt_describe("Check that we can combine manual placement with automatic"
" GTT placement and reserves/unreserves space for objects.");
- igt_subtest("allocator-nopin-reserve")
+ igt_subtest("allocator-nopin-reserve") {
+ gem_quiescent_gpu(fd);
test_allocator_nopin(fd, true);
+ }
igt_describe("Check if multiple processes can use alloctor.");
igt_subtest("allocator-fork")
--
GitLab