Quoting Zhao Yakui (2018-06-22 07:09:10) > @@ -2728,6 +2729,7 @@ populate_lr_context(struct i915_gem_context *ctx, > struct intel_engine_cs *engine, > struct intel_ring *ring) > { > + enum i915_map_type map = HAS_LLC(ctx->i915) ? I915_MAP_WB : I915_MAP_WC; > void *vaddr; > u32 *regs; > int ret; > @@ -2738,13 +2740,12 @@ populate_lr_context(struct i915_gem_context *ctx, > return ret; > } > > - vaddr = i915_gem_object_pin_map(ctx_obj, I915_MAP_WB); > + vaddr = i915_gem_object_pin_map(ctx_obj, map); As this uses the cpu domain and flushed afterwards, this one is correct in its usage of MAP_WB. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx