Module: Mesa
Branch: master
Commit: eb291d7013eef64c33826f9cc0006c89adcf4e53
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=eb291d7013eef64c33826f9cc0006c89adcf4e53
Author: Neil Roberts <n...@linux.intel.com>
Date: Tue Nov 24 17:59:28 2015 +0100
i965/gen8+: Don't upload the MCS buffer for single-sampled textures
For single-sampled textures the MCS buffer is only used to implement
fast clears. However the surface always needs to be resolved before
being used as a texture anyway so the the MCS buffer doesn't actually
achieve anything. This is important for Gen9 because in that case SRGB
surfaces are not supported for fast clears and we don't want the
hardware to see the MCS buffer in that case.
Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com>
---
src/mesa/drivers/dri/i965/gen8_surface_state.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/gen8_surface_state.c
b/src/mesa/drivers/dri/i965/gen8_surface_state.c
index b062bf0..904950d 100644
--- a/src/mesa/drivers/dri/i965/gen8_surface_state.c
+++ b/src/mesa/drivers/dri/i965/gen8_surface_state.c
@@ -225,7 +225,11 @@ gen8_emit_texture_surface_state(struct brw_context *brw,
pitch = mt->pitch;
}
- if (mt->mcs_mt) {
+ /* The MCS is not uploaded for single-sampled surfaces because the color
+ * buffer should always have been resolved before it is used as a texture
+ * so there is no need for it.
+ */
+ if (mt->mcs_mt && mt->num_samples > 1) {
aux_mt = mt->mcs_mt;
aux_mode = GEN8_SURFACE_AUX_MODE_MCS;
_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-commit
