[Mesa-dev] [PATCH] scons: Don't build the assembly sources on Mac OS X.

2012-02-18 Thread Vinson Lee
This patch allows the Mac OS X SCons build to complete. The assembly sources contain psuedo-ops that not are supported on Mac OS X. Signed-off-by: Vinson Lee --- src/mapi/glapi/SConscript |2 +- src/mesa/SConscript |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [Mesa-dev] [PATCH 2/2] i965: handle gl_PointCoord for Gen4 and Gen5 platform

2012-02-18 Thread Liu Aleaxander
On Sun, Feb 19, 2012 at 8:54 AM, Eric Anholt wrote: > On Sat, 18 Feb 2012 23:07:32 +0800, Liu Aleaxander > wrote: >> This patch add the support of gl_PointCoord gl builtin var for platform >> gen4 and gen5(ILK). >> >> We can get the point start coord and the current pixel coord from PS >> payloa

[Mesa-dev] Building mesa with libdrm

2012-02-18 Thread Satyajit Sarangi
I posted this message to mesa-users but didn't get an answer so I am posting this to mesa dev. The MesaLib 7.5.1 tar.gz files does compile fine however git:master doesn't seem to compile. I tried older commits yet they still don't compile at all. -- I have been trying to build mesa with drm from

Re: [Mesa-dev] [PATCH 2/2] i965: Make the dummy fragment shader work in SIMD16 mode.

2012-02-18 Thread Eric Anholt
On Sat, 18 Feb 2012 13:16:04 -0800, Kenneth Graunke wrote: > If you're resorting to the dummy shader, you've probably already turned > off SIMD16 mode. But if you didn't, it would die in a fire. > > We could either fail to compile in SIMD16 mode...or just fix it. > > Signed-off-by: Kenneth Gra

Re: [Mesa-dev] [PATCH 2/2] i965: handle gl_PointCoord for Gen4 and Gen5 platform

2012-02-18 Thread Eric Anholt
On Sat, 18 Feb 2012 23:07:32 +0800, Liu Aleaxander wrote: > This patch add the support of gl_PointCoord gl builtin var for platform > gen4 and gen5(ILK). > > We can get the point start coord and the current pixel coord from PS > payload, and the only left element needed is the point size. Then ha

[Mesa-dev] [PATCH] Set close on exec flag FD_CLOEXEC

2012-02-18 Thread David Fries
Set the close on exec flag when opening dri character devices, so they will be closed and free any resouces allocated in exec. Signed-off-by: David Fries Reviewed-by: Adam Jackson --- I had posted this in November/December, got feedback, redid the patch, posted an update, but now I don't see it

[Mesa-dev] [PATCH 2/2] i965: Make the dummy fragment shader work in SIMD16 mode.

2012-02-18 Thread Kenneth Graunke
If you're resorting to the dummy shader, you've probably already turned off SIMD16 mode. But if you didn't, it would die in a fire. We could either fail to compile in SIMD16 mode...or just fix it. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 12 +++---

[Mesa-dev] [PATCH 1/2] i965: Fix GPU hangs in the dummy fragment shader.

2012-02-18 Thread Kenneth Graunke
The dummy FB write failed to specify EOT and a message length, causing the GPU to hang. Now we can enjoy "everyone's favorite color" again. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) This is occasi

Re: [Mesa-dev] [PATCH 3/3] gallivm: add integer and unsigned mod arit functions.

2012-02-18 Thread Tom Stellard
On Sat, Feb 18, 2012 at 4:20 AM, Jose Fonseca wrote: > - Original Message - >> On Fri, Feb 17, 2012 at 9:46 PM, Jose Fonseca >> wrote: >> > Dave, >> > >> > Ideally there should be only one lp_build_mod() which will invoke >> > LLVMBuildSRem or LLVMBuildURem depending on the value of >> >

[Mesa-dev] [PATCH v2] st/egl: Move drm_display_authenticate into HAVE_WAYLAND_BACKEND section.

2012-02-18 Thread Vinson Lee
Fixes this GCC warning. native_drm.c:153:1: warning: ‘drm_display_authenticate’ defined but not used [-Wunused-function] Signed-off-by: Vinson Lee --- src/gallium/state_trackers/egl/drm/native_drm.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/gal

[Mesa-dev] [Bug 41152] [glsl] Shader backend in Regnum Online does not work

2012-02-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41152 --- Comment #2 from Sven Arvidsson 2012-02-18 08:54:42 PST --- No difference here. Bug 40361 seems to be a regression causing problems in the fixed function rendering, this bug is about the shader backend that has never worked correctly AFAIK.

[Mesa-dev] [PATCH 2/2] i965: handle gl_PointCoord for Gen4 and Gen5 platform

2012-02-18 Thread Liu Aleaxander
This patch add the support of gl_PointCoord gl builtin var for platform gen4 and gen5(ILK). We can get the point start coord and the current pixel coord from PS payload, and the only left element needed is the point size. Then handle that in SF thread. Bugzilla: https://bugs.freedesktop.org/show_

[Mesa-dev] [PATCH 1/2] i965: fix wrong urb entries per attr for platform under SNB

2012-02-18 Thread Liu Aleaxander
For SNB+, One attribute takes 2 urb entries. While for platform under SNB, we write 4 MRF registers for each attribute, thus it takes 4 urb entries, right? NOTE: this is a candidate for stable release branches. I didn't see this fix anything, meanwhile I didn't see this broke anything so

Re: [Mesa-dev] [PATCH] i965: handle gl_PointCoord for Gen4 and Gen5 platform

2012-02-18 Thread Liu Aleaxander
On Sat, Feb 18, 2012 at 3:45 AM, Eric Anholt wrote: > On Fri, 17 Feb 2012 16:09:37 +0800, Yuanhan Liu > wrote: >> This patch add the support of gl_PointCoord gl builtin var for platform >> gen4 and gen5(ILK). >> >> We can get the point start coord and the current pixel coord, and the >> only lef

Re: [Mesa-dev] [PATCH] st/egl: Define drm_display_authenticate only with HAVE_WAYLAND_BACKEND.

2012-02-18 Thread Jose Fonseca
It's ok like this. But I think it would be nice to move this function inside the existing #ifdef HAVE_WAYLAND_BACKEND ... #endif, where it is used, to keep all related wayland code close. Jose - Original Message - > Fixes this GCC warning. > native_drm.c:153:1: warning: ‘drm_display_au

Re: [Mesa-dev] [PATCH] st/vega: Remove unused variable.

2012-02-18 Thread Jose Fonseca
Looks good. This logic is already in mask_resource_fill so there seems to be no mistake, just dead code. Jose - Original Message - > Fixes this GCC warning. > mask.c: In function ‘mask_layer_fill’: > mask.c:387:12: warning: variable ‘alpha_color’ set but not used > [-Wunused-but-set-vari

Re: [Mesa-dev] [PATCH] st/glx: Remove unused variables.

2012-02-18 Thread Jose Fonseca
LGTM. Jose - Original Message - > Fixes these GCC warnings. > glx_api.c: In function ‘choose_visual’: > glx_api.c:678:8: warning: variable ‘trans_value’ set but not used > [-Wunused-but-set-variable] > glx_api.c:677:8: warning: variable ‘trans_type’ set but not used > [-Wunused-but-set-va

Re: [Mesa-dev] [PATCH 2/3] gallivm: add uint to float arit function.

2012-02-18 Thread Jose Fonseca
- Original Message - > On Fri, Feb 17, 2012 at 10:01 PM, Jose Fonseca > wrote: > > Note that SSE doesn't have vectorized UI to FP. Only SI to FP. At > > one point LLVM was breaking down the conversion to scalar, which > > is overkill. I think that since I look at it, LLVM has been > > up

Re: [Mesa-dev] [PATCH 3/3] gallivm: add integer and unsigned mod arit functions.

2012-02-18 Thread Jose Fonseca
- Original Message - > On Fri, Feb 17, 2012 at 9:46 PM, Jose Fonseca > wrote: > > Dave, > > > > Ideally there should be only one lp_build_mod() which will invoke > > LLVMBuildSRem or LLVMBuildURem depending on the value of > > bld->type.sign.  The point being that this allows the same code

[Mesa-dev] [PATCH] st/vega: Remove unused variable.

2012-02-18 Thread Vinson Lee
Fixes this GCC warning. mask.c: In function ‘mask_layer_fill’: mask.c:387:12: warning: variable ‘alpha_color’ set but not used [-Wunused-but-set-variable] Signed-off-by: Vinson Lee --- src/gallium/state_trackers/vega/mask.c |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --gi

Re: [Mesa-dev] [PATCH 2/3] gallivm: add uint to float arit function.

2012-02-18 Thread Dave Airlie
On Fri, Feb 17, 2012 at 10:01 PM, Jose Fonseca wrote: > Note that SSE doesn't have vectorized UI to FP. Only SI to FP. At one point > LLVM was breaking down the conversion to scalar, which is overkill. I think > that since I look at it, LLVM has been updated to generate better code. > Please ta

Re: [Mesa-dev] [PATCH 3/3] gallivm: add integer and unsigned mod arit functions.

2012-02-18 Thread Dave Airlie
On Fri, Feb 17, 2012 at 9:46 PM, Jose Fonseca wrote: > Dave, > > Ideally there should be only one lp_build_mod() which will invoke > LLVMBuildSRem or LLVMBuildURem depending on the value of bld->type.sign.  The > point being that this allows the same code generation logic to seemingly > target

[Mesa-dev] [PATCH] st/glx: Remove unused variables.

2012-02-18 Thread Vinson Lee
Fixes these GCC warnings. glx_api.c: In function ‘choose_visual’: glx_api.c:678:8: warning: variable ‘trans_value’ set but not used [-Wunused-but-set-variable] glx_api.c:677:8: warning: variable ‘trans_type’ set but not used [-Wunused-but-set-variable] glx_api.c:663:8: warning: variable ‘min_ci’ se

[Mesa-dev] [PATCH] st/egl: Define drm_display_authenticate only with HAVE_WAYLAND_BACKEND.

2012-02-18 Thread Vinson Lee
Fixes this GCC warning. native_drm.c:153:1: warning: ‘drm_display_authenticate’ defined but not used [-Wunused-function] Signed-off-by: Vinson Lee --- src/gallium/state_trackers/egl/drm/native_drm.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/gallium/state_track