On Wed, 2018-01-10 at 08:54 -0800, Kenneth Graunke wrote:
> On Wednesday, January 10, 2018 5:11:21 AM PST Iago Toral wrote:
> > On Mon, 2018-01-08 at 13:30 -0800, Kenneth Graunke wrote:
> > > Growing the batch/state buffer is a lot more dangerous than I
> > > thought.
> > >
> > > A number of place
Am Mittwoch, den 10.01.2018, 15:27 -0500 schrieb Ilia Mirkin:
[...]
>
> If your hardware executes all the vertices in parallel, then a
> barrier should be unnecessary.
My first try for this patch did not include forcing the barrier into
slot x, which in turn resulted in failing piglits, e.g.
On 01/11/2018 12:43 AM, Jordan Justen wrote:
On 2018-01-08 23:48:19, Tapani Pälli wrote:
Cache set and get are called in similar fashion as what is happening
with disk cache. Functionality requires ARB_get_program_binary and
EGL_ANDROID_blob_cache support.
Signed-off-by: Tapani Pälli
---
s
On 01/10/2018 07:04 AM, Tapani Pälli wrote:
Hi Marek;
This one works but only if you add
DRI_CONF_ALLOW_RGB10_CONFIGS("false")
to the DRI_CONF_SECTION_MISCELLANEOUS section in intel_screen. With that
change: Reviewed-by: Tapani Pälli
With this patch now committed to master, rgb10 visuals
---
src/amd/common/ac_llvm_build.c | 1 +
src/amd/common/ac_llvm_build.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 07044142b0..75cc0dc63c 100644
--- a/src/amd/common/ac_llvm_build.c
+++ b/src/amd/common/ac_llvm_build
Without this we en up with the llvm error message:
"Both operands to a binary operator are not of the same type!"
---
src/amd/common/ac_nir_to_llvm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 8d759
https://bugs.freedesktop.org/show_bug.cgi?id=104214
Evangelos Foutras changed:
What|Removed |Added
CC||fdb...@fratti.ch
CC
Hi Bas,
If I read it correctly, you are trying to interpret the ClipRule as a mask.
You set the bit i if the ith rectangle is need to be inclusive.
Am I right?
IIRC, It is not a right interpretation.
Where did you read the register spec? I would like to take a look.
Thanks.
Best Regards,
Davi
https://bugs.freedesktop.org/show_bug.cgi?id=104214
--- Comment #36 from Evangelos Foutras ---
*** Bug 104342 has been marked as a duplicate of this bug. ***
--
You are receiving this mail because:
You are the QA Contact for the bug.___
mesa-dev maili
From: Dave Airlie
This just moves some calcs from the emission to the binding.
Probably won't affect much, just reduced some time spent in binding
---
src/amd/vulkan/radv_cmd_buffer.c | 11 ++-
src/amd/vulkan/radv_private.h| 1 +
2 files changed, 7 insertions(+), 5 deletions(-)
di
From: Dave Airlie
When chaining buffers if we see an empty one (just padded)
skip over it.
---
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c
b/src/amd/vulkan/wi
From: Dave Airlie
This should make the merging of cmd buffers less CPU intensive,
note I said *should* :)
---
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 47 ---
1 file changed, 20 insertions(+), 27 deletions(-)
diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_c
Am 10.01.2018 um 07:48 schrieb Dave Airlie:
> From: Dave Airlie
>
> This stops them being optimised out.
> ---
> src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 5 -
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
> b/src/galliu
https://bugs.freedesktop.org/show_bug.cgi?id=104214
--- Comment #35 from fdb...@fratti.ch ---
That patch fixes https://bugs.freedesktop.org/show_bug.cgi?id=104301 too, so
I'm guessing this was the same issue.
--
You are receiving this mail because:
You are the QA Contact for the bug.
Tested-by: Dieter Nützel
on RX580 with UH, UV, glmark2 and Blender 2.79 with and without nir.
Dieter
Am 10.01.2018 20:49, schrieb Marek Olšák:
From: Marek Olšák
Cc: 17.2 17.3
---
src/gallium/drivers/radeon/r600_texture.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-
For the series:
Tested-by: Dieter Nützel
on RX580 with UH, UV, glmark2 and Blender 2.79 with and without nir.
Dieter
Am 10.01.2018 20:49, schrieb Marek Olšák:
From: Marek Olšák
Cc: 17.2 17.3
---
src/gallium/auxiliary/util/u_threaded_context.c | 13 +
src/gallium/auxiliary/uti
For the series:
Tested-by: Dieter Nützel
Dieter
Am 08.01.2018 01:35, schrieb Timothy Arceri:
---
src/gallium/drivers/radeonsi/si_shader_nir.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader_nir.c
b/src/gallium/drivers/radeonsi/si_shader_nir.c
i
Tested-by: Dieter Nützel
Woohoo,
this one shot 'smoketest' for the first time over 1200 fps on my system
(RX580, Xeon X3470, 4/8, ~3 GHz, 24 GB)!
Comming from ~1060-1080 fps.
6154 presents in 5.00069 seconds (FPS: 1230.63)
6183 presents in 5.00108 seconds (FPS: 1236.33)
6173 presents in 5.0
Am 10.01.2018 um 07:48 schrieb Dave Airlie:
> From: Dave Airlie
>
> This adds support for GDS ops to sb backend.
>
> This seems to work for atomics and tess factor writes.
>
> Signed-off-by: Dave Airlie
> diff --git a/src/gallium/drivers/r600/sb/sb_bc_finalize.cpp
> b/src/gallium/drivers/r60
From: Dave Airlie
Instead of putting the push constants into the upload buffer,
if we have space in the sgprs we can upload the per-stage
constants into the shaders directly.
This saves a few reads from memory in the meta shaders,
we should also be able to inline other objects like
descriptors.
From: Dave Airlie
This gathers the min/max push consts used, and whether
the shader uses indirect push constants.
It also adds an init function to handle the combined
shaders.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 2 ++
src/amd/common/ac_shader_info.c | 25
Am 11.01.2018 um 03:28 schrieb Dave Airlie:
> On 11 January 2018 at 12:19, Roland Scheidegger wrote:
>> Am 10.01.2018 um 07:48 schrieb Dave Airlie:
>>> From: Dave Airlie
>>>
>>> This stops them being optimised out.
>>> ---
>>> src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 5 -
>>> 1 file ch
Lowering these to temps makes a big mess, and results in some
piglit test failures. Also the radeonsi backend (the only backend
to support tess) has support for indirects so there is no need to
lower them anyway.
Fixes the following piglit tests on radeonsi:
tests/spec/arb_tessellation_shader/exe
On 11 January 2018 at 12:19, Roland Scheidegger wrote:
> Am 10.01.2018 um 07:48 schrieb Dave Airlie:
>> From: Dave Airlie
>>
>> This stops them being optimised out.
>> ---
>> src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 5 -
>> 1 file changed, 4 insertions(+), 1 deletion(-)
>>
>> diff --gi
From: Dave Airlie
Running dota2 since the below commit crashes with an llvm assert.
Trim the vector like the other user. This possible could also be
avoided by not padding inside the load vec3->vec4.
Fixes: 41c36c4549 (amd/common: use ac_build_buffer_load() for emitting UBO
loads)
Signed-off-b
For patches 1, 4-7:
Reviewed-by: Marek Olšák
Marek
On Mon, Jan 8, 2018 at 1:35 AM, Timothy Arceri wrote:
> ---
> src/gallium/drivers/radeonsi/si_shader_nir.c | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/gallium/drivers/radeonsi/si_shader_nir.c
> b/src/gallium/drivers/rad
Reviewed-by: Marek Olšák
Marek
On Fri, Jan 5, 2018 at 5:49 PM, Guillaume Charifi
wrote:
> Signed-off-by: Guillaume Charifi
> ---
> src/mesa/state_tracker/st_atom_framebuffer.c | 18 +-
> 1 file changed, 5 insertions(+), 13 deletions(-)
>
> diff --git a/src/mesa/state_tracker/s
Patches 1 and 3 are
Reviewed-by: Ian Romanick
On 01/09/2018 02:58 PM, Grazvydas Ignotas wrote:
> Signed-off-by: Grazvydas Ignotas
> ---
> src/util/crc32.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/util/crc32.c b/src/util/crc32.c
> index 44d637c..f2e01c6 1006
Am 11.01.2018 um 00:27 schrieb Marek Olšák:
> On Thu, Jan 11, 2018 at 12:12 AM, Roland Scheidegger
> wrote:
>> Am 10.01.2018 um 20:49 schrieb Marek Olšák:
>>> From: Marek Olšák
>>>
>>> ---
>>> src/gallium/drivers/llvmpipe/lp_screen.c | 2 +-
>>> src/gallium/drivers/llvmpipe/lp_state_fs.c | 1
Reviewed-by: Marek Olšák
Marek
On Tue, Jan 9, 2018 at 11:58 PM, Grazvydas Ignotas wrote:
> zlib provides a faster slice-by-4 CRC32 implementation than the
> traditional single byte lookup one used by mesa. As most supported
> platforms now link zlib unconditionally, we can easily use it.
>
> Im
On Wed, Jan 10, 2018 at 3:54 AM, Karol Herbst wrote:
> From: Rob Clark
>
> This was handled for VS, but not for GS.
>
> Fixes for gallium drivers using nir:
> spec@arb_gpu_shader5@arb_gpu_shader5-xfb-streams-without-invocations
> spec@arb_gpu_shader5@arb_gpu_shader5-xfb-streams*
> spec@arb_transf
From: Roland Scheidegger
ARB_ubo requires 12 UBOs (per stage) at least, but this limit has been
raised by GL 4.3 to 14, so don't advertize GL 4.3 without it (only checking
the vertex stage since all drivers probably have the same limit anyway for
other stages). (piglit has minmax tests for that k
Reviewed-by: Marek Olšák
Marek
On Wed, Jan 10, 2018 at 7:28 AM, Timothy Arceri wrote:
> Fixes the follow test for radeonsi nir:
>
> tests/spec/arb_tessellation_shader/execution/quads.shader_test
>
> Also stops 8 other tests from crashing, they now just fail e.g.
>
> tcs-output-array-float-index
https://bugs.freedesktop.org/show_bug.cgi?id=104214
--- Comment #34 from Evangelos Foutras ---
I built mesa/lib32-mesa packages for Arch Linux which include the patch from
comment 29 (not sure if lib32-mesa is needed but it can't hurt either):
https://pkgbuild.com/~foutrelis/test-builds/mesa/bug
On Tue, Jan 09, 2018 at 09:41:02PM +, Anuj Phogat wrote:
> More Coffee Lake PCI IDs have been added to the spec.
>
> Cc: Rodrigo Vivi
> Signed-off-by: Anuj Phogat
Reviewed-by: Rodrigo Vivi
> ---
> include/pci_ids/i965_pci_ids.h | 10 +-
> 1 file changed, 9 insertions(+), 1 delet
Reviewed-by: Bas Nieuwenhuizen
On Wed, Jan 10, 2018 at 11:14 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> If the number of instances hasn't changed and we've already
> emitted it, don't emit it again.
>
> If the vertex shader is the same and the first_instance, vertex_offset
> haven't changed
BTW, this is technically not a radv patch since it's shared code.
Marek
On Thu, Jan 11, 2018 at 12:30 AM, Marek Olšák wrote:
> I also recommend porting lp_create_builder, which might also improve things.
>
> Marek
>
> On Wed, Jan 10, 2018 at 10:31 PM, Samuel Pitoiset
> wrote:
>> This is an opti
I also recommend porting lp_create_builder, which might also improve things.
Marek
On Wed, Jan 10, 2018 at 10:31 PM, Samuel Pitoiset
wrote:
> This is an optimisation that is recommended by Matt Arsenault,
> and used by RadeonSI.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/common/ac_nir_
On Thu, Jan 11, 2018 at 12:12 AM, Roland Scheidegger wrote:
> Am 10.01.2018 um 20:49 schrieb Marek Olšák:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/llvmpipe/lp_screen.c | 2 +-
>> src/gallium/drivers/llvmpipe/lp_state_fs.c | 18 +-
>> 2 files changed, 18 insertions
Am 10.01.2018 um 20:49 schrieb Marek Olšák:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/llvmpipe/lp_screen.c | 2 +-
> src/gallium/drivers/llvmpipe/lp_state_fs.c | 18 +-
> 2 files changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/llvmpipe/l
Reviewed-by: Timothy Arceri
On 09/01/18 18:48, Tapani Pälli wrote:
Signed-off-by: Tapani Pälli
---
src/compiler/glsl/shader_cache.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/compiler/glsl/shader_cache.h b/src/compiler/glsl/shader_cache.h
index c0598f37e8.
On 11/01/18 08:19, Dieter Nützel wrote:
Ping!
Shouldn't land this, soon
I'd love to land it but still waiting on a review.
Am 08.12.2017 10:57, schrieb Timothy Arceri:
Core support in now in master so this is just a resend of the gallium
patches.
Resolves crash in Dead Island:
Bugzilla:
On 2018-01-08 23:48:19, Tapani Pälli wrote:
> Cache set and get are called in similar fashion as what is happening
> with disk cache. Functionality requires ARB_get_program_binary and
> EGL_ANDROID_blob_cache support.
>
> Signed-off-by: Tapani Pälli
> ---
> src/mesa/Makefile.sources |
On 11 January 2018 at 06:13, Gert Wollny wrote:
> Am Mittwoch, den 10.01.2018, 16:36 +0100 schrieb Gert Wollny:
>> This seems to satisfy the sb optimizer, i.e. no regressions in the
>> piglits compared to disabling sb for tesselation shaders with
>> barriers but enabling them in general.
>> ---
>
From: Dave Airlie
If the number of instances hasn't changed and we've already
emitted it, don't emit it again.
If the vertex shader is the same and the first_instance, vertex_offset
haven't changed don't emit them again.
This increases the fps in GL_vs_VK -t 1 -m -api vk from around 40
to aroun
On Wed, Jan 10, 2018 at 9:36 PM, Rob Clark wrote:
> On Wed, Jan 10, 2018 at 3:33 PM, Rob Clark wrote:
>> On Wed, Jan 10, 2018 at 2:49 PM, Marek Olšák wrote:
>>> From: Marek Olšák
>>>
>>> ---
>>> src/gallium/drivers/freedreno/freedreno_screen.c | 4 +---
>>> src/gallium/drivers/freedreno/freed
This is an optimisation that is recommended by Matt Arsenault,
and used by RadeonSI.
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_nir_to_llvm.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 1520342d05..b47
Reviewed-by: Jason Ekstrand
On Wed, Jan 10, 2018 at 1:11 PM, Andres Gomez wrote:
> anv_extensions usage from anv_icd was bringing the unwanted dependency
> of mako templates for the latter. We don't want that since it will
> force the dependency even for distributable tarballs which was not
> n
On Wed, Jan 10, 2018 at 3:50 PM, Connor Abbott wrote:
> On Wed, Jan 10, 2018 at 3:27 PM, Ilia Mirkin wrote:
>> On Wed, Jan 10, 2018 at 3:13 PM, Gert Wollny wrote:
>>> Am Mittwoch, den 10.01.2018, 16:36 +0100 schrieb Gert Wollny:
This seems to satisfy the sb optimizer, i.e. no regressions in
Ping!
Shouldn't land this, soon
Am 08.12.2017 10:57, schrieb Timothy Arceri:
Core support in now in master so this is just a resend of the gallium
patches.
Resolves crash in Dead Island:
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=85564
Branch available here:
https://github.com/t
Reviewed-by: Timothy Arceri
On 11/01/18 03:24, Bas Nieuwenhuizen wrote:
If they were promoted from inputs/outputs, they could have a
non-zero value left over, which messed with our store handling.
Fixes: 06f05040eb "radv: Link shaders."
---
src/amd/common/ac_nir_to_llvm.c | 1 +
1 file chan
anv_extensions usage from anv_icd was bringing the unwanted dependency
of mako templates for the latter. We don't want that since it will
force the dependency even for distributable tarballs which was not
needed until now.
Jason suggested this approach.
v2: Patch simplification (Jason).
Bugzilla
On Wed, Jan 10, 2018 at 3:27 PM, Ilia Mirkin wrote:
> On Wed, Jan 10, 2018 at 3:13 PM, Gert Wollny wrote:
>> Am Mittwoch, den 10.01.2018, 16:36 +0100 schrieb Gert Wollny:
>>> This seems to satisfy the sb optimizer, i.e. no regressions in the
>>> piglits compared to disabling sb for tesselation sh
On Wed, Jan 10, 2018 at 3:33 PM, Rob Clark wrote:
> On Wed, Jan 10, 2018 at 2:49 PM, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/freedreno/freedreno_screen.c | 4 +---
>> src/gallium/drivers/freedreno/freedreno_state.c | 13 +
>> 2 files changed, 14 in
On Wed, Jan 10, 2018 at 2:49 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/freedreno/freedreno_screen.c | 4 +---
> src/gallium/drivers/freedreno/freedreno_state.c | 13 +
> 2 files changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/dr
On Wed, Jan 10, 2018 at 3:13 PM, Gert Wollny wrote:
> Am Mittwoch, den 10.01.2018, 16:36 +0100 schrieb Gert Wollny:
>> This seems to satisfy the sb optimizer, i.e. no regressions in the
>> piglits compared to disabling sb for tesselation shaders with
>> barriers but enabling them in general.
>> --
Am Mittwoch, den 10.01.2018, 16:36 +0100 schrieb Gert Wollny:
> This seems to satisfy the sb optimizer, i.e. no regressions in the
> piglits compared to disabling sb for tesselation shaders with
> barriers but enabling them in general.
> ---
Actually, it seems this is not enough, at least for Tomb
From: Marek Olšák
---
src/gallium/drivers/llvmpipe/lp_screen.c | 2 +-
src/gallium/drivers/llvmpipe/lp_state_fs.c | 18 +-
2 files changed, 18 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_screen.c
b/src/gallium/drivers/llvmpipe/lp_screen.c
index
From: Marek Olšák
---
src/mesa/state_tracker/st_atom_constbuf.c | 19 +++
src/mesa/state_tracker/st_context.c | 2 --
src/mesa/state_tracker/st_context.h | 1 -
src/mesa/state_tracker/st_pbo.c | 18 +++---
4 files changed, 6 insertions(+), 34 d
From: Marek Olšák
---
src/gallium/auxiliary/util/u_tests.c | 111 ++-
1 file changed, 110 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/util/u_tests.c
b/src/gallium/auxiliary/util/u_tests.c
index 2548b46..cb337a2 100644
--- a/src/gallium/auxili
From: Marek Olšák
---
src/gallium/state_trackers/nine/device9.c| 4 -
src/gallium/state_trackers/nine/device9.h| 3 -
src/gallium/state_trackers/nine/nine_ff.c| 28 +-
src/gallium/state_trackers/nine/nine_state.c | 125 +--
4 files changed, 4 insertio
From: Marek Olšák
Cc: 17.2 17.3
---
src/gallium/auxiliary/util/u_threaded_context.c | 13 +
src/gallium/auxiliary/util/u_threaded_context.h | 8
2 files changed, 21 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_threaded_context.c
b/src/gallium/auxiliary/util/u_
From: Marek Olšák
---
src/gallium/docs/source/screen.rst | 4
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 -
src/gallium/drivers/freedreno/freedreno_screen.c | 1 -
src/gallium/drivers/i915/i915_screen.c | 1 -
src/gallium/drivers/llvmpipe/lp_screen.c
Reviewed-by: Bruce Cherniak
> On Jan 9, 2018, at 11:50 AM, George Kyriazis
> wrote:
>
> BuilderSWR::swr_gs_llvm_fetch_input() (and consequently
> swr_gs_llvm_fetch_input()), did not handle the case where
> is_vindex_indirect or is_aindex_direct is set.
>
> Implement it, using the code in dra
From: Marek Olšák
---
src/gallium/docs/source/screen.rst | 2 --
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 -
src/gallium/drivers/freedreno/freedreno_screen.c | 1 -
src/gallium/drivers/i915/i915_screen.c | 1 -
src/gallium/drivers/llvmpipe/lp_screen.c
From: Marek Olšák
All drivers support it.
---
src/mesa/state_tracker/st_extensions.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index 2827c59..ea49abf 100644
--- a/src/mesa/state_trac
From: Marek Olšák
---
src/gallium/docs/source/screen.rst | 3 ---
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 -
src/gallium/drivers/freedreno/freedreno_screen.c | 1 -
src/gallium/drivers/i915/i915_screen.c | 1 -
src/gallium/drivers/llvmpipe/lp_screen.c
From: Marek Olšák
---
src/gallium/drivers/freedreno/freedreno_screen.c | 4 +---
src/gallium/drivers/freedreno/freedreno_state.c | 13 +
2 files changed, 14 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/freedreno/freedreno_screen.c
b/src/gallium/drivers/freedreno
From: Marek Olšák
Cc: 17.2 17.3
---
src/gallium/drivers/radeon/r600_texture.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 34b3ab0..36eff40 100644
--- a/src/galliu
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
index 0bbd661..df0df8c 100644
--- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.
---
src/mesa/drivers/dri/i965/brw_draw.c | 41
1 file changed, 41 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
b/src/mesa/drivers/dri/i965/brw_draw.c
index 4945dec..9fd44e4 100644
--- a/src/mesa/drivers/dri/i965/brw_draw.c
+++ b/src/mesa/dr
This commit unifies the CCS_E and CCS_D cases. This should fix a couple
of subtle issues. One is that when you use INTEL_DEBUG=norbc to disable
CCS_E, we don't get the sRGB blending workaround. By unifying the code,
we give CCS_D that workaround as well.
The second issue fixed by this refactor
This reverts commit ee57b15ec764736e2d5360beaef9fb2045ed0f68, "i965:
Disable regular fast-clears (CCS_D) on gen9+". How taht we've fixed the
issue with too many different aux usages in the render cache, it should
be safe to re-enable CCS_D for sRGB.
Bugzilla: https://bugs.freedesktop.org/show_bug
This lets us perform render cache flushes whenever a surface goes from
being used with one aux+format to a different aux+format.
This is the "proper" fix for https://bugs.freedesktop.org/102435.
ee57b15ec764736e2d5360beaef9fb2045ed0f68 which was really just a partial
revert of 3e57e9494c2279580ad6
This makes sure we flush things out such as depth writes prior to using
a surface through the render cache. Besides the overhead of a hash
table look-up, this function is harmless to call repeatedly with the
same arguments.
Cc: "17.3"
---
src/mesa/drivers/dri/i965/brw_draw.c | 2 ++
1 file chan
This series contains the fixes required to fully re-enable CCS for sRGB
render buffers. A bunch of these patches have been sent out individually
or in smaller series but they sort-of all go together.
Jason Ekstrand (6):
i965: Call brw_cache_flush_for_render in predraw_resolve_framebuffer
i965
This allows to reduce the number of dwords that are loaded
with buffer_load_format_xyzw. For example, when the only used
channel is 1, the driver will emit buffer_load_format_x instead.
Shader stats for DOW3 (with some local hacky scripts for SPIRV):
143 shaders in 143 tests
Totals:
SGPRS: 5344 -
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c| 16
src/amd/common/ac_llvm_build.h| 1 +
src/amd/common/ac_nir_to_llvm.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.c | 2 +-
src/gallium
For both versions of llvm.amdgcn.buffer.load.{format}.*.
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c | 61 +++---
1 file changed, 40 insertions(+), 21 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_buil
On Wed, Jan 10, 2018 at 9:55 AM, Matt Turner wrote:
> Bizarre. Hope we can get more info on what's going on, but if UW makes
> it work, let's do it.
>
> Both are:
>
> Reviewed-by: Matt Turner
>
Thanks! Based on the limited information, do you have any suggestions how
how to enforce this in the
Bizarre. Hope we can get more info on what's going on, but if UW makes
it work, let's do it.
Both are:
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=104214
Thomas Hellström changed:
What|Removed |Added
Assignee|i...@freedesktop.org |thellst...@vmware.com
--- Comment #
https://bugs.freedesktop.org/show_bug.cgi?id=104376
Thomas Hellström changed:
What|Removed |Added
Status|NEEDINFO|RESOLVED
QA Contact|intel-3d
https://bugs.freedesktop.org/show_bug.cgi?id=104214
Thomas Hellström changed:
What|Removed |Added
CC||ta...@outlook.com
--- Comment #32 fr
On Wed, Jan 10, 2018 at 1:35 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> "The maxDescriptorSet* limit is n times the corresponding
> maxPerStageDescriptor* limit, where n is the number of shader stages
> supported by the VkPhysicalDevice. If all shader stages are supported,
> n
I forgot to CC you on this like you asked,
Dylan
Quoting Dylan Baker (2018-01-05 12:00:57)
> This is a fifth iteration of the meson build system for libdrm. This
> version is significantly cleaned up from the last version and uses a
> style more like the build system in mesa.
>
> It builds all o
Currently meson implements the same logic as SCons for translations,
namely it doesn't do them. This patch changes meson to use logic more
like autotools, and generate translations. To do this we have to go
behind meson's back a bit, and wrap the whole thing up in a single
python script.
Meson has
I'm not sure off the top of my head. I don't have an r600 anymore, but I have an
SI and a nouveau machine, so I'll see if I can reproduce the problem there and
fix it.
Dylan
Quoting Marc Dietrich (2018-01-09 02:38:33)
> Hi Dylan,
>
> just found that vdpau does not work on r600 with meson build.
On Wed, Jan 10, 2018 at 4:09 AM, Andres Gomez wrote:
> anv_extensions usage from anv_icd was bringing the unwanted dependency
> of mako templates for the latter. We don't want that since it will
> force the dependency even for distributable tarballs which was not
> needed until now.
>
> Jason sug
On Wednesday, January 10, 2018 5:11:21 AM PST Iago Toral wrote:
> On Mon, 2018-01-08 at 13:30 -0800, Kenneth Graunke wrote:
> > Growing the batch/state buffer is a lot more dangerous than I
> > thought.
> >
> > A number of places emit multiple state buffer sections, and then
> > write
> > data to
On 01/10/2018 05:11 PM, Marek Olšák wrote:
On Mon, Jan 8, 2018 at 10:31 PM, Samuel Pitoiset
wrote:
On 01/06/2018 12:12 PM, Marek Olšák wrote:
From: Marek Olšák
SGPRS: 2170102 -> 2158430 (-0.54 %)
VGPRS: 1645656 -> 1641516 (-0.25 %)
Spilled SGPRs: 9078 -> 8810 (-2.95 %)
Spilled VGPRs: 13
If they were promoted from inputs/outputs, they could have a
non-zero value left over, which messed with our store handling.
Fixes: 06f05040eb "radv: Link shaders."
---
src/amd/common/ac_nir_to_llvm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/co
On Mon, Jan 8, 2018 at 10:31 PM, Samuel Pitoiset
wrote:
>
>
> On 01/06/2018 12:12 PM, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> SGPRS: 2170102 -> 2158430 (-0.54 %)
>> VGPRS: 1645656 -> 1641516 (-0.25 %)
>> Spilled SGPRs: 9078 -> 8810 (-2.95 %)
>> Spilled VGPRs: 130 -> 114 (-12.31 %)
>> Scr
This seems to satisfy the sb optimizer, i.e. no regressions in the piglits
compared to disabling sb for tesselation shaders with barriers but enabling
them in general.
---
The series goes on top of the r600 sb tessellation support patches.
Best,
Gert
src/gallium/drivers/r600/sb/sb_bc.h
---
src/gallium/drivers/r600/r600_shader.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_shader.c
b/src/gallium/drivers/r600/r600_shader.c
index 3973669509..6a0f3cd8c2 100644
--- a/src/gallium/drivers/r600/r600_shader.c
+++ b/src/gallium/drivers/r600/r600_sha
On 01/10/2018 02:24 PM, Grazvydas Ignotas wrote:
On Wed, Jan 10, 2018 at 1:57 PM, Samuel Pitoiset
wrote:
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c| 12 ++-
src/amd/common/ac_llvm_util.c | 18
src/amd
On Wed, Jan 10, 2018 at 1:57 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/common/ac_llvm_build.c| 12 ++-
> src/amd/common/ac_llvm_util.c | 18
> src/amd/common/ac_llvm_util.h | 6
On Mon, 2018-01-08 at 13:30 -0800, Kenneth Graunke wrote:
> Growing the batch/state buffer is a lot more dangerous than I
> thought.
>
> A number of places emit multiple state buffer sections, and then
> write
> data to the returned pointer, or save a pointer to brw-
> >batch.state.bo
> and then u
My last status was that Leo wanted to commit this.
Leo are you still waiting for something?
Christian.
Am 10.01.2018 um 10:29 schrieb Julien Isorce:
Hi Christian,
>> Otherwise I'm going to give it a few more days on the list and push
it if nobody objects.
Gentle ping ? So far the series ar
https://bugs.freedesktop.org/show_bug.cgi?id=104214
--- Comment #31 from Thomas Hellström ---
(In reply to Evangelos Foutras from comment #30)
> FWIW the "buf->busy == 0" condition seems to always evaluate to true in my
> tests; if I omit it and print !!buf->busy before the
> dri3_free_render_bu
1 - 100 of 128 matches
Mail list logo
