From: Kenneth Graunke Date: Wed, 10 Jul 2013 20:18:34 +0000 (-0700) Subject: i965: Cite the 965 PRM for "the data cache is the sampler cache". X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3f64cfabfc0bae7f5dba9d541680edcc6b745cea;p=mesa.git i965: Cite the 965 PRM for "the data cache is the sampler cache". Presumably, this comment exists to justify the usage of I915_GEM_DOMAIN_SAMPLER for this relocation. At one point, this was necessary to ensure that the right flushing was done to keep caches coherent. These days, the kernel just flushes everything, so I don't think it matters. Still, the comment is interesting, so leave it in place. Signed-off-by: Kenneth Graunke --- diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c index c678919cc15..8847f916536 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c @@ -347,9 +347,9 @@ brw_create_constant_surface(struct brw_context *brw, surf[4] = 0; surf[5] = 0; - /* Emit relocation to surface contents. Section 5.1.1 of the gen4 - * bspec ("Data Cache") says that the data cache does not exist as - * a separate cache and is just the sampler cache. + /* Emit relocation to surface contents. The 965 PRM, Volume 4, section + * 5.1.2 "Data Cache" says: "the data cache does not exist as a separate + * physical cache. It is mapped in hardware to the sampler cache." */ drm_intel_bo_emit_reloc(brw->batch.bo, *out_offset + 4,