Re: [Mesa-dev] [PATCH 00/26] i965: Tessellation shaders for Gen8+!

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

[Mesa-dev] [PATCH] gallium/aux../util: Make u_prims_for_vertices() safe

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

[Mesa-dev] [Bug 91888] EGL Wayland software rendering no longer work after regression

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.

Re: [Mesa-dev] [PATCH] clover: Fix build against LLVM 3.8 SVN >= r255078

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

Re: [Mesa-dev] [PATCH 1/8] glsl: pass stage into mark function

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

[Mesa-dev] [Bug 1626] X server should not poll() on DRM fd

https://bugs.freedesktop.org/show_bug.cgi?id=1626 Michel Dänzer changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH v4 15/44] i965: Work around L3 state leaks during context switches.

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

Re: [Mesa-dev] [PATCH] svga: initialize pipe_driver_query_info entries with a macro

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

Re: [Mesa-dev] [PATCH] Fix locking of GLsync objects

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

[Mesa-dev] [PATCH] i965: Separate base offset/constant offset combining from remapping.

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(-)

Re: [Mesa-dev] [Mesa-stable] [PATCH v2] configure.ac: use pkg-config for libelf

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:

Re: [Mesa-dev] [PATCH 7/8] glsl: fix transform feedback for 64-bit outupts.

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

[Mesa-dev] [RFC PATCH 5/5] i965: Skip execution size adjustment for instructions of width 4

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

[Mesa-dev] [RFC PATCH 3/5] i965/eu: set execution size for SEND message in brw_send_indirect_message

--- 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 +++

[Mesa-dev] [RFC PATCH 1/5] i965/eu: set correct execution size in brw_NOP

--- 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 +++

[Mesa-dev] [RFC PATCH 0/5] Skip automatic execsize for instructions with a width of 4

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

[Mesa-dev] [Bug 91888] EGL Wayland software rendering no longer work after regression

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.

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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 >>

Re: [Mesa-dev] [PATCH 3/8] glsl: use dual slot helper in the linker code.

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

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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: >> >>

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965: Fix crash when calling glViewport with no surface bound

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

Re: [Mesa-dev] [PATCH 1/2] glsl: do not loose always_active_io when packing varyings

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:

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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

Re: [Mesa-dev] [RFC] i965: Resolve color for all active shader images in intel_update_state().

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

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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

Re: [Mesa-dev] [PATCH 2/8] glsl/fp64: add helper for dual slot double detection.

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

Re: [Mesa-dev] [PATCH 8/8] glsl: only divide left components when it is a dual slot double.

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

Re: [Mesa-dev] [Mesa-stable] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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

[Mesa-dev] [RFC PATCH 2/5] i965/fs: set execution size for SEND messages in generate_uniform_pull_constant_load_gen7

--- 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 +++

[Mesa-dev] [RFC PATCH 4/5] i965: set correct execsize for MOVS with a width of 4 in brw_find_live_channel

--- 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 +++

Re: [Mesa-dev] [PATCH 01/10] gallium/pb_cache: add a copy of cache bufmgr independent of pb_manager

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

Re: [Mesa-dev] [PATCH v4 15/44] i965: Work around L3 state leaks during context switches.

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

[Mesa-dev] [PATCH] gallium/util: handle patches in u_prims_for_vertices to fix a radeonsi crash

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

Re: [Mesa-dev] [PATCH shader-db 1/3] split-to-files: deal with minimum versions, other shader types

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

[Mesa-dev] [Bug 91888] EGL Wayland software rendering no longer work after regression

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] [RFC] glapi: Build gl_gentable.c only on Darwin

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

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965: Fix crash when calling glViewport with no surface bound

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); >> -

Re: [Mesa-dev] [PATCH] i965: Separate base offset/constant offset combining from remapping.

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

Re: [Mesa-dev] [RFC PATCH 5/5] i965: Skip execution size adjustment for instructions of width 4

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 >

[Mesa-dev] [Bug 91888] EGL Wayland software rendering no longer work after regression

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.

[Mesa-dev] [PATCH 4/4] st/osmesa: Fix a typo in a comment

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

[Mesa-dev] [PATCH 3/4] meta: Fix a typo in a print message

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 ---

