i965: Cite the 965 PRM for "the data cache is the sampler cache". 07/7507/1
authorKenneth Graunke <kenneth@whitecape.org>
Wed, 10 Jul 2013 20:18:34 +0000 (13:18 -0700)
committerKenneth Graunke <kenneth@whitecape.org>
Tue, 16 Jul 2013 02:40:51 +0000 (19:40 -0700)
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 <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_wm_surface_state.c

index c678919..8847f91 100644 (file)
@@ -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,