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

Iago Toral itoral at igalia.com
Mon Sep 18 08:53:36 UTC 2017


Hi Jason,
thanks for the quick review! I agree with all your points and will
update the series accordingly before pushing.
Iago
On Fri, 2017-09-15 at 17:11 -0700, Jason Ekstrand wrote:
> 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.co
> m> 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/20170918/a30d15ef/attachment.html>


More information about the mesa-dev mailing list