Can anyone review this one?
On Wed, 2017-03-29 at 08:58 +0200, Iago Toral Quiroga wrote:
> Writing and testing are two different things and they can be set
> separately
> by the application. If an application wants to record depth data
> without
> caring for the depth test, it can enable depth tes
On Mon, Apr 3, 2017 at 2:42 PM, Tapani Pälli wrote:
>
>
> On 04/02/2017 08:12 PM, Tomasz Figa wrote:
>>
>> Sorry for replying to myself, just got enlightened...
>>
>> On Mon, Apr 3, 2017 at 2:07 AM, Tomasz Figa wrote:
>>>
>>> Hi Mauro,
>>>
>>> On Mon, Apr 3, 2017 at 1:38 AM, Mauro Rossi
>>> wrot
On 04/02/2017 08:12 PM, Tomasz Figa wrote:
Sorry for replying to myself, just got enlightened...
On Mon, Apr 3, 2017 at 2:07 AM, Tomasz Figa wrote:
Hi Mauro,
On Mon, Apr 3, 2017 at 1:38 AM, Mauro Rossi wrote:
2017-03-30 16:17 GMT+02:00 Emil Velikov :
On 30 March 2017 at 11:55, Tomasz
From: Dave Airlie
This fixes an old bug that seems to get triggered by
dEQP-VK.memory.requirements.image.sparse_tiling_optimal
We return early when allocating S8_UINT due to there being
no color or depth, and end up with image size of 0.
Signed-off-by: Dave Airlie
---
src/amd/vulkan/winsys/am
On 04/02/2017 07:38 PM, Mauro Rossi wrote:
2017-03-30 16:17 GMT+02:00 Emil Velikov mailto:emil.l.veli...@gmail.com>>:
On 30 March 2017 at 11:55, Tomasz Figa mailto:tf...@chromium.org>> wrote:
> Android buffer queues can be abandoned, which results in failing to
> dequeue next buf
Hi,
2017-04-02 22:44 GMT+02:00 Rhys Kidd :
> On 2 April 2017 at 08:42, Christian Gmeiner
> wrote:
>>
>> At the moment travis ci failes with the following error:
>>
>> checking for RADEON... yes
>> checking for RADEON... yes
>> configure: error: LLVM 3.8.0 or newer is required for r600
>> make: **
---
src/gallium/state_trackers/omx/vid_dec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/omx/vid_dec.c
b/src/gallium/state_trackers/omx/vid_dec.c
index 9a6efb8e28..94664eba04 100644
--- a/src/gallium/state_trackers/omx/vid_dec.c
+++ b/src/gallium
On Sun, Apr 2, 2017 at 11:31 AM, Bartosz Tomczyk <
bartosz.tomczy...@gmail.com> wrote:
> ---
> src/mesa/state_tracker/st_shader_cache.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_shader_cache.c
> b/src/mesa/state_tracker/st_shader_cache.c
> index e8c7289e
From: Dave Airlie
This fixes:
dEQP-VK.glsl.texture_functions.texture.samplercubearray*
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index
From: Dave Airlie
The ordering NIR gives us is correct for the hw, this fixes:
dEQP-VK.glsl.texture_functions.texturegrad.* (mainly trigged
on isampler/usampler 3d textures.).
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(
From: Dave Airlie
Interp at sample needs to use the center, since the sample
positions it retrieves are relative to the center.
This fixes a bunch of CTS tests with multisample_interpolation.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 4 +---
1 file changed, 1 insertion(
From: Dave Airlie
The current code was broken, and I decided to redesign it instead.
This puts the sample positions for all samples into the queue
constant descriptor buffer after all the spill/ring descriptors.
It then uses a single offset register to point how far into the
samples the samples
Trailing whitespace and style issue with else fixed and pushed. Thanks!
On 31/03/17 17:22, Nicolai Hähnle wrote:
On 30.03.2017 22:31, Bartosz Tomczyk wrote:
Call it directly when batch queue is empty. This avoids costly thread
synchronisation. This commit improves performance of games that have
That changes to the param in si_llvm_pack_two_int16() and
si_llvm_pack_two_int32_as_int16() should probably have been split out.
Anyway with or without that change series is:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freede
The changes to the param in si_llvm_pack_two_int16() and
si_llvm_pack_two_int32_as_int16() should probably have been split out.
Anyway with or without that change series is:
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedes
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 108 ++-
1 file changed, 48 insertions(+), 60 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 3876010..2da00f9 100644
--- a/src/galli
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index e514d61..21efd9a 100644
--- a/src/galli
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 270 ++---
src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 2 +-
.../drivers/radeonsi/si_shader_tgsi_setup.c| 24 +-
3 files changed, 145 insertions(+), 151 deletions(-)
diff --git a/src/g
From: Marek Olšák
Don't clear state[7] on SI-CIK, and only do the meta stuff on VI+.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100531
---
src/gallium/drivers/radeonsi/si_descriptors.c | 32 ++-
1 file changed, 17 insertions(+), 15 deletions(-)
diff --git a/
For the series:
Reviewed-by: Marek Olšák
Marek
On Fri, Mar 31, 2017 at 7:14 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Require LLVM 5.0 or later because LLVM 4.0 is easily fooled into
> putting the lane select of llvm.amdgcn.readlane into a VGPR and then
> fails to continue to compi
On 03/04/17 09:52, Timothy Arceri wrote:
On 03/04/17 03:31, Bartosz Tomczyk wrote:
---
src/compiler/glsl/shader_cache.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/glsl/shader_cache.cpp
b/src/compiler/glsl/shader_cache.cpp
index ea1bc01f02..8c42a95664 100644
--- a/s
On 03/04/17 03:31, Bartosz Tomczyk wrote:
---
src/compiler/glsl/shader_cache.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/glsl/shader_cache.cpp
b/src/compiler/glsl/shader_cache.cpp
index ea1bc01f02..8c42a95664 100644
--- a/src/compiler/glsl/shader_cache.cpp
+++ b/src
Pushed. Thanks.
Marek
On Sun, Apr 2, 2017 at 7:22 PM, Grazvydas Ignotas wrote:
> All the -Wunused-but-set-variable ones.
> Found a way to do it with a oneliner.
>
> Signed-off-by: Grazvydas Ignotas
> ---
> no commit access (currently pending)
>
> src/amd/addrlib/core/addrcommon.h | 2 +-
> 1 f
Pushed. Thanks.
Marek
On Sun, Apr 2, 2017 at 7:22 PM, Grazvydas Ignotas wrote:
> si_state.c: In function ‘si_make_texture_descriptor’:
> si_state.c:3240:25: warning: ‘num_format’ may be used uninitialized
> si_state.c:3240:12: warning: ‘data_format’ may be used uninitialized
>
> Signed-off-by: G
Reviewed-by: Marek Olšák
Marek
On Sun, Apr 2, 2017 at 6:02 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/gallium/docs/source/tgsi.rst | 32
> 1 file changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/src/gallium/docs/source/tgsi.rst
For the series:
Reviewed-by: Marek Olšák
Marek
On Fri, Mar 31, 2017 at 12:48 PM, Samuel Pitoiset
wrote:
> Should be used by the state tracker when glGetImageHandleARB()
> is called in order to create a pipe_image_view template.
>
> v3: - move the comment to *.c
> v2: - make 'st' const
> -
Pushed, thanks.
Marek
On Sat, Apr 1, 2017 at 10:55 AM, Constantine Kharlamov
wrote:
> Thank you, please, could you push them too? I don't have commit access.
>
> On 28.03.2017 22:36, Marek Olšák wrote:
>> Actually nevermind.
>>
>> Patches 1-3:
>> Reviewed-by: Marek Olšák
>>
>> Patches 4-5:
>> T
On 2 April 2017 at 22:43, Bas Nieuwenhuizen wrote:
> We want the guardband_x/y to be the largerst scalars such that each
> viewport scaled by that amount is still a subrange of [-32767, 32767].
>
> The old code has a couple of issues:
> 1) It used scissor instead of viewport_scissor, potentially t
On 3 April 2017 at 06:44, Rhys Kidd wrote:
> On 2 April 2017 at 08:42, Christian Gmeiner
> wrote:
>>
>> At the moment travis ci failes with the following error:
>>
>> checking for RADEON... yes
>> checking for RADEON... yes
>> configure: error: LLVM 3.8.0 or newer is required for r600
>> make: **
Per comments by Travis-CI, the apt addon is only really needed for the
container-based Precise builds, as they don't yet support Trusty on that
platform.
Mesa currently uses Trusty fully-virtualized environment (due to sudo:
required).
See further:
https://docs.travis-ci.com/user/trusty-ci-envi
On 2 April 2017 at 08:42, Christian Gmeiner
wrote:
> At the moment travis ci failes with the following error:
>
> checking for RADEON... yes
> checking for RADEON... yes
> configure: error: LLVM 3.8.0 or newer is required for r600
> make: *** No targets specified and no makefile found. Stop.
>
>
On Sun, Apr 2, 2017 at 8:31 PM, Bartosz Tomczyk wrote:
> ---
> src/compiler/glsl/shader_cache.cpp | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/compiler/glsl/shader_cache.cpp
> b/src/compiler/glsl/shader_cache.cpp
> index ea1bc01f02..8c42a95664 100644
> --- a/src/compiler/glsl/s
From: Marek Olšák
---
src/gallium/auxiliary/indices/u_primconvert.c | 10 --
src/gallium/include/pipe/p_state.h| 2 +-
2 files changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/gallium/auxiliary/indices/u_primconvert.c
b/src/gallium/auxiliary/indices/u_primconvert.
From: Marek Olšák
---
src/gallium/include/pipe/p_state.h | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index ce19b92..5d995dc 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/gallium/in
From: Marek Olšák
Some other changes needed here.
---
src/gallium/auxiliary/rbug/rbug_texture.c | 4 ++--
src/gallium/auxiliary/rbug/rbug_texture.h | 4 ++--
src/gallium/auxiliary/vl/vl_video_buffer.h | 2 +-
src/gallium/drivers/rbug/rbug_core.c | 2 +-
src/gallium/include/pipe/p_sta
From: Marek Olšák
---
src/gallium/include/pipe/p_state.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index 54bcf07..62f5192 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/gallium/i
From: Marek Olšák
Also:
pipe_transfer: 48 -> 40 bytes.
pipe_blit_info = 176 -> 160 bytes.
---
src/gallium/include/pipe/p_state.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index 392bb8b..6a
From: Marek Olšák
---
src/gallium/include/pipe/p_state.h | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index a4692b6..392bb8b 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/gall
From: Marek Olšák
---
src/gallium/include/pipe/p_state.h | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index ae7f1c3..d68a4d4 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/
From: Marek Olšák
---
src/gallium/include/pipe/p_state.h | 9 -
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index 62f5192..a4692b6 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/gallium/
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.h | 2 +-
src/gallium/drivers/radeon/r600_streamout.c | 2 +-
src/gallium/include/pipe/p_state.h| 4 ++--
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h
2017-03-31 13:05 GMT+02:00 Tapani Pälli :
>
>
> On 03/31/2017 10:12 AM, Tapani Pälli wrote:
>
>>
>>
>> On 03/31/2017 09:06 AM, Tapani Pälli wrote:
>>
>>>
>>>
>>> On 03/31/2017 08:24 AM, Rob Clark wrote:
>>>
On Fri, Mar 31, 2017 at 12:22 AM, Tapani Pälli
wrote:
>
>
> On
Also change gs_output_prim type: unsigned → pipe_prim_type. The idea of
the code is mostly taken from radeonsi. The new code operating on
prev/curr rast_primitives saves ≈15 reloads of PA_SC_LINE_STIPPLE per
frame in Kane&Lynch2
Signed-off-by: Constantine Kharlamov
---
src/gallium/drivers/r600/r
The patches are based on top of the other 3 reviewed but not yet pushed from
"Small r600_draw_vbo optimizations". Here, amongst all, I replaced some
"unsigned"s with the enums they represent. It helps understanding the code, and
even allowed to remove a short comment with the sole purpose of just d
Signed-off-by: Constantine Kharlamov
---
src/gallium/drivers/r600/r600_state_common.c | 41 +---
1 file changed, 19 insertions(+), 22 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_state_common.c
b/src/gallium/drivers/r600/r600_state_common.c
index 096caed8d3..d
Specifically, non-line primitives skipped, and defaulting to reset on
each packet.
The skip of non-line primitives saves ≈110 resetting of
PA_SC_LINE_STIPPLE register per frame in Kane&Lynch2.
Signed-off-by: Constantine Kharlamov
---
src/gallium/drivers/r600/r600_state_common.c | 21 +++
Note: si_shader.h has also "type" variable that should be changed to
"enum pipe_prim_type", however it triggers a bunch of warnings about
unhandled switches, so due not knowing the correct way to handle them, I
decided to leave it as is.
Signed-off-by: Constantine Kharlamov
---
src/gallium/drive
---
src/mesa/state_tracker/st_shader_cache.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/state_tracker/st_shader_cache.c
b/src/mesa/state_tracker/st_shader_cache.c
index e8c7289ec6..5dbcb74f73 100644
--- a/src/mesa/state_tracker/st_shader_cache.c
+++ b/src/mesa/state_tracker/st
---
src/compiler/glsl/shader_cache.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/glsl/shader_cache.cpp
b/src/compiler/glsl/shader_cache.cpp
index ea1bc01f02..8c42a95664 100644
--- a/src/compiler/glsl/shader_cache.cpp
+++ b/src/compiler/glsl/shader_cache.cpp
@@ -1273,6 +12
All the -Wunused-but-set-variable ones.
Found a way to do it with a oneliner.
Signed-off-by: Grazvydas Ignotas
---
no commit access (currently pending)
src/amd/addrlib/core/addrcommon.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/addrlib/core/addrcommon.h
b/src/
si_state.c: In function ‘si_make_texture_descriptor’:
si_state.c:3240:25: warning: ‘num_format’ may be used uninitialized
si_state.c:3240:12: warning: ‘data_format’ may be used uninitialized
Signed-off-by: Grazvydas Ignotas
---
no commit access (currently pending)
src/gallium/drivers/radeonsi/s
Sorry for replying to myself, just got enlightened...
On Mon, Apr 3, 2017 at 2:07 AM, Tomasz Figa wrote:
> Hi Mauro,
>
> On Mon, Apr 3, 2017 at 1:38 AM, Mauro Rossi wrote:
>>
>>
>> 2017-03-30 16:17 GMT+02:00 Emil Velikov :
>>>
>>> On 30 March 2017 at 11:55, Tomasz Figa wrote:
>>> > Android buff
Hi Mauro,
On Mon, Apr 3, 2017 at 1:38 AM, Mauro Rossi wrote:
>
>
> 2017-03-30 16:17 GMT+02:00 Emil Velikov :
>>
>> On 30 March 2017 at 11:55, Tomasz Figa wrote:
>> > Android buffer queues can be abandoned, which results in failing to
>> > dequeue next buffer. Currently this would fail somewhere
2017-03-30 16:17 GMT+02:00 Emil Velikov :
> On 30 March 2017 at 11:55, Tomasz Figa wrote:
> > Android buffer queues can be abandoned, which results in failing to
> > dequeue next buffer. Currently this would fail somewhere deep within
> > the DRI stack calling loader's getBuffers*(), without any
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/tgsi.rst | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/src/gallium/docs/source/tgsi.rst b/src/gallium/docs/source/tgsi.rst
index 05b06ce..ca31924 100644
--- a/src/gallium/docs/source/tg
https://bugs.freedesktop.org/show_bug.cgi?id=100402
--- Comment #2 from Ilia Mirkin ---
Please retest with
https://cgit.freedesktop.org/mesa/mesa/commit/?id=7a0c1eee0c25e343d8c276e4471e4a113d61f233
and let me know if that's fixed now.
--
You are receiving this mail because:
You are the assignee
Some packets like 3DSTATE_VF_STATISTICS, 3DSTATE_DRAWING_RECTANGLE,
3DPRIMITIVE, PIPELINE_SELECT, etc... have configurable fields in
dword0, we probably want to print those.
Signed-off-by: Lionel Landwerlin
---
src/intel/common/gen_decoder.c| 21 +
src/intel/c
Signed-off-by: Lionel Landwerlin
---
src/intel/common/gen_decoder.c | 50 +-
src/intel/common/gen_decoder.h | 1 +
2 files changed, 26 insertions(+), 25 deletions(-)
diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
index 7b04ac
This series is :
Reviewed-by: Lionel Landwerlin
On 01/04/17 06:37, Jason Ekstrand wrote:
I've had these patches lying around for a while now waiting for the extension
to go public. It went public, so here they are. This series adds all of the
core window system bits for doing a trivial imple
We want the guardband_x/y to be the largerst scalars such that each
viewport scaled by that amount is still a subrange of [-32767, 32767].
The old code has a couple of issues:
1) It used scissor instead of viewport_scissor, potentially taking into
account a viewport that is too small and theref
At the moment travis ci failes with the following error:
checking for RADEON... yes
checking for RADEON... yes
configure: error: LLVM 3.8.0 or newer is required for r600
make: *** No targets specified and no makefile found. Stop.
Signed-off-by: Christian Gmeiner
---
.travis.yml | 2 +-
1 file
Reviewed-by: Bas Nieuwenhuizen
On Sun, Apr 2, 2017 at 6:47 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> y is vert, x is horiz.
>
> Noticed in visual inspection compared to radeonsi.
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/vulkan/si_cmd_buffer.c | 4 ++--
> 1 file changed, 2 insertio
62 matches
Mail list logo