On Monday, September 25, 2017 5:46:32 AM PDT Lionel Landwerlin wrote: > I'm genuinely surprised we didn't noticed this problem before :| > > Fixes: 71bfb44005bf ("i965: Port brw_cs_state tracked state to genxml.") > Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> > Cc: "17.2" <mesa-sta...@lists.freedesktop.org>
I'm not - these fields are just used for prefetching the sampler states, so programming too small a number will just prefetch fewer of them. No big deal, certainly not going to hurt correctness. I probably wouldn't bother Cc'ing it to stable. That said, nice find :) We should definitely fix this.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev