Re: [Mesa-dev] [PATCH 4/8] radv: add all dependencies from external to the first subpass

2018-06-28 Thread Fredrik Höglund
On Wednesday 27 June 2018, Samuel Pitoiset wrote: > > On 06/27/2018 02:12 AM, Bas Nieuwenhuizen wrote: > > Reviewed-by: Bas Nieuwenhuizen > > > > for patch 3-4. Not sure they should go to stable though, since they > > are optimizations? > > Isn't the whole series for optimization purposes? No,

Re: [Mesa-dev] [PATCH 4/8] radv: add all dependencies from external to the first subpass

2018-06-27 Thread Bas Nieuwenhuizen
On Wed, Jun 27, 2018 at 9:46 AM, Samuel Pitoiset wrote: > > > On 06/27/2018 02:12 AM, Bas Nieuwenhuizen wrote: >> >> Reviewed-by: Bas Nieuwenhuizen >> >> for patch 3-4. Not sure they should go to stable though, since they >> are optimizations? > > > Isn't the whole series for optimization purpose

Re: [Mesa-dev] [PATCH 4/8] radv: add all dependencies from external to the first subpass

2018-06-27 Thread Samuel Pitoiset
On 06/27/2018 02:12 AM, Bas Nieuwenhuizen wrote: Reviewed-by: Bas Nieuwenhuizen for patch 3-4. Not sure they should go to stable though, since they are optimizations? Isn't the whole series for optimization purposes? On Tue, Jun 26, 2018 at 11:49 PM, Fredrik Höglund wrote: This is to a

Re: [Mesa-dev] [PATCH 4/8] radv: add all dependencies from external to the first subpass

2018-06-26 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen for patch 3-4. Not sure they should go to stable though, since they are optimizations? On Tue, Jun 26, 2018 at 11:49 PM, Fredrik Höglund wrote: > This is to avoid repeating dependencies when more than one subpass > has a dependency from external. > > Cc: > Signed

[Mesa-dev] [PATCH 4/8] radv: add all dependencies from external to the first subpass

2018-06-26 Thread Fredrik Höglund
This is to avoid repeating dependencies when more than one subpass has a dependency from external. Cc: Signed-off-by: Fredrik Höglund --- src/amd/vulkan/radv_pass.c | 4 1 file changed, 4 insertions(+) diff --git a/src/amd/vulkan/radv_pass.c b/src/amd/vulkan/radv_pass.c index 2827f5f1a8d.