Re: [Mesa-dev] [PATCH] intel: Fix initial MakeCurrent for single-buffer drawables

2014-01-27 Thread Zhang, Xiong Y
hi, Kristian: This patch block the startup of desktop-shell client in weston. Panel surface and background surface share a common context. First when client redraw panel surface, _mesa_make_current will set brw->ctx.ViewportInitialized to true. Later when client redraw background surface

[Mesa-dev] [PATCH] Correct test for depth parameter for checking if dimension is legal

2014-01-27 Thread Kevin Rogovin
Fixes the tests for the depth parameter for TexImage3D calls when the target type is GL_TEXTURE_2D_ARRAY or GL_TEXTURE_CUBE_MAP_ARRAY so that a depth value of 0 is accepted. Previously, the check incorrectly required the depth argument to be atleast 1. --- src/mesa/main/teximage.c | 4 ++-- 1 fi

[Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Topi Pohjolainen
This is really not needed as blorp blit programs already sample XRGB normally and get alpha channel set to 1.0 automatically by the sampler engine. This is simply copied directly to the payload of the render target write message and hence there is no need for any additional blending support from th

Re: [Mesa-dev] [PATCH 14/16] radeonsi: We don't support indirect addressing of shader inputs/outputs

2014-01-27 Thread Marek Olšák
The function "emit_fetch" in radeon_setup_tgsi_llvm.c seems to implement indirect addressing of everything but constants. Marek On Mon, Jan 27, 2014 at 3:14 AM, Michel Dänzer wrote: > On Fre, 2014-01-24 at 23:54 +0100, Marek Olšák wrote: >> Please can this be done for the geometry shader only? >

[Mesa-dev] [Bug 73946] scanout broken on radeon SI (OLAND)

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73946 --- Comment #9 from Marek Olšák --- (In reply to comment #8) > (In reply to comment #7) > > Could you tell me at which point exactly you added the PIPE_BIND_SCANOUT > > flag? > > I didn't. I modified > src/gallium/drivers/radeon/r600_texture.c:r

Re: [Mesa-dev] [PATCH] gallium/rtasm: handle mmap failures appropriately

2014-01-27 Thread Emil Velikov
Ping :) On 19/01/14 23:53, Emil Velikov wrote: > For a variety of reasons mmap (selinux and pax to name > a few) and can fail and with current code. This will > result in a crash in the driver, if not worse. > > This has been the case since the inception of the > gallium copy of rtasm. > > Cc: 9

[Mesa-dev] [Bug 73631] es2tri fragment shader did not compile AMD ARUBA

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73631 Emil Velikov changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH] gallium/rtasm: handle mmap failures appropriately

2014-01-27 Thread Jakob Bornecrantz
On Mon, Jan 27, 2014 at 12:57 PM, Emil Velikov wrote: > Ping :) > > On 19/01/14 23:53, Emil Velikov wrote: >> For a variety of reasons mmap (selinux and pax to name >> a few) and can fail and with current code. This will >> result in a crash in the driver, if not worse. >> >> This has been the cas

Re: [Mesa-dev] [PATCH demos] opengles2/es2tri: add precision qualifier to the fragment shader

2014-01-27 Thread Emil Velikov
On 21/01/14 17:26, Emil Velikov wrote: > On 20/01/14 15:16, Brian Paul wrote: >> On 01/19/2014 08:34 AM, Emil Velikov wrote: >>> The missing qualifier causes failure during the compilation stage. >>> >>> Bugzilla: >>> https://urldefense.proofpoint.com/v1/url?u=https://bugs.freedesktop.org/show_bug.

Re: [Mesa-dev] [PATCH] loader: Get driver name from udev hwdb when available

2014-01-27 Thread Emil Velikov
Hi Kristian, On 20/01/14 20:42, Kristian Høgsberg wrote: > The udev hwdb is a mechanism for applying udev properties to devices at > hotplug time. The hwdb text files are compiled into a binary database > that lets udev efficiently look up and apply properties to devices that > match a given moda

Re: [Mesa-dev] [PATCH 5/7] dri: Fix the logger error message handling.

2014-01-27 Thread Emil Velikov
On 23/01/14 23:12, Eric Anholt wrote: > Since the loader changes, there has been a compiler warning that the > prototype didn't match. It turns out that if a loader error message was > ever thrown, you'd segfault because of trying to use the warning level as > a format string. Truly sorry about th

