Re: [Mesa-dev] [PATCH] r300/compiler: Avoid generating MOV instructions for invalid IMM swizzles

2012-10-14 Thread son_of_the_osiris
Here is what valgrind says: [root@tomek-pc pc-linux64]# valgrind /root/Desktop/Lightsmark2008.2.0/bin/pc-linux64/backend ==699== Memcheck, a memory error detector ==699== Copyright (C) 2002-2012, and GNU GPL'd, by Julian Seward et al. ==699== Using Valgrind-3.8.1 and LibVEX; rerun with -h for c

Re: [Mesa-dev] [PATCH] Don't fail if libX11 isn't installed

2012-10-14 Thread Kenneth Graunke
On 10/14/2012 06:26 PM, Daniel Stone wrote: Hi, On 10 October 2012 01:07, Dan Nicholson wrote: On Oct 8, 2012 10:49 PM, "Matt Turner" wrote: Wow, it's been like that since 2008. Reviewed-by: Matt Turner Originally configure basically supported only GLX either through DRI or Xlib, so Xlib

Re: [Mesa-dev] [PATCH] Don't fail if libX11 isn't installed

2012-10-14 Thread Daniel Stone
Hi, On 10 October 2012 01:07, Dan Nicholson wrote: > On Oct 8, 2012 10:49 PM, "Matt Turner" wrote: >> Wow, it's been like that since 2008. >> >> Reviewed-by: Matt Turner > > Originally configure basically supported only GLX either through DRI or > Xlib, so Xlib was basically required. Not so mu

[Mesa-dev] [PATCH 23/23] mesa: Remove PROGRAM_WRITE_ONLY register type.

2012-10-14 Thread Kenneth Graunke
More dead code. I'm not sure what it was for. --- src/mesa/drivers/dri/i965/brw_vs_emit.c | 1 - src/mesa/drivers/dri/r200/r200_vertprog.c | 1 - src/mesa/program/prog_execute.c | 3 --- src/mesa/program/prog_print.c | 2 -- 4 files changed, 7 deletions(-) diff --git a/sr

[Mesa-dev] [PATCH 22/23] mesa: Remove dead _mesa_num_parameters_of_type() function.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/program/prog_parameter.c | 18 -- src/mesa/program/prog_parameter.h | 5 - 2 files changed, 23 deletions(-) diff --git a/src/mesa/program/prog_parameter.c b/src/mesa/program/prog_parameter.c index 9fa67b8..8713d57 100644 --- a/src/mesa/program/prog_parameter.c +

[Mesa-dev] [PATCH 21/23] mesa: Remove dead _mesa_add_attribute() function.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/program/prog_parameter.c | 30 -- src/mesa/program/prog_parameter.h | 4 2 files changed, 34 deletions(-) diff --git a/src/mesa/program/prog_parameter.c b/src/mesa/program/prog_parameter.c index f1cc989..9fa67b8 100644 --- a/src/mesa/program/prog_pa

[Mesa-dev] [PATCH 20/23] mesa: Remove remnants of PROGRAM_VARYING.

2012-10-14 Thread Kenneth Graunke
The previous patch removed the producer of things in this file. Since there aren't any, we can remove it. --- src/mesa/main/mtypes.h | 1 - src/mesa/program/prog_parameter_layout.c | 2 +- src/mesa/program/prog_print.c| 5 - src/mesa/program/programopt.c

[Mesa-dev] [PATCH 19/23] mesa: Remove dead _mesa_add_varying() function.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/program/prog_parameter.c | 21 - src/mesa/program/prog_parameter.h | 5 - 2 files changed, 26 deletions(-) diff --git a/src/mesa/program/prog_parameter.c b/src/mesa/program/prog_parameter.c index 852af71..f1cc989 100644 --- a/src/mesa/program/prog_parameter.

[Mesa-dev] [PATCH 18/23] mesa: Remove dead program_parameter::Flags field.

2012-10-14 Thread Kenneth Graunke
All flags are now gone, so we can stop storing and passing this around. --- src/mesa/program/ir_to_mesa.cpp | 2 +- src/mesa/program/prog_parameter.c | 23 +-- src/mesa/program/prog_parameter.h | 4 +--- src/mesa/program/program_parse.y | 2 +- 4 files changed, 12 inserti

[Mesa-dev] [PATCH 17/23] st/mesa: Remove the PROG_PARAM_BIT_CYL_WRAP flag.

2012-10-14 Thread Kenneth Graunke
Nobody ever set the flag, which makes this dead code. Also remove the ureg_DECL_fs_input_cyl helper function, since it's now dead too. --- src/gallium/auxiliary/tgsi/tgsi_ureg.h | 15 --- src/mesa/program/prog_parameter.h| 8 src/mesa/state_tracker/st_mesa_to_tgsi.

[Mesa-dev] [PATCH 16/23] mesa: Remove GLSL-related PROG_PARAM_BIT flags.

2012-10-14 Thread Kenneth Graunke
GLSL doesn't use the program code anymore. Accordingly, there were no consumers of these flags, so there's no need to define them. --- src/mesa/program/prog_parameter.h | 4 src/mesa/program/prog_print.c | 8 2 files changed, 12 deletions(-) diff --git a/src/mesa/program/prog_p

[Mesa-dev] [PATCH 15/23] mesa: Remove support for named parameters.

2012-10-14 Thread Kenneth Graunke
These were only part of NV_fragment_program, so we can kill them. The fact that PROGRAM_NAMED_PARAM appears in r200_vertprog.c is rather comedic, but also demonstrates that people just spam the various types of parameters everywhere because they're confusing. --- src/mesa/drivers/dri/i915/i915_fr

[Mesa-dev] [PATCH 14/23] driconf: Remove force enable for NV_vertex_program.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/drivers/dri/common/xmlpool/de.po | 4 src/mesa/drivers/dri/common/xmlpool/es.po | 5 - src/mesa/drivers/dri/common/xmlpool/fr.po | 4 src/mesa/drivers/dri/common/xmlpool/nl.po | 4 src/mesa/drivers/dri/common/xmlpool/options.h | 10 -- sr

[Mesa-dev] [PATCH 13/23] mesa: Remove yet more remnants of NV_fragment_program.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/drivers/dri/r200/r200_vertprog.c | 1 - src/mesa/main/extensions.c| 1 - src/mesa/main/getstring.c | 3 +-- src/mesa/main/mtypes.h| 3 +-- src/mesa/program/prog_instruction.h | 13 + src/mesa/state_tracker/st_cb

[Mesa-dev] [PATCH 12/23] mesa: Remove some miscellaneous NV program stuff from arbprogram.c.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/main/arbprogram.c | 26 +- 1 file changed, 1 insertion(+), 25 deletions(-) diff --git a/src/mesa/main/arbprogram.c b/src/mesa/main/arbprogram.c index ef1f620..9be86c2 100644 --- a/src/mesa/main/arbprogram.c +++ b/src/mesa/main/arbprogram.c @@ -152,14 +152,13 @

[Mesa-dev] [PATCH 11/23] mesa: Simplify _mesa_BindProgram() by removing NV program remnants.

2012-10-14 Thread Kenneth Graunke
Without NV programs, there's no need for the compatible_program_targets function. A simple (non-)equality check will do. --- src/mesa/main/arbprogram.c | 43 +-- 1 file changed, 9 insertions(+), 34 deletions(-) diff --git a/src/mesa/main/arbprogram.c b/src

[Mesa-dev] [PATCH 10/23] mesa: Remove get and enable bits for NV_fragment_program.

2012-10-14 Thread Kenneth Graunke
Also remove a leftover remnant from NV_vertex_program. --- src/mesa/main/enable.c | 16 src/mesa/main/get.c| 23 ++- 2 files changed, 2 insertions(+), 37 deletions(-) diff --git a/src/mesa/main/enable.c b/src/mesa/main/enable.c index 297b495..be105b6 10064

[Mesa-dev] [PATCH 09/23] mesa: Remove prog_print support for NV programs.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/program/prog_print.c | 60 --- src/mesa/program/prog_print.h | 1 - 2 files changed, 5 insertions(+), 56 deletions(-) diff --git a/src/mesa/program/prog_print.c b/src/mesa/program/prog_print.c index a560b53..99623c1 100644 --- a/src/mesa/progr

[Mesa-dev] [PATCH 07/23] mesa: Remove the gl_program::Resident flag.

2012-10-14 Thread Kenneth Graunke
It apparently was only used for NV programs. --- src/mesa/main/mtypes.h | 1 - src/mesa/program/program.c | 1 - 2 files changed, 2 deletions(-) diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index 43f0689..fdfc436 100644 --- a/src/mesa/main/mtypes.h +++ b/src/mesa/main/mtypes.h

[Mesa-dev] [PATCH 08/23] mesa: Remove support for parsing NV fragment programs.

2012-10-14 Thread Kenneth Graunke
--- src/mesa/SConscript|1 - src/mesa/main/arbprogram.c |8 - src/mesa/program/nvfragparse.c | 1591 src/mesa/program/nvfragparse.h | 47 -- src/mesa/sources.mak |1 - 5 files changed, 1648 deletions(-) delete mode 1

[Mesa-dev] [PATCH 06/23] mesa: Remove the EmitNVTempInitialization shader compiler option.

2012-10-14 Thread Kenneth Graunke
Nobody uses it anymore. --- src/mesa/drivers/dri/i965/brw_context.c | 1 - src/mesa/main/mtypes.h | 1 - 2 files changed, 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c index c12ebcc..4196e02 100644 --- a/src/mesa/dr

[Mesa-dev] [PATCH 05/23] mesa: Remove the NV program API functions.

2012-10-14 Thread Kenneth Graunke
These are all unused now. --- src/mesa/SConscript | 1 - src/mesa/main/api_exec.c | 1 - src/mesa/main/dlist.c | 1 - src/mesa/main/nvprogram.c | 489 -- src/mesa/main/nvprogram.h | 86 src/mesa/sources.mak | 1 - 6 file

[Mesa-dev] [PATCH 04/23] mesa: Switch to the other glGetVertexAttribPointervARB implementation.

2012-10-14 Thread Kenneth Graunke
Previously, Mesa used nvprogram.c's _mesa_GetVertexAttribPointervNV() function to implement this GL call. There was also a second implementation in varray.c, _mesa_GetVertexAttribPointervARB(), which was entirely unused. The varray.c variant has an additional assertion and checks the index agains

[Mesa-dev] [PATCH 03/23] mesa: Remove some shared NV_vp/fp functions from the dispatch table.

2012-10-14 Thread Kenneth Graunke
Also kill the resulting dead code for display list handling. --- src/mesa/main/api_exec.c | 9 -- src/mesa/main/dlist.c| 71 2 files changed, 80 deletions(-) diff --git a/src/mesa/main/api_exec.c b/src/mesa/main/api_exec.c index 52866b1..e

[Mesa-dev] [PATCH 02/23] mesa: Unhook NV_fragment_program API from the dispatch table.

2012-10-14 Thread Kenneth Graunke
The NamedParameter functions were introduced in NV_fragment_program, and are not shared with any other extensions. Although this patch appears to remove the LocalParameter functions, it does not: the ARB_fragment_program section also set them up. Now we simply initialize them a single time. ---

[Mesa-dev] [PATCH 01/23] swrast: Remove support for the NV_fragment_program extension.

2012-10-14 Thread Kenneth Graunke
No hardware drivers support this, it's obsolete, and unlikely to be useful without NV_vertex_program, which is gone now. --- src/mesa/main/extensions.c | 1 - src/mesa/swrast/s_context.c | 4 +--- src/mesa/swrast/s_fragprog.c | 5 - 3 files changed, 1 insertion(+), 9 deletions(-) diff --gi

[Mesa-dev] Nuking GL_NV_fragment_program

2012-10-14 Thread Kenneth Graunke
This series removes support for the the old NV_fragment_program extension. No hardware drivers implemented it (only swrast), we aren't aware of any applications that use it, and we already dropped NV_vertex_program. You can obtain it in the 'nukefp' branch of: http://cgit.freedesktop.org/~kwg/mesa

[Mesa-dev] [PATCH] gallium/docs: update some distro information

2012-10-14 Thread Andreas Boll
--- src/gallium/docs/source/conf.py|2 +- src/gallium/docs/source/distro.rst | 43 ++-- 2 files changed, 37 insertions(+), 8 deletions(-) diff --git a/src/gallium/docs/source/conf.py b/src/gallium/docs/source/conf.py index 0846e7d..1288666 100644 --- a/sr

[Mesa-dev] [PATCH 2/2] mesa/vbo: Fix scaling issue in 2-bit signed normalized packing.

2012-10-14 Thread Kenneth Graunke
Since a signed 2-bit integer can only represent -1, 0, or 1, it is tempting to simply to convert it directly to a float. This maps it onto the correct range of [-1.0, 1.0]. However, it gives different values compared to the usual equation: (2.0 * 1.0 + 1.0) * (1.0 / 3.0) = +1.0 (same)

[Mesa-dev] [PATCH 1/2] mesa/vbo: Fix scaling issue in 10-bit signed normalized packing.

2012-10-14 Thread Kenneth Graunke
For the 10-bit components, the divisor was incorrect. A 10-bit signed integer can represent -2^9 through 2^9 - 1, which leads to the following ranges: (float)value.x -> [ -512, 511] 2.0F * (float)value.x -> [-1024, 1022] 2.0F * (float)value.x + 1.0F -> [-1023, 1023] S

Re: [Mesa-dev] [RFC PATCH] Add support for ARB_vertex_type_2_10_10_10_rev on i965

2012-10-14 Thread Kenneth Graunke
On 10/14/2012 01:12 AM, Chris Forbes wrote: On Sun, Oct 14, 2012 at 8:52 PM, Kenneth Graunke wrote: Right, suspected as much -- although hoped the situation was better than that, since the Sandybridge PRM mentions a lot of these in the format conversion table. OK, so I need to hack the shader

[Mesa-dev] [Bug 47248] autogen missing dependency on flex and bison, causes infinite loop in glsl build

2012-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47248 Simos Xenitellis changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

[Mesa-dev] [Bug 47248] autogen missing dependency on flex and bison, causes infinite loop in glsl build

2012-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47248 --- Comment #12 from Simos Xenitellis --- (In reply to comment #11) > Thanks. I'll check both issues tomorrow night. I think these changes have not been pushed yet to master. I got an error while compiling Mesa from git. Autogen correctly did

[Mesa-dev] [Bug 47248] autogen missing dependency on flex and bison, causes infinite loop in glsl build

2012-10-14 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=47248 Simos Xenitellis changed: What|Removed |Added CC||simos.bugzi...@gmail.com -- You are

Re: [Mesa-dev] [RFC PATCH] Add support for ARB_vertex_type_2_10_10_10_rev on i965

2012-10-14 Thread Kenneth Graunke
On 10/13/2012 07:13 PM, Kenneth Graunke wrote: On 10/09/2012 01:06 PM, Kenneth Graunke wrote: On 10/09/2012 02:33 AM, Chris Forbes wrote: This series almost adds support for ARB_vertex_type_2_10_10_10_rev on i965. Working from the Sandybridge and Ivybridge PRMs, this ought to work (what look li