On Wednesday, January 10, 2018 11:22:38 AM PST Jason Ekstrand wrote: > This commit unifies the CCS_E and CCS_D cases. This should fix a couple > of subtle issues. One is that when you use INTEL_DEBUG=norbc to disable > CCS_E, we don't get the sRGB blending workaround. By unifying the code, > we give CCS_D that workaround as well. > > The second issue fixed by this refactor is that the blending workaround > was appears to be enabled on all gens but really only applies on gen9. > Due to a happy accident in the way code was laid out, it was only > getting enabled on gen9: gen8 and earlier don't support non-zero-one > clear colors, and gen10 supports sRGB for CCS_E so it got caught in the > format_ccs_e_compat_with_miptree case. This refactor moves it above the > format_ccs_e_compat_with_miptree case so it's an explicit early exit and > makes it explicitly only on gen9. > > Reviewed-by: Nanley Chery <[email protected]> > Cc: "17.3" <[email protected]> > --- > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 26 +++++++++++++------------- > 1 file changed, 13 insertions(+), 13 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > index abdaa6a..0bbd661 100644 > --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > @@ -2685,28 +2685,28 @@ intel_miptree_render_aux_usage(struct brw_context > *brw, > return ISL_AUX_USAGE_MCS; > > case ISL_AUX_USAGE_CCS_D: > - return mt->mcs_buf ? ISL_AUX_USAGE_CCS_D : ISL_AUX_USAGE_NONE; > - > - case ISL_AUX_USAGE_CCS_E: { > - /* If the format supports CCS_E and is compatible with the miptree, > - * then we can use it. > - */ > - if (format_ccs_e_compat_with_miptree(&brw->screen->devinfo, > - mt, render_format)) > - return ISL_AUX_USAGE_CCS_E; > - > - /* Otherwise, we have to fall back to CCS_D */ > + case ISL_AUX_USAGE_CCS_E: > + if (!mt->mcs_buf) { > + assert(mt->aux_usage == ISL_AUX_USAGE_CCS_D); > + return ISL_AUX_USAGE_NONE; > + } > > /* gen9 hardware technically supports non-0/1 clear colors with sRGB > * formats. However, there are issues with blending where it doesn't > * properly apply the sRGB curve to the clear color when blending. > */ > - if (blend_enabled && isl_format_is_srgb(render_format) && > + if (brw->screen->devinfo.gen == 9 && blend_enabled &&
I personally hate these run-on chains, can we make a devinfo pointer? Patches 3-4 are: Reviewed-by: Kenneth Graunke <[email protected]> > + isl_format_is_srgb(render_format) && > !isl_color_value_is_zero_one(mt->fast_clear_color, render_format)) > return ISL_AUX_USAGE_NONE; > > + if (mt->aux_usage == ISL_AUX_USAGE_CCS_E && > + format_ccs_e_compat_with_miptree(&brw->screen->devinfo, > + mt, render_format)) > + return ISL_AUX_USAGE_CCS_E; > + > + /* Otherwise, we have to fall back to CCS_D */ > return ISL_AUX_USAGE_CCS_D; > - } > > default: > return ISL_AUX_USAGE_NONE; >
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
