drm/i915/execlists: Ignore tracek for nop process_csb
authorChris Wilson <chris@chris-wilson.co.uk>
Sun, 9 Feb 2020 13:19:16 +0000 (13:19 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Sun, 9 Feb 2020 13:47:09 +0000 (13:47 +0000)
Recording the frequent inspection of CSB head/tail when there is
expected to be no update adds noise to the debug trace. (Not entirely
useless, but since we know the sequence of function calls, we can
surmise the function was called -- so redundant.)

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200209131922.180287-2-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/gt/intel_lrc.c

index 2138507..230470c 100644 (file)
@@ -2248,7 +2248,6 @@ static void process_csb(struct intel_engine_cs *engine)
         */
        head = execlists->csb_head;
        tail = READ_ONCE(*execlists->csb_write);
-       ENGINE_TRACE(engine, "cs-irq head=%d, tail=%d\n", head, tail);
        if (unlikely(head == tail))
                return;
 
@@ -2262,6 +2261,7 @@ static void process_csb(struct intel_engine_cs *engine)
         */
        rmb();
 
+       ENGINE_TRACE(engine, "cs-irq head=%d, tail=%d\n", head, tail);
        do {
                bool promote;