This series is,
Reviewed-by: Edward O'Callaghan
On 2016-01-06 12:46, Marek Olšák wrote:
From: Marek Olšák
This will be used by radeonsi.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 2 ++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 3 insertions(+)
diff --git a/src/galliu
On 2016-01-06 10:30, Brian Paul wrote:
Series looks OK to me. Reviewed-by: Brian Paul
Do you need someone to commit/push for you?
I do yes, thank you kindly.
Edward.
-Brian
On 01/05/2016 03:07 AM, Edward O'Callaghan wrote:
Fix silly issue with MSVC case fall-though support to need
a ex
https://bugs.freedesktop.org/show_bug.cgi?id=93524
--- Comment #1 from Michel Dänzer ---
Please send the patch to the mesa-dev mailing list for review, with the commit
log changed to something like:
clover: Add libelf cflags
Fixes the build when libelf is in a non default location.
--
You a
https://bugs.freedesktop.org/show_bug.cgi?id=93577
Michel Dänzer changed:
What|Removed |Added
Component|Drivers/Gallium/r600|Mesa core
Assignee|dri-devel@l
From: Nicolai Hähnle
This is more future-proof than the current code.
---
src/mesa/state_tracker/st_cb_bufferobjects.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_bufferobjects.c
b/src/mesa/state_tracker/st_cb_bufferobjects.c
index 5d20b26
From: Nicolai Hähnle
This is more future-proof, plugs the memory leak of Label and properly
destroys the buffer mutex.
---
src/mesa/drivers/dri/radeon/radeon_buffer_objects.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/radeon/radeon_buffer_objects.c
From: Nicolai Hähnle
This is more future-proof, plugs the memory leak of Label and properly
destroys the buffer mutex.
---
src/mesa/drivers/dri/i965/intel_buffer_objects.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/intel_buffer_objects.c
b/src/
From: Nicolai Hähnle
This is more future-proof, plugs the memory leak of Label and properly
destroys the buffer mutex.
---
src/mesa/drivers/dri/i915/intel_buffer_objects.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i915/intel_buffer_objects.c
b/src/
From: Nicolai Hähnle
gl_buffer_object has grown more complicated and requires cleanup. Using this
function from drivers will be more future-proof.
---
src/mesa/main/bufferobj.c | 2 +-
src/mesa/main/bufferobj.h | 4
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main
I have been holding off v2 with correct spacing before \ (seemingly
missed the first time) waiting for a review to confirm that this is
the right approach as agreed in IRC.
Let me know if anything else.
--
Jimmy
On Sat, Dec 19, 2015 at 10:57 PM, Jimmy Berry wrote:
> Removes the need to set LIB
Whoops. I think that was my fault.
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Marek Olšák
This will be used by radeonsi.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 2 ++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 3 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index e04f407..e3feed9
From: Marek Olšák
This will be used by radeonsi.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 30 ++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 23 insertions(+), 8 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/au
From: Marek Olšák
This has no users in Mesa.
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index e3feed9..e3a6fb0 100644
--- a/src/gallium/auxili
Even if re-linking fails rendering shouldn't fail as the previous
succesfully linked program will still be available. It also shouldn't
be possible to have an unlinked program as part of the current rendering
state.
This fixes a subtest in:
ES31-CTS.sepshaderobjs.StateInteraction
This change shou
On Jan 4, 2016 8:30 PM, "Ilia Mirkin" wrote:
>
> gcc 4.9.3 shows the following error:
>
> brw_vue_map.c:260:20: warning: array subscript is above array bounds
> [-Warray-bounds]
> return brw_names[slot - VARYING_SLOT_MAX];
>
> This is because BRW_VARYING_SLOT_COUNT is a valid value for the enu
On Jan 5, 2016 5:34 AM, "Kenneth Graunke" wrote:
>
> While GLSL restricts bitfieldInsert's offset and bits parameters to
> be scalars, we shouldn't require this in the IR.
>
> In particular, opt_vectorize() tries to combine
>
> result.x = bitfieldInsert(src0.x, src1.x, src2.x, src3.x);
> result.y
Am 03.01.2016 um 16:17 schrieb Oded Gabbay:
> This patch converts the SSE-optimized lp_rast_triangle_32_3_16()
> to VMX/VSX.
>
> I measured the results on POWER8 machine with 32 cores at 3.4GHz and
> 16GB of RAM.
>
> FPS/Score
> NameBefore AfterDelta
> -
On Tue, Jan 5, 2016 at 8:34 AM, Kenneth Graunke wrote:
> While GLSL restricts bitfieldInsert's offset and bits parameters to
> be scalars, we shouldn't require this in the IR.
>
> In particular, opt_vectorize() tries to combine
>
> result.x = bitfieldInsert(src0.x, src1.x, src2.x, src3.x);
> resul
Reviewed-by: Jason Ekstrand
On Tue, Jan 5, 2016 at 4:25 PM, Kenneth Graunke
wrote:
> The nir_opt_algebraic rule
>
> (('fadd', ('flog2', a), ('fneg', ('flog2', b))), ('flog2', ('fdiv', a,
> b))),
>
> can produce new fdiv operations, which need to be lowered on i965,
> as we don't actually implem
The nir_opt_algebraic rule
(('fadd', ('flog2', a), ('fneg', ('flog2', b))), ('flog2', ('fdiv', a, b))),
can produce new fdiv operations, which need to be lowered on i965,
as we don't actually implement fdiv. (Normally, we handle this in
GLSL IR's lower_instructions pass, but in the above case we
From: Roland Scheidegger
The trick here is to recognize that in the c + n * dcdx calculations,
not only can the lower FIXED_ORDER bits not change (as the dcdx values
have those all zero) but that this means the sign bit of the calculations
cannot be different as well, that is
sign(c + n*dcdx) ==
From: Roland Scheidegger
Otherwise some planes we get in rasterization have subpixel precision, others
not. Doesn't matter so far, but will soon. (OpenGL actually supports viewports
with subpixel accuracy, so could even do bounding box calcs with that).
---
src/gallium/drivers/llvmpipe/lp_setup_
Negating that expression would make it more readable. There are also a
lot of unnecessary parentheses. If you at least remove the
parentheses:
Reviewed-by: Marek Olšák
Marek
On Tue, Jan 5, 2016 at 11:23 PM, Timothy Arceri
wrote:
> ping
>
> This fixes some piglit subtests for ARB_program_interf
On Mon, Dec 28, 2015 at 9:00 PM, Timothy Arceri
wrote:
> Previously we would pack varyings before trying to remove them, this
> relied on the packing pass not packing varyings with a location of -1
> to avoid packing varyings that should be removed.
> However this meant unused varyings with an exp
https://bugs.freedesktop.org/show_bug.cgi?id=93003
Sinclair Yeh changed:
What|Removed |Added
CC||s...@vmware.com
--
You are receiving thi
Series looks OK to me. Reviewed-by: Brian Paul
Do you need someone to commit/push for you?
-Brian
On 01/05/2016 03:07 AM, Edward O'Callaghan wrote:
Fix silly issue with MSVC case fall-though support to need
a extra 'break;'
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by
On Mon, Dec 28, 2015 at 9:00 PM, Timothy Arceri
wrote:
> There is a function dedicated to demoting unused varyings lets
> trust it to do its job.
> ---
> src/glsl/link_varyings.cpp | 5 -
> 1 file changed, 5 deletions(-)
>
> diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
On Mon, Dec 28, 2015 at 9:00 PM, Timothy Arceri
wrote:
> After lowering the matching flag is_unmatched_generic_inout is lost so
> we need to move this validation before lowering.
> ---
> src/glsl/link_varyings.cpp | 22 +++---
> 1 file changed, 11 insertions(+), 11 deletions(-)
>
On Mon, Dec 28, 2015 at 9:00 PM, Timothy Arceri
wrote:
> An SSO program can have multiple stages and we only want to add the externally
> facing varyings. The current code was adding both the packed inputs and
> outputs
> for the first and last stage of each program.
> ---
> src/glsl/linker.cpp
ping
This fixes some piglit subtests for ARB_program_interface_query.
On Thu, 2015-12-24 at 13:16 +1100, Timothy Arceri wrote:
> V2: fix alignment
> ---
> src/mesa/main/shader_query.cpp | 7 ++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/shader_query.cp
https://bugs.freedesktop.org/show_bug.cgi?id=93003
--- Comment #2 from le...@thefnords.org ---
1) Mesa version was originally 11.0.6, and I know I've tested the update to
11.0.7/8. I am not sure if I've tried it with the current F23 version
(11.1.0). I'll check kernel version as soon as I'm home
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #14 from Michael Harder ---
(In reply to Patrick Baggett from comment #11)
> Created attachment 120822 [details] [review]
> Use prefetcht1 instead of prefetch[w]
>
> This should fix the SIGILL when running this code. It replaces 3DNo
https://bugs.freedesktop.org/show_bug.cgi?id=93003
Sinclair Yeh changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #1 from Sinclair Yeh
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #13 from Roland Scheidegger ---
(In reply to Patrick Baggett from comment #11)
> Created attachment 120822 [details] [review]
> Use prefetcht1 instead of prefetch[w]
>
> This should fix the SIGILL when running this code. It replaces
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #12 from Roland Scheidegger ---
(In reply to Patrick Baggett from comment #10)
> Created attachment 120821 [details]
> attachment-30621-0.html
>
> Given that there is a _mesa_3dnow_transform_points4_2d in the x86-64 asm
> (using MMX/
https://bugs.freedesktop.org/show_bug.cgi?id=27512
Lars Kellogg-Stedman changed:
What|Removed |Added
CC|l...@oddbit.com |
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #11 from Patrick Baggett ---
Created attachment 120822
--> https://bugs.freedesktop.org/attachment.cgi?id=120822&action=edit
Use prefetcht1 instead of prefetch[w]
This should fix the SIGILL when running this code. It replaces 3DNow
Am 04.01.2016 um 22:00 schrieb Samuel Pitoiset:
>
>
> On 01/04/2016 09:25 PM, Ilia Mirkin wrote:
>> On Mon, Jan 4, 2016 at 3:13 PM, Samuel Pitoiset
>> wrote:
>>> On 01/03/2016 05:37 AM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
diff --git a/src/gallium/auxiliary/tgsi/tgsi_strings.
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #10 from Patrick Baggett ---
Given that there is a _mesa_3dnow_transform_points4_2d in the x86-64 asm
(using MMX/3DNow! is deprecated in x86-64), it appears that this code was
copy-pasted. I wrote a quick patch to change prefetch[w] t
Given that there is a _mesa_3dnow_transform_points4_2d in the x86-64 asm
(using MMX/3DNow! is deprecated in x86-64), it appears that this code was
copy-pasted. I wrote a quick patch to change prefetch[w] to prefetcht1,
which is more or less the equivalent in SSE. However, I'm not actually sure
thos
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #9 from Roland Scheidegger ---
(In reply to Michael Harder from comment #8)
> From 'cat /proc/cpuinfo'
> model name : Intel(R) Pentium(R) 4 CPU 3.00GHz
Using any special build flags? As said I can't see how this code could work
This series looks fine to me.
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Monday, January 4, 2016 4:49 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee
Subject: [PATCH 8/8] mesa: minor clean-up of some memcpy/sizeof() calls in
m_matrix.c
--
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #8 from Michael Harder ---
>From 'cat /proc/cpuinfo'
model name : Intel(R) Pentium(R) 4 CPU 3.00GHz
and from 'lspci'
VGA compatible controller: Intel Corporation 82915G/GV/910GL Integrated
Graphics Controller (rev 04)
--
You a
On Tue, Jan 5, 2016 at 5:19 PM, Axel Davy wrote:
> Hi Nicolas,
>
> I believe this patch doesn't fully solve the problem and could introduce
> some bugs.
>
> For example you change dri2_create_image_from_name pitch argument to stride,
> whereas this function is used as is to implement createImageFr
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue, Jan 5, 2016 at 8:34 AM, Kenneth Graunke wrote:
> While GLSL restricts bitfieldInsert's offset and bits parameters to
> be scalars, we shouldn't require this in the IR.
>
> In particular, opt_vectorize() tries to combine
>
> result.x = bitfieldInsert(src0.x, src1.x, src2.x, src3.x);
> resul
On Tue, Jan 5, 2016 at 6:04 AM, Kenneth Graunke wrote:
> ---
> run.py | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> in case you want to use the old runner to produce .out files.
Thanks :)
All four look good to me.
Reviewed-by: Matt Turner
_
Both look good to me. Thanks!
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Hi Nicolas,
I believe this patch doesn't fully solve the problem and could introduce
some bugs.
For example you change dri2_create_image_from_name pitch argument to stride,
whereas this function is used as is to implement createImageFromName
from __DRIimageExtensionRec,
which takes a pitch. T
Le mardi 05 janvier 2016 à 17:19 +0100, Axel Davy a écrit :
> Hi Nicolas,
>
> I believe this patch doesn't fully solve the problem and could
> introduce some bugs.
>
> For example you change dri2_create_image_from_name pitch argument to
> stride,
> whereas this function is used as is to implement
On Tue, Jan 05, 2016 at 05:33:09AM -0800, Kenneth Graunke wrote:
> That's what it's for. Plus, we actually implement rcp.
Reviewed-by: Topi Pohjolainen
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_lower_texture_gradients.cpp | 2 +-
> 1 file changed, 1 insertion(+
Le me know if this patch needs an update.
cheers,
Nicolas
Le jeudi 24 décembre 2015 à 15:15 -0500, Nicolas Dufresne a écrit :
> In order to convert from stride to pitch, few functions were diving
> by 4
> the stride. This is not valid for RGB565 and this conversion was not
> needed anyway in this
Kenneth Graunke writes:
> Compute shaders require reconfiguring the L3 for shared local memory
> support. We have to be able to write the L3 registers to do that.
>
> This effectively turns off compute shaders prior to Kernel 4.2.
>
> (Previously, the extension enable was in an API_OPENGL_CORE c
On Tue, Jan 5, 2016 at 4:04 PM, Matt Arsenault wrote:
>
>> On Jan 5, 2016, at 07:28, Marek Olšák wrote:
>>
>> Hi,
>>
>> I'd like us to do this computation in Mesa, because it can be more
>> accurate there. The pixel shader wave count depends heavily on LDS,
>> because each interpolated input occu
https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #7 from Roland Scheidegger ---
(In reply to Michael Harder from comment #6)
> I am experiencing a crash with a SIGILL, Illegal instruction in Debian when
> using Kodi.
>
> [Thread debugging using libthread_db enabled]
> Using host li
> On Jan 5, 2016, at 07:28, Marek Olšák wrote:
>
> Hi,
>
> I'd like us to do this computation in Mesa, because it can be more
> accurate there. The pixel shader wave count depends heavily on LDS,
> because each interpolated input occupies 12 dwords of LDS per
> primitive and there can be 1-16 p
Thanks Samuel,
I confirm that your patch fixes the testcase for both BDW and SKL.
/Marta
From: Samuel Iglesias Gonsálvez [sigles...@igalia.com]
Sent: Tuesday, January 05, 2016 2:32 PM
To: mesa-dev@lists.freedesktop.org
Cc: Lofstedt, Marta; Samuel Iglesias
vector_insert takes a vector, a scalar location, and a scalar value,
and produces a new vector with that component updated. As such, it
can't be vectorized properly.
vector_extract takes a vector and a scalar location, and returns
that scalar component of the vector. Vectorization doesn't really
While GLSL restricts bitfieldInsert's offset and bits parameters to
be scalars, we shouldn't require this in the IR.
In particular, opt_vectorize() tries to combine
result.x = bitfieldInsert(src0.x, src1.x, src2.x, src3.x);
result.y = bitfieldInsert(src0.y, src1.y, src2.y, src3.y);
result.z = bit
That's what it's for. Plus, we actually implement rcp.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_lower_texture_gradients.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_lower_texture_gradients.cpp
b/src/mesa/drivers/d
Fixes CTS test:
ES31-CTS.shader_image_load_store.negative-linkErrors
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93410
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/glsl/linker.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
Hi,
I'd like us to do this computation in Mesa, because it can be more
accurate there. The pixel shader wave count depends heavily on LDS,
because each interpolated input occupies 12 dwords of LDS per
primitive and there can be 1-16 primitives per wave. With 32 inputs
and 16 primitives, you can ge
On 04.01.2016 23:31, Julien Isorce wrote:
From: Julien Isorce
The counter was not set but used by the nouveau driver.
It is required otherwise visual output is garbage.
Signed-off-by: Julien Isorce
Reviewed-by: Christian König
---
src/gallium/state_trackers/va/picture.c| 8
---
run.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
in case you want to use the old runner to produce .out files.
diff --git a/run.py b/run.py
index 58bdbbf..ced8710 100755
--- a/run.py
+++ b/run.py
@@ -48,7 +48,7 @@ def run_test(filename):
lines = (line for line in res
VS/GS/FS match the regex '.S' but TCS/TES do not. Just match \S+.
---
report.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/report.py b/report.py
index 877e4ff..f9b1776 100755
--- a/report.py
+++ b/report.py
@@ -10,7 +10,7 @@ def get_results(filename):
results = {}
This will prevent more python2isms from creeping in.
---
report.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/report.py b/report.py
index 1352f22..877e4ff 100755
--- a/report.py
+++ b/report.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
import re
im
I ported this to Python 3 back in 2012. It's not supposed to be run
with Python 2.
---
report.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/report.py b/report.py
index 409474a..1352f22 100755
--- a/report.py
+++ b/report.py
@@ -73,7 +73,7 @@ def main():
affected_
Compute shaders require reconfiguring the L3 for shared local memory
support. We have to be able to write the L3 registers to do that.
This effectively turns off compute shaders prior to Kernel 4.2.
(Previously, the extension enable was in an API_OPENGL_CORE conditional.
However, that isn't nece
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/drivers/r600/r600_shader.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_shader.c
b/src/gallium/drivers/r600/r600_shader.c
index
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/drivers/i915/i915_state.c | 2 +-
src/gallium/drivers/ilo/shader/ilo_shader_vs.c| 2 +-
src/gallium/drivers/llvmpipe/lp_test_blend.c | 2 +-
src/gallium/drivers/llvmpipe/lp_test_co
Fix silly issue with MSVC case fall-though support to need
a extra 'break;'
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/auxiliary/draw/draw_pipe_aaline.c | 2 +-
src/gallium/auxiliary/gallivm/lp_bld_swizzle.c | 2 +-
src/gallium/auxiliary/nir/
Fix a 's/unsigned int/unsigned/' consistency case while here.
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/drivers/svga/svga_resource_buffer_upload.c | 2 +-
src/gallium/drivers/svga/svga_tgsi_insn.c | 12 ++--
2 files chan
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_tgsi_soa.c
b/src/gallium/auxiliary/gallivm/lp_bld_tgs
Found-by: Coccinelle
Signed-off-by: Edward O'Callaghan
Reviewed-by: Brian Paul
---
src/gallium/drivers/ilo/core/ilo_builder.c | 8
src/gallium/drivers/ilo/shader/ilo_shader_fs.c | 16
src/gallium/drivers/ilo/shader/ilo_shader_vs.c | 4 ++--
src/gallium/drive
76 matches
Mail list logo
