I'm working on rebasing these patches on Jason's NIR input/output
intrinsic changes. Patches 17, 18, 19, 22, and 24 are probably not
worth reviewing in their current form.
10, 12-16, and 23 still apply in their current form.
--Ken
signature.asc
Description: This is a digitally signed message
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
being initialized in the expected manner.
Further, we also fix a possibly NULL pointer
https://bugs.freedesktop.org/show_bug.cgi?id=91888
--- Comment #16 from Pekka Paalanen ---
(In reply to nerdopolis1 from comment #15)
> Doesn't seem that break _mesa_error works, it's not defined...
It should become defined once all the related Mesa libraries get loaded.
Michel Dänzer writes:
> From: Michel Dänzer
>
> Signed-off-by: Michel Dänzer
Looks OK to me,
Reviewed-by: Francisco Jerez
> ---
> src/gallium/state_trackers/clover/llvm/invocation.cpp | 4
> 1
On Wed, Dec 9, 2015 at 8:06 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Don't use a bool here, as for some 64-bit fixes we need
> the stage.
>
> Signed-off-by: Dave Airlie
> ---
> src/glsl/ir_set_program_inouts.cpp | 8
> 1
https://bugs.freedesktop.org/show_bug.cgi?id=1626
Michel Dänzer changed:
What|Removed |Added
Status|NEW |RESOLVED
On 2015-12-08 08:43:53, Francisco Jerez wrote:
> This is going to require some rather intrusive kernel changes to fix
> properly, in the meantime (and forever on at least pre-v4.1 kernels)
> we'll have to restore the hardware defaults at the end of every batch
> in which the L3 configuration was
On 9 December 2015 at 00:35, Brian Paul wrote:
> To be safe, set all the fields in case the enums ordering/values
> ever change.
Since you guys are using MVSC2013 to build svga one can even use c99
initializers. This should be more robust wrt the issues mentioned.
-Emil
Hello Steinar,
On 8 December 2015 at 01:01, Steinar H. Gunderson
wrote:
> Hi,
>
> 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
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 changed, 50 insertions(+), 28 deletions(-)
On 9 December 2015 at 05:55, Jonathan Gray wrote:
> Use PKG_CHECK_MODULES to get the flags to link libelf
>
> v2: keep AC_CHECK_LIB as a fallback for elfutils provided
> libelf that doesn't install a pkg-config file.
>
> Signed-off-by: Jonathan Gray
> Reviewed-by:
On Wed, 2015-12-09 at 16:06 +1000, Dave Airlie wrote:
> From: Dave Airlie
>
> This fixes the calculations for transform feedback for doubles.
>
> Signed-off-by: Dave Airlie
Patches 4-7 are also:
Reviewed-by: Timothy Arceri
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 of 8 (for SIMD8). Unfortunately, we can't just check
the size
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 9543d5e..13c8c36 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
+++
---
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/drivers/dri/i965/brw_eu_emit.c
+++
Right now we rely on the code at the bottom of brw_set_dest to set the correct
execution size for anything that does not operate on a full SIMD register
(dst.width < BRW_EXECUTE_8). However, this won't work with fp64, where operands
are twice as big and we see instructions with a horizontal
https://bugs.freedesktop.org/show_bug.cgi?id=91888
--- Comment #17 from nerdopol...@verizon.net ---
Argh, I tried to recompile mesa master with all of the symbols, and now SDL is
working, not sure what to think now...
--
You are receiving this mail because:
You are the QA Contact for the bug.
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 optimise
>> all of them out with -O1 or greater. The _mm_set1_epi32/pinsrd calls
>>
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: Timothy Arceri
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:
>> >>
On 8 December 2015 at 16:35, Neil Roberts wrote:
> If EGL_KHR_surfaceless_context is used then glViewport can be called
> with NULL for the draw and read surfaces. This was previously causing
> a crash because the i965 driver tries to use this point to invalidate
> the
On Wed, 2015-12-09 at 09:48 +0200, Tapani Pälli wrote:
> Otherwise packed and inactive varyings get optimized away. This needs
> to be prevented when using separate shader objects where interface
> needs to be preserved.
>
> Signed-off-by: Tapani Pälli
Reviewed-by:
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 optimise
> >> all
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
On 9 December 2015 at 05:37, 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
On Wed, Dec 9, 2015 at 8:06 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> The old function didn't work for matrices, and we need this
> in other places to fix some other problems, so move to a helper
> in glsl type and fix the one user so far.
>
> A dual
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
On 9 December 2015 at 11:39, 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 optimise
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
index 8d24883..36a7329 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
+++
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 13c8c36..78f2c8c 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
+++
On Dec 8, 2015 10:08 PM, "Nicolai Hähnle" wrote:
>
> On 06.12.2015 19:00, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> This simplified (basically duplicated) version of pb_cache_manager will
>> allow removing some ugly hacks from radeon and amdgpu
Jordan Justen writes:
> On 2015-12-08 08:43:53, Francisco Jerez wrote:
>> This is going to require some rather intrusive kernel changes to fix
>> properly, in the meantime (and forever on at least pre-v4.1 kernels)
>> we'll have to restore the hardware defaults at the
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
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
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
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
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);
>> -
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
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
>
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/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
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
---
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
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)) {
>>> -
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
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
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
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
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:
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.
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
---
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(-)
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
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
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
>
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
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 +
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
>>
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
https://bugs.freedesktop.org/show_bug.cgi?id=90821
Nanley Chery changed:
What|Removed |Added
Status|NEW |RESOLVED
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, );
> pipe->set_vertex_buffers(pipe, 1, 1, );
> pipe->set_vertex_buffers(pipe, 2, 1, );
>
> num_vtxbufs is now 3 as it should be.
>
> Now you
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
>
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
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
Ok, first of all bind some buffers:
pipe->set_vertex_buffers(pipe, 0, 1, );
pipe->set_vertex_buffers(pipe, 1, 1, );
pipe->set_vertex_buffers(pipe, 2, 1, );
num_vtxbufs is now 3 as it should be.
Now you are unbinding buffers, one after another starting at slot 0:
pipe->set_vertex_buffers(pipe,
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
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.
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
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
---
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
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 +-
>
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
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
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
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
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
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
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
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
---
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
---
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
---
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
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
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
...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
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
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
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,
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
---
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
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
>
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
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
+++
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,
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
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
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;
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
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
1 - 100 of 130 matches
Mail list logo