Re: [Mesa-dev] [PATCH 9/9] i965: Enable flush control

2017-11-04 Thread Kenneth Graunke
On Thursday, November 2, 2017 12:01:48 PM PDT Adam Jackson wrote: > From: Neil Roberts > > Reviewed-by: Adam Jackson > Signed-off-by: Neil Roberts Patches 2 and 9 are: Reviewed-by: Kenneth Graunke signature.asc Description: This is a digitally signed message part. __

[Mesa-dev] [PATCH] radv: add initial copy descriptor support. (v2)

2017-11-04 Thread Dave Airlie
From: Dave Airlie It appears the latest dota2 vulkan uses this, and we get a hang in VR mode without it. v2: remove finishme I left in after finishing. Cc: "17.2 17.3" Signed-off-by: Dave Airlie --- src/amd/vulkan/radv_descriptor_set.c | 55 ++-- 1 file change

[Mesa-dev] [PATCH 1/2] radv: Use an array to store descriptor sets.

2017-11-04 Thread Bas Nieuwenhuizen
The vram_list linked list resulted in lots of pointer chasing. Replacing this with an array instead improves descriptor set allocation CPU usage by 3x at least (when also considering the free), because it had to iterate through 300-400 sets on average. Not a huge improvement as the pre-improvement

[Mesa-dev] [PATCH 2/2] radv: Optimize calling radv_save_descriptors.

2017-11-04 Thread Bas Nieuwenhuizen
uint32_t data[MAX_SETS * 2] = {}; was getting executed before the exit and took significant amounts of time. By having the check outside the function, we skip the execution of the clear. --- src/amd/vulkan/radv_cmd_buffer.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/

Re: [Mesa-dev] [PATCH 10/12] glsl: Minor cleanups after previous commit

2017-11-04 Thread Thomas Helland
Patches 6-10 are: Reviewed-by: Thomas Helland 2017-11-02 21:25 GMT+01:00 Ian Romanick : > From: Ian Romanick > > I think it's more clear to only call emit_access once. The only > difference between the two calls is the value of size_mul used for the > offset parameter... but you really have to

[Mesa-dev] [PATCH 3/3] radeonsi: don't map big VRAM buffers for the first upload directly

2017-11-04 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/r600_buffer_common.c | 20 src/gallium/drivers/radeon/r600_pipe_common.h | 1 + 2 files changed, 21 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_buffer_common.c b/src/gallium/drivers/radeon/r600_buffer_commo

[Mesa-dev] [PATCH 2/3] gallium/u_threaded: don't map big VRAM buffers for the first upload directly

2017-11-04 Thread Marek Olšák
From: Marek Olšák This improves Paraview "many spheres" performance 4x along with the radeonsi commit. --- src/gallium/auxiliary/util/u_threaded_context.c | 14 ++ src/gallium/auxiliary/util/u_threaded_context.h | 6 ++ src/gallium/drivers/radeon/r600_buffer_common.c | 10 ++

[Mesa-dev] [PATCH 1/3] gallium/u_threaded: clean up tc_improve_map_buffer_flags and prevent reentry

2017-11-04 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/util/u_threaded_context.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/src/gallium/auxiliary/util/u_threaded_context.c b/src/gallium/auxiliary/util/u_threaded_context.c index 7e28b87..0f23258 100644 --- a/sr