diff mbox

[i-g-t] lib/igt_gt: Allow non-default contexts to hang non-render rings

Message ID 1507679991-26908-1-git-send-email-vinay.belgaumkar@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Vinay Belgaumkar Oct. 10, 2017, 11:59 p.m. UTC 
This limitation does not exist for gen8+.

Cc: Michel Thierry <michel.thierry@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Petri Latvala <petri.latvala@intel.com>
Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
---
 lib/igt_gt.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Chris Wilson Oct. 11, 2017, 12:51 a.m. UTC | #1 
Quoting Vinay Belgaumkar (2017-10-11 00:59:51)
> This limitation does not exist for gen8+.

Oh yes it did. "It did" being the operative phrase there. Read the
comment, find the commit that changed the ABI.
-Chris
diff mbox

Patch

diff --git a/lib/igt_gt.c b/lib/igt_gt.c
index b3f3b38..abf789d 100644
--- a/lib/igt_gt.c
+++ b/lib/igt_gt.c
@@ -214,6 +214,7 @@  void igt_disallow_hang(int fd, igt_hang_t arg)
  * @ctx: the contxt specifier
  * @ring: execbuf ring flag
  * @flags: set of flags to control execution
+ * @offset: The resultant gtt offset of the exec obj
  *
  * This helper function injects a hanging batch associated with @ctx into @ring.
  * It returns a #igt_hang_t structure which must be passed to
@@ -239,8 +240,11 @@  igt_hang_t igt_hang_ctx(int fd,
 
 	igt_require_hang_ring(fd, ring);
 
-	/* One day the kernel ABI will be fixed! */
-	igt_require(ctx == 0 || ring == I915_EXEC_RENDER);
+	if (!(intel_gen(intel_get_drm_devid(fd)) >= 8))
+	{
+		/* One day the kernel ABI will be fixed! */
+		igt_require(ctx == 0 || ring == I915_EXEC_RENDER);
+	}
 
 	param.context = ctx;
 	param.size = 0;