[Mesa-dev] [PATCH] loader: Add missing \n on message printing

2014-01-27 Thread Emil Velikov
Cover both loader and glx/dri_glx Drop \n from the default loader logger Signed-off-by: Emil Velikov --- Hi Eric The missing \n was inherited from the dri3_common.c which on itself seems to have come from from the legacy dri_glx.c codebase. This patch should cover all the leftout cases. Feel fr

Re: [Mesa-dev] [PATCH 02/16] radeonsi: Initial geometry shader support

2014-01-27 Thread Marek Olšák
On Sat, Jan 25, 2014 at 2:21 AM, Marek Olšák wrote: > On Fri, Jan 24, 2014 at 10:19 PM, Marek Olšák wrote: >> 1) Do we really have to have so much code inside #if...#endif? Can we >> always compile everything and just not set the pipe_context functions >> if LLVM < 3.5 and the same for get_(shade

[Mesa-dev] [Bug 74098] [clover] adaptor.hpp:47:16: error: function cannot return function type 'std::vector > (unsigned long, unsigned long)'

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74098 --- Comment #1 from Tom Stellard --- Which version of clang were you using? -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-dev@lists.freedeskt

[Mesa-dev] [PATCH] r600g/compute: Emit DEALLOC_STATE on cayman after dispatching a compute shader.

2014-01-27 Thread Tom Stellard
From: Tom Stellard This is necessary to prevent the next SURFACE_SYNC packet from hanging the GPU. https://bugs.freedesktop.org/show_bug.cgi?id=73418 CC: "9.2" "10.0" --- src/gallium/drivers/r600/evergreen_compute.c | 9 - src/gallium/drivers/r600/evergreend.h| 1 + src/gall

Re: [Mesa-dev] [PATCH] r600g/compute: Emit DEALLOC_STATE on cayman after dispatching a compute shader.

2014-01-27 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, Jan 27, 2014 at 3:05 PM, Tom Stellard wrote: > From: Tom Stellard > > This is necessary to prevent the next SURFACE_SYNC packet from > hanging the GPU. > > https://bugs.freedesktop.org/show_bug.cgi?id=73418 > > CC: "9.2" "10.0" > --- > src/gallium/drive

Re: [Mesa-dev] [PATCH] R600/SI: Add pattern for truncating i32 to i1

2014-01-27 Thread Tom Stellard
On Mon, Jan 27, 2014 at 04:43:14PM +0900, Michel Dänzer wrote: > On Fre, 2014-01-24 at 07:40 -0800, Tom Stellard wrote: > > On Fri, Jan 24, 2014 at 01:27:00PM +0900, Michel Dänzer wrote: > > > From: Michel Dänzer > > > > > > Fixes half a dozen piglit tests with radeonsi. > > > > > > Signed-off-b

Re: [Mesa-dev] [PATCH] r600g/compute: Emit DEALLOC_STATE on cayman after dispatching a compute shader.

2014-01-27 Thread Alex Deucher
On Mon, Jan 27, 2014 at 9:05 AM, Tom Stellard wrote: > From: Tom Stellard > > This is necessary to prevent the next SURFACE_SYNC packet from > hanging the GPU. > > https://bugs.freedesktop.org/show_bug.cgi?id=73418 > > CC: "9.2" "10.0" Reviewed-by: Alex Deucher > --- > src/gallium/drivers/r

[Mesa-dev] [PATCH 1/2] pipe-loader: Add auth_x parameter to pipe_loader_drm_probe_fd()

2014-01-27 Thread Tom Stellard
From: Tom Stellard The caller can use this boolean to parameter to tell the pipe-loader to authenticate with the X server when probing a file descriptor. --- src/gallium/auxiliary/pipe-loader/pipe_loader.h | 6 +- src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 8 +--- src/gall

[Mesa-dev] [PATCH 2/2] pipe-loader: Add support for render nodes v2

2014-01-27 Thread Tom Stellard
From: Tom Stellard v2: - Add missing call to pipe_loader_drm_release() - Fix render node macros - Drop render-node configure option --- For reference, version 1 of this patch: http://lists.freedesktop.org/archives/mesa-dev/2013-October/047296.html .../auxiliary/pipe-loader/pipe_loader

