This fixes
dEQP-GLES31.functional.draw_indirect.draw_elements_indirect.*.default_attribute
These tests were causing a const vbo to be set up, and were small enough
draws that the logic was trying to go via the push path (which emits
data directly into the cmd stream rather than uploading a user v
https://bugs.freedesktop.org/show_bug.cgi?id=95395
--- Comment #2 from Jonathan Gray ---
None of the builds were with LLVM enabled. Interestingly I can't reproduce
this on sparc64 which requires strict alignment.
--
You are receiving this mail because:
You are the assignee for the bug.
Giuseppe Bilotta writes:
> Code generation (kernel compilation) may sometimes hit LLVM-specific
> bugs. Adding the used LLVM version to the version string may make bug
> triaging easier.
>
> Signed-off-by: Giuseppe Bilotta
Acked-by: Francisco Jerez
> ---
> configure.ac
Series is:
Reviewed-by: Leo Liu
On 05/14/2016 11:33 AM, Emil Velikov wrote:
Add separate labels and jump to the correct one as needed.
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/dri/dri2.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-
Jan Vesely writes:
> Signed-off-by: Jan Vesely
> ---
> src/gallium/state_trackers/clover/Makefile.am | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/gallium/state_trackers/clover/Makefile.am
> b/src/gallium/state_trackers/clover/Makefile.am
> index 4c9d7d9..26ebd3b 100644
> --
Jan Vesely writes:
> Signed-off-by: Jan Vesely
> ---
> src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> b/src/gallium/state_trackers/clover/llvm/invocation.cp
This was handled in handleTEX(), however the way the logic works, those
extra arguments aren't added on by then, so it did nothing. Instead we
must duplicate that bit here. GK110 appears to complain about
MISALIGNED_GPR, however it's reasonable to believe that GK104 has the
same requirements.
Bugz
Samuel Iglesias Gonsálvez writes:
> On 14/05/16 01:16, Francisco Jerez wrote:
>> Samuel Iglesias Gonsálvez writes:
>>
>>> From: Iago Toral Quiroga
>>>
>>> UBO loads with constant offset use the UNIFORM_PULL_CONSTANT_LOAD
>>> instruction, which reads 16 bytes (a vec4) of data from memory. For d
https://bugs.freedesktop.org/show_bug.cgi?id=95374
Vedran Miletić changed:
What|Removed |Added
CC||riva...@gmail.com
--
You are receiving
On Sat, May 14, 2016 at 4:43 PM, Jason Ekstrand wrote:
> On Sat, May 14, 2016 at 12:23 PM, Rob Clark wrote:
>>
>> On Thu, May 12, 2016 at 10:17 PM, Jason Ekstrand
>> wrote:
>> >
>> >
>> > On Mon, May 9, 2016 at 12:33 PM, Rob Clark wrote:
>> >>
>> >> From: Rob Clark
>> >>
>> >> Signed-off-by: R
On Sat, May 14, 2016 at 12:23 PM, Rob Clark wrote:
> On Thu, May 12, 2016 at 10:17 PM, Jason Ekstrand
> wrote:
> >
> >
> > On Mon, May 9, 2016 at 12:33 PM, Rob Clark wrote:
> >>
> >> From: Rob Clark
> >>
> >> Signed-off-by: Rob Clark
> >> Reviewed-by: Connor Abbott
> >> ---
> >> src/compile
On Tuesday, May 10, 2016 4:16:20 PM PDT Jason Ekstrand wrote:
> When Paul originally wrote blorp he hand-rolled a shader builder that
> builds i965 shaders directly. This has caused headaches because every time
> we make a change to the back-end compiler, we have to update blorp. NIR on
> the oth
Reviewed-by: Jason Ekstrand
On Sat, May 14, 2016 at 1:10 PM, Rob Clark wrote:
> From: Rob Clark
>
> Drop extra #include which is otherwise unneeded (and makes this header
> difficult to include from outside of src/mesa).
>
> Signed-off-by: Rob Clark
> ---
> src/compiler/nir/glsl_to_nir.h | 3
On Sat, May 14, 2016 at 2:58 PM, Roland Scheidegger wrote:
> On 05/14/2016 04:24 PM, Ilia Mirkin wrote:
>>
>> On Sat, May 14, 2016 at 10:23 AM, Roland Scheidegger
>> wrote:
>>>
>>> Am 14.05.2016 um 14:55 schrieb Marek Olšák:
Dave,
It should be noted that clip distances can be disab
From: Rob Clark
use glsl/libstandalone.la to add support for taking glsl src files (in
addition to .tgsi) as input. Then glsl->nir and feed the result into
the ir3 backend as normal.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/Makefile.am | 2 +
src/gallium/drivers/freed
From: Rob Clark
Split standalone glsl_compiler into a libstandalone.la and a thin
main.cpp. This way drivers can re-use the glsl standalone frontend in
their own standalone compilers.
Signed-off-by: Rob Clark
---
There is one kinda ugly hack (#including a .cpp file) to work around
an automake
From: Rob Clark
Drop extra #include which is otherwise unneeded (and makes this header
difficult to include from outside of src/mesa).
Signed-off-by: Rob Clark
---
src/compiler/nir/glsl_to_nir.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/compiler/nir/glsl_to_nir.
On Sat, May 14, 2016 at 12:20 PM, Rob Clark wrote:
> On Thu, May 12, 2016 at 10:55 PM, Jason Ekstrand
> wrote:
> >
> >
> > On Tue, May 10, 2016 at 11:57 AM, Rob Clark wrote:
> >>
> >> From: Rob Clark
> >>
> >> Some optimizations, like converting integer multiply/divide into left/
> >> right sh
On Fri, May 13, 2016 at 10:49 AM, Jason Ekstrand
wrote:
>
>
> On Wed, May 11, 2016 at 7:42 PM, Jason Ekstrand
> wrote:
>
>> The current MSAA resolve code has a special-case for if the MCS value is
>> 0.
>> In this case we can only sample once because we know that all values are
>> in
>> slice 0.
On Saturday, May 14, 2016 3:26:41 PM PDT Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
> ---
> src/compiler/nir/nir_opcodes.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/nir/nir_opcodes.py b/src/compiler/nir/
nir_opcodes.py
> ind
From: Rob Clark
Caller can pass a hashtable mapping NIR object (currently instr or var,
but I guess others could be added as needed) to annotation msg to print
inline with the shader dump. As the annotation msg is printed, it is
removed from the hashtable to give the caller a way to know about a
From: Rob Clark
Log all the errors, and at the end dump the shader w/ error annotations
to make it easier to see where the problems are.
Signed-off-by: Rob Clark
---
src/compiler/nir/nir_validate.c | 58 +
1 file changed, 58 insertions(+)
diff --git a/s
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 96f6a48..e2cadda 100644
--
Signed-off-by: Jan Vesely
---
src/gallium/state_trackers/clover/Makefile.am | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/clover/Makefile.am
b/src/gallium/state_trackers/clover/Makefile.am
index 4c9d7d9..26ebd3b 100644
--- a/src/gallium/state_trackers/clover/
From: Rob Clark
Signed-off-by: Rob Clark
---
src/compiler/nir/nir_opcodes.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/compiler/nir/nir_opcodes.py b/src/compiler/nir/nir_opcodes.py
index 24ffc31..9d05594 100644
--- a/src/compiler/nir/nir_opcodes.py
+++ b/src/co
On Thu, May 12, 2016 at 10:17 PM, Jason Ekstrand wrote:
>
>
> On Mon, May 9, 2016 at 12:33 PM, Rob Clark wrote:
>>
>> From: Rob Clark
>>
>> Signed-off-by: Rob Clark
>> Reviewed-by: Connor Abbott
>> ---
>> src/compiler/Makefile.sources| 1 +
>> src/compiler/nir/nir.h
On Thu, May 12, 2016 at 10:55 PM, Jason Ekstrand wrote:
>
>
> On Tue, May 10, 2016 at 11:57 AM, Rob Clark wrote:
>>
>> From: Rob Clark
>>
>> Some optimizations, like converting integer multiply/divide into left/
>> right shifts, have additional constraints on the search expression.
>> Like requi
On 05/14/2016 04:24 PM, Ilia Mirkin wrote:
On Sat, May 14, 2016 at 10:23 AM, Roland Scheidegger wrote:
Am 14.05.2016 um 14:55 schrieb Marek Olšák:
Dave,
It should be noted that clip distances can be disabled by
pipe_rasterizer_state::clip_plane_enable, but cull distances can't.
(same as GL)
On Sun, May 1, 2016 at 6:42 AM, Emil Velikov wrote:
> The drv is no longer used/needed as of last commit.
>
> Cc: Rob Herring
> Signed-off-by: Emil Velikov
> ---
> src/egl/drivers/dri2/platform_android.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Acked-by: Rob Herring
___
On Sun, May 1, 2016 at 6:42 AM, Emil Velikov wrote:
> Make the function non static so that we can use it directly from the
> android platform code.
>
> Cc: Rob Herring
> Signed-off-by: Emil Velikov
> ---
> src/egl/drivers/dri2/egl_dri2.c | 2 +-
> src/egl/drivers/dri2/egl_dri2.h
https://bugs.freedesktop.org/show_bug.cgi?id=95395
--- Comment #1 from Ilia Mirkin ---
Another thing to check is whether you can reproduce on amd64 with
DRAW_USE_LLVM=0 - otherwise softpipe uses llvm for the vertex stages when
available.
--
You are receiving this mail because:
You are the assig
https://bugs.freedesktop.org/show_bug.cgi?id=95395
Kenneth Graunke changed:
What|Removed |Added
Assignee|i...@freedesktop.org |mesa-dev@lists.freedesktop.
2016-05-13 15:30 GMT+08:00 Pohjolainen, Topi :
> On Thu, May 12, 2016 at 12:25:25AM +0800, Chih-Wei Huang wrote:
>> Testing android-x86 with mesa 11.2.2,
>> I found the Google Play crashed forever on
>> a device with Intel Gen9 GPU (e.g., Skylake).
>>
>> After analyzing, the i965 driver seems to as
Hi, there are minor rendering glitches in Shadow of Mordor on R9 390.
git-59156b2 + this patch v2. I can send a screenshot if you are unable to
reproduce this.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/
On Saturday, May 14, 2016 8:55:12 AM PDT Vanja Z wrote:
> Hi all,
>
> I'm sorry if this is the wrong place to post this. Upgrading from mesa
11.2.1 to 11.2.2 on Arch Linux results in several programs not working. I am
getting the following errors when launching Paraview for example,
>
> libGL e
Any comments on the patch and/or discussion after the --- line ?
On 24 April 2016 at 16:14, Emil Velikov wrote:
> From: Emil Velikov
>
> If the mutexattrs are the default one can just pass NULL to
> pthread_mutex_init. As the compiler does not know this detail it
> unnecessarily creates/destroys
On 1 May 2016 at 12:42, Emil Velikov wrote:
> Make the function non static so that we can use it directly from the
> android platform code.
>
> Cc: Rob Herring
> Signed-off-by: Emil Velikov
> ---
> src/egl/drivers/dri2/egl_dri2.c | 2 +-
> src/egl/drivers/dri2/egl_dri2.h | 4 +++
On 1 May 2016 at 13:48, Emil Velikov wrote:
> The build systems already add this as applicable. There's no need to
> have this in the source file.
>
> Signed-off-by: Emil Velikov
> ---
> src/gbm/main/gbm.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/gbm/main/gbm.c b/src/gbm/main/
On 12 May 2016 at 16:25, Leo Liu wrote:
> On 05/12/2016 11:08 AM, Emil Velikov wrote:
>>
>> On 12 May 2016 at 15:01, Leo Liu wrote:
>>>
>>>
>>> On 05/12/2016 09:47 AM, Emil Velikov wrote:
Hi Leo,
On 11 May 2016 at 22:14, Leo Liu wrote:
>
> On 05/11/2016 04:20 PM, Axel
On 12 May 2016 at 16:19, Leo Liu wrote:
> On 05/12/2016 11:10 AM, Emil Velikov wrote:
>>
>> On 12 May 2016 at 15:10, Leo Liu wrote:
>>>
>>> fd should be set to -1 only if it got closed by pipe_loader_release.
>>>
>>> Signed-off-by: Leo Liu
>>> ---
>>> src/gallium/auxiliary/vl/vl_winsys_dri.c |
Analogous to previous commit.
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/xa/xa_tracker.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/xa/xa_tracker.c
b/src/gallium/state_trackers/xa/xa_tracker.c
index f09baed..e091b083 1006
Analogous to previous commits.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/vl/vl_winsys_drm.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/vl/vl_winsys_drm.c
b/src/gallium/auxiliary/vl/vl_winsys_drm.c
index 6d9d947..6a759ae 100644
Add separate labels and jump to the correct one as needed.
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/dri/dri2.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/dri/dri2.c
b/src/gallium/state_tracker
On 4 May 2016 at 03:02, Rob Herring wrote:
> In preparation to add public map/unmap functions, rename the existing
> gbm_dri_bo_{map,unmap} functions to indicate that they are only for dumb
> buffers.
>
> Signed-off-by: Rob Herring
> ---
> v2:
> - moved into new patch
>
> src/gbm/backends/dri/gb
On 29 April 2016 at 14:07, Iago Toral wrote:
> On Fri, 2016-04-29 at 14:01 +0100, Emil Velikov wrote:
>> On 29 April 2016 at 13:19, Iago Toral wrote:
>> > On Fri, 2016-04-29 at 00:55 -0700, Kenneth Graunke wrote:
>> >> On Thursday, April 28, 2016 4:06:49 PM PDT Emil Velikov wrote:
>> >> > Hi all,
Except for patches 1 and 6, this series is
Reviewed-by: Ilia Mirkin
On Sat, May 14, 2016 at 9:54 AM, 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/gall
On Sat, May 14, 2016 at 9:54 AM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 117
> +
> .../nouveau/codegen/nv50_ir_lowering_nvc0.h| 2 +
> 2 files changed, 119 insertions(+)
>
> diff --git a/
On 05/13/2016 05:45 AM, Ilia Mirkin wrote:
INTERP is defined (by me) to have to have a INPUT source. However the
state tracker does not always obey this. This happens due to varying
packing logic introducing additional mov's which can't always be undone.
Instead of just giving up, we instead tr
On Sat, May 14, 2016 at 9:54 AM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 53
> src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 1 +
> src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 8 +-
> src/gallium/dr
On Sat, May 14, 2016 at 10:23 AM, Roland Scheidegger wrote:
> Am 14.05.2016 um 14:55 schrieb Marek Olšák:
>> Dave,
>> It should be noted that clip distances can be disabled by
>> pipe_rasterizer_state::clip_plane_enable, but cull distances can't.
>> (same as GL)
>
> That only applies to user clip
Am 14.05.2016 um 14:55 schrieb Marek Olšák:
> Dave,
> It should be noted that clip distances can be disabled by
> pipe_rasterizer_state::clip_plane_enable, but cull distances can't.
> (same as GL)
That only applies to user clip planes, not shader clip distances.
>
> Roland,
> Our hardware only
From: Tobias Klausmann
Cull distances are just a special case of clip distances as far as the
hardware is concerned. Make sure that the relevant "planes" are enabled,
and flip the clip mode to cull for those.
Signed-off-by: Tobias Klausmann
[imirkin: add enables on nvc0, add nv50 support]
Signe
The pass that st/mesa relies on to combine clip and cull distances has
been reverted, so we can't expose ARB_cull_distance until that is
resolved.
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_extensions.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/st
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 58 ++
1 file changed, 58 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
index 14f4be4..f
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_compute.c | 53
src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 1 +
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 8 +-
src/gallium/drivers/nouveau/nvc0/nvc0_tex.c | 154 +++-
4 f
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp
index 27883a0..b893996 100644
--- a/src/gallium/dri
Signed-off-by: Samuel Pitoiset
---
.../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 117 +
.../nouveau/codegen/nv50_ir_lowering_nvc0.h| 2 +
2 files changed, 119 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_lowering_nvc0.cpp
b/src/gall
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 46 +-
1 file changed, 44 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c
index eaf9c78..bd68ca9 100644
--- a/src/gallium/drivers/nouveau
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 bd68ca9..40e5a9d 100644
--- a/src/gallium/dr
Hi there,
This series implements both ARB_shader_image_load_store (GL 4.2) and
ARB_shader_image_size (GL 4.3) which allows us to enable OpenGL 4.2 on Fermi
GPUS. (GL3.txt won't be updated until images are also implemented on Maxwell)
3D images are fully not supported because we don't think they a
From: Ilia Mirkin
This makes sure that rIndirectSrc and other things stay updated.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp
b/s
Hi Ken,
On 14 May 2016 at 01:44, Kenneth Graunke wrote:
> Commit 5310bca024f77da40ea6f4c275455f9cb0528f9e added a new "double df"
> field to the brw_reg struct, adding an extra 4 bytes of data that isn't
> usually initialized (or may contain irrelevant garbage if the struct is
> mutated). This m
Dave,
It should be noted that clip distances can be disabled by
pipe_rasterizer_state::clip_plane_enable, but cull distances can't.
(same as GL)
Roland,
Our hardware only has 2 vec4 outputs. Each component can be configured
to be "clip distance", "cull distance", or "disabled" independently.
Mare
On 14/05/16 01:16, Francisco Jerez wrote:
> Samuel Iglesias Gonsálvez writes:
>
>> From: Iago Toral Quiroga
>>
>> UBO loads with constant offset use the UNIFORM_PULL_CONSTANT_LOAD
>> instruction, which reads 16 bytes (a vec4) of data from memory. For dvec
>> types this only provides components
On Tue, May 10, 2016 at 1:21 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This avoids allocating giant IBs from the outset, especially for CE and DMA.
>
> With this change, we also never flush prematurely due to the CE IB: as long
> as there is space in the buffer, we will use it.
> ---
>
Hi all,
I'm sorry if this is the wrong place to post this. Upgrading from mesa 11.2.1
to 11.2.2 on Arch Linux results in several programs not working. I am getting
the following errors when launching Paraview for example,
libGL error: unable to load driver: i965_dri.so
libGL error: driver point
On 2016-05-13 19:26:37, Kenneth Graunke wrote:
> I don't know what the intention was here, but this function returns
> void. We can't assert anything about its return value.
c1bbaff1e83f901d67d78f9e1ddfe8291dd09bfa seems to be related, and
appears to have changed this file similarly for some othe
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
With Curro's comment addressed,
Reviewed-by: Samuel Iglesias Gonsálvez
On 14/05/16 02:44, Kenneth Graunke wrote:
> Commit 5310bca024f77da40ea6f4c275455f9cb0528f9e added a new "double
> df" field to the brw_reg struct, adding an extra 4 bytes of da
69 matches
Mail list logo