https://bugs.freedesktop.org/show_bug.cgi?id=27512
--- Comment #6 from mhard...@gmail.com ---
I am experiencing a crash with a SIGILL, Illegal instruction in Debian when
using Kodi.
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.s
On Mon, 2016-01-04 at 20:01 -0500, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> Cc: Samuel Iglesias Gonsálvez
> Cc: Marta Lofstedt
> ---
>
> I have not tested whether this fixes your CTS tests (as I don't have
> access
> to them). But it should... and if it doesn't, it'll be a variation of
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 enum
type. Adding an assert will generate no additional code bu
All indirect draws are passed to the new draw function. By default
there's a fallback implementation which pipes it right back to
draw_prims, but eventually both the fallback and draw_prim's support for
indirect drawing should be removed.
This should allow a backend to properly support ARB_multi_d
On Mon, Jan 4, 2016 at 10:07 PM, Ian Romanick wrote:
> On 12/31/2015 11:55 AM, Ilia Mirkin wrote:
>> The current support in mesa for ARB_multi_draw_indirect is pretty
>> shameful... it just does N separate draws. Instead make it possible
>> for drivers to handle this in a more efficient manner. Al
On Mon, Jan 4, 2016 at 10:10 PM, Ian Romanick wrote:
> On 12/31/2015 11:55 AM, Ilia Mirkin wrote:
>> This is optional for now in the transition period, but optimally all
>> backends that support indirect draws would switch over to it and we can
>> remove the support for indirect in the "regular" d
On 12/31/2015 11:55 AM, Ilia Mirkin wrote:
> This is optional for now in the transition period, but optimally all
> backends that support indirect draws would switch over to it and we can
> remove the support for indirect in the "regular" draw function.
>
> This should allow a backend to properly
On 12/31/2015 11:55 AM, Ilia Mirkin wrote:
> The current support in mesa for ARB_multi_draw_indirect is pretty
> shameful... it just does N separate draws. Instead make it possible
> for drivers to handle this in a more efficient manner. Also create the
> backend interfaces necessary to support ARB
---
si-report.py | 56 ++--
1 file changed, 54 insertions(+), 2 deletions(-)
diff --git a/si-report.py b/si-report.py
index ec88112..e717af0 100755
--- a/si-report.py
+++ b/si-report.py
@@ -65,6 +65,12 @@ def get_scratch_str(value, suffixes = Tr
On 12/31/2015 09:36 AM, Matt Turner wrote:
> On Thu, Dec 31, 2015 at 12:25 PM, Ilia Mirkin wrote:
>> On Thu, Dec 31, 2015 at 12:17 PM, Matt Turner wrote:
>>> On Thu, Dec 31, 2015 at 9:45 AM, Ilia Mirkin wrote:
On Dec 31, 2015 9:32 AM, "Matt Turner" wrote:
>
> On Wed, Dec 30, 2015 a
On Mon, Jan 4, 2016 at 8:32 PM, Ian Romanick wrote:
> On 01/03/2016 10:02 AM, Ilia Mirkin wrote:
>> On Sun, Jan 3, 2016 at 12:33 PM, Roland Scheidegger
>> wrote:
>>> Am 03.01.2016 um 01:37 schrieb Ilia Mirkin:
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/tgsi.rst | 10
On Wed, Dec 30, 2015 at 09:32:38PM -0500, Nicolai Hähnle wrote:
> For Gallium-based drivers, this is required for receiving shader information
> via debug messages.
Patches 2-5 are
Acked-by: Tom Stellard
> ---
> run.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/run.c b/run.c
> in
On 01/03/2016 10:02 AM, Ilia Mirkin wrote:
> On Sun, Jan 3, 2016 at 12:33 PM, Roland Scheidegger
> wrote:
>> Am 03.01.2016 um 01:37 schrieb Ilia Mirkin:
>>> Signed-off-by: Ilia Mirkin
>>> ---
>>> src/gallium/docs/source/tgsi.rst | 10 --
>>> 1 file changed, 8 insertions(+), 2 deletions(
[adding Ian and Ken, who may or may not be interested]
I'd like to push this out in the next couple of days (along with my
ARB_indirect_parameters series), but I'd really like to get an ack
from someone on the Intel team that such an interface would be
acceptable to them. You can see the full seri
Signed-off-by: Ilia Mirkin
Cc: Samuel Iglesias Gonsálvez
Cc: Marta Lofstedt
---
I have not tested whether this fixes your CTS tests (as I don't have access
to them). But it should... and if it doesn't, it'll be a variation of the
logic used here.
src/mesa/drivers/dri/i965/brw_wm_surface_state
This reverts commit 8cf2e892fca20c4776b4a07c39918343cb2d4e0e. It's
entirely bogus to attempt to store anything about the binding the buffer
object itself, which might be bound any number of times.
Signed-off-by: Ilia Mirkin
Cc: Samuel Iglesias Gonsálvez
Cc: Marta Lofstedt
---
src/mesa/drivers/
We only want to set the SVGA_NEW_STIPPLE dirty flag when the polygon
stipple state changes. Before, we only set the flag when we were
enabling stipple, but not disabling.
We don't really have to add SVGA_NEW_STIPPLE to the dirty FS state
set since it's a subset of SVGA_NEW_RAST, but let's be expl
---
src/mesa/math/m_matrix.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/mesa/math/m_matrix.c b/src/mesa/math/m_matrix.c
index 6522200..b3cfcd2 100644
--- a/src/mesa/math/m_matrix.c
+++ b/src/mesa/math/m_matrix.c
@@ -654,7 +654,7 @@ static GLboolean inver
It's very rare that a GL app calls glVertex3dv(), but one in particular
calls it lot, always with Z = 0. Check for that condition and convert
the call into glVertex2f. This reduces VBO memory used and reduces
the number of times we have to switch between float[2] and float[3]
vertex formats in th
Since it's a true/false value.
---
src/gallium/drivers/svga/svga_context.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_context.h
b/src/gallium/drivers/svga/svga_context.h
index d21b071..e4f29b8 100644
--- a/src/gallium/drivers/svga/svga_contex
---
src/gallium/drivers/svga/svga_context.h | 5 +
src/gallium/drivers/svga/svga_draw.c| 26 +-
2 files changed, 26 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_context.h
b/src/gallium/drivers/svga/svga_context.h
index c282932..d21b071
and svga_set_sampler_views(). If there's no change, return early
and don't set a SVGA_NEW_x dirty state flag.
---
src/gallium/drivers/svga/svga_pipe_sampler.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_pipe_sampler.c
b/src/g
---
src/gallium/drivers/svga/svga_state_vs.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/svga/svga_state_vs.c
b/src/gallium/drivers/svga/svga_state_vs.c
index 24574c1..a103dab 100644
--- a/src/gallium/drivers/svga/svga_state_vs.c
+++ b/src/gallium/drivers/svga/svga_
Like debug_dump_float_rgba_bmp() but takes ubyte values.
---
src/gallium/auxiliary/util/u_debug.c | 59
src/gallium/auxiliary/util/u_debug.h | 4 +++
2 files changed, 63 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_debug.c
b/src/gallium/auxiliary/
Reviewed-by: Ilia Mirkin
On Mon, Jan 4, 2016 at 5:47 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Instead, keep track of GL_DEBUG_OUTPUT and (un)install the pipe_debug_callback
> accordingly. Hardware drivers can still use the absence of the callback to
> skip more expensive operations
From: Nicolai Hähnle
Instead, keep track of GL_DEBUG_OUTPUT and (un)install the pipe_debug_callback
accordingly. Hardware drivers can still use the absence of the callback to
skip more expensive operations in the normal case, and users can no longer be
surprised by the need to set the debug flag
On Mon, Jan 4, 2016 at 5:34 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Instead, keep track of GL_DEBUG_OUTPUT and (un)install the pipe_debug_callback
> accordingly. Hardware drivers can still use the absence of the callback to
> skip more expensive operations in the normal case, and use
From: Nicolai Hähnle
Instead, keep track of GL_DEBUG_OUTPUT and (un)install the pipe_debug_callback
accordingly. Hardware drivers can still use the absence of the callback to
skip more expensive operations in the normal case, and users can no longer be
surprised by the need to set the debug flag
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
---
src/gallium/state_trackers/va/picture.c| 8
src/gallium/state_trackers/va/picture_h264.c | 2 ++
src/gallium/state_
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.c
b/src/gallium/auxiliary/tgsi/tgsi_strings.c
index fd926b3..ae30399 1
On Mon, Jan 4, 2016 at 3: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.c
>>> b/src/gallium/auxiliary/tgsi/tgsi_strings.c
>>
RES is split into two -- BUFFER and IMAGE.
On Mon, Jan 4, 2016 at 3:22 PM, Samuel Pitoiset
wrote:
> I assume this is going to replace the old RES stuff?
>
>
> On 01/03/2016 05:37 AM, Ilia Mirkin wrote:
>>
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/gallium/auxiliary/tgsi/tgsi_dump.c | 5 +
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.c
>> b/src/gallium/auxiliary/tgsi/tgsi_strings.c
>> index fd926b3..ae30399 100644
>> --- a/src/gallium/auxiliar
On 01/04/2016 09:18 PM, Ilia Mirkin wrote:
On Mon, Jan 4, 2016 at 3:04 PM, Samuel Pitoiset
wrote:
Did you write piglit tests for this new extension?
I don't see any tests which use MultiDrawArraysIndirectCountARB() or
MultiDrawElementsIndirectCountARB().
http://patchwork.freedesktop.org/pat
I assume this is going to replace the old RES stuff?
On 01/03/2016 05:37 AM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_dump.c | 5 +++
src/gallium/auxiliary/tgsi/tgsi_strings.c | 1 +
src/gallium/auxiliary/tgsi/tgsi_text.c | 5 +++
src/ga
On Mon, Jan 4, 2016 at 3:04 PM, Samuel Pitoiset
wrote:
> Did you write piglit tests for this new extension?
> I don't see any tests which use MultiDrawArraysIndirectCountARB() or
> MultiDrawElementsIndirectCountARB().
http://patchwork.freedesktop.org/patch/69334/
>
>
> Are you going to submit th
On 01/03/2016 05:37 AM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_build.c| 62 +
src/gallium/auxiliary/tgsi/tgsi_dump.c | 10 +--
src/gallium/auxiliary/tgsi/tgsi_parse.c| 4 +-
src/gallium/auxilia
Reviewed-by: Edward O'Callaghan
On 2016-01-05 05:27, Rob Clark wrote:
From: Rob Clark
nir.h is a bit inconsistent about 'typedef struct {} nir_foo' vs
'typedef struct nir_foo {} nir_foo'. But missing struct name tags is
inconvenient when you need a fwd declaration without pulling in all
of n
Did you write piglit tests for this new extension?
I don't see any tests which use MultiDrawArraysIndirectCountARB() or
MultiDrawElementsIndirectCountARB().
Are you going to submit the rest of the series for nvc0?
On 01/02/2016 09:38 PM, Ilia Mirkin wrote:
I chose to make separate macros for
On 02/01/16 20:39, srol...@vmware.com wrote:
From: Roland Scheidegger
This is quite a few less instructions, albeit still do the 2 64bit muls
with scalar c code (they'd need way more shuffles, plus fixup for the signed
mul so it totally doesn't seem worth it - x86 can do 32x32->64bit signed
sca
On Mon, Jan 4, 2016 at 12:52 PM, Ilia Mirkin wrote:
> On Mon, Jan 4, 2016 at 12:44 PM, Matt Turner wrote:
>> On Wed, Dec 30, 2015 at 4:26 PM, Ilia Mirkin wrote:
>>> On Wed, Dec 30, 2015 at 3:26 PM, Matt Turner wrote:
The OpenGL specifications for these functions say:
The resul
From: Rob Clark
nir.h is a bit inconsistent about 'typedef struct {} nir_foo' vs
'typedef struct nir_foo {} nir_foo'. But missing struct name tags is
inconvenient when you need a fwd declaration without pulling in all
of nir.
So add missing struct name tag for nir_variable, and a couple other
s
On Mon, Jan 4, 2016 at 12:52 PM, Ilia Mirkin wrote:
> On Mon, Jan 4, 2016 at 12:44 PM, Matt Turner wrote:
>> On Wed, Dec 30, 2015 at 4:26 PM, Ilia Mirkin wrote:
>>> On Wed, Dec 30, 2015 at 3:26 PM, Matt Turner wrote:
The OpenGL specifications for these functions say:
The resul
https://bugs.freedesktop.org/show_bug.cgi?id=93570
Rob Clark changed:
What|Removed |Added
CC||robcl...@freedesktop.org
--
You are receivi
On Mon, Jan 4, 2016 at 12:44 PM, Matt Turner wrote:
> On Wed, Dec 30, 2015 at 4:26 PM, Ilia Mirkin wrote:
>> On Wed, Dec 30, 2015 at 3:26 PM, Matt Turner wrote:
>>> The OpenGL specifications for these functions say:
>>>
>>>The result will be undefined if or is negative, or if
>>>the su
Looks OK to me.
For the series, Reviewed-by: Brian Paul
On 01/02/2016 01:39 PM, srol...@vmware.com wrote:
From: Roland Scheidegger
eo, just like dcdx and dcdy, cannot overflow 32bit.
Store it as unsigned though just in case (it cannot be negative, but
in theory twice as big as dcdx or dcdy
On Wed, Dec 30, 2015 at 8:55 PM, Roland Scheidegger wrote:
> Am 30.12.2015 um 21:26 schrieb Matt Turner:
>> The OpenGL specifications for these functions say:
>>
>>The result will be undefined if or is negative, or if
>>the sum of and is greater than the number of bits
>>used to st
On Wed, Dec 30, 2015 at 4:26 PM, Ilia Mirkin wrote:
> On Wed, Dec 30, 2015 at 3:26 PM, Matt Turner wrote:
>> The OpenGL specifications for these functions say:
>>
>>The result will be undefined if or is negative, or if
>>the sum of and is greater than the number of bits
>>used to
I didn't do a detailed review, but these changes look like good clean-ups.
For the series, Reviewed-by: Brian Paul
On 12/21/2015 08:00 PM, srol...@vmware.com wrote:
From: Roland Scheidegger
Previously the code would just redirect requests for attributes which
don't exist to use output 0. Re
https://bugs.freedesktop.org/show_bug.cgi?id=93570
Peter changed:
What|Removed |Added
CC||pbrobin...@gmail.com
--
You are receiving this
On Mon, Jan 4, 2016 at 11:14 AM, Rob Clark wrote:
> On Mon, Jan 4, 2016 at 11:03 AM, Ilia Mirkin wrote:
>> On Mon, Jan 4, 2016 at 10:30 AM, Rob Clark wrote:
>>> On Sat, Jan 2, 2016 at 11:37 PM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_bu
https://bugs.freedesktop.org/show_bug.cgi?id=93570
--- Comment #12 from Icenowy Zheng ---
Yes. And it's still faster than softpipe.
It seems that loading a scene cost too much time, as when the animation
started, it's smooth. (but the result is low fps number)
bugzilla-dae...@freedesktop.o
https://bugs.freedesktop.org/show_bug.cgi?id=93570
--- Comment #11 from Rob Clark ---
(In reply to Icenowy Zheng from comment #10)
> I've finally built a git version of mesa with a svn version of llvm.
>
> Now the image is fixed, but the performace is 1/4 to the original.
just a sanity check, b
On Mon, Jan 4, 2016 at 11:03 AM, Ilia Mirkin wrote:
> On Mon, Jan 4, 2016 at 10:30 AM, Rob Clark wrote:
>> On Sat, Jan 2, 2016 at 11:37 PM, Ilia Mirkin wrote:
>>> Signed-off-by: Ilia Mirkin
>>> ---
>>> src/gallium/auxiliary/tgsi/tgsi_build.c| 62 +
>>> src/gallium/a
On Mon, Jan 4, 2016 at 10:30 AM, Rob Clark wrote:
> On Sat, Jan 2, 2016 at 11:37 PM, Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/gallium/auxiliary/tgsi/tgsi_build.c| 62 +
>> src/gallium/auxiliary/tgsi/tgsi_dump.c | 10 +--
>> src/galli
https://bugs.freedesktop.org/show_bug.cgi?id=93570
--- Comment #10 from Icenowy Zheng ---
I've finally built a git version of mesa with a svn version of llvm.
Now the image is fixed, but the performace is 1/4 to the original.
--
You are receiving this mail because:
You are the QA Contact for t
On Sat, Jan 2, 2016 at 11:37 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/gallium/auxiliary/tgsi/tgsi_build.c| 62 +
> src/gallium/auxiliary/tgsi/tgsi_dump.c | 10 +--
> src/gallium/auxiliary/tgsi/tgsi_parse.c| 4 +-
> src/g
On 01/04/2016 07:24 AM, Jose Fonseca wrote:
On 04/01/16 14:03, Marek Olšák wrote:
> Also, do we have
know any real apps using glBitmap?
Yes, Brian Paul came across real world apps that use glBitmap for text
drawing or thin wrappers around them like wglUseFontBitmaps/glXUseXFont,
and IIRC p
On 04/01/16 14:03, Marek Olšák wrote:
On Mon, Jan 4, 2016 at 2:09 PM, Jose Fonseca wrote:
On 04/01/16 12:25, Jose Fonseca wrote:
On 21/12/15 22:35, Marek Olšák wrote:
Hi,
This patch series adds more flexibility to u_upload_mgr. First, it
adds the ability to specify the alignment per suball
On Mon, Jan 4, 2016 at 2:09 PM, Jose Fonseca wrote:
> On 04/01/16 12:25, Jose Fonseca wrote:
>>
>> On 21/12/15 22:35, Marek Olšák wrote:
>>>
>>> Hi,
>>>
>>> This patch series adds more flexibility to u_upload_mgr. First, it
>>> adds the ability to specify the alignment per suballocation. The idea
Hi everyone, and Happy New Year!
As most of you are probably aware I participated in GSoC last summer.
(Loop analysis and range propagation for NIR).
This was a very valuable and fun experience,
and something I'd like to have another shot at.
So why am I writing in this early? Last year I was a b
On 04/01/16 12:25, Jose Fonseca wrote:
On 21/12/15 22:35, Marek Olšák wrote:
Hi,
This patch series adds more flexibility to u_upload_mgr. First, it
adds the ability to specify the alignment per suballocation. The idea
is that several users can use the same upload buffer, but each may
need a dif
For the series:
Reviewed-by: Marek Olšák
Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Buffer range is a property of the binding, not the buffer. What if one
buffer is bound twice with diff ranges to diff bindings?
Also please don't make core changes in commits with a driver prefix... I
nearly missed this one because it said i965.
On Dec 15, 2015 6:52 AM, "Iago Toral Quiroga" wrote
On Sun, Jan 3, 2016 at 5:37 AM, Ilia Mirkin wrote:
> Each load/store on most hardware can specify what caching to do. Since
> SSBO allows individual variables to also have separate caching modes,
> allow loads/stores to have the qualifiers instead of attempting to
> encode them in declarations.
>
On 21/12/15 22:35, Marek Olšák wrote:
Hi,
This patch series adds more flexibility to u_upload_mgr. First, it adds the
ability to specify the alignment per suballocation. The idea is that several
users can use the same upload buffer, but each may need a different alignment.
Finally, it allows
Patches 1-4:
Reviewed-by: Marek Olšák
Marek
On Sat, Jan 2, 2016 at 9:38 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/state_tracker/st_cb_bufferobjects.c | 1 +
> src/mesa/state_tracker/st_extensions.c | 1 +
> 2 files changed, 2 insertions(+)
>
> diff --git a/sr
On Sunday, January 3, 2016 3:41:08 PM PST Francisco Jerez wrote:
> Kenneth Graunke writes:
>
> > On Saturday, January 2, 2016 10:48:02 PM PST Francisco Jerez wrote:
> >> Switching the current pipeline while it's not completely idle or the
> >> read and write caches aren't flushed can lead to corr
Reviewed-by: Marek Olšák
Marek
On Mon, Jan 4, 2016 at 8:55 AM, Tapani Pälli wrote:
> Patch changes linker to allocate gl_shader_variable instead of using
> ir_variable. This makes it possible to get rid of ir_variables and ir
> in memory after linking.
>
> v2: check that we do not create duplic
https://bugs.freedesktop.org/show_bug.cgi?id=93570
--- Comment #9 from Icenowy Zheng ---
I'm sorry, but my device is not capable to build a svn version of LLVM. (The
original version of LLVM (3.7) is built on a buildbot, so does Mesa 11.0)
Can you provide me a simple testsuite to check whether i
On Thu, 2015-12-31 at 11:20 -0800, Jordan Justen wrote:
> Previously we were treating the binding index for Uniform Buffer
> Objects and Shader Storage Buffer Objects as being part of the
> combined BufferInterfaceBlocks array.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93322
> Cc:
On Thu, 2015-12-31 at 11:20 -0800, Jordan Justen wrote:
> v2:
> * Fill UboInterfaceBlockIndex and SsboInterfaceBlockIndex in
>split_ubos_and_ssbos (Iago)
>
> Cc: Samuel Iglesias Gonsálvez
> Cc: Iago Toral
> Signed-off-by: Jordan Justen
> Reviewed-by: Juha-Pekka Heikkila
Reviewed-by: Iago
72 matches
Mail list logo
