Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index 1f79fef..ca22df2 100644
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/opengl/eglinfo.c
@@ -93,6 +93,8 @@
On 04/20/2016 09:37 PM, Nicolai Hähnle wrote:
On 20.04.2016 11:13, Samuel Pitoiset wrote:
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
The missing break caused the IB size to be overwritten with
the size of IB_CONST.
This was introduced in: 7201230582e060aa2eb79c825d3188b437ef7bb8
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
https://bugs.freedesktop.org/show_bug.cgi?id=95038
--- Comment #1 from Aaron Watry ---
Are the atomic_add/atomic_or operations causing the crash operating against
local or global memory?
Last I checked, I think that atomics for r600g-based cards were only
implemented in LLVM
On Tue, Apr 19, 2016 at 12:44 PM, Matt Turner wrote:
> On Mon, Apr 18, 2016 at 11:52 PM, Kenneth Graunke
> wrote:
>> Previously, opt_vector_float() always interpreted MOV sources as
>> floating point, and always created a MOV with a F-type destination.
On Wed, Apr 20, 2016 at 4:10 PM, Adam Jackson wrote:
> Signed-off-by: Adam Jackson
Series is:
Reviewed-by: Alex Deucher
> ---
> src/egl/opengl/eglinfo.c | 41 +++--
> 1 file changed, 35
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 23
> ++
> 1 file
https://bugs.freedesktop.org/show_bug.cgi?id=95038
Bug ID: 95038
Summary: atomic_add and atomic_or cause a OpenCL crashes
Product: Mesa
Version: 11.2
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
On 04/20/2016 07:47 PM, Ilia Mirkin wrote:
Presumably you'd want to touch up the non-limm side of this as well?
Although TBH, I can't really think of a time when it'd matter. I think
you're pretty much guaranteed to have stype == dtype for mul's. Maybe
I'm wrong?
Well, the non-limm part is
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Vinson Lee changed:
What|Removed |Added
Keywords||bisected
On Wed, Apr 20, 2016 at 8:18 PM, Grigori Goronzy wrote:
> On 2016-04-20 02:20, Nicolai Hähnle wrote:
>>
>> This is just a slight massaging of the patch you sent previously. What
>> happened to the discussion we had about how to do this properly?
>>
>
> This already provides
Am 20.04.2016 um 21:29 schrieb Matt Turner:
> Float suffixes are allowed in all subsequent GLSL specifications, and
> it's obvious what the user meant if they specify one. Accept it with a
> warning to avoid breaking applications, like Planeshift.
> ---
> src/compiler/glsl/glsl_lexer.ll | 4 ++--
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 41 +++--
1 file changed, 35 insertions(+), 6 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index 875e407..75d9fe5 100644
---
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 29 ++---
1 file changed, 18 insertions(+), 11 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index b044eaa..875e407 100644
--- a/src/egl/opengl/eglinfo.c
+++
These are independent of the display, and I want to iterate over those.
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index ca22df2..b044eaa
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 1:47 PM, Samuel Pitoiset
wrote:
> This is only valid for other atomic operations (including CAS). This
> fixes an invalid opcode error from dmesg. While we are it, make sure
> to initialize global
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87
> ++
> 1 file changed, 87 insertions(+)
>
> diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=95005
--- Comment #6 from Christoph Haag ---
Just FYI: the patch from comment 1 helps for the (very) old unreal demos that
can be downloaded from the unreal wiki, but the current unreal engine has a
similar (something with
On 04/20/2016 07:49 PM, Ilia Mirkin wrote:
The SM20/SM30 logic does this for exch || cas. Are you *sure* this
shouldn't be the same? It's pretty silly to do a CAS without a dst,
but it's definitely possible (through DCE).
Yeah, I'm sure. It's definitely not the same logic between gk104 and
Reviewed-by: Ilia Mirkin
Might also want to see what happens on GK20A. Should Just Work (tm).
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> This exposes 8 images for all shader types.
>
> Signed-off-by: Samuel Pitoiset
Reviewed-by: Ilia Mirkin
Same comment re GK20A.
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +-
> 1 file
On Wed, Apr 20, 2016 at 3:14 PM, Samuel Pitoiset
wrote:
>
>
> On 04/20/2016 07:47 PM, Ilia Mirkin wrote:
>>
>> Presumably you'd want to touch up the non-limm side of this as well?
>> Although TBH, I can't really think of a time when it'd matter. I think
>> you're pretty
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 153 +
1 file changed, 153 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
This exposes 8 images for all shader types.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index f8d74d5..c017f4f
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87 ++
1 file changed, 87 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
Hi,
This series is based on the previous one for GK104. Between GK104 and GK110
only the ISA has changed. This adds code emission for all instructions that
are needed to support surfaces/images.
I have exactly the same passrate for both deqp and piglit as GK104.
Please review,
Thanks!
Samuel
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 23 ++
1 file changed, 23 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 58 ++
1 file changed, 58 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 153
> +
> 1 file changed, 153 insertions(+)
>
> diff --git
Signed-off-by: Adam Jackson
---
src/egl/opengl/eglinfo.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
index 75d9fe5..f7da372 100644
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/opengl/eglinfo.c
@@ -178,6 +178,7 @@
On 04/20/2016 10:13 PM, Ilia Mirkin wrote:
On Wed, Apr 20, 2016 at 3:14 PM, Samuel Pitoiset
wrote:
On 04/20/2016 07:47 PM, Ilia Mirkin wrote:
Presumably you'd want to touch up the non-limm side of this as well?
Although TBH, I can't really think of a time when
I am fine with this, it would be nice if we could modify the following
piglit tests to pass when a warning is emitted:
https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/compiler/literals/invalid-float-suffix-capital-f.vert
Adam Jackson writes:
> Signed-off-by: Adam Jackson
> ---
> src/egl/opengl/eglinfo.c | 29 ++---
> 1 file changed, 18 insertions(+), 11 deletions(-)
>
> diff --git a/src/egl/opengl/eglinfo.c b/src/egl/opengl/eglinfo.c
> index
On Wed, Apr 20, 2016 at 4:41 PM, Samuel Pitoiset
wrote:
>
>
> On 04/20/2016 10:35 PM, Ilia Mirkin wrote:
>>
>> On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
>> wrote:
>>>
>>> Signed-off-by: Samuel Pitoiset
>>>
This reverts commit b449366587b5f3f64c6fb45fe22c39e4bc8a4309.
I removed the pass thinking that it was now not useful, but that was not
true. I believe I ran shader-db on HSW and saw no results, but HSW does
not use the unlit centroid workaround code and as a result does not emit
redundant
On Wed, Apr 20, 2016 at 4:55 PM, Adam Jackson wrote:
> Signed-off-by: Adam Jackson
Reviewed-by: Alex Deucher
> ---
> src/egl/opengl/eglinfo.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/egl/opengl/eglinfo.c
Float suffixes are allowed in all subsequent GLSL specifications, and
it's obvious what the user meant if they specify one. Accept it with a
warning to avoid breaking applications, like Planeshift.
---
src/compiler/glsl/glsl_lexer.ll | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff
From: Marek Olšák
---
src/gallium/drivers/r600/r600_pipe.c | 3 +
src/gallium/drivers/radeon/Makefile.sources | 1 +
src/gallium/drivers/radeon/r600_pipe_common.c | 2 +
src/gallium/drivers/radeon/r600_pipe_common.h | 5 +
On 20.04.2016 11:13, Samuel Pitoiset wrote:
On 04/20/2016 05:43 PM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_cb_queryobj.c | 55
On Wed, Apr 20, 2016 at 12:48 PM, Lars Hamre wrote:
> I am fine with this, it would be nice if we could modify the following
> piglit tests to pass when a warning is emitted:
>
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 12
> 1 file changed,
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 58
> ++
> 1 file
On 04/20/2016 10:35 PM, Ilia Mirkin wrote:
On Wed, Apr 20, 2016 at 4:25 PM, Samuel Pitoiset
wrote:
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87 ++
1 file changed, 87
https://bugs.freedesktop.org/show_bug.cgi?id=95038
ros...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 2016-04-20 11:46, Nicolai Hähnle wrote:
On 19.04.2016 17:50, Bas Nieuwenhuizen wrote:
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 5 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 5 -
2 files changed, 8
On 20.04.2016 01:32, Bas Nieuwenhuizen wrote:
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=464cef5b06e65aa740704e4adac68b7f5fee1b88
> Author: Bas Nieuwenhuizen
> Date: Sat Mar 19 15:16:50 2016 +0100
>
> radeonsi: enable TGSI support cap for
On 19/04/16 23:52, Jason Ekstrand wrote:
> On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
> sigles...@igalia.com> wrote:
>
>> From: Connor Abbott
>>
>> v2: Move to compiler/nir (Iago)
>>
>> Signed-off-by: Iago Toral Quiroga
>> ---
>>
On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote:
>
>
> On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez
> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for
>
On 19.04.2016 00:55, Nicolas Dufresne wrote:
> Le lundi 18 avril 2016 à 11:40 +0900, Michel Dänzer a écrit :
>> On 17.04.2016 09:49, nico...@ndufresne.ca wrote:
>>>
>>> From: Nicolas Dufresne
>>>
>>> Sorry for the long delay breaking down this patch. I have now
>>>
On 20.04.2016 17:48, Oded Gabbay wrote:
> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
>> On 20.04.2016 03:13, Oded Gabbay wrote:
>>> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay
On Wed, Apr 20, 2016 at 11:14 AM, Oded Gabbay wrote:
> On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote:
>> On 20.04.2016 17:48, Oded Gabbay wrote:
>>> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
On 20.04.2016
On Wed, 2016-04-20 at 08:37 +0200, Iago Toral wrote:
> On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote:
> >
> >
> > On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez
> > wrote:
> > From: Iago Toral Quiroga
> >
> >
https://bugs.freedesktop.org/show_bug.cgi?id=95003
--- Comment #3 from Serge Martin ---
That would be CL_KERNEL_PREFERRED_WORK_GROUP_SIZE_MULTIPLE from
clGetKernelWorkGroupInfo
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA
On Wed, Apr 20, 2016 at 12:04 PM, Michel Dänzer wrote:
> On 20.04.2016 17:48, Oded Gabbay wrote:
>> On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
>>> On 20.04.2016 03:13, Oded Gabbay wrote:
On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák
On 20.04.2016 03:39, Emil Velikov wrote:
> On 19 April 2016 at 15:47, Chuck Atkins wrote:
>> This still doesn't quite give what you want. One can also have an llvm with
>> component shared libs. So there's three different options for llvm library
>> configurations: a
On 20.04.2016 03:13, Oded Gabbay wrote:
> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
>> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote:
>>> On Mon, Apr 18, 2016 at 6:03 PM,
>>> Ilia Mirkin wrote:
On Mon, Apr 18,
In order to implement the "uninitialized variable" warning it was
needed to hande several corner cases manually. Recently I sent a series
to fix the last false positive:
https://lists.freedesktop.org/archives/mesa-dev/2016-April/113614.html
At this point I think that it would be better to include
On Wed, Apr 20, 2016 at 11:28 AM, Michel Dänzer wrote:
> On 20.04.2016 03:13, Oded Gabbay wrote:
>> On Tue, Apr 19, 2016 at 5:59 PM, Marek Olšák wrote:
>>> On Tue, Apr 19, 2016 at 3:11 PM, Oded Gabbay wrote:
On Mon, Apr 18, 2016
---
Those are basically the shaders I used while developing the warning,
plus some cleaning and some extra ones.
The only reason they are vertex shaders is that the standalone compiler
needs to receive a shader with a valid shader extension.
.../glsl/tests/warnings/000-basic-test.vert
Add an option in order to ask to just print the InfoLog, without any
header or separator. Useful if we want to use the standalone compiler
to track only the warning/error messages.
---
src/compiler/glsl/main.cpp | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff
It executes compiler-glsl on all the available shaders, and it checks
that the outcome is the expected.
Bash code based on the already existing optimization-test
---
src/compiler/Makefile.glsl.am | 3 ++-
src/compiler/glsl/tests/warnings-test | 31 +++
2
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #5 from Timo Aaltonen ---
llvm-3.8 misdetects skylake features, this is fixed in 3.9-snapshot..
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the
On 20.04.2016 02:52, Tom Stellard wrote:
> The range metadata tells LLVM the range of expected values for this intrinsic,
> so it can do some additional optimizations on the result.
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 29 ++---
> 1 file changed, 26
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #6 from Jose Fonseca ---
It's not the first time LLVM misidentifies modern CPUs.
I thought that all the logic in src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
for setting +/-foo mattrs would save us from this
Hi,
On 15-04-16 00:29, Samuel Pitoiset wrote:
On 04/12/2016 12:04 PM, Hans de Goede wrote:
Hi,
On 08-04-16 18:14, Samuel Pitoiset wrote:
On 04/08/2016 12:17 PM, Hans de Goede wrote:
Hi,
On 23-03-16 23:10, Samuel Pitoiset wrote:
Are you sure this won't break compute shaders on fermi?
On 19/04/16 19:39, Emil Velikov wrote:
Hi Chuck,
Thanks for chipping in.
On 19 April 2016 at 15:47, Chuck Atkins wrote:
This still doesn't quite give what you want. One can also have an llvm with
component shared libs. So there's three different options for llvm
Patches 1-3:
Reviewed-by: Marek Olšák
Patch 4:
Acked-by: Marek Olšák
Marek
On Tue, Apr 19, 2016 at 7:52 PM, Tom Stellard wrote:
> The ds_bpermute instruction allows threads to transfer data directly
> to or from the vgprs of
On 20 April 2016 at 03:27, Jonathan Gray wrote:
> On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote:
>> On 19 April 2016 at 04:03, Jonathan Gray wrote:
>> > Any objections to this?
>> >
>> Absolutely none. I simply missed it.
>>
>> > On Mon, Dec 21,
On Wed, Apr 20, 2016 at 1:29 AM, Bas Nieuwenhuizen
wrote:
> I retract patch 1 and 2. Large scratch buffers are nice, but the
> hardware only supports a 32-bit offset into it.
You can still allocate a smaller scratch buffer. This should limit the
number of waves in hw.
Removes the need to set LIBVA_DRIVER_NAME=gallium for supported targets and is
consistent with vdpau and general gallium drivers.
Author: Jimmy Berry
Signed-off-by: Jimmy Berry
---
src/gallium/targets/va/Makefile.am | 14 ++
1 file
According to the comments in src/glx/glxcurrent.c __glXGetCurrentContext()
always returns a valid pointer. If no context is made current, it will
contain dummyContext. Thus a test for NULL will always fail.
Patch author: Egbert Eich
On Tue, Apr 19, 2016 at 11:02 PM, Rob Herring wrote:
>> It would ofc be good to make sure we don't break things for
>> android-x86, but I think they would stick on dri2 + pre-atomic state
>> w/ compile time build decisions until enough of the desktop gpu's
>> support atomic..
On Wed, Apr 20, 2016 at 4:56 PM, Aaron Watry wrote:
> Hi Marek,
>
> This patch ended up breaking make check for me (src/egl/egl-symbols-check).
>
> Also, while bisecting, it occurred to me that you pushed v2 of the series,
> while you had previously sent a v3 to the mesa-dev
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #8 from Timo Aaltonen ---
Actually it wasn't avx512, that was the first one I tried :) It's enabled also
on 3.7 and that version works fine. Only one that was added in 3.8 is PKU, but
dropping just that didn't
From: Nicolai Hähnle
They can only indicate out of memory conditions, since the other error
conditions are caught earlier.
---
src/mesa/state_tracker/st_cb_queryobj.c | 55 -
1 file changed, 33 insertions(+), 22 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_blit.c| 12 +--
src/gallium/drivers/radeonsi/si_descriptors.c | 110 +-
src/gallium/drivers/radeonsi/si_pipe.h| 4 +-
src/gallium/drivers/radeonsi/si_state.h | 4 +-
4
From: Marek Olšák
add it to the RW_BUFFERS descriptor array
now the slot masks don't have to have 64 bits
---
src/gallium/drivers/radeonsi/si_descriptors.c | 23 +
src/gallium/drivers/radeonsi/si_pipe.c| 2 -
src/gallium/drivers/radeonsi/si_pipe.h|
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c| 19 +++
src/gallium/drivers/radeonsi/si_shader.h| 6 ++
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.h | 1 +
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_descriptors.c | 14 +++---
src/gallium/drivers/radeonsi/si_shader.c | 5 +++--
src/gallium/drivers/radeonsi/si_state.c | 3 ++-
src/gallium/drivers/radeonsi/si_state.h | 4
4 files
Hi Marek,
This patch ended up breaking make check for me (src/egl/egl-symbols-check).
Also, while bisecting, it occurred to me that you pushed v2 of the series,
while you had previously sent a v3 to the mesa-dev list. Was pushing v2
intentional?
--Aaron
On Tue, Mar 8, 2016 at 4:52 PM, Marek
Reviewed-by: Nicolai Hähnle
On 20.04.2016 08:35, Bas Nieuwenhuizen wrote:
Some CAPs are stored as 64-bit value while Mesa stores
the related constant as 32-bit value.
Signed-off-by: Bas Nieuwenhuizen
---
For the series:
Reviewed-by: Marek Olšák
You might want to wait for others to comment.
Mare
On Tue, Apr 19, 2016 at 9:38 PM, Rob Herring wrote:
> Add MESA_FORMAT_R8G8B8A8_UNORM and MESA_FORMAT_R8G8B8X8_UNORM formats as
> these are the preferred formats
On 20 April 2016 at 01:46, Elie TOURNIER wrote:
> Now Windows users have the same doxygen files than *nix users
> Not tested (I don't have a windows)
>
> Signed-off-by: Elie TOURNIER
> ---
> doxygen/doxy.bat | 7 +++
> 1 file changed, 7
From: Nicolai Hähnle
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94984
---
src/gallium/drivers/radeon/r600_query.c | 40 ++---
src/gallium/drivers/radeon/r600_query.h | 2 +-
2 files changed, 33 insertions(+), 9 deletions(-)
diff
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/si_state.h | 1 +
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_descriptors.c | 50 +--
src/gallium/drivers/radeonsi/si_pipe.h| 2 +-
2 files changed, 25 insertions(+), 27 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
Hi,
This moves all private shader resources to the RW buffer bindings, including
all driver constant buffers, and the poly stipple image (which is converted
into a constant buffer).
RW buffer descriptors are made global, not per shader stage, so all shaders
receive the same pointer.
Finally,
From: Marek Olšák
- use an enum
- use a unique slot number regardless of the shader stage
(the per-stage slots will go away for RW buffers)
---
src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++---
src/gallium/drivers/radeonsi/si_shader.c| 13 ++
From: Marek Olšák
this will be used in the next commit
---
src/gallium/drivers/radeonsi/si_descriptors.c | 27 +--
1 file changed, 17 insertions(+), 10 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
From: Marek Olšák
const buffers are no longer used since the clip plane const buffer was
moved to RW buffers
---
src/gallium/drivers/radeonsi/si_shader.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 4 ++--
src/gallium/drivers/radeonsi/si_shader.c | 6 +++---
src/gallium/drivers/radeonsi/si_state.h | 25 +++--
3 files changed, 12 insertions(+), 23 deletions(-)
diff --git
On Tue, Apr 19, 2016 at 8:45 PM, Mauro Rossi wrote:
> LOCAL_C_INCLUDES needs an additional path to fix the following build error:
>
> external/mesa/src/compiler/glsl/glcpp/glcpp-lex.l:30:25: fatal error:
> glcpp-parse.h: No such file or directory
> #include
While i never got this error in the first place, I can attest that it
certainly hasn't broken any use in paraview, so...
Tested-by: Chuck Atkins
- Chuck
On Tue, Apr 19, 2016 at 1:45 AM, Markus Wick wrote:
> Am 2016-04-19 01:12, schrieb Tim Rowley:
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 3:35 PM, Bas Nieuwenhuizen
wrote:
> Some CAPs are stored as 64-bit value while Mesa stores
> the related constant as 32-bit value.
>
> Signed-off-by: Bas Nieuwenhuizen
The mode should stay the same as the original struct. In
particular, shared should not be changed to temporary.
Signed-off-by: Bas Nieuwenhuizen
---
src/compiler/glsl/opt_structure_splitting.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 19 April 2016 at 20:59, Tom Stellard wrote:
> On Tue, Apr 19, 2016 at 07:39:13PM +0100, Emil Velikov wrote:
>> Hi Chuck,
>>
>> Thanks for chipping in.
>>
>> On 19 April 2016 at 15:47, Chuck Atkins wrote:
>> > This still doesn't quite give what you
Patched 3-5 are:
Reviewed-by: Iago Toral Quiroga
On Mon, 2016-04-18 at 19:04 -0700, Jason Ekstrand wrote:
> We shouldn't be reading the const_index directly
> ---
> src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff
On Wed, Apr 20, 2016 at 01:33:24PM +0100, Emil Velikov wrote:
> On 20 April 2016 at 03:27, Jonathan Gray wrote:
> > On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote:
> >> On 19 April 2016 at 04:03, Jonathan Gray wrote:
> >> > Any objections to this?
>
1 - 100 of 161 matches
Mail list logo