The only purpose of this function is to disable aux on texture surfaces when the corresponding renderbuffer has aux disabled. However, the act of disabling aux on the renderbuffer will cause it to be resolved and intel_miptree_texture_aux_usage will already check the resolved status of a texture and return ISL_AUX_USAGE_NONE for it. Even if we used CCS for it, that wouldn't really be a problem because the CCS will be in the pass-through state and so it would effectively be ignored.
Cc: [email protected] --- src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 20 -------------------- 1 file changed, 20 deletions(-) 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 fd2ee36..10bd812 100644 --- a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c +++ b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c @@ -441,23 +441,6 @@ swizzle_to_scs(GLenum swizzle, bool need_green_to_blue) return (need_green_to_blue && scs == HSW_SCS_GREEN) ? HSW_SCS_BLUE : scs; } -static bool -brw_aux_surface_disabled(const struct brw_context *brw, - const struct intel_mipmap_tree *mt) -{ - const struct gl_framebuffer *fb = brw->ctx.DrawBuffer; - - for (unsigned i = 0; i < fb->_NumColorDrawBuffers; i++) { - const struct intel_renderbuffer *irb = - intel_renderbuffer(fb->_ColorDrawBuffers[i]); - - if (irb && irb->mt == mt) - return brw->draw_aux_buffer_disabled[i]; - } - - return false; -} - static void brw_update_texture_surface(struct gl_context *ctx, unsigned unit, @@ -588,9 +571,6 @@ brw_update_texture_surface(struct gl_context *ctx, enum isl_aux_usage aux_usage = intel_miptree_texture_aux_usage(brw, mt, format); - if (brw_aux_surface_disabled(brw, mt)) - aux_usage = ISL_AUX_USAGE_NONE; - brw_emit_surface_state(brw, mt, mt->target, view, aux_usage, surf_offset, surf_index, 0); -- 2.5.0.400.gff86faf _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
