[Mesa-dev] [PATCH 04/11] i965/miptree Remove layout_flags parameter form is_mcs_supported

Jordan Justen jordan.l.justen at intel.com
Thu Aug 3 17:22:28 UTC 2017


subect: form => from

Commit message? Why is this being removed?

On 2017-08-02 13:35:29, Jason Ekstrand wrote:
> ---
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> index daa49f9..0e0aaa8 100644
> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
> @@ -59,8 +59,7 @@ intel_miptree_alloc_aux(struct brw_context *brw,
>                          struct intel_mipmap_tree *mt);
>  
>  static bool
> -is_mcs_supported(const struct brw_context *brw, mesa_format format,
> -                 uint32_t layout_flags)
> +is_mcs_supported(const struct brw_context *brw, mesa_format format)
>  {
>     /* Prior to Gen7, all MSAA surfaces used IMS layout. */
>     if (brw->gen < 7)
> @@ -86,11 +85,6 @@ is_mcs_supported(const struct brw_context *brw, mesa_format format,
>         */
>        if (brw->gen == 7 && _mesa_get_format_datatype(format) == GL_INT) {
>           return false;
> -      } else if (layout_flags & MIPTREE_LAYOUT_DISABLE_AUX) {
> -         /* We can't use the CMS layout because it uses an aux buffer, the MCS
> -          * buffer. So fallback to UMS, which is identical to CMS without the
> -          * MCS. */
> -         return false;

I see that intel_miptree_create_for_dri_image will set this flag for
'winsys' images, so why is this change safe to make? (Maybe it isn't
quite 'safe' on it's own?)

I think the answer is in the next patch 'i965/miptree: Refactor
is_mcs_supported' where is_mcs_supported is refactored to
intel_miptree_supports_mcs. I think maybe it move from asking if the
format potentially supports MCS to whether the miptree actually has
MCS enabled. ?

If so, maybe a small commit message comment would help? Or, maybe
sqaush it with the next patch?

-Jordan

>        } else {
>           return true;
>        }
> @@ -329,8 +323,7 @@ intel_miptree_choose_aux_usage(struct brw_context *brw,
>  {
>     assert(mt->aux_usage == ISL_AUX_USAGE_NONE);
>  
> -   const unsigned no_flags = 0;
> -   if (mt->surf.samples > 1 && is_mcs_supported(brw, mt->format, no_flags)) {
> +   if (mt->surf.samples > 1 && is_mcs_supported(brw, mt->format)) {
>        assert(mt->surf.msaa_layout == ISL_MSAA_LAYOUT_ARRAY);
>        mt->aux_usage = ISL_AUX_USAGE_MCS;
>     } else if (intel_tiling_supports_ccs(brw, mt->surf.tiling) &&
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list