Re: [Mesa-dev] [PATCH V2 2/2] i965: Change 8X MSAA sample mapping

2016-08-11 Thread Jason Ekstrand
On Aug 11, 2016 5:34 PM, "Anuj Phogat" wrote: > > Thanks for the review. Can you also review the piglit patch? > "multisample_blit_scaled: Change 8X MSAA sample mapping" Done > > On Thu, Aug 11, 2016 at 3:56 PM, Jason Ekstrand wrote: > > This looks good. Thanks! I think we should probably squ

Re: [Mesa-dev] [PATCH V2 2/2] i965: Change 8X MSAA sample mapping

2016-08-11 Thread Anuj Phogat
Thanks for the review. Can you also review the piglit patch? "multisample_blit_scaled: Change 8X MSAA sample mapping" On Thu, Aug 11, 2016 at 3:56 PM, Jason Ekstrand wrote: > This looks good. Thanks! I think we should probably squash it in to the > other one because there's technically a regre

Re: [Mesa-dev] [PATCH V2 2/2] i965: Change 8X MSAA sample mapping

2016-08-11 Thread Jason Ekstrand
This looks good. Thanks! I think we should probably squash it in to the other one because there's technically a regression between the two. Both are Reviewed-by: Jason Ekstrand On Thu, Aug 11, 2016 at 12:28 PM, Anuj Phogat wrote: > This is required following the change in 8X sample position

[Mesa-dev] [PATCH V2 2/2] i965: Change 8X MSAA sample mapping

2016-08-11 Thread Anuj Phogat
This is required following the change in 8X sample positions. Fixes the recently modified multisample-scaled-blit piglit tests. Signed-off-by: Anuj Phogat --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 10 +- src/mesa/drivers/dri/i965/gen6_multisample_state.c | 2 +- 2 files c