Re: [Mesa-dev] software implementation of vulkan for gsoc/evoc

2017-06-10 Thread Jose Fonseca
I know this is an old thread. I completely missed it the first time, but recently rediscovered after reading http://www.phoronix.com/scan.php?page=news_item&px=Vulkan-CPU-Repository , and perhaps it's not too late for a couple comments FWIW. On 13/02/17 02:17, Jacob Lifshay wrote: forgot to

[Mesa-dev] [PATCH 3/3] radv: Rename winsys enums.

2017-06-10 Thread Bas Nieuwenhuizen
Don't rename the enums and constants used for metadata. Signed-off-by: Bas Nieuwenhuizen --- src/amd/vulkan/radv_cmd_buffer.c | 4 +-- src/amd/vulkan/radv_descriptor_set.c | 2 +- src/amd/vulkan/radv_device.c | 48 +-- src/amd/vulk

[Mesa-dev] [PATCH 2/3] radv: Rename winsys interface structures from radeon* to radv*.

2017-06-10 Thread Bas Nieuwenhuizen
For preventing confusion with a radeon winsys. Signed-off-by: Bas Nieuwenhuizen --- src/amd/vulkan/radv_cmd_buffer.c | 26 ++--- src/amd/vulkan/radv_cs.h | 24 ++--- src/amd/vulkan/radv_descriptor_set.c | 18 ++-- src/amd/vulkan/radv_d

[Mesa-dev] [PATCH 1/3] radv: Rename radv_radeon_winsys.h to radv_winsys.h.

2017-06-10 Thread Bas Nieuwenhuizen
The amdgpu winsys has radv_amdgpu_winsys.h, and getting another radv_radeon_winsys.h in there for a radeon winsys would be awkward. Signed-off-by: Bas Nieuwenhuizen --- src/amd/vulkan/Makefile.sources| 2 +- src/amd/vulkan/radv_cmd_buffer.c | 2 +-

[Mesa-dev] [Bug 101374] Worms Clan Wars hangs on loading screen

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101374 --- Comment #1 from Hi-Angel --- It's unclear if OP at the second link described own system (which doesn't have the problem), or the system of the friends with the problem. Probably the first, because a single GPU is mentioned, whilst the number

[Mesa-dev] [Bug 101374] Worms Clan Wars hangs on loading screen

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101374 Bug ID: 101374 Summary: Worms Clan Wars hangs on loading screen Product: Mesa Version: unspecified Hardware: All OS: All Status: NEW Severity: normal

[Mesa-dev] [Bug 101334] Any vulkan app seems to freeze the system

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101334 --- Comment #8 from John --- Grazvydas, I've rebuilt mesa at the faulty commit with your 2 patches and it worked as well as before that commit. Thank you for the quick fix! Now if possible I'd love to look at the freezes I've had since my fi

[Mesa-dev] [PATCH] ac: Use mov_dpp for derivatives.

2017-06-10 Thread Bas Nieuwenhuizen
Slightly faster than bpermute, and seems supported since at least LLVM 3.9. v2: Since this supersedes bpermute, remove the bpermute code. Signed-off-by: Bas Nieuwenhuizen --- src/amd/common/ac_llvm_build.c | 47 src/amd/common/ac_llvm_build.h

[Mesa-dev] [PATCH] ac: Use mov_dpp for derivatives.

2017-06-10 Thread Bas Nieuwenhuizen
Slightly faster than bpermute, and seems supported since at least LLVM 3.9. Signed-off-by: Bas Nieuwenhuizen --- src/amd/common/ac_llvm_build.c | 78 +- 1 file changed, 54 insertions(+), 24 deletions(-) diff --git a/src/amd/common/ac_llvm_build.c b/src/am

Re: [Mesa-dev] [PATCH 1/2] radv: assert on CP_DMA_USE_L2 for SI

2017-06-10 Thread Bas Nieuwenhuizen
The series is Reviewed-by: Bas Nieuwenhuizen On Sat, Jun 10, 2017 at 5:53 PM, Grazvydas Ignotas wrote: > The register header (and radeonsi comment) states V_411_SRC_ADDR_TC_L2 > is for CIK+ only, so let's assert on earlier ASICs. > > Signed-off-by: Grazvydas Ignotas > --- > src/amd/vulkan/si_

[Mesa-dev] [Bug 94168] Incorrect rendering when running Populous 3 on wine using DDraw->WineD3D->OpenGL wrapper [apitrace]

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=94168 --- Comment #10 from Sven Arvidsson --- Gabriel Knight 3 is another Wine direct draw game with a similar problem. When restoring a saved game the screen stops refreshing, but the game still runs. Applying the suggested patch or using LIBGL_ALWA

[Mesa-dev] [PATCH 10/13] radeonsi: pack si_buffer_resources better

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state.h | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h index b616757..d8bf13e 100644 --- a/src/gallium/drivers/radeonsi/si_state.

[Mesa-dev] [PATCH 12/13] radeonsi: pack si_framebuffer better

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeon/r600_pipe_common.h | 2 +- src/gallium/drivers/radeon/r600_texture.c | 2 +- src/gallium/drivers/radeonsi/si_pipe.h| 12 ++-- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/drivers/radeon/r600_pipe

[Mesa-dev] [PATCH 07/13] radeonsi: replace si_vertex_elements::elements with separate fields

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_descriptors.c | 9 - src/gallium/drivers/radeonsi/si_state.c | 8 +--- src/gallium/drivers/radeonsi/si_state.h | 4 +++- src/gallium/drivers/radeonsi/si_state_shaders.c | 7 ++- 4 files changed, 14 insertions