Re: [Mesa-dev] softpipe: V.2 implement some support for multiple viewports

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

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965: Fix crash when calling glViewport with no surface bound

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)) { >>> -

[Mesa-dev] [PATCH 2/4] mesa: Fix typos in print messages

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

Re: [Mesa-dev] [PATCH] i965: handle stencil_bits parameter for MESA_FORMAT_B8G8R8X8_UNORM format.

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

[Mesa-dev] [PATCH 2/2] mesa: Fix a typo in a comment

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

[Mesa-dev] [PATCH 1/2] glx: Fix a typo in a comment

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

Re: [Mesa-dev] [PATCH] i965: handle stencil_bits parameter for MESA_FORMAT_B8G8R8X8_UNORM format.

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:

Re: [Mesa-dev] [PATCH] Fix locking of GLsync objects

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.

[Mesa-dev] [PATCH 1/4] glsl: Fix a typo in a comment

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 ---

Re: [Mesa-dev] [PATCH 1/2] glx: Fix a typo in a comment

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(-)

Re: [Mesa-dev] [PATCH 1/4] glsl: Fix a typo in a comment

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

Re: [Mesa-dev] [RFC] glapi: Build gl_gentable.c only on Darwin

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

Re: [Mesa-dev] [PATCH 2/2] util/u_helpers: return correct number of bound buffers

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 >

Re: [Mesa-dev] [RFC] glapi: Build gl_gentable.c only on Darwin

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

Re: [Mesa-dev] [PATCH] gallium/util: handle patches in u_prims_for_vertices to fix a radeonsi crash

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 +

Re: [Mesa-dev] [PATCH] i965: handle stencil_bits parameter for MESA_FORMAT_B8G8R8X8_UNORM format.

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 >>

Re: [Mesa-dev] [PATCH 1/2] nvc0: fix use after free of pipe_resource

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

[Mesa-dev] [Bug 90821] Segfault when calling glViewport on surfaceless EGL context without bound FBO

https://bugs.freedesktop.org/show_bug.cgi?id=90821 Nanley Chery changed: What|Removed |Added Status|NEW |RESOLVED

Re: [Mesa-dev] [PATCH 2/2] util/u_helpers: return correct number of bound buffers

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

Re: [Mesa-dev] [PATCH] configure.ac: fix test for SSE4.1 assembler support

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 >

Re: [Mesa-dev] [RFC PATCH 1/5] i965/eu: set correct execution size in brw_NOP

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

Re: [Mesa-dev] softpipe: V.2 implement some support for multiple viewports

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

Re: [Mesa-dev] [PATCH] mesa: detect inefficient buffer use and report through debug output

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

Re: [Mesa-dev] [PATCH 2/2] util/u_helpers: return correct number of bound buffers

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-dev] Mesa 11.0.7

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

Re: [Mesa-dev] [PATCH] mesa: detect inefficient buffer use and report through debug output

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.

[Mesa-dev] [PATCH] radeonsi: implement RB+ for Stoney (v2)

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

[Mesa-dev] [PATCH] nvc0: optimize coherent buffer checking at draw time

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 ---

Re: [Mesa-dev] [PATCH 14/26] i965: Add Gen8+ tessellation evaluation shader state (3DSTATE_DS).

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

Re: [Mesa-dev] [PATCH 12/26] i965: Add tessellation shader sampler support.

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 +- >

[Mesa-dev] [ANNOUNCE] mesa-demos 8.3.0

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

[Mesa-dev] [PATCH] radeonsi: don't call of u_prims_for_vertices for patches and rectangles

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

Re: [Mesa-dev] [PATCH 2/2] st/mesa: fix GLSL uniform updates for glBitmap & glDrawPixels

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

Re: [Mesa-dev] [PATCH 15/26] i965: Add Gen7+ tessellation engine state (3DSTATE_TE).

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

[Mesa-dev] [PATCH v2] nv50, nvc0: optimize coherent buffer checking at draw time

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

Re: [Mesa-dev] [PATCH] gallium/aux../util: Make u_prims_for_vertices() safe

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

[Mesa-dev] [PATCH 3/4] winsys/radeon: clear the buffer cache on mmap failure and try again

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

[Mesa-dev] [PATCH 2/4] winsys/amdgpu: clear the buffer cache on allocation failure and try again

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 ---

[Mesa-dev] [PATCH 4/4] winsys/amdgpu: clear the buffer cache on mmap failure and try again

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 ---

[Mesa-dev] [PATCH 1/4] winsys/radeon: clear the buffer cache on allocation failure and try again

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 ---

Re: [Mesa-dev] softpipe: V.2 implement some support for multiple viewports

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

Re: [Mesa-dev] [PATCH] mesa: detect inefficient buffer use and report through debug output

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

Re: [Mesa-dev] [PATCH] i965: Separate base offset/constant offset combining from remapping.

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

Re: [Mesa-dev] [PATCH] mesa/varray: set double arrays to non-normalised.

...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

Re: [Mesa-dev] [PATCH] i965: Separate base offset/constant offset combining from remapping.

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

[Mesa-dev] [PATCH 1/2] gallium/ddebug: add GALLIUM_DDEBUG_SKIP option

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

[Mesa-dev] [PATCH 2/2] gallium/ddebug: regularly log the total number of draw calls

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,

[Mesa-dev] [PATCH] radeonsi: also print hexadecimal values for register fields in the IB parser

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 ---

Re: [Mesa-dev] [PATCH] mesa: fix ID usage for buffer warnings

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

Re: [Mesa-dev] [PATCH v2] nv50, nvc0: optimize coherent buffer checking at draw time

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 >

Re: [Mesa-dev] [PATCH 8/8] glsl: only divide left components when it is a dual slot double.

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

[Mesa-dev] [PATCH] mesa: fix ID usage for buffer warnings

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 +++

[Mesa-dev] [Bug 91888] EGL Wayland software rendering no longer work after regression

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,

[Mesa-dev] [PATCH] mesa/varray: set double arrays to non-normalised.

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

Re: [Mesa-dev] [PATCH 1/3] mesa: docs: Add link to planet.freedesktop.org

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

Re: [Mesa-dev] [PATCH 2/3] mesa: docs: i965: Use correct doxygen groupings syntax

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;

Re: [Mesa-dev] [PATCH 03/15] i965/fs: Use MOV_INDIRECT for all indirect uniform loads

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

Re: [Mesa-dev] [PATCH v3 44/44] docs: Add ARB_compute_shader to 11.2.0 release notes

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   2   >