[Mesa-dev] [PATCH v2 0/6] Flag new aux state on aux state updates

Jason Ekstrand jason at jlekstrand.net
Sat Sep 16 00:11:38 UTC 2017


I don't think we need patches 2 or 6 and the commit message from 2 should
probably go on 5.  I left a comment on patch 4.  Other than that, 1, 3, 4,
and 5 are

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

On Fri, Sep 15, 2017 at 3:02 AM, Iago Toral Quiroga <itoral at igalia.com>
wrote:

> Jason, Ken: I think this series addresses all your feedback, let me know
> if you think I missed anything.
>
> Maybe you also prefer to squash some of the patches, let me know if that
> is the case.
>
> Iago Toral Quiroga (6):
>   i965: rename BRW_NEW_FAST_CLEAR_COLOR to BRW_NEW_AUX_STATE
>   i965: emit BRW_NEW_AUX_STATE when we allocate aux surfaces
>   i965: emit BRW_NEW_AUX_STATE if we drop the aux surface
>   i965: emit BRW_NEW_AUX_STATE when we change the fast clear value
>   i965: emit BRW_NEW_AUX_STATE on aux state changes
>   i965: emit BRW_NEW_AUX_STATE for textures without unresolved colors
>
>  src/mesa/drivers/dri/i965/brw_blorp.c             | 12 ++--------
>  src/mesa/drivers/dri/i965/brw_clear.c             |  2 +-
>  src/mesa/drivers/dri/i965/brw_context.h           |  4 ++--
>  src/mesa/drivers/dri/i965/brw_gs_surface_state.c  |  2 +-
>  src/mesa/drivers/dri/i965/brw_state_upload.c      |  2 +-
>  src/mesa/drivers/dri/i965/brw_tcs_surface_state.c |  2 +-
>  src/mesa/drivers/dri/i965/brw_tes_surface_state.c |  2 +-
>  src/mesa/drivers/dri/i965/brw_vs_surface_state.c  |  2 +-
>  src/mesa/drivers/dri/i965/brw_wm_surface_state.c  | 12 +++++-----
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.c     | 19 ++++++++++++---
>  src/mesa/drivers/dri/i965/intel_mipmap_tree.h     | 28
> +++++++++++++++++++++++
>  11 files changed, 60 insertions(+), 27 deletions(-)
>
> --
> 2.11.0
>
> _______________________________________________
> 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/20170915/bbea8b98/attachment.html>


More information about the mesa-dev mailing list