On Dec 9, 2015 11:47 PM, "Iago Toral" wrote:
>
> On Wed, 2015-12-09 at 08:10 -0800, Jason Ekstrand wrote:
> >
> > On Dec 9, 2015 4:16 AM, "Iago Toral Quiroga"
> > wrote:
> > >
> > > This code in brw_set_dest adjusts the execution size of any
> > instruction
> > > with a dst.width < 8. However, we
On Wed, 2015-12-09 at 08:10 -0800, Jason Ekstrand wrote:
>
> On Dec 9, 2015 4:16 AM, "Iago Toral Quiroga"
> wrote:
> >
> > This code in brw_set_dest adjusts the execution size of any
> instruction
> > with a dst.width < 8. However, we don't want to do this with
> instructions
> > operating on dou
On 12/10/2015 05:41 AM, Dave Airlie wrote:
From: Dave Airlie
If we have a dmat2[4], then dmat2[0] is at 17, dmat2[1] at 19,
dmat2[2] at 21 etc. The old code was returning 17,18,19.
I think this code is also wrong for float matricies as well.
This partly fixes:
GL41-CTS.vertex_attrib_64bit.li
On Tue, Dec 01, 2015 at 12:20:02AM -0800, Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> Cc: Iago Toral Quiroga
For the series:
Reviewed-by: Kristian Høgsberg
Admittedly, light review on patches 17-25 but Iago covered that and
the overall refactoring looks sounds.
For the new shared l
On Wed, Dec 9, 2015 at 9:36 PM, Jason Ekstrand wrote:
> This is an initial implementation of the MOV_INDIRECT opcode in the vec4
> backend. Unfortunately, I haven't had a chance to test it in the wild yet,
> but I think review would still be good. In particular, the approach I took
> to handling
This is an initial implementation of the MOV_INDIRECT opcode in the vec4
backend. Unfortunately, I haven't had a chance to test it in the wild yet,
but I think review would still be good. In particular, the approach I took
to handling swizzles.
Unfortunately, the only indirect MOV instructions y
On Thu, 2015-12-10 at 13:41 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> If we have a dmat2[4], then dmat2[0] is at 17, dmat2[1] at 19,
> dmat2[2] at 21 etc. The old code was returning 17,18,19.
>
> I think this code is also wrong for float matricies as well.
Would be good to have a piglit
On Wed, Dec 9, 2015 at 8:23 PM, Jason Ekstrand wrote:
> Instead of using reladdr, this commit changes the FS backend to emit a
> MOV_INDIRECT whenever we need an indirect uniform load. We also have to
> rework some of the other bits of the backend to handle this new form of
> uniform load. The o
From: Roland Scheidegger
Discovered this when working on other clip code, apparently didn't work
correctly - the combination of linear interpolated values and using
gl_ClipVertex produced wrong values (failing all such combinations
in piglits glsl-1.30 interpolation tests).
Use the pre-clip-pos v
On Tue, Dec 8, 2015 at 6:25 PM, Kenneth Graunke wrote:
> On Tuesday, December 08, 2015 01:46:22 PM Jason Ekstrand wrote:
>> diff --git a/src/glsl/nir/nir_lower_io.c b/src/glsl/nir/nir_lower_io.c
>> index f64ac69..a2723d5 100644
>> --- a/src/glsl/nir/nir_lower_io.c
>> +++ b/src/glsl/nir/nir_lower_i
This is to prevent shader-db regressions from D <-> UD conversions in deref
add+mul chains caused by using MOV_INDIRECT.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/br
Now that we have MOV_INDIRECT opcodes, we have all of the size information
we need directly in the opcode. With a little restructuring of the
algorithm used in assign_constant_locations we don't need param_size
anymore. The big thing to watch out for now, however, is that you can have
two ranges
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 9b06ed2..de33c1d 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw
We aren't using it anymore.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 7 +--
src/mesa/drivers/dri/i965/brw_ir_fs.h | 5 +
2 files changed, 2 insertions(+), 10 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 7cc03c5..786c5fb 10
This commit moves us to an instruction based model rather than a
register-based model for indirects. This is more accurate anyway as we
have to emit instructions to resolve the reladdr. It's also a lot simpler
because it gets rid of the recursive reladdr problem by design.
One side-effect of thi
Instead of using reladdr, this commit changes the FS backend to emit a
MOV_INDIRECT whenever we need an indirect uniform load. We also have to
rework some of the other bits of the backend to handle this new form of
uniform load. The obvious change is that demote_pull_constants now acts
more like
---
src/glsl/nir/nir_intrinsics.h | 7 +--
src/glsl/nir/nir_lower_io.c | 5 +
2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/glsl/nir/nir_intrinsics.h b/src/glsl/nir/nir_intrinsics.h
index 63df21e..c329a82 100644
--- a/src/glsl/nir/nir_intrinsics.h
+++ b/src/glsl/nir
It should work fine without it and the visitor can set it if it wants.
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
index c25da07..d86eee1 10064
It's not really doing enough anymore to justify a helper function.
---
src/mesa/drivers/dri/i965/brw_vec4.h | 2 --
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 37 ++
2 files changed, 14 insertions(+), 25 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/
---
src/mesa/drivers/dri/i965/brw_vec4.cpp| 8
src/mesa/drivers/dri/i965/brw_vec4.h | 2 --
src/mesa/drivers/dri/i965/brw_vec4_nir.cpp| 9 -
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp| 11 ---
src/mesa/drivers/dri/i965/brw_vec4_v
Unfortunately, this also means that we need to use a slightly different
algorithm for assign_constant_locations. The old algorithm worked based on
the assumption that each read of a uniform value read exactly one float.
If it encountered a MOV_INDIRECT, it would immediately bail and push the
whole
The subnr field is in bytes so we don't need to multiply by type_sz.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index de33c1d..9eaf8d0 100644
--- a/src/mes
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 -
src/mesa/drivers/dri/i965/brw_fs.h | 2 --
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 9 -
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 ---
4 files changed, 15 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/br
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++--
src/mesa/drivers/dri/i965/brw_fs.h | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index a2ec03e..f3cf129 100644
--- a/src/mesa/drivers/dri/i965/
While we're at it, we also add support for the possibility that the
indirect is, in fact, a constant. This shouldn't happen in the common case
(if it does, that means NIR failed to constant-fold something), but it's
possible so we should handle it.
---
src/mesa/drivers/dri/i965/brw_fs.cpp
This series is kind-of two series in one. First, we rework the way we do
uniform handling all throughout both backends. In particular, we stop
using reladdr and, instead, start using MOV_INDIRECT. Doing this allows us
some nice simplifications: First, we no longer have the recursive reladdr
pro
From: Dave Airlie
If we have a dmat2[4], then dmat2[0] is at 17, dmat2[1] at 19,
dmat2[2] at 21 etc. The old code was returning 17,18,19.
I think this code is also wrong for float matricies as well.
This partly fixes:
GL41-CTS.vertex_attrib_64bit.limits_test
---
src/mesa/main/shader_query.cpp
On 10.12.2015 06:58, Marek Olšák wrote:
> From: Marek Olšák
>
> Both caused a crash due to a division by zero in that function.
> This is an alternative fix.
>
> Cc: 11.0 11.1
> ---
> src/gallium/drivers/radeonsi/si_state_draw.c | 14 +-
> 1 file changed, 13 insertions(+), 1 deleti
On 10.12.2015 06:58, Marek Olšák wrote:
> From: Marek Olšák
The series is
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
__
On 10.12.2015 07:54, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/radeonsi/si_debug.c | 11 +++
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_debug.c
> b/src/gallium/drivers/radeonsi/si_debug.c
> index cce665e
On 12/07/2015 12:18 PM, Sarah Sharp wrote:
> When reading the source code, it's useful to indicate that a group of
> fields in a struct are related in someway. The convention in Mesa seems
> to be:
>
> struct foo {
> /**
> * Related fields description
> * @{
> */
> int bar;
This patch is
Reviewed-by: Ian Romanick
On 12/07/2015 12:18 PM, Sarah Sharp wrote:
> The freedesktop.org blog feeds aren't mentioned on either mesa3d.org or
> any of the graphics project wikis (including the DRI wiki) on
> freedeskop.org. Fix that by linking to it from the sidebar.
>
> Signed-
From: Nicolai Hähnle
When we know that hangs occur only very late in a reproducible run (e.g.
apitrace), we can save a lot of debugging time by skipping the flush and hang
detection for earlier draw calls.
---
src/gallium/drivers/ddebug/dd_draw.c | 39 +-
src/ga
From: Nicolai Hähnle
This helps in the use of GALLIUM_DDEBUG_SKIP: first run a target application
with skip set to a very large number and note how many draw calls happen
before the bug. Then re-run, skipping the corresponding number of calls.
Despite the additional run, this can still be much fa
...and it matches what we do for single precision.
Reviewed-by: Ian Romanick
Presumably this should also be a candidate for 11.0 and 11.1?
On 12/09/2015 04:37 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> Doesn't have any effect in practice I don't think, but
> CTS reads back using GetVertex
On 12/09/2015 04:07 AM, Timothy Arceri wrote:
> On Wed, 2015-12-09 at 16:06 +1000, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> Signed-off-by: Dave Airlie
>> ---
>> src/glsl/lower_packed_varyings.cpp | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/glsl/lower_pack
From: Dave Airlie
Doesn't have any effect in practice I don't think, but
CTS reads back using GetVertexAttrib.
This fixes: GL41-CTS.vertex_attrib_64bit.get_vertex_attrib
Signed-off-by: Dave Airlie
---
src/mesa/main/varray.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/s
https://bugs.freedesktop.org/show_bug.cgi?id=91888
--- Comment #20 from nerdopol...@verizon.net ---
And now all the examples from qtbase/examples/opengl are working as well even
after exporting LIBGL_ALWAYS_SOFTWARE as well with mesa master.
including contextinfo, qopenglwidget, qopenglwindow, and
On Wed, Dec 9, 2015 at 12:08 PM, Kenneth Graunke wrote:
> On Wednesday, December 09, 2015 08:03:25 AM Jason Ekstrand wrote:
>> On Dec 9, 2015 2:51 AM, "Kenneth Graunke" wrote:
>> >
>> > My tessellation branch has two additional remap functions. I don't want
>> > to replicate this logic there.
>>
Reviewed-by: Ilia Mirkin
On Wed, Dec 9, 2015 at 6:02 PM, Brian Paul wrote:
> We need a different ID pointer for each call site.
> ---
> src/mesa/main/bufferobj.c | 18 --
> 1 file changed, 12 insertions(+), 6 deletions(-)
>
> diff --git a/src/mesa/main/bufferobj.c b/src/mesa/mai
We need a different ID pointer for each call site.
---
src/mesa/main/bufferobj.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 6bc1b5e..e0639c8 100644
--- a/src/mesa/main/bufferobj.c
+++ b/src/mes
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_debug.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_debug.c
b/src/gallium/drivers/radeonsi/si_debug.c
index cce665e..034acf5 100644
--- a/src/gallium/drivers/radeonsi/si_deb
On Wed, Dec 9, 2015 at 5:40 PM, Samuel Pitoiset
wrote:
> Instead of iterating over all the buffer resources looking for coherent
> buffers, we keep track of a context-wide count. This will save some
> iterations (and CPU cycles) in 99.99% case because usually coherent
> buffers are not so used.
>
Instead of iterating over all the buffer resources looking for coherent
buffers, we keep track of a context-wide count. This will save some
iterations (and CPU cycles) in 99.99% case because usually coherent
buffers are not so used.
Changes from v2:
- forgot to apply some changes for nv50 (textur
From: Marek Olšák
v2: fix dual source blending
---
src/gallium/drivers/radeon/r600_pipe_common.c | 1 +
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +
src/gallium/drivers/radeon/r600_texture.c | 6 +
src/gallium/drivers/radeonsi/si_state.c | 159 +-
On Wed, Dec 9, 2015 at 5:23 PM, Brian Paul wrote:
> On 12/09/2015 11:43 AM, Ilia Mirkin wrote:
>>
>> On Mon, Dec 7, 2015 at 8:42 PM, Brian Paul wrote:
>>>
>>> When a buffer is created with GL_STATIC_DRAW, its contents should not
>>> be changed frequently. But that's exactly what one application
On 12/09/2015 11:43 AM, Ilia Mirkin wrote:
On Mon, Dec 7, 2015 at 8:42 PM, Brian Paul wrote:
When a buffer is created with GL_STATIC_DRAW, its contents should not
be changed frequently. But that's exactly what one application I'm
debugging does. This patch adds code to try to detect inefficie
Am 09.12.2015 um 19:59 schrieb eocallag...@alterapraxis.com:
> Roland,
>
> I could not due to ml size limit or something, it just bounces hence the
> pull request.
Due to size limit? Didn't look that big to me...
Otherwise, looks good to me, though the second commit should mention it
actually ena
On Mon, Dec 7, 2015 at 5:36 PM, Brian Paul wrote:
> On 12/06/2015 04:34 PM, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> Spotted by luck. The GLSL uniform storage is only associated once
>> in LinkShader and can't be reallocated afterwards, because that would
>> break the association.
>> ---
From: Marek Olšák
---
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
index 41efbcb..674482e 100644
--- a/src/gallium/winsys/amdgpu/drm/am
From: Marek Olšák
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
b/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
index a5f8aeb..5ba01b9 100644
--- a/src/gallium/wi
From: Marek Olšák
---
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
index 674482e..adea376 100644
--- a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
+++ b/src/
From: Marek Olšák
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
b/src/gallium/winsys/radeon/drm/radeon_drm_bo.c
index 3fd233c..a5f8aeb 100644
--- a/src/gallium/winsys/r
From: Marek Olšák
Both caused a crash due to a division by zero in that function.
This is an alternative fix.
Cc: 11.0 11.1
---
src/gallium/drivers/radeonsi/si_state_draw.c | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_
Pushed, thanks.
Marek
On Wed, Dec 9, 2015 at 10:07 AM, Edward O'Callaghan
wrote:
> Let us avoid trapping in hardware from a SIGFPE and instead
> assert on a zero divisor.
>
> Hint: This can occur if a PIPE_PRIM_? is not handled in
> u_prim_vertex_count() that results in ' info ' not
>
On 2015-12-02 16:15:56, Kenneth Graunke wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/gen7_te_state.c | 36
> ---
> 1 file changed, 28 insertions(+), 8 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen7_te_state.c
> b/src/mesa/d
On 2015-12-02 16:15:55, Kenneth Graunke wrote:
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/gen8_ds_state.c | 66
> +++
> 1 file changed, 59 insertions(+), 7 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/gen8_ds_state.c
> b/src/mesa/d
This new mesa-demos release fixes the build issue against mesa 10.6 (Bug #91643)
and picks up the latest glxinfo changes. For the misc changes see below.
Andreas.
Adam Jackson (1):
glxinfo: Add support for GLX_MESA_query_renderer
Andreas Boll (3):
demos: add missing binaries to .git
12 & 13 Reviewed-by: Jordan Justen
On 2015-12-02 16:15:53, Kenneth Graunke wrote:
> Based on code by Chris Forbes and Fabian Bieler.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_context.h | 2 +-
> src/mesa/drivers/dri/i965/brw_sampler_state.c | 46
> +++
Mesa 11.0.7 is now available.
This release brings substantial amount of fixes in meta (affecting i965),
some driver fixes for i965, nouveau, r600 and llvm. The video encoding
for Stoney has been disabled, as it isn't working properly.
There are also build fixes for DragonFly and other *BSD platfo
Instead of iterating over all the buffer resources looking for coherent
buffers, we keep track of a context-wide count. This will save some
iterations (and CPU cycles) in 99.99% case because usually coherent
buffers are not so used.
Signed-off-by: Samuel Pitoiset
---
I didn't test the patch, but
On Wednesday, December 09, 2015 08:03:25 AM Jason Ekstrand wrote:
> On Dec 9, 2015 2:51 AM, "Kenneth Graunke" wrote:
> >
> > My tessellation branch has two additional remap functions. I don't want
> > to replicate this logic there.
> >
> > Signed-off-by: Kenneth Graunke
> > ---
> > src/mesa/dri
On 9 December 2015 at 21:39, Timothy Arceri wrote:
> On Wed, 2015-12-09 at 16:06 +1000, Dave Airlie wrote:
>> From: Dave Airlie
>>
>> Signed-off-by: Dave Airlie
>
> Great timing :) I was going to have to look into fixing this stuff for
> enhanced layouts.
>
> Patches 1 & 2 are:
> Reviewed-by: Ti
Reviewed-by: Jason Ekstrand
On Thu, Nov 19, 2015 at 7:25 AM, Neil Roberts wrote:
> The tiled memcpy doesn't work for copying from RGBX to RGBA because it
> doesn't override the alpha component to 1.0. Commit 2cebaac479d4 added
> a check to disable it for RGBX formats by looking at the TexFormat.
On Wed, Dec 9, 2015 at 2:01 PM, Patrick Rudolph wrote:
> Ok, first of all bind some buffers:
>
> pipe->set_vertex_buffers(pipe, 0, 1, &vtxbuf);
> pipe->set_vertex_buffers(pipe, 1, 1, &vtxbuf);
> pipe->set_vertex_buffers(pipe, 2, 1, &vtxbuf);
>
> num_vtxbufs is now 3 as it should be.
>
> Now you ar
https://bugs.freedesktop.org/show_bug.cgi?id=90821
Nanley Chery changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
I pushed a slightly modified version of this:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=432a798cf5c7fab18a3e32d4073840df7d0d37cb
Thanks for the patch! I hope this will resolve some weird crashes
people have seen with various buffers being null unexpectedly.
On Sun, Dec 6, 2015 at 4:11 AM,
On Wed, Dec 9, 2015 at 11:23 AM, Ilia Mirkin wrote:
> On Wed, Dec 9, 2015 at 11:18 AM, Deve wrote:
>> This patch indeed seems to not have a sense. I just added it to the bug
>> report as a suggestion that it works for me after this modification. Emil
>> Velikov said that I should send it to the m
On 2015-12-10 01:47, Marek Olšák wrote:
From: Marek Olšák
I guess the crash was because of divison by zero.
Cc: 11.0 11.1
---
src/gallium/auxiliary/util/u_prim.h | 17 +
src/gallium/drivers/radeonsi/si_state_draw.c | 3 ++-
2 files changed, 15 insertions(+), 5 delet
Ok, first of all bind some buffers:
pipe->set_vertex_buffers(pipe, 0, 1, &vtxbuf);
pipe->set_vertex_buffers(pipe, 1, 1, &vtxbuf);
pipe->set_vertex_buffers(pipe, 2, 1, &vtxbuf);
num_vtxbufs is now 3 as it should be.
Now you are unbinding buffers, one after another starting at slot 0:
pipe->set_ve
Roland,
I could not due to ml size limit or something, it just bounces hence the
pull request.
Cheers,
Edward.
On 2015-12-10 02:38, Roland Scheidegger wrote:
Am 09.12.2015 um 05:16 schrieb Edward O'Callaghan:
This fixes my initial attempt so that piglit now passes 14/14. Thanks
to a couple
On Mon, Dec 7, 2015 at 8:42 PM, Brian Paul wrote:
> When a buffer is created with GL_STATIC_DRAW, its contents should not
> be changed frequently. But that's exactly what one application I'm
> debugging does. This patch adds code to try to detect inefficient
> buffer use in a couple places. The
On Wed, Dec 9, 2015 at 4:15 AM, Iago Toral Quiroga wrote:
> ---
> src/mesa/drivers/dri/i965/brw_eu_emit.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
> b/src/mesa/drivers/dri/i965/brw_eu_emit.c
> index f8c0f80..9543d5e 100644
> --- a/src/mesa/
On Tue, Dec 8, 2015 at 9:37 PM, Jonathan Gray wrote:
> Change the __m128i variables to be volatile so gcc 4.9 won't optimise
> all of them out with -O1 or greater. The _mm_set1_epi32/pinsrd calls
> still get optimised out but now there is at least one SSE4.1 instruction
> generated via _mm_max_ep
I'm probably just being dense... can you provide an exact sequence of
calls that would cause this logic to fail? Seems like it should work
as-is...
On Sun, Dec 6, 2015 at 4:12 AM, Patrick Rudolph wrote:
> In case a state tracker unbinds every slot by a seperate
> pipe->set_vertex_buffers() call,
The general concept of this change seems fine to me.
Given the desire to keep glapi as similar as possible across platforms, would
it be better to just move this into glx/apple rather than leaving it in glapi?
> On Dec 9, 2015, at 09:07, Emil Velikov wrote:
>
> On 9 December 2015 at 14:11, And
On 9 December 2015 at 14:11, Andreas Boll wrote:
> Removes the public symbol _glapi_create_table_from_handle from
> libGL.so.1 on all plattforms except Darwin.
>
typo -> platforms
> Since the symbol is not used on other plattforms it makes sense to
ditto
> build gl_gentable.c only on Darwin.
>
I
On 12/09/2015 09:20 AM, Andreas Boll wrote:
s/suports/supports/
Signed-off-by: Andreas Boll
---
src/glsl/glsl_parser_extras.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/glsl_parser_extras.h b/src/glsl/glsl_parser_extras.h
index 6bded3e..a4bda77 100644
--- a/
On 12/09/2015 09:29 AM, Andreas Boll wrote:
s/suports/supports/
Signed-off-by: Andreas Boll
---
Found two more "suports" typos.
I could squash all patches together if that's preferred.
src/glx/dri2_glx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glx/dri2_glx.c
s/suports/supports/
Signed-off-by: Andreas Boll
---
src/mesa/main/extensions.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/extensions.h b/src/mesa/main/extensions.h
index 1615e1c..b5e0350 100644
--- a/src/mesa/main/extensions.h
+++ b/src/mesa/main/extensions
s/suports/supports/
Signed-off-by: Andreas Boll
---
Found two more "suports" typos.
I could squash all patches together if that's preferred.
src/glx/dri2_glx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c
index 27ea952..651915a 1006
On Wed, Dec 9, 2015 at 11:18 AM, Deve wrote:
> This patch indeed seems to not have a sense. I just added it to the bug
> report as a suggestion that it works for me after this modification. Emil
> Velikov said that I should send it to the mailing list.
>
> Here is how it works in Supertuxkart:
> W
https://bugs.freedesktop.org/show_bug.cgi?id=91888
--- Comment #19 from nerdopol...@verizon.net ---
I didn't see any thing in the changelog for 'egl' that looked like it might be
a fix... Not 100% sure though
--
You are receiving this mail because:
You are the QA Contact for the bug.
___
s/inconsistant/inconsistent/
s/occurences/occurrences/
Signed-off-by: Andreas Boll
---
src/mesa/main/teximage.c | 2 +-
src/mesa/main/transformfeedback.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 60fc
s/suports/supports/
Signed-off-by: Andreas Boll
---
src/glsl/glsl_parser_extras.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/glsl_parser_extras.h b/src/glsl/glsl_parser_extras.h
index 6bded3e..a4bda77 100644
--- a/src/glsl/glsl_parser_extras.h
+++ b/src/glsl/gls
s/suport/support/
Signed-off-by: Andreas Boll
---
src/gallium/state_trackers/osmesa/osmesa.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/osmesa/osmesa.c
b/src/gallium/state_trackers/osmesa/osmesa.c
index 0285cb0..0f27ba8 100644
--- a/src/galliu
s/Unkown/Unknown/
Signed-off-by: Andreas Boll
---
src/mesa/drivers/common/meta_blit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/common/meta_blit.c
b/src/mesa/drivers/common/meta_blit.c
index c5faf61..4dbf0a7 100644
--- a/src/mesa/drivers/common/meta_bl
This patch indeed seems to not have a sense. I just added it to the bug
report as a suggestion that it works for me after this modification.
Emil Velikov said that I should send it to the mailing list.
Here is how it works in Supertuxkart:
We create rtt with following parameters:
DepthStencilT
On Wed, Dec 09, 2015 at 10:35:25AM +, Emil Velikov wrote:
>> I was told that it's easier for people to review my patch if it comes in via
>> email than being stuck in the bug tracker; FWIW, this is for bug 120238.
> Which bugtracker it this ? bugs.fd.o does not like the number
> mentioned. Plea
On Dec 9, 2015 4:16 AM, "Iago Toral Quiroga" wrote:
>
> This code in brw_set_dest adjusts the execution size of any instruction
> with a dst.width < 8. However, we don't want to do this with instructions
> operating on doubles, since these will have a width of 4, but still
> need an execution size
On Dec 9, 2015 2:51 AM, "Kenneth Graunke" wrote:
>
> My tessellation branch has two additional remap functions. I don't want
> to replicate this logic there.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_nir.c | 78
-
> 1 file chang
On 9 December 2015 at 14:57, Neil Roberts wrote:
> Emil Velikov writes:
>
>> Worth throwing in 11.0 as well ?
>
> Yeah, that would probably be sensible.
>
>>> if (_mesa_is_winsys_fbo(ctx->DrawBuffer)) {
>>> - dri2InvalidateDrawable(driContext->driDrawablePriv);
>>> - dri2InvalidateD
Am 09.12.2015 um 05:16 schrieb Edward O'Callaghan:
> This fixes my initial attempt so that piglit now passes 14/14. Thanks
> to a couple of tips from Roland in the previous patch I was able to
> fix the remaining issue. This should be golden now.
>
Great that you got it working!
Please send the p
Hi,
On 11/09/2015 08:47 PM, Matt Turner wrote:
On Mon, Nov 9, 2015 at 10:46 AM, Ilia Mirkin wrote:
I used this script in conjunction with ST_DUMP_SHADERS. What other way is there?
Some local hack and we should probably finish and upstream.
Did anything happen with this?
I had to rewrite s
Emil Velikov writes:
> Worth throwing in 11.0 as well ?
Yeah, that would probably be sensible.
>> if (_mesa_is_winsys_fbo(ctx->DrawBuffer)) {
>> - dri2InvalidateDrawable(driContext->driDrawablePriv);
>> - dri2InvalidateDrawable(driContext->driReadablePriv);
>> + if (driContex
https://bugs.freedesktop.org/show_bug.cgi?id=91888
--- Comment #18 from Daniel Stone ---
How about 'hooray, it's fixed'? :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedeskto
From: Marek Olšák
I guess the crash was because of divison by zero.
Cc: 11.0 11.1
---
src/gallium/auxiliary/util/u_prim.h | 17 +
src/gallium/drivers/radeonsi/si_state_draw.c | 3 ++-
2 files changed, 15 insertions(+), 5 deletions(-)
diff --git a/src/gallium/auxiliar
Removes the public symbol _glapi_create_table_from_handle from
libGL.so.1 on all plattforms except Darwin.
Since the symbol is not used on other plattforms it makes sense to
build gl_gentable.c only on Darwin.
A little bit of history:
_glapi_create_table_from_handle was introduced in
commit 859
On Wed, Dec 9, 2015 at 2:34 PM, Jonathan Gray wrote:
> On Wed, Dec 09, 2015 at 01:39:30PM +0200, Oded Gabbay wrote:
>> On Wed, Dec 9, 2015 at 1:09 PM, Emil Velikov
>> wrote:
>> > On 9 December 2015 at 05:37, Jonathan Gray wrote:
>> >> Change the __m128i variables to be volatile so gcc 4.9 won't
Kristian Høgsberg writes:
> On Sat, Sep 5, 2015 at 11:30 AM, Jordan Justen
> wrote:
>> From: Francisco Jerez
>>
>> Fixes
>> arb_shader_image_load_store/execution/load-from-cleared-image.shader_test
>>
>> Cc: Chris Wilson
>> Cc: Jason Ekstrand
>> Tested-by: Jordan Justen
>
> This patch is re
1 - 100 of 130 matches
Mail list logo
