Quoting Matthew Auld (2019-08-09 23:26:22) > @@ -1017,10 +1020,14 @@ static void reloc_cache_reset(struct reloc_cache *cache) > } else { > struct i915_ggtt *ggtt = cache_to_ggtt(cache); > > - intel_gt_flush_ggtt_writes(ggtt->vm.gt); > + if (!cache->is_lmem) > + intel_gt_flush_ggtt_writes(ggtt->vm.gt); I love an optimist. At the least you might need the wmb(). But we have yet to see how many mistakes they've carried over into the new implementation ;) -Chris _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel