drm/i915/guc: Ensure H2G buffer updates visible before tail update
authorMatthew Brost <matthew.brost@intel.com>
Thu, 3 Jun 2021 05:16:25 +0000 (22:16 -0700)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 4 Jun 2021 08:39:04 +0000 (10:39 +0200)
Ensure H2G buffer updates are visible before descriptor tail updates by
inserting a barrier between the H2G buffer update and the tail. The
barrier is simple wmb() for SMEM and is register write for LMEM. This is
needed if more than 1 H2G can be inflight at once.

If this barrier is not inserted it is possible the descriptor tail
update is scene by the GuC before H2G buffer update which results in the
GuC reading a corrupt H2G value. This can bring down the H2G channel
among other bad things.

Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: John Harrison <John.C.Harrison@Intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20210603051630.2635-16-matthew.brost@intel.com
drivers/gpu/drm/i915/gt/uc/intel_guc_ct.c

index 5b9f694..b07d22d 100644 (file)
@@ -328,6 +328,28 @@ static u32 ct_get_next_fence(struct intel_guc_ct *ct)
        return ++ct->requests.last_fence;
 }
 
+static void write_barrier(struct intel_guc_ct *ct)
+{
+       struct intel_guc *guc = ct_to_guc(ct);
+       struct intel_gt *gt = guc_to_gt(guc);
+
+       if (i915_gem_object_is_lmem(guc->ct.vma->obj)) {
+               GEM_BUG_ON(guc->send_regs.fw_domains);
+               /*
+                * This register is used by the i915 and GuC for MMIO based
+                * communication. Once we are in this code CTBs are the only
+                * method the i915 uses to communicate with the GuC so it is
+                * safe to write to this register (a value of 0 is NOP for MMIO
+                * communication). If we ever start mixing CTBs and MMIOs a new
+                * register will have to be chosen.
+                */
+               intel_uncore_write_fw(gt->uncore, GEN11_SOFT_SCRATCH(0), 0);
+       } else {
+               /* wmb() sufficient for a barrier if in smem */
+               wmb();
+       }
+}
+
 /**
  * DOC: CTB Host to GuC request
  *
@@ -411,6 +433,12 @@ static int ct_write(struct intel_guc_ct *ct,
        }
        GEM_BUG_ON(tail > size);
 
+       /*
+        * make sure H2G buffer update and LRC tail update (if this triggering a
+        * submission) are visible before updating the descriptor tail
+        */
+       write_barrier(ct);
+
        /* now update desc tail (back in bytes) */
        desc->tail = tail * 4;
        return 0;