Filter out warnings for the kernel context that is used to flush
inactive contexts, as they do no not pose a risk.

Signed-off-by: Chris Wilson <[email protected]>
---
 drivers/gpu/drm/i915/gt/intel_lrc.c | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/gt/intel_lrc.c 
b/drivers/gpu/drm/i915/gt/intel_lrc.c
index 51aef2a233cb..ca2d3416fa4d 100644
--- a/drivers/gpu/drm/i915/gt/intel_lrc.c
+++ b/drivers/gpu/drm/i915/gt/intel_lrc.c
@@ -1236,7 +1236,8 @@ assert_pending_valid(const struct intel_engine_execlists 
*execlists,
 
        for (port = execlists->pending; (rq = *port); port++) {
                if (ce == rq->hw_context) {
-                       GEM_TRACE_ERR("Duplicate context in pending[%zd]\n",
+                       GEM_TRACE_ERR("Dup context:%llx in pending[%zd]\n",
+                                     ce->timeline->fence_context,
                                      port - execlists->pending);
                        return false;
                }
@@ -1245,20 +1246,24 @@ assert_pending_valid(const struct 
intel_engine_execlists *execlists,
                if (i915_request_completed(rq))
                        continue;
 
-               if (i915_active_is_idle(&ce->active)) {
-                       GEM_TRACE_ERR("Inactive context in pending[%zd]\n",
+               if (i915_active_is_idle(&ce->active) &&
+                   !i915_gem_context_is_kernel(ce->gem_context)) {
+                       GEM_TRACE_ERR("Inactive context:%llx in pending[%zd]\n",
+                                     ce->timeline->fence_context,
                                      port - execlists->pending);
                        return false;
                }
 
                if (!i915_vma_is_pinned(ce->state)) {
-                       GEM_TRACE_ERR("Unpinned context in pending[%zd]\n",
+                       GEM_TRACE_ERR("Unpinned context:%llx in pending[%zd]\n",
+                                     ce->timeline->fence_context,
                                      port - execlists->pending);
                        return false;
                }
 
                if (!i915_vma_is_pinned(ce->ring->vma)) {
-                       GEM_TRACE_ERR("Unpinned ringbuffer in pending[%zd]\n",
+                       GEM_TRACE_ERR("Unpinned ring:%llx in pending[%zd]\n",
+                                     ce->timeline->fence_context,
                                      port - execlists->pending);
                        return false;
                }
-- 
2.24.0.rc2

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

Reply via email to