Re: [Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Don't temporarily enable CCS_E within a render pass

2017-01-23 Thread Nanley Chery
On Mon, Jan 23, 2017 at 05:42:44PM -0800, Jason Ekstrand wrote: > On Mon, Jan 23, 2017 at 5:40 PM, Nanley Chery wrote: > > > On Mon, Jan 23, 2017 at 05:30:22PM -0800, Jason Ekstrand wrote: > > > On Mon, Jan 23, 2017 at 4:55 PM, Nanley Chery > > wrote: > > > > > > > Compressing a render target an

Re: [Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Don't temporarily enable CCS_E within a render pass

2017-01-23 Thread Jason Ekstrand
On Mon, Jan 23, 2017 at 5:40 PM, Nanley Chery wrote: > On Mon, Jan 23, 2017 at 05:30:22PM -0800, Jason Ekstrand wrote: > > On Mon, Jan 23, 2017 at 4:55 PM, Nanley Chery > wrote: > > > > > Compressing a render target and decompressing it in the same > > > single-subpass render pass may waste band

Re: [Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Don't temporarily enable CCS_E within a render pass

2017-01-23 Thread Nanley Chery
On Mon, Jan 23, 2017 at 05:30:22PM -0800, Jason Ekstrand wrote: > On Mon, Jan 23, 2017 at 4:55 PM, Nanley Chery wrote: > > > Compressing a render target and decompressing it in the same > > single-subpass render pass may waste bandwidth. While this may be > > beneficial in some circumstances, it

Re: [Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Don't temporarily enable CCS_E within a render pass

2017-01-23 Thread Jason Ekstrand
On Mon, Jan 23, 2017 at 4:55 PM, Nanley Chery wrote: > Compressing a render target and decompressing it in the same > single-subpass render pass may waste bandwidth. While this may be > beneficial in some circumstances, it does not help in all. > > Cc: "13.0 17.0" > This doesn't really fix a bu

[Mesa-dev] [PATCH 1/2] anv/cmd_buffer: Don't temporarily enable CCS_E within a render pass

2017-01-23 Thread Nanley Chery
Compressing a render target and decompressing it in the same single-subpass render pass may waste bandwidth. While this may be beneficial in some circumstances, it does not help in all. Cc: "13.0 17.0" Signed-off-by: Nanley Chery --- src/intel/vulkan/genX_cmd_buffer.c | 6 -- 1 file changed