From: Michel Dänzer
Fixes make check.
Signed-off-by: Michel Dänzer
---
src/egl/egl-symbols-check | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/egl/egl-symbols-check b/src/egl/egl-symbols-check
index 5d46fed..fd20948 100755
--- a/src/egl/egl-symbols-check
+++ b/src/egl/egl-symbols-c
It is written in spec, that if config doesn't support pbuffer
then the EGL_BAD_MATCH should be set.
---
src/egl/drivers/dri2/egl_dri2_fallbacks.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/egl/drivers/dri2/egl_dri2_fallbacks.h
b/src/egl/drivers/dri2/egl_dri2_fallbacks.h
index e7
On Thu, Apr 21, 2016 at 08:03:02AM +0300, Pohjolainen, Topi wrote:
> On Wed, Apr 20, 2016 at 09:49:02PM -0700, Ben Widawsky wrote:
> > From: Ben Widawsky
> >
> > Starting with Skylake, the display engine is capable of scanning out from
> > Y-tiled buffers. As such, we can and should use Y-tiling
On Wednesday, April 20, 2016 9:49:02 PM PDT Ben Widawsky wrote:
> From: Ben Widawsky
>
> Starting with Skylake, the display engine is capable of scanning out from
> Y-tiled buffers. As such, we can and should use Y-tiling for better
efficiency.
> This also has the added benefit of being able to
On Wed, Apr 20, 2016 at 09:49:02PM -0700, Ben Widawsky wrote:
> From: Ben Widawsky
>
> Starting with Skylake, the display engine is capable of scanning out from
> Y-tiled buffers. As such, we can and should use Y-tiling for better
> efficiency.
> This also has the added benefit of being able to
On Saturday, April 16, 2016 4:42:28 PM PDT Topi Pohjolainen wrote:
> This series adds blorp pipeline upload support for gen8 and gen9,
> switches over to blorp blits (except for 2X and 16X msaa which don't
> have support in blorp yet) and finally re-introduces blorp clears
> for gen6-9. This makes
From: Ben Widawsky
Starting with Skylake, the display engine is capable of scanning out from
Y-tiled buffers. As such, we can and should use Y-tiling for better efficiency.
This also has the added benefit of being able to fast clear the winsys buffer.
Note that the buffer allocation done for mip
On Thursday, April 21, 2016 7:14:58 AM PDT Pohjolainen, Topi wrote:
> On Wed, Apr 20, 2016 at 09:06:57PM -0700, Kenneth Graunke wrote:
> > On Saturday, April 16, 2016 4:42:52 PM PDT Topi Pohjolainen wrote:
> > > Signed-off-by: Topi Pohjolainen
> > > ---
> > > src/mesa/drivers/dri/i965/brw_meta_fa
On Wed, Apr 20, 2016 at 09:06:57PM -0700, Kenneth Graunke wrote:
> On Saturday, April 16, 2016 4:42:52 PM PDT Topi Pohjolainen wrote:
> > Signed-off-by: Topi Pohjolainen
> > ---
> > src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 44 +
> +---
> > src/mesa/drivers/dri/i965/br
On Saturday, April 16, 2016 4:42:52 PM PDT Topi Pohjolainen wrote:
> Signed-off-by: Topi Pohjolainen
> ---
> src/mesa/drivers/dri/i965/brw_meta_fast_clear.c | 44 +
+---
> src/mesa/drivers/dri/i965/brw_meta_util.h | 8 +
> 2 files changed, 33 insertions(+), 19 delet
On 21.04.2016 02:42, Marek Olšák wrote:
> On Thu, Apr 14, 2016 at 9:29 AM, Michel Dänzer wrote:
>> On 14.04.2016 11:37, Michel Dänzer wrote:
>>> On 12.04.2016 21:33, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=5a4b74d1ba2c156766a
From: Dave Airlie
This adds support for indirect samplers to the texture size query.
v2: use first active sampler unit.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/gallivm/lp_bld_sample.h | 2 ++
src/gallium/auxiliary/gallivm/lp_bld_sample_soa.c | 3 +++
src/gallium/auxiliary/g
From: Dave Airlie
This is a first attempt at adding support for dynamic indexing
of samplers to llvmpipe. This is needed for ARB_gpu_shader5 support.
This uses the sampler function generator to generate functions
for all samplers, then uses if statements to pick which one to call.
This passes a
I've tried to use the exec mask to work out which sampler we should
be using, this still fails the two tests concerned, maybe I'm doing
something silly.
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman
On 04/20/2016 07:03 PM, Ian Romanick wrote:
> On 04/20/2016 12:29 PM, Matt Turner wrote:
>> 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.
>
On 04/20/2016 12:29 PM, Matt Turner wrote:
> 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.
Did we ever figure out if Planeshift still uses F s
On Wed, Apr 20, 2016 at 06:38:32PM -0700, Kenneth Graunke wrote:
> In commit cda886a4851ab767fba40e8474d6fa8190347e4f, Neil made us stop
> advertising RGBX formats on Gen9+, as the hardware apparently no longer
> has working fast clear support for those formats. Instead, we just
> fall back to RGB
On Wednesday, April 20, 2016 12:29:23 PM PDT Matt Turner wrote:
> 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
Rob Clark writes:
> diff --git a/src/mesa/state_tracker/st_glsl_to_nir.cpp
> b/src/mesa/state_tracker/st_glsl_to_nir.cpp
> new file mode 100644
> index 000..c15c537
> --- /dev/null
> +++ b/src/mesa/state_tracker/st_glsl_to_nir.cpp
> +static void
> +st_nir_assign_uniform_locations(struct gl_
In commit cda886a4851ab767fba40e8474d6fa8190347e4f, Neil made us stop
advertising RGBX formats on Gen9+, as the hardware apparently no longer
has working fast clear support for those formats. Instead, we just
fall back to RGBA formats, and use SCS to override alpha to 1.0.
This is fine, but had o
Rob Clark writes:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
> ---
> src/mesa/Makefile.sources | 2 +
> src/mesa/state_tracker/st_nir.h | 31
> src/mesa/state_tracker/st_nir_lower_builtin.c | 245
> ++
> 3 files changed, 27
When the draw_buffer_enum_to_bitmask function is called while Mesa is
in it's meta state, the wrong GLbitfield is returned sometimes for
OpenGL ES contexts. This is because the _mesa_meta_begin function
makes the ctx->API always equal to the desktop GL during the meta
state.
In the case of GL_BAC
Rob Herring writes:
> Add support for 32-bit RGBX/RGBA formats which are preferred for Android.
>
> Signed-off-by: Rob Herring
> ---
> v2:
> - Rebase to current master after introduction of new
> dri2_format_to_pipe_format function.
Still gets my r-b.
signature.asc
Description: PGP signatu
From: Roland Scheidegger
Screwed up since 0753b135f6e83b171d8a1b08aea967374f3542bc.
(Only an issue with different min/mag filters, and then only in some cases,
which is probably why it went unnoticed for quite a while.
The effect should have simply been nearest mip filter instead of linear, iff
On 20.04.2016 18:49, Bas Nieuwenhuizen wrote:
We need to enable a bit in the CONTEXT_CONTROL packet for the
loads to work.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++
src/gallium/drivers/radeonsi/si_hw_context.c | 5 +
src/gallium/driv
On 20.04.2016 18:49, Bas Nieuwenhuizen wrote:
I have no source for the actual name of these fields, as these are
not in the kernel headers. I hope they are clear though.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/sid.h
v2: varient -> variant
---
src/gallium/drivers/swr/swr_shader.cpp | 41 --
src/gallium/drivers/swr/swr_state.cpp | 6 ++---
src/gallium/drivers/swr/swr_state.h| 17 --
3 files changed, 41 insertions(+), 23 deletions(-)
diff --git a/src/gallium/dri
On Wed, Apr 20, 2016 at 8:31 PM, Tim Rowley wrote:
> ---
> src/gallium/drivers/swr/swr_shader.cpp | 41
> --
> src/gallium/drivers/swr/swr_state.cpp | 6 ++---
> src/gallium/drivers/swr/swr_state.h| 17 --
> 3 files changed, 41 insertions(+), 23
---
src/gallium/drivers/swr/swr_shader.cpp | 41 --
src/gallium/drivers/swr/swr_state.cpp | 6 ++---
src/gallium/drivers/swr/swr_state.h| 17 --
3 files changed, 41 insertions(+), 23 deletions(-)
diff --git a/src/gallium/drivers/swr/swr_shader.cpp
We need to enable a bit in the CONTEXT_CONTROL packet for the
loads to work.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++
src/gallium/drivers/radeonsi/si_hw_context.c | 5 +
src/gallium/drivers/radeonsi/si_state.h | 1 +
3 files change
I have no source for the actual name of these fields, as these are
not in the kernel headers. I hope they are clear though.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_state.c | 4 ++--
src/gallium/drivers/radeonsi/sid.h | 3 +++
2 files changed, 5 insertions(+), 2
https://bugs.freedesktop.org/show_bug.cgi?id=95005
--- Comment #7 from Ilia Mirkin ---
(In reply to Christoph Haag from comment #6)
> 32283 @2 glLinkProgram(program = 3071)
> 32283: warning: link failed
> 32283: warning: error: vertex shader output `out_TEXCOORD0' declared as type
> `#anon_struct
Add support for 32-bit RGBX/RGBA formats which are preferred for Android.
Signed-off-by: Rob Herring
---
v2:
- Rebase to current master after introduction of new
dri2_format_to_pipe_format function.
src/gallium/state_trackers/dri/dri2.c | 6 ++
src/gallium/state_trackers/dri/dri_sc
On 20-Apr-16 6:59 PM, Sinclair Yeh wrote:
Minor comment below. Either way:
Reviewed-by: Sinclair Yeh
On Wed, Apr 20, 2016 at 05:52:17PM +0200, Mircea Gherzan wrote:
This header should not blindly include the GLX and should also
conditionally define functions that use GLX parameters. The
MESA
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 b/src/egl/opengl/eglinfo.c
> index 75d9fe5..f7da372 100644
> --- a/sr
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 MOV_DISP
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 TEXCOORD0 again), but differe
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 it'd matter. I think
you're
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 @@ doOneDisplay(EGLDisplay
https://bugs.freedesktop.org/show_bug.cgi?id=95038
ros...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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 b044eaa..875e407 100644
> --- a/src/egl/opengl
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
>>> ---
>>> .../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 87
>>> +
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 insertions(+)
diff --git a/src/gallium/drivers/nouveau/
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 changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/nouveau/
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
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 +-
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 changed, 58 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/n
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 a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
> b/
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 for the local memory
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
b/src/gal
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 a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
> b
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 changed, 23 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/n
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, 12 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/codegen/nv50_i
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
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index 56c28e8.
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
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index 5f93df2..c
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 100644
--- a/src/gallium/dr
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
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index 16d9421
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
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index c8cb266.
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 Pi
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
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index
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
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
index 15280df.
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 insertions(+), 6 deletions(-)
>
> diff --git a/src/egl/opengl/eglinfo.c
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 addr to 0 for other atomic operations.
>
> Signed-
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.
---
src/mesa/state_tracker/st_
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 much guaranteed to have sty
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
+++ b/src/egl/opengl/eglinf
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 100644
--- a/src/
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
--- a/src/egl/opengl/eglinfo.c
+++ b/src/egl/op
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 @@ PrintConfigs(EGLDisplay
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:
> https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/compiler/literals/invalid-float-suffix-capital-f.vert
> htt
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 ++--
>
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
https://cgit.freedesktop.org/piglit/tree/tests/spec/glsl-1.10/co
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
--
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
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 good value as-is and it
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Vinson Lee changed:
What|Removed |Added
Keywords||bisected
CC|
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 wr
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.
>>
>> This meant that we could mess up sequ
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 gk
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
Se
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 +
src/gallium/drivers/radeon/r600_test_dma.c| 360 ++
On Wed, Apr 20, 2016 at 11:38 AM, Kristian Høgsberg
wrote:
> On Mon, Apr 18, 2016 at 5:10 PM, Jason Ekstrand
> wrote:
> > The first patch in this series adds a short style guide for the Vulkan
> > driver. The rest adds a new emit macro and updates the entire driver to
> > use it and, while we'r
On Mon, Apr 18, 2016 at 5:10 PM, Jason Ekstrand wrote:
> The first patch in this series adds a short style guide for the Vulkan
> driver. The rest adds a new emit macro and updates the entire driver to
> use it and, while we're there, makes the style more consistent.
>
> Jason Ekstrand (18):
>
https://bugs.freedesktop.org/show_bug.cgi?id=95036
Bug ID: 95036
Summary: make check egl-symbols-check regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: r
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 good value as-is and it is (IMHO) reasonably
clean, so why not include it for the time being
https://bugs.freedesktop.org/show_bug.cgi?id=94291
--- Comment #10 from Timo Aaltonen ---
Oh, I didn't know they were subvariants :)
I've dropped them from our llvm-3.8 for now at least..
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the b
Pushed!
On Sat, Apr 16, 2016 at 9:52 PM, Jason Ekstrand
wrote:
> Reviewed-by: Jason Ekstrand
> On Apr 16, 2016 12:50 PM, "Laurent Carlier" wrote:
>
>> mcpu=generic doesn't enable sse2, and anvil definitly needs it
>> ---
>> src/intel/vulkan/Makefile.am | 2 +-
>> 1 file changed, 1 insertion(+
Pushed. Thanks!
On Sat, Apr 16, 2016 at 1:48 PM, Marcin Ślusarz
wrote:
>
> ---
> src/intel/vulkan/Makefile.am | 9 +
> src/intel/vulkan/anv_private.h | 3 ---
> 2 files changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.
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).
On Wed, Apr 20, 2016 at 1:47 PM, Samuel Pitoiset
wrote:
> This is only valid for other atomic operations (including CA
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 addr to 0 for other atomic operations.
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110
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?
-ilia
On Wed, Apr 20, 2016 at 1:06 PM, Samuel Pitoiset
wrote:
> Forcing t
Reviewed-by: Ilia Mirkin
On Wed, Apr 20, 2016 at 1:06 PM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> Cc: "11.1 11.2"
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/n
On Thu, Apr 14, 2016 at 9:29 AM, Michel Dänzer wrote:
> On 14.04.2016 11:37, Michel Dänzer wrote:
>> On 12.04.2016 21:33, Marek =?UNKNOWN?B?T2zFocOhaw==?= wrote:
>>>
>>> URL:
>>> http://cgit.freedesktop.org/mesa/mesa/commit/?id=5a4b74d1ba2c156766a7a5dbfef099c7db5d6694
>>> Author: Marek Olšák
For patches 1,3,4:
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 5:43 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94984
> ---
> src/gallium/drivers/radeon/r600_query.c | 40
> ++---
> src/ga
https://bugs.freedesktop.org/show_bug.cgi?id=95035
Bug ID: 95035
Summary: Gallium OSMesa driver is far from being thread-safe
Product: Mesa
Version: 11.2
Hardware: Other
OS: All
Status: NEW
Severity: norma
Forcing the destination type to be signed when the source is signed
is not totally correct.
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/no
Signed-off-by: Samuel Pitoiset
Cc: "11.1 11.2"
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gk110.c
Minor comment below. Either way:
Reviewed-by: Sinclair Yeh
On Wed, Apr 20, 2016 at 05:52:17PM +0200, Mircea Gherzan wrote:
> This header should not blindly include the GLX and should also
> conditionally define functions that use GLX parameters. The
> MESA_EGL_NO_X11_HEADERS macro is used to ch
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 20, 2016 at 5:52 PM, Mircea Gherzan
wrote:
> This header should not blindly include the GLX and should also
> conditionally define functions that use GLX parameters. The
> MESA_EGL_NO_X11_HEADERS macro is used to check if we're building without
> GLX s
Rob Herring writes:
> The RGBX/RGBA pixel formats used in the Android EGL don't get configs
> created due to the missing formats in the DRI state tracker. This series
> adds the necessary formats for configs and DRI images. Support in GBM is
> also added as it will be needed soon for Android.
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 -
1 file changed, 33 insertions(+), 22
1 - 100 of 159 matches
Mail list logo