Re: [Mesa-dev] [PATCH 1/2] pipe-loader: Add auth_x parameter to pipe_loader_drm_probe_fd()

2014-01-27 Thread Erik Faye-Lund
On Mon, Jan 27, 2014 at 5:13 PM, Tom Stellard wrote: > From: Tom Stellard > > The caller can use this boolean to parameter to tell the pipe-loader Boolean to parameter? A superfluous "to", perhaps? ___ mesa-dev mailing list mesa-dev@lists.freedesktop.o

Re: [Mesa-dev] [PATCH 7/7] dri: Reuse dri_message to implement our other message handlers.

2014-01-27 Thread Eric Anholt
Kristian Høgsberg writes: > On Thu, Jan 23, 2014 at 3:12 PM, Eric Anholt wrote: > > Entire series > > Reviewed-by: Kristian Høgsberg > > The only thing that came to mind was that we should move the code to > open with O_CLOEXEC to its own wrapper function. Of course there's > really no reason

[Mesa-dev] [Bug 74098] [clover] adaptor.hpp:47:16: error: function cannot return function type 'std::vector > (unsigned long, unsigned long)'

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74098 Chad Harris changed: What|Removed |Added CC||chad.har...@lmco.com -- You are receiving

Re: [Mesa-dev] [PATCH 5/7] glsl: add gl_InvocationID variable for ARB_gpu_shader5

2014-01-27 Thread Jordan Justen
Paul pointed out that that gl_InvocationID should be a system value variable. Therefore, patches 5 and 6 are self-NAK'd. -Jordan On Sun, Jan 26, 2014 at 12:52 PM, Jordan Justen wrote: > Signed-off-by: Jordan Justen > --- > src/glsl/builtin_variables.cpp | 2 ++ > src/mesa/main/mtypes.h

Re: [Mesa-dev] [PATCH] Correct test for depth parameter for checking if dimension is legal

2014-01-27 Thread Matt Turner
Okay if we make this the subject? > mesa: Allow depth = 0 parameter for TexImage3D. I'll also mark it for stable when I commit it. Reviewed-by: Matt Turner On Mon, Jan 27, 2014 at 2:16 AM, Kevin Rogovin wrote: > Fixes the tests for the depth parameter for TexImage3D calls when the > target ty

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Eric Anholt
Topi Pohjolainen writes: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence there is no need for > an

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Eric Anholt
Topi Pohjolainen writes: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence there is no need for > an

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 02:46 AM, Topi Pohjolainen wrote: > This is really not needed as blorp blit programs already sample > XRGB normally and get alpha channel set to 1.0 automatically by > the sampler engine. This is simply copied directly to the payload > of the render target write message and hence ther

[Mesa-dev] [Bug 74122] New: loader.c:71:19: fatal error: dlfcn.h: No such file or directory

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74122 Priority: medium Bug ID: 74122 Keywords: regression CC: alexandre.f.dem...@gmail.com, e...@anholt.net, kei...@keithp.com, m...@fireburn.co.uk Assignee: mesa-dev@lists.fr

[Mesa-dev] [Bug 74122] loader.c:71:19: fatal error: dlfcn.h: No such file or directory

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74122 --- Comment #1 from Eric Anholt --- Why is DRI code being built for a windows target? Sounds like scons's windows build is wrong. -- You are receiving this mail because: You are the assignee for the bug. ___

[Mesa-dev] [PATCH] glsl: Avoid combining statements from different basic blocks.

2014-01-27 Thread Matt Turner
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74113 --- src/glsl/opt_vectorize.cpp | 19 +++ 1 file changed, 19 insertions(+) diff --git a/src/glsl/opt_vectorize.cpp b/src/glsl/opt_vectorize.cpp index 9ca811a..6d2bd13 100644 --- a/src/glsl/opt_vectorize.cpp +++ b/src/glsl

[Mesa-dev] [Bug 74098] [clover] adaptor.hpp:47:16: error: function cannot return function type 'std::vector > (unsigned long, unsigned long)'

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74098 --- Comment #2 from Vinson Lee --- (In reply to comment #1) > Which version of clang were you using? $ clang++ --version Ubuntu clang version 3.4-1ubuntu2 (tags/RELEASE_34/final) (based on LLVM 3.4) Target: x86_64-pc-linux-gnu Thread model: posi

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Pohjolainen, Topi
On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: > Topi Pohjolainen writes: > > > This is really not needed as blorp blit programs already sample > > XRGB normally and get alpha channel set to 1.0 automatically by > > the sampler engine. This is simply copied directly to the payload >

