Re: [Mesa-dev] [PATCH 04/10] radv: do not rebind the same pipeline when doing meta operations

2017-10-03 Thread Samuel Pitoiset
On 10/03/2017 08:55 PM, Bas Nieuwenhuizen wrote: Doesn't it make more sense to do the check in radv_CmdBindPipeline? It does. On Tue, Oct 3, 2017 at 8:52 PM, Samuel Pitoiset wrote: This might save some usless state changes, and it improves consistency with the other meta operations. Sig

Re: [Mesa-dev] [PATCH 04/10] radv: do not rebind the same pipeline when doing meta operations

2017-10-03 Thread Bas Nieuwenhuizen
Doesn't it make more sense to do the check in radv_CmdBindPipeline? On Tue, Oct 3, 2017 at 8:52 PM, Samuel Pitoiset wrote: > This might save some usless state changes, and it improves > consistency with the other meta operations. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulkan/radv_me

[Mesa-dev] [PATCH 04/10] radv: do not rebind the same pipeline when doing meta operations

2017-10-03 Thread Samuel Pitoiset
This might save some usless state changes, and it improves consistency with the other meta operations. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_meta_buffer.c | 18 -- src/amd/vulkan/radv_meta_resolve_fs.c | 7 +-- 2 files changed, 17 insertions(+), 8 deleti