[Mesa-dev] [PATCH v3 09/13] i965/blorp: Update the fast clear color entry buffer.

[Jason Ekstrand]

On Wed, Feb 21, 2018 at 1:45 PM, Rafael Antognolli <
rafael.antognolli at intel.com> wrote:

> On Gen10, whenever the fast clear color changes, update it on the clear
> color entry buffer. This allow us to use it directly when emitting the
> surface state.
>
> Signed-off-by: Rafael Antognolli <rafael.antognolli at intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_blorp.c | 26 ++++++++++++++++++++++++++
>  1 file changed, 26 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c
> b/src/mesa/drivers/dri/i965/brw_blorp.c
> index 1d586e5ef38..25bfd118e7e 100644
> --- a/src/mesa/drivers/dri/i965/brw_blorp.c
> +++ b/src/mesa/drivers/dri/i965/brw_blorp.c
> @@ -1142,6 +1142,27 @@ set_write_disables(const struct intel_renderbuffer
> *irb,
>     return disables;
>  }
>
> +static void
> +update_fast_clear_color(struct brw_context *brw,
> +                        struct blorp_surf *surf,
> +                        const union isl_color_value clear_color)
> +{
> +   assert(surf);
> +   /* Clear values are stored at the same bo as the aux surface, right
> +    * after the surface.
> +    */
> +   uint32_t clear_offset = surf->aux_addr.offset + surf->aux_surf->size;
> +   for (int i = 0; i < brw->isl_dev.ss.clear_value_size / 4; i++) {
> +      brw_store_data_imm32(brw, surf->aux_addr.buffer,
> +                           clear_offset + i * 4, clear_color.u32[i]);
> +   }
> +
> +   surf->clear_color_addr = (struct blorp_address) {
> +      .buffer = surf->aux_addr.buffer,
> +      .offset = clear_offset,
> +   };
>

I kind-of wonder if we don't want to just make core blorp do this.
Something like

if (GEN_GEN >= 10 && params->dst.clear_color_addr.buffer &&
    params->fast_clear_op == ISL_AUX_OP_FAST_CLEAR) {
   /* Fill out the clear color */
}

I'm not actually sure that we do, but it's an idea.  Other than that, this
patch looks fine by me.


> +}
> +
>  static void
>  do_single_blorp_clear(struct brw_context *brw, struct gl_framebuffer *fb,
>                        struct gl_renderbuffer *rb, unsigned buf,
> @@ -1238,6 +1259,11 @@ do_single_blorp_clear(struct brw_context *brw,
> struct gl_framebuffer *fb,
>        blorp_surf_for_miptree(brw, &surf, irb->mt, irb->mt->aux_usage,
> true,
>                               &level, irb->mt_layer, num_layers, isl_tmp);
>
> +      /* update clear color */
> +      const struct gen_device_info *devinfo = &brw->screen->devinfo;
> +      if (devinfo->gen >= 10 && !same_clear_color)
> +         update_fast_clear_color(brw, &surf, clear_color);
> +
>        /* Ivybrigde PRM Vol 2, Part 1, "11.7 MCS Buffer for Render
> Target(s)":
>         *
>         *    "Any transition from any value in {Clear, Render, Resolve} to
> a
> --
> 2.14.3
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20180227/526edf39/attachment.html>