Re: [Mesa-dev] [PATCH] anv/gen7_pipeline: Use MSDISPMODE_PERSAMPLE for non-multisampled fbo

2016-09-29 Thread Jason Ekstrand
On Sep 28, 2016 12:10 PM, "Anuj Phogat" wrote: > > On Wed, Sep 21, 2016 at 12:49 PM, Anuj Phogat wrote: > > On Wed, Sep 21, 2016 at 11:49 AM, Jason Ekstrand wrote: > >> This seems odd... When can it even happen that we have

Re: [Mesa-dev] [PATCH] anv/gen7_pipeline: Use MSDISPMODE_PERSAMPLE for non-multisampled fbo

2016-09-28 Thread Anuj Phogat
On Wed, Sep 21, 2016 at 12:49 PM, Anuj Phogat wrote: > On Wed, Sep 21, 2016 at 11:49 AM, Jason Ekstrand wrote: >> This seems odd... When can it even happen that we have persample_dispatch >> set in wm_surface_state and have only one sample? Does this

Re: [Mesa-dev] [PATCH] anv/gen7_pipeline: Use MSDISPMODE_PERSAMPLE for non-multisampled fbo

2016-09-21 Thread Anuj Phogat
On Wed, Sep 21, 2016 at 11:49 AM, Jason Ekstrand wrote: > This seems odd... When can it even happen that we have persample_dispatch > set in wm_surface_state and have only one sample? Does this fix a test > case? > No, It just fixes a simulator warning. It's recommended in

Re: [Mesa-dev] [PATCH] anv/gen7_pipeline: Use MSDISPMODE_PERSAMPLE for non-multisampled fbo

2016-09-21 Thread Jason Ekstrand
This seems odd... When can it even happen that we have persample_dispatch set in wm_surface_state and have only one sample? Does this fix a test case? On Sep 21, 2016 9:14 PM, "Anuj Phogat" wrote: > Signed-off-by: Anuj Phogat > --- >

[Mesa-dev] [PATCH] anv/gen7_pipeline: Use MSDISPMODE_PERSAMPLE for non-multisampled fbo

2016-09-21 Thread Anuj Phogat
Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen7_pipeline.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c index 878308b..5150ef9 100644 --- a/src/intel/vulkan/gen7_pipeline.c