On Tue, May 16, 2017 at 03:44:59PM -0700, Jason Ekstrand wrote: > The guts of blorp and ISL don't understand i965's partial miptrees. > Instead, we need to subtract off first_level before we hand anything off > to blorp. > --- > src/mesa/drivers/dri/i965/brw_blorp.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c > b/src/mesa/drivers/dri/i965/brw_blorp.c > index ebc4612..64aa4c9 100644 > --- a/src/mesa/drivers/dri/i965/brw_blorp.c > +++ b/src/mesa/drivers/dri/i965/brw_blorp.c > @@ -271,6 +271,9 @@ blorp_surf_for_miptree(struct brw_context *brw, > } > assert((surf->aux_usage == ISL_AUX_USAGE_NONE) == > (surf->aux_addr.buffer == NULL));
Here it would be nice to have an assert: assert(*level >= mt->first_level); > + > + /* ISL wants real levels, not offset ones. */ > + *level -= mt->first_level; > } > > static enum isl_format > -- > 2.5.0.400.gff86faf > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev