[Mesa-dev] [PATCH 2/3] freedreno: change to goto fail

2015-12-06 Thread Serge Martin
in fd_resource_transfer_map, like the others error cases --- src/gallium/drivers/freedreno/freedreno_resource.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c b/src/gallium/drivers/freedreno/freedreno_resource.c index

[Mesa-dev] [PATCH 3/3] freedreno: little clean up in fd_create_surface

2015-12-06 Thread Serge Martin
in order to avoid returing invalid adress if CALLOC_STRUCT return NULL. --- src/gallium/drivers/freedreno/freedreno_surface.c | 31 --- 1 file changed, 16 insertions(+), 15 deletions(-) diff --git a/src/gallium/drivers/freedreno/freedreno_surface.c

[Mesa-dev] [PATCH 1/3] freedreno: fix bind_sampler_states when hwcso is NULL

2015-12-06 Thread Serge Martin
src/gallium/tests/trivial/compute.c expects samplers to be cleaned when the samplers list is NULL. Like in radeon, the function behave like when the number of samplers parameter is set to 0. --- src/gallium/drivers/freedreno/a2xx/fd2_texture.c | 3 +++

Re: [Mesa-dev] [Pull Request]: Coccinelle cleanups

2015-12-06 Thread Marek Olšák
Pushed, thanks. Marek On Sun, Dec 6, 2015 at 2:40 PM, wrote: > The following changes since commit 7e43a280793af50e826fb6183670de8e3d427d74: > > gallium/radeon*: Remove useless casts (2015-12-06 11:52:16 +0100) > > are available in the git repository at: > >

[Mesa-dev] [Bug 92850] Segfault loading War Thunder

2015-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92850 --- Comment #48 from higu...@gmx.net --- -O3 -fno-inline-small-functions also works for me -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug.

[Mesa-dev] [PATCH demos 2/2] demos: add missing binaries to .gitignore

2015-12-06 Thread Andreas Boll
Signed-off-by: Andreas Boll --- src/tests/.gitignore | 1 + src/trivial/.gitignore | 1 + 2 files changed, 2 insertions(+) diff --git a/src/tests/.gitignore b/src/tests/.gitignore index 8f9cb58..1e86e68 100644 --- a/src/tests/.gitignore +++ b/src/tests/.gitignore

[Mesa-dev] [PATCH demos 1/2] util/glstate.[ch]: Update FSF address in licence header

2015-12-06 Thread Andreas Boll
Found via the following Lintian (Debian package checker) warning: The /usr/share/doc/pkg/copyright file refers to the old postal address of the Free Software Foundation (FSF). The new address is: Free Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA.

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

2015-12-06 Thread Patrick Rudolph
Always reset the vertex bufctx to make sure there's no pointer to an already freed pipe_resource left after unbinding buffers. Fixes use after free crash in nvc0_bufctx_fence(). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93004 Signed-off-by: Patrick Rudolph ---

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

2015-12-06 Thread Patrick Rudolph
In case a state tracker unbinds every slot by a seperate pipe->set_vertex_buffers() call, starting from slot zero, the number of bound buffers would not reach zero at all. Unbinding all buffers at once or starting at the top-most slot results in correct behaviour. Calculating the correct number

Re: [Mesa-dev] [PATCH] gallium/radeon*: Remove useless casts

2015-12-06 Thread Marek Olšák
Pushed, thanks. Marek On Sun, Dec 6, 2015 at 8:37 AM, Edward O'Callaghan wrote: > These are unnecessary and are likely just left overs from prior > work. > > Signed-off-by: Edward O'Callaghan > --- >

[Mesa-dev] [Bug 93271] libgles2 exports new symbols

2015-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93271 Bug ID: 93271 Summary: libgles2 exports new symbols Product: Mesa Version: unspecified Hardware: Other OS: All Status: NEW Severity: normal

[Mesa-dev] [Pull Request]: Coccinelle cleanups

2015-12-06 Thread eocallaghan
The following changes since commit 7e43a280793af50e826fb6183670de8e3d427d74: gallium/radeon*: Remove useless casts (2015-12-06 11:52:16 +0100) are available in the git repository at: https://github.com/victoredwardocallaghan/mesa-GLwork.git coccinelle for you to fetch changes up to

Re: [Mesa-dev] [PATCH] gallium: Remove redundant NULL ptr checks

2015-12-06 Thread Marek Olšák
This doesn't apply, because master has "if(map)", but your patch has "if (map)". Marek On Sun, Dec 6, 2015 at 6:00 AM, Edward O'Callaghan wrote: > Signed-off-by: Edward O'Callaghan > --- >

Re: [Mesa-dev] [PATCH][mesa-demos] configure.ac: fix AC_WITH(glut) so that --without-glut works

2015-12-06 Thread Andreas Boll
Reviewed-by: Andreas Boll Do you need someone to push this for you? 2015-11-11 12:39 GMT+01:00 Ross Burton : > Currently if --without-glut is used on a system that has the GLUT libraries > installed, GLUT is used regardless. > > Change the

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

2015-12-06 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sun, Dec 6, 2015 at 10:12 AM, Patrick Rudolph wrote: > In case a state tracker unbinds every slot by a seperate > pipe->set_vertex_buffers() call, starting from slot zero, the number > of bound buffers would not reach

Re: [Mesa-dev] [PATCH] mesa-demos: xeglthreads needs to add "-lX11 -lpthread" itself

2015-12-06 Thread Andreas Boll
Is this still an issue? Could you give us the build log of a verbose build? Thanks, Andreas 2015-08-21 10:37 GMT+02:00 Stefan Dirsch : > Hi > > This is just a simple buildfix for current mesa-demos git master branch. > > Thanks, > Stefan > > Author: Stefan Dirsch

Re: [Mesa-dev] [PATCH 0/7] DRI3 support for EGL (v4)

2015-12-06 Thread Mike Lothian
Hi Have you had any success with this? Cheers Mike On Wed, 18 Nov 2015 at 02:27 Boyan Ding wrote: > 2015-11-18 8:04 GMT+08:00 Martin Peres : > > > > > > On 18/11/15 01:37, Mike Lothian wrote: > >> > >> > >> Hi > >> > >> Out of interest have any

[Mesa-dev] [PATCH] r600g: fix outputing to non-0 buffers for stream 0.

2015-12-06 Thread Dave Airlie
From: Dave Airlie This fixes: arb_transform_feedback3-ext_interleaved_two_bufs_gs arb_transform_feedback3-ext_interleaved_two_bufs_gs_max transform-feedback-builtins If we are only emitting one ring, then emit all output buffers on it. ---

[Mesa-dev] [PATCH] mesa: enable tessellation on GL 4.0 and above

2015-12-06 Thread Timothy Arceri
--- src/mesa/main/context.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h index 8b64f45..f3d42cb 100644 --- a/src/mesa/main/context.h +++ b/src/mesa/main/context.h @@ -361,7 +361,7 @@ static inline GLboolean

[Mesa-dev] [Bug 93278] configure wants libdrm_intel

2015-12-06 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93278 Bug ID: 93278 Summary: configure wants libdrm_intel Product: Mesa Version: 10.5 Hardware: ARM OS: Linux (All) Status: NEW Severity: normal

Re: [Mesa-dev] [PATCH] mesa: enable tessellation on GL 4.0 and above

2015-12-06 Thread Ilia Mirkin
On Sun, Dec 6, 2015 at 10:05 PM, Timothy Arceri wrote: > --- > src/mesa/main/context.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h > index 8b64f45..f3d42cb 100644 > ---

[Mesa-dev] [PATCH 1/2] program: add _mesa_reserve_parameter_storage

2015-12-06 Thread Marek Olšák
From: Marek Olšák The next commit will use this. --- src/mesa/program/prog_parameter.c | 47 ++- src/mesa/program/prog_parameter.h | 4 2 files changed, 36 insertions(+), 15 deletions(-) diff --git

[Mesa-dev] [PATCH 03/10] winsys/amdgpu: use pb_cache instead of pb_cache_manager

2015-12-06 Thread Marek Olšák
From: Marek Olšák This is a prerequisite for the removal of radeon_winsys_cs_handle. --- src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 209 -- src/gallium/winsys/amdgpu/drm/amdgpu_bo.h | 14 +- src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c |

[Mesa-dev] [PATCH 05/10] winsys/radeon: remove redundant radeon_bomgr::va

2015-12-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c b/src/gallium/winsys/radeon/drm/radeon_drm_bo.c index 7f395b7..7f46955 100644

[Mesa-dev] [PATCH 08/10] winsys/radeon: use radeon_bomgr less

2015-12-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 23 --- src/gallium/winsys/radeon/drm/radeon_drm_bo.h | 1 - 2 files changed, 8 insertions(+), 16 deletions(-) diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c

[Mesa-dev] [PATCH 07/10] winsys/radeon: rename radeon_bomgr_init_functions

2015-12-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 2 +- src/gallium/winsys/radeon/drm/radeon_drm_bo.h | 2 +- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git

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

2015-12-06 Thread Marek Olšák
From: Marek Olšák This simplified (basically duplicated) version of pb_cache_manager will allow removing some ugly hacks from radeon and amdgpu winsyses and flatten simplify their design. The difference is that winsyses must manually add buffers to the cache in "destroy"

[Mesa-dev] [PATCH 10/10] gallium/radeon: remove radeon_winsys_cs_handle

2015-12-06 Thread Marek Olšák
From: Marek Olšák "radeon_winsys_cs_handle *cs_buf" is now equivalent to "pb_buffer *buf". --- src/gallium/drivers/r300/r300_context.h | 4 src/gallium/drivers/r300/r300_cs.h | 4 ++-- src/gallium/drivers/r300/r300_emit.c| 22

[Mesa-dev] [PATCH 06/10] winsys/radeon: move variables from radeon_bomgr to radeon_drm_winsys

2015-12-06 Thread Marek Olšák
From: Marek Olšák radeon_bomgr is going away. --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 210 +- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 29 +++ src/gallium/winsys/radeon/drm/radeon_drm_winsys.h | 16 ++ 3 files changed, 129

[Mesa-dev] [PATCH 04/10] winsys/amdgpu: don't use the "rws" abbreviation for amdgpu_winsys

2015-12-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 28 +-- src/gallium/winsys/amdgpu/drm/amdgpu_bo.h | 2 +- src/gallium/winsys/amdgpu/drm/amdgpu_cs.h | 2 +- src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 8 4

[Mesa-dev] [PATCH 09/10] winsys/radeon: use pb_cache instead of pb_cache_manager

2015-12-06 Thread Marek Olšák
From: Marek Olšák This is a prerequisite for the removal of radeon_winsys_cs_handle. --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 212 +++--- src/gallium/winsys/radeon/drm/radeon_drm_bo.h | 14 +-

[Mesa-dev] [PATCH 02/10] gallium/pb_bufmgr_cache: use the new pb_cache module

2015-12-06 Thread Marek Olšák
From: Marek Olšák --- src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c | 232 +++-- 1 file changed, 34 insertions(+), 198 deletions(-) diff --git a/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c b/src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c

[Mesa-dev] [PATCH 00/10] Rework of pb_cache_manager for removal of radeon_winsys_cs_handle

2015-12-06 Thread Marek Olšák
Hi, This series addresses the weirdness in radeon drivers that every buffer has 2 handles: - pb_buffer (== pb_cache_buffer) - radeon_winsys_cs_handle (winsys-specific pb_buffer) The inefficiency of converting pb_cache_buffer into the winsys-specific buffer made me introduce

[Mesa-dev] [PATCH] r600g: fix geom shader input indirect indexing.

2015-12-06 Thread Dave Airlie
From: Dave Airlie This fixes: gs-input-array-vec4-index-rd The others run out of gprs unfortunately. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/r600_shader.c | 32 ++-- 1 file changed, 30 insertions(+), 2

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

2015-12-06 Thread Marek Olšák
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. --- src/mesa/state_tracker/st_cb_bitmap.c | 6 +-

Re: [Mesa-dev] [PATCH] mesa: enable tessellation on GL 4.0 and above

2015-12-06 Thread Timothy Arceri
On Sun, 2015-12-06 at 22:07 -0500, Ilia Mirkin wrote: > On Sun, Dec 6, 2015 at 10:05 PM, Timothy Arceri > wrote: > > --- > > src/mesa/main/context.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/mesa/main/context.h

Re: [Mesa-dev] [PATCH] mesa: enable tessellation on GL 4.0 and above

2015-12-06 Thread Ilia Mirkin
On Sun, Dec 6, 2015 at 10:19 PM, Timothy Arceri wrote: > On Sun, 2015-12-06 at 22:07 -0500, Ilia Mirkin wrote: >> On Sun, Dec 6, 2015 at 10:05 PM, Timothy Arceri >> wrote: >> > --- >> > src/mesa/main/context.h | 2 +- >> > 1 file

Re: [Mesa-dev] r600 tess branches updated

2015-12-06 Thread Grazvydas Ignotas
Hi, On Fri, Dec 4, 2015 at 6:19 AM, Dave Airlie wrote: > Hey all, > > I've pushed an updated version of the r600g tess support to my > r600g-tess-submit branch. FWIW: Tested-by: Grazvydas Ignotas on JUNIPER XT with heaven and piglit, no issues noticed.

Re: [Mesa-dev] [PATCH 2/2] mesa: move GL_INVALID_OPERATION error to rendering call

2015-12-06 Thread Tapani Pälli
Yes, nice cleanup Reviewed-by: Tapani Pälli On 12/06/2015 06:36 AM, Timothy Arceri wrote: The validation api doesn't trigger this error so just move it to the code called during rendering. Cc: Tapani Pälli Cc: Kenneth Graunke

Re: [Mesa-dev] [PATCH] glsl: re-validate program pipeline after sampler change

2015-12-06 Thread Tapani Pälli
Reviewed-by: Tapani Pälli On 12/06/2015 05:42 AM, Timothy Arceri wrote: Cc: "11.1" Cc: Tapani Pälli Cc: Kenneth Graunke https://bugs.freedesktop.org/show_bug.cgi?id=93180 ---

Re: [Mesa-dev] [PATCH 1/2] mesa: move pipeline input/output validation inside _mesa_validate_program_pipeline()

2015-12-06 Thread Tapani Pälli
On 12/07/2015 08:12 AM, Tapani Pälli wrote: Does this fix something in the remaining OpenGL ES 3.1 CTS failures? If Oops, yes it does. Looks like we need this now, let's try to optimize it a bit more light-weight later. One thing that will make it tiny bit faster is when resource list will

[Mesa-dev] [PATCH] r600: handle geometry dynamic input array index

2015-12-06 Thread Dave Airlie
From: Dave Airlie This fixes: glsl-1.50/execution/geometry/dynamic_input_array_index.shader_test my profanity. We need to load the AR register with the value from the index reg Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/r600_shader.c | 13

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

2015-12-06 Thread Michel Dänzer
On 06.12.2015 13:58, Jonathan Gray wrote: > On Tue, Dec 01, 2015 at 04:56:54PM +, Emil Velikov wrote: >> Hi Jonathan, >> >> On 23 November 2015 at 03:24, Jonathan Gray wrote: >>> Use PKG_CHECK_MODULES to get the flags to link libelf. >>> >>> Signed-off-by: Jonathan Gray

Re: [Mesa-dev] [PATCH] mesa: enable tessellation on GL 4.0 and above

2015-12-06 Thread Timothy Arceri
On Sun, 2015-12-06 at 22:26 -0500, Ilia Mirkin wrote: > On Sun, Dec 6, 2015 at 10:19 PM, Timothy Arceri > wrote: > > On Sun, 2015-12-06 at 22:07 -0500, Ilia Mirkin wrote: > > > On Sun, Dec 6, 2015 at 10:05 PM, Timothy Arceri > > >

Re: [Mesa-dev] [PATCH 1/2] mesa: move pipeline input/output validation inside _mesa_validate_program_pipeline()

2015-12-06 Thread Tapani Pälli
Does this fix something in the remaining OpenGL ES 3.1 CTS failures? If not I think we should think about this more, how can we move this (maybe just partially) to happen already during UseProgramStages, it feels too expensive for draw time. FYI I'm working on to fix remaining SSO CTS