From: Jason Ekstrand <jason@jlekstrand.net>
Date: Mon, 27 Jan 2020 20:42:20 +0000 (-0600)
Subject: intel/mi_builder: Force write completion on Gen12+
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=997040e4b8353fe9b71a5e9fde2f933eae09c7a3;p=mesa.git

intel/mi_builder: Force write completion on Gen12+

Otherwise, we have no guarantee that the write actually lands before we
move on to other things.  Doing this on every SDI is probably a bit
harsh but it's safe.  We should figure out a good way to avoid this when
we can.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3593>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3593>
---

diff --git a/src/intel/common/gen_mi_builder.h b/src/intel/common/gen_mi_builder.h
index dd7d87fc67a..bcd685754a3 100644
--- a/src/intel/common/gen_mi_builder.h
+++ b/src/intel/common/gen_mi_builder.h
@@ -358,6 +358,9 @@ _gen_mi_copy_no_unref(struct gen_mi_builder *b,
       case GEN_MI_VALUE_TYPE_IMM:
          gen_mi_builder_emit(b, GENX(MI_STORE_DATA_IMM), sdi) {
             sdi.Address = dst.addr;
+#if GEN_GEN >= 12
+            sdi.ForceWriteCompletionCheck = true;
+#endif
             sdi.ImmediateData = src.imm;
          }
          break;