On Monday, April 30, 2018 10:25:50 AM PDT Scott D Phillips wrote: > Removes a place where gtt mapping is used. > > Reviewed-by: Nanley Chery <[email protected]> > Reviewed-by: Chris Wilson <[email protected]> > --- > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > index 498eebd2f86..b3b27c50de3 100644 > --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c > @@ -3770,7 +3770,8 @@ intel_miptree_map(struct brw_context *brw, > #if defined(USE_SSE41) > } else if (!(mode & GL_MAP_WRITE_BIT) && > !mt->compressed && cpu_has_sse4_1 && > - (mt->surf.row_pitch % 16 == 0)) { > + (mt->surf.row_pitch % 16 == 0) && > + (mt->surf.tiling == ISL_TILING_LINEAR)) { > intel_miptree_map_movntdqa(brw, mt, map, level, slice); > #endif > } else if (mt->surf.tiling != ISL_TILING_LINEAR && >
You added intel_miptree_map_tiled_memcpy near the end, but couldn't you just put it before intel_miptree_map_movntdqa? Then it would try that first and fall back to this if it couldn't be handled. --Ken
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