Re: [Mesa-dev] What use do swap interval > 1 and OML_sync_control divisor and remainder have?

2014-01-27 Thread Ian Romanick
On 01/24/2014 04:32 AM, Pekka Paalanen wrote: > Hi, > > I am investigating what kind of Wayland protocol extensions would be > needed to support proper presentation timing. Looking at existing > works, I am wondering about two things whether they have any real use. > > Where is swap interval (e.g

Re: [Mesa-dev] [PATCH] Correct test for depth parameter for checking if dimension is legal

2014-01-27 Thread Ian Romanick
On 01/27/2014 03:16 AM, Kevin Rogovin wrote: > Fixes the tests for the depth parameter for TexImage3D calls when the > target type is GL_TEXTURE_2D_ARRAY or GL_TEXTURE_CUBE_MAP_ARRAY > so that a depth value of 0 is accepted. Previously, the check > incorrectly required the depth argument to be atle

[Mesa-dev] [PATCH 1/2] glsl: Add constant evaluation of ir_binop_bfm.

2014-01-27 Thread Matt Turner
--- src/glsl/ir_constant_expression.cpp | 17 + 1 file changed, 17 insertions(+) diff --git a/src/glsl/ir_constant_expression.cpp b/src/glsl/ir_constant_expression.cpp index f811fd1..7fa5a09 100644 --- a/src/glsl/ir_constant_expression.cpp +++ b/src/glsl/ir_constant_expression.cp

[Mesa-dev] [PATCH 2/2] glsl: Use bitfieldInsert in ldexp() lowering.

2014-01-27 Thread Matt Turner
Shaves a few instructions off of lowered ldexp(). --- src/glsl/lower_instructions.cpp | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/glsl/lower_instructions.cpp b/src/glsl/lower_instructions.cpp index 8f8d448..44a6e80 100644 --- a/src/glsl/lower_instruction

[Mesa-dev] [PATCH] nv30: don't overwrite blend color setting for r32/r16 float formats

2014-01-27 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- Untested. Happened to see this when I was reading this file (don't ask). I can't imagine the current situation to be the intended one, based on this code... src/gallium/drivers/nouveau/nv30/nv30_state_validate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[Mesa-dev] [Bug 74127] New: [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 Priority: medium Bug ID: 74127 Assignee: mesa-dev@lists.freedesktop.org Summary: [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed." S

Re: [Mesa-dev] [PATCH 1/2] glsl: Add constant evaluation of ir_binop_bfm.

2014-01-27 Thread Ian Romanick
On 01/27/2014 01:02 PM, Matt Turner wrote: > --- > src/glsl/ir_constant_expression.cpp | 17 + > 1 file changed, 17 insertions(+) > > diff --git a/src/glsl/ir_constant_expression.cpp > b/src/glsl/ir_constant_expression.cpp > index f811fd1..7fa5a09 100644 > --- a/src/glsl/ir_const

Re: [Mesa-dev] [PATCH 2/2] glsl: Use bitfieldInsert in ldexp() lowering.

2014-01-27 Thread Ian Romanick
On 01/27/2014 01:02 PM, Matt Turner wrote: > Shaves a few instructions off of lowered ldexp(). > --- > src/glsl/lower_instructions.cpp | 14 ++ > 1 file changed, 10 insertions(+), 4 deletions(-) > > diff --git a/src/glsl/lower_instructions.cpp b/src/glsl/lower_instructions.cpp > index

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Carl Worth
Matt Turner writes: > The check was in the wrong place, such that if a shader incorrectly put > a preprocessor token before the #version declaration, the version would > be resolved twice, leading to a segmentation fault when attempting to > redefine the __VERSION__ macro. This fix looks good. An

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Ian Romanick
On 01/26/2014 07:14 PM, Matt Turner wrote: > The check was in the wrong place, such that if a shader incorrectly put > a preprocessor token before the #version declaration, the version would > be resolved twice, leading to a segmentation fault when attempting to > redefine the __VERSION__ macro. >

[Mesa-dev] [PATCH] glsl: Avoid combining statements from different basic blocks.

2014-01-27 Thread Matt Turner
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74113 --- src/glsl/opt_vectorize.cpp | 35 +++ 1 file changed, 35 insertions(+) diff --git a/src/glsl/opt_vectorize.cpp b/src/glsl/opt_vectorize.cpp index ac43a29..5ad1320 100644 --- a/src/glsl/opt_vectorize.cp

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Matt Turner
On Mon, Jan 27, 2014 at 3:14 PM, Ian Romanick wrote: > On 01/26/2014 07:14 PM, Matt Turner wrote: >> The check was in the wrong place, such that if a shader incorrectly put >> a preprocessor token before the #version declaration, the version would >> be resolved twice, leading to a segmentation fa

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Ian Romanick
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/27/2014 03:34 PM, Carl Worth wrote: > Matt Turner writes: >> The check was in the wrong place, such that if a shader >> incorrectly put a preprocessor token before the #version >> declaration, the version would be resolved twice, leading to a >>

Re: [Mesa-dev] [PATCH 2/2] glcpp: Resolve implicit GLSL version to 100 if the API is ES.

2014-01-27 Thread Ian Romanick
On 01/26/2014 07:14 PM, Matt Turner wrote: > Fixes a regression since b2d1c579 where ES shaders without a #version > declaration would fail to compile if their precision declaration was > wrapped in the standard #ifdef GL_ES check. > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73978 >

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Ian Romanick
On 01/27/2014 04:18 PM, Matt Turner wrote: > On Mon, Jan 27, 2014 at 3:14 PM, Ian Romanick wrote: >> On 01/26/2014 07:14 PM, Matt Turner wrote: >>> The check was in the wrong place, such that if a shader incorrectly put >>> a preprocessor token before the #version declaration, the version would >>

Re: [Mesa-dev] [PATCH 1/2] glcpp: Check version_resolved in the proper place.

2014-01-27 Thread Carl Worth
Ian Romanick writes: >> (The first "is" should be "if"). > > That's in patch #2. :) Yeah, that's what I get for reviewing patches in the order they arrive on the list... :-) -Carl pgpWgJ3mrTEsx.pgp Description: PGP signature ___ mesa-dev mailing list

[Mesa-dev] [PATCH 1/2] r200: move driContextSetFlags(ctx) call after ctx var is initialized

2014-01-27 Thread Brian Paul
From: Brian Paul Otherwise, ctx was a garbage value. CC: "10.0" --- src/mesa/drivers/dri/r200/r200_context.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/r200/r200_context.c b/src/mesa/drivers/dri/r200/r200_context.c index a7021f2..88be4f0 1006

[Mesa-dev] [PATCH 2/2] radeon: move driContextSetFlags(ctx) call after ctx var is initialized

2014-01-27 Thread Brian Paul
From: Brian Paul CC: "10.0" --- src/mesa/drivers/dri/radeon/radeon_context.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/radeon/radeon_context.c b/src/mesa/drivers/dri/radeon/radeon_context.c index 73fd188..14dc3b2 100644 --- a/src/mesa/drivers

[Mesa-dev] [PATCH 6/6] mesa: remove unused _mesa_select_tex_object() function

2014-01-27 Thread Brian Paul
From: Brian Paul The _mesa_get_current_tex_object() function is now used everywhere that _mesa_select_tex_object() was formerly used. --- src/mesa/main/teximage.c | 28 ++-- src/mesa/main/teximage.h | 5 - 2 files changed, 6 insertions(+), 27 deletions(-) diff --git

[Mesa-dev] [PATCH 4/6] st/mesa: use _mesa_get_current_tex_object() in st_context_teximage()

2014-01-27 Thread Brian Paul
From: Brian Paul --- src/mesa/state_tracker/st_manager.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/state_tracker/st_manager.c b/src/mesa/state_tracker/st_manager.c index 8158450..ac37fa2 100644 --- a/src/mesa/state_tracker/st_manager.c +++ b/src/mesa/state

[Mesa-dev] [PATCH 1/6] r200: use _mesa_get_current_tex_object() in r200SetTexBuffer2()

2014-01-27 Thread Brian Paul
From: Brian Paul --- src/mesa/drivers/dri/r200/r200_texstate.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/r200/r200_texstate.c b/src/mesa/drivers/dri/r200/r200_texstate.c index f655bc1..311fc1f 100644 --- a/src/mesa/drivers/dri/r200/r200_texst

[Mesa-dev] [PATCH 5/6] swrast: use _mesa_get_current_tex_object() in swrastSetTexBuffer2()

2014-01-27 Thread Brian Paul
From: Brian Paul --- src/mesa/drivers/dri/swrast/swrast.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c index 30e6805..a25013a 100644 --- a/src/mesa/drivers/dri/swrast/swrast.c +++ b/src/mesa/dr

[Mesa-dev] [PATCH 3/6] mesa: use _mesa_get_current_tex_object() in GetTexLevelParameteriv()

2014-01-27 Thread Brian Paul
From: Brian Paul And update a related comment. --- src/mesa/main/texparam.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c index 7c59d11..dbcc6c8 100644 --- a/src/mesa/main/texparam.c +++ b/src/mesa/main/texparam.c

[Mesa-dev] [PATCH 2/6] radeon: use _mesa_get_current_tex_object() in radeonSetTexBuffer2()

2014-01-27 Thread Brian Paul
From: Brian Paul --- src/mesa/drivers/dri/radeon/radeon_texstate.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/radeon/radeon_texstate.c b/src/mesa/drivers/dri/radeon/radeon_texstate.c index 09a7ccb..ee87f82 100644 --- a/src/mesa/drivers/dri/rad

Re: [Mesa-dev] [PATCH 1/2] r200: move driContextSetFlags(ctx) call after ctx var is initialized

2014-01-27 Thread Alex Deucher
On Mon, Jan 27, 2014 at 3:08 PM, Brian Paul wrote: > From: Brian Paul > > Otherwise, ctx was a garbage value. > > CC: "10.0" For the series: Reviewed-by: Alex Deucher > --- > src/mesa/drivers/dri/r200/r200_context.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git

Re: [Mesa-dev] [V4 PATCH 0/7] mesa: Naming MESA_FORMATs to a specification

2014-01-27 Thread Marek Olšák
Tested with piglit and pushed. Nice work! Marek On Mon, Jan 27, 2014 at 1:10 AM, Mark Mueller wrote: > Thanks Marek, > I sent out a new set of patches, otherwise there is a branch called > NewStartAMF at fdo:~mmueller/mesa with everything in. > > Mark > > > On Sun, Jan 26, 2014 at 3:03 PM, Marek

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 --- Comment #1 from Scott Moreau --- I discovered this stems from the fact that libudev.so.1 does not exist on the system, since there is no systemd installed. -- You are receiving this mail because: You are the assignee for the bug. __

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 Emil Velikov changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 --- Comment #3 from Scott Moreau --- Should there be a configure check added to make it clear? -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 --- Comment #4 from Emil Velikov --- (In reply to comment #3) > Should there be a configure check added to make it clear? Currently we check if libudev is available during compile/build time, and build the "offending" code only if it's present.

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 --- Comment #5 from Scott Moreau --- (In reply to comment #4) > (In reply to comment #3) > > Should there be a configure check added to make it clear? > Currently we check if libudev is available during compile/build time, and > build the "offend

[Mesa-dev] [Bug 74127] [bisected] GL programs fail with "loader.c:112: asserted_dlsym: Assertion `result' failed."

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=74127 Emil Velikov changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|NOTABUG

Re: [Mesa-dev] [PATCH] i965/blorp: do not use unnecessary hw-blending support

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 11:06 AM, Pohjolainen, Topi wrote: > On Mon, Jan 27, 2014 at 10:20:54AM -0800, Eric Anholt wrote: >> Topi Pohjolainen writes: >> >>> This is really not needed as blorp blit programs already sample >>> XRGB normally and get alpha channel set to 1.0 automatically by >>> the sampler eng

[Mesa-dev] [PATCH] mesa: Generate correct error code in glDrawBuffers()

2014-01-27 Thread Anuj Phogat
OpenGL 3.3 spec expects GL_INVALID_OPERATION: "For both the default framebuffer and framebuffer objects, the constants FRONT, BACK, LEFT, RIGHT, and FRONT AND BACK are not valid in the bufs array passed to DrawBuffers, and will result in the error INVALID OPERATION." But OpenGL 4.0 spec cha

[Mesa-dev] [Bug 73946] scanout broken on radeon SI (OLAND)

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73946 --- Comment #10 from Michel Dänzer --- (In reply to comment #9) > I fixed Wayland/Weston with this patch: > > http://cgit.freedesktop.org/mesa/mesa/commit/?id=0b37737cc3e7042bffb7c8a0e6a5c822bb806977 I had that fix when I reproduced the problem

[Mesa-dev] [Bug 73746] Gallum HUD not always visible with Rust

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73746 Thomas Rohloff changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH] glsl: Avoid combining statements from different basic blocks.

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 03:17 PM, Matt Turner wrote: > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74113 > --- > src/glsl/opt_vectorize.cpp | 35 +++ > 1 file changed, 35 insertions(+) > > diff --git a/src/glsl/opt_vectorize.cpp b/src/glsl/opt_vectorize.cpp > inde

Re: [Mesa-dev] [PATCH 3/5] glcpp: Set extension defines after resolving the GLSL version.

2014-01-27 Thread Matt Turner
On Fri, Jan 17, 2014 at 9:08 PM, Matt Turner wrote: > Cc: mesa-sta...@lists.freedesktop.org Unless there are objections, I think we should probably skip this patch going to the stable branches. It wound up causing a couple of regressions, and fixed a pretty minor (at least in my opinion) issue. I

Re: [Mesa-dev] [PATCH RFC 04/11] glsl: add dead branch analysis

2014-01-27 Thread Paul Berry
On 22 January 2014 09:16, Connor Abbott wrote: > Dead branch analysis determines when the then or else branches of an > if statement will always terminate in a loop jump or return statement, > and hence once we enter that branch we will never get to the statements > after the if. This is useful f

Re: [Mesa-dev] [PATCH RFC 05/11] glsl: add loop jump visitor

2014-01-27 Thread Paul Berry
On 22 January 2014 09:16, Connor Abbott wrote: > +class ir_loop_jumps_visitor : public ir_hierarchical_visitor > +{ > +public: > + ir_loop_jumps_visitor(); > + ~ir_loop_jumps_visitor(); > + > + virtual ir_visitor_status visit_enter(ir_loop *); > + virtual ir_visitor_status visit(ir_loop_j

[Mesa-dev] [PATCH] loader: fix running with --disable-egl builds

2014-01-27 Thread Dave Airlie
From: Dave Airlie I sometimes build without EGL just for speed purposes, however it no longer finds my drivers when I do due to the HAVE_LIBUDEV defines being wrong. Signed-off-by: Dave Airlie --- configure.ac | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/configure

[Mesa-dev] [PATCH] gbm: Make libgbm.so.1 symlink.

2014-01-27 Thread Matt Turner
--- src/gbm/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gbm/Makefile.am b/src/gbm/Makefile.am index 4030cc6..de5126c 100644 --- a/src/gbm/Makefile.am +++ b/src/gbm/Makefile.am @@ -50,3 +50,4 @@ endif all-local: libgbm.la $(MKDIR_P) $(top_builddir)/$(LIB_DIR);

Re: [Mesa-dev] [PATCH] gbm: Make libgbm.so.1 symlink.

2014-01-27 Thread Kenneth Graunke
On 01/27/2014 10:19 PM, Matt Turner wrote: > --- > src/gbm/Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/gbm/Makefile.am b/src/gbm/Makefile.am > index 4030cc6..de5126c 100644 > --- a/src/gbm/Makefile.am > +++ b/src/gbm/Makefile.am > @@ -50,3 +50,4 @@ endif > all-local:

[Mesa-dev] glsl: vectorize pass probably needs to change types of scalar constants as well?

2014-01-27 Thread Aras Pranckevicius
The new vectorization pass (added in 4bd6e0d7c69) properly changes types of scalar dereferences. Doesn't it need to change types of scalar constants as well? Consider this shader: uniform sampler2D maintex; uniform float factor; varying vec2 uv; void main() { vec4 c = texture2D(maintex, uv);

[Mesa-dev] can we have gl_framebuffer _NumColorDrawBuffers = 1 and all the entries in the array NULL

2014-01-27 Thread Dave Airlie
I've just been playing with layered rendering on r600g and the depth test Marek wrote, and I noticed I'm sometimes getting a gallium framebuffer state with 1 cbuf but no defined cbuf, this comes direct from the Mesa state in the gl_framebuffer object, is this legal? Dave. ___

[Mesa-dev] [Bug 71543] [mesa] Source-based games segfault

2014-01-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71543 Fabio Pedretti changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---