[Mesa-dev] [PATCH 02/13] radeonsi: use uint32_t to declare si_shader_key.opt.kill_outputs

2017-06-10 Thread Marek Olšák
From: Marek Olšák the next patch will benefit from this --- src/gallium/drivers/radeonsi/si_shader.c| 8 +--- src/gallium/drivers/radeonsi/si_shader.h| 3 ++- src/gallium/drivers/radeonsi/si_state_shaders.c | 5 +++-- 3 files changed, 10 insertions(+), 6 deletions(-) diff --

[Mesa-dev] [PATCH 01/13] radeonsi: remove 8 bytes from si_shader_key by flattening opt.hw_vs

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_shader.c| 10 +- src/gallium/drivers/radeonsi/si_shader.h| 7 +++ src/gallium/drivers/radeonsi/si_state.c | 2 +- src/gallium/drivers/radeonsi/si_state_shaders.c | 12 ++-- 4 files changed, 15 ins

[Mesa-dev] [PATCH 05/13] radeonsi: allocate si_state_rasterizer::pm4_poly_offset only when needed

2017-06-10 Thread Marek Olšák
From: Marek Olšák Each element has over 700 bytes. --- src/gallium/drivers/radeonsi/si_state.c | 14 +- src/gallium/drivers/radeonsi/si_state.h | 2 +- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/rade

[Mesa-dev] [PATCH 04/13] radeonsi: pack si_state_rasterizer fields

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state.h | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h index 390e16f..dabe9b9 100644 --- a/src/gallium/

[Mesa-dev] [PATCH 08/13] radeonsi: pack struct si_vertex_elements better

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state.h | 19 ++- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h index 99c8ee6..77fa467 100644 --- a/src/gallium/drivers/radeon

[Mesa-dev] [PATCH 13/13] radeonsi: pack si_context better

2017-06-10 Thread Marek Olšák
From: Marek Olšák there isn't much to gain here --- src/gallium/drivers/radeonsi/si_pipe.h | 36 +- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.h b/src/gallium/drivers/radeonsi/si_pipe.h index 388f6e0..eef0

[Mesa-dev] [PATCH 06/13] radeonsi: rename si_vertex_element -> si_vertex_elements

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_descriptors.c | 2 +- src/gallium/drivers/radeonsi/si_pipe.h| 2 +- src/gallium/drivers/radeonsi/si_state.c | 6 +++--- src/gallium/drivers/radeonsi/si_state.h | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff

[Mesa-dev] [PATCH 11/13] radeonsi: pack si_sampler_view better

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_pipe.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.h b/src/gallium/drivers/radeonsi/si_pipe.h index 46b095d..55fda4d 100644 --- a/src/gallium/drivers/radeonsi/si_pipe.h +++ b/s

[Mesa-dev] [PATCH 03/13] radeonsi: remove 8 bytes from si_shader_key with uint32_t ff_tcs_inputs_to_copy

2017-06-10 Thread Marek Olšák
From: Marek Olšák The previous patch helps with this. --- src/gallium/drivers/radeonsi/si_shader.c| 8 ++-- src/gallium/drivers/radeonsi/si_shader.h| 3 ++- src/gallium/drivers/radeonsi/si_state_shaders.c | 8 ++-- 3 files changed, 14 insertions(+), 5 deletions(-) diff -

[Mesa-dev] [PATCH 09/13] radeonsi: pack struct si_descriptors better

2017-06-10 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/radeonsi/si_state.h | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_state.h b/src/gallium/drivers/radeonsi/si_state.h index 77fa467..b616757 100644 --- a/src/gallium/dr

[Mesa-dev] [Bug 101334] Any vulkan app seems to freeze the system

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101334 --- Comment #7 from Grazvydas Ignotas --- I've sent different version to ML, testing that one would be preferred: https://lists.freedesktop.org/archives/mesa-dev/2017-June/158700.html -- You are receiving this mail because: You are the assigne

[Mesa-dev] [PATCH 2/2] radv: don't even attempt to prefetch on SI

2017-06-10 Thread Grazvydas Ignotas
Before bcae327469 this was emitting CP DMA packet even on SI, but apparently hasn't caused too many problems. After that commit the CP DMA code now always sets the CIK+ only bit for prefetch. Just follow radeonsi there and don't try to prefetch at all. Bugzilla: https://bugs.freedesktop.org/show_b

[Mesa-dev] [PATCH 1/2] radv: assert on CP_DMA_USE_L2 for SI

2017-06-10 Thread Grazvydas Ignotas
The register header (and radeonsi comment) states V_411_SRC_ADDR_TC_L2 is for CIK+ only, so let's assert on earlier ASICs. Signed-off-by: Grazvydas Ignotas --- src/amd/vulkan/si_cmd_buffer.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cm

Re: [Mesa-dev] [PATCH] gallium: fixed modulo zero crashes in tgsi interpreter (v2)

2017-06-10 Thread Roland Scheidegger
Pushed, thanks! Roland Am 09.06.2017 um 15:39 schrieb Marius Gräfe: > softpipe throws integer division by zero exceptions on windows > when using % with integers in a geometry shader. > > v2: Made error results consistent with existing div/mod zero handling in > tgsi. 64 bit signed integer d

[Mesa-dev] [Bug 101334] Any vulkan app seems to freeze the system

2017-06-10 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=101334 Grazvydas Ignotas changed: What|Removed |Added CC||nota...@gmail.com --- Comment #6 fr