[Mesa-dev] [PATCH v4 00/18] Use clear color address in surface state.

Rafael Antognolli rafael.antognolli at intel.com
Thu Mar 8 16:48:53 UTC 2018


New revision of this series.

Major differences are:
 - I tried to incorporate most of the review feedback.
 - While adding some code in i965 to update the clear color state
   buffer, I seemd to avoid some bugs by actually doing it in blorp.
   However, anv already updates the same buffer, so in order to avoid
   having both places doing that, I tried to centralize that code in
   blorp.
 - Now blorp updates the clear color whenever we are doing a fast clear
   and have the clear_color_addr field set with a valid buffer.

Cc: Jason Ekstrand <jason at jlekstrand.net>
Cc: Jordan Justen <jordan.l.justen at intel.com>

Rafael Antognolli (18):
  anv/image: Do not override lower bits of dword.
  genxml: Preserve fields that share dword space with addresses.
  intel/genxml: Use a single field for clear color address on gen10.
  intel/genxml: Add Clear Color struct.
  intel/isl: Add support to emit clear value address.
  intel/blorp: Add support for fast clear address.
  intel/blorp: Only copy clear color when doing a resolve.
  intel/blorp: Update clear color state buffer during fast clears.
  i965/miptree: Add space to store the clear value in the aux surface.
  i965/miptree: Add new BO for clear color.
  i965/blorp: Add aux_buf variable to simplify code.
  i965/blorp: Update the fast clear color address.
  i965/surface_state: Emit the clear color address instead of value.
  anv: Add a helper to extract clear color from the attachment.
  anv: Emit the fast clear color address, instead of value.
  anv: Use clear address for HiZ fast clears too.
  anv: Make blorp update the clear color.
  intel: Remove use_clear_address flag from isl_surf_fill_state_info.

 src/intel/blorp/blorp_genX_exec.h                | 75 ++++++++++++++++++++---
 src/intel/genxml/gen10.xml                       | 15 +++--
 src/intel/genxml/gen11.xml                       | 17 ++++--
 src/intel/genxml/gen_pack_header.py              |  9 ++-
 src/intel/isl/isl.c                              | 23 ++++---
 src/intel/isl/isl.h                              | 14 +++++
 src/intel/isl/isl_surface_state.c                |  5 +-
 src/intel/vulkan/anv_blorp.c                     | 69 +++++++++++++++------
 src/intel/vulkan/anv_device.c                    | 19 ++++++
 src/intel/vulkan/anv_image.c                     | 47 +++++++++++----
 src/intel/vulkan/anv_private.h                   | 38 +++++++++++-
 src/intel/vulkan/genX_cmd_buffer.c               | 76 +++++++++++++-----------
 src/mesa/drivers/dri/i965/brw_blorp.c            | 26 ++++----
 src/mesa/drivers/dri/i965/brw_clear.c            | 22 +++++++
 src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 26 +++++++-
 src/mesa/drivers/dri/i965/intel_mipmap_tree.c    | 34 +++++++++++
 src/mesa/drivers/dri/i965/intel_mipmap_tree.h    | 16 +++++
 17 files changed, 423 insertions(+), 108 deletions(-)

-- 
2.14.3



More information about the mesa-dev mailing list