On Sun, Jun 25, 2017 at 10:29 PM, Dave Airlie <airl...@gmail.com> wrote: > On 26 June 2017 at 06:19, Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl> wrote: >> v2: Don't pass layout to image view usage mask. >> >> Signed-off-by: Bas Nieuwenhuizen <ba...@google.com> >> Fixes: 0628580eff6 "radv: Specify semantics of HTILE layout helpers." > > Just wondering what this actually fixes, I've mostly left GENERAL in > use in meta as it seems like it would avoid transitions in meta paths, > which mostly seems like the correct thing to do, the app should be > transitioning things before/after the entrypoint that causes the meta > path to be hit. > > Though of course there are problems knowing exactly what to flush > before we know which meta path we want to use.
Because HTILE is enabled for TRANSFER_DST_OPTIMAL, but not for GENERAL. Ignoring a compressed HTILE is not a good idea. So I think it makes sense to at least keep these as TRANSFER_DST_OPTIMAL to keep them in sync with whatever we decide the compression to be. This doesn't result in any transitions either, as the initial_layout/final_layout are also set to TRANSFER_DST_OPTIMAL. > > Dave. _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev