[Mesa-dev] [PATCH] glapi: add output info to GetProgramiv's params

2013-10-04 Thread Emilio Pozuelo Monfort
Signed-off-by: Emilio Pozuelo Monfort --- src/mapi/glapi/gen/gl_API.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml index 3ffa817..48fce36 100644 --- a/src/mapi/glapi/gen/gl_API.xml +++ b/src/mapi/glapi/gen/gl_A

[Mesa-dev] [PATCH] nouveau_vieux: add AllocTextureImageBuffer implementation

2013-10-04 Thread Ilia Mirkin
This fixes issues where get_rt_format would see a 0 format because the nouveau_surface had not been properly initialized. Fixes crash on supertuxkart startup (which still fails due to out-of-vram issues). Signed-off-by: Ilia Mirkin --- src/mesa/drivers/dri/nouveau/nouveau_texture.c | 9 +

[Mesa-dev] [PATCH 1/2] gallium: add PIPE_CAP_MIXED_FRAMEBUFFER_SIZES

2013-10-04 Thread Ilia Mirkin
This CAP will determine whether ARB_framebuffer_object can be enabled. The nv30 driver does not allow mixing swizzled and linear zsbuf/cbuf textures. Signed-off-by: Ilia Mirkin --- src/gallium/docs/source/screen.rst | 3 +++ src/gallium/drivers/freedreno/freedreno_screen.c | 1 + s

[Mesa-dev] [PATCH 2/2] mesa/st: disable ARB_framebuffer_object when no driver support.

2013-10-04 Thread Ilia Mirkin
When PIPE_CAP_MIXED_FRAMEBUFFER_SIZES is not provided, parts of ARB_framebuffer_object can't be supported, such as on NV30. Signed-off-by: Ilia Mirkin --- src/mesa/state_tracker/st_extensions.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/mesa/state_tracker/st_e

[Mesa-dev] [PATCH 0/8] gallium: create a generic way to validate fb params

2013-10-04 Thread Ilia Mirkin
NV30 is blessed with all manners of hardware restrictions. One of them is that the render target format's color and depth outputs need to be the same bit-ness, i.e. either both 16 or both 32 bits. (In addition to all color attachments needing to be the same, and everything needing to be the same si

[Mesa-dev] [PATCH 1/8] mesa/st: create interface to verify fb format, reject bad fbs

2013-10-04 Thread Ilia Mirkin
Extract format parameters in st_validate_framebuffer and call the new optional pipe->is_fb_format_supported function to check that parameters are supported by the driver. Signed-off-by: Ilia Mirkin --- I'm a bit weirded out that no attempt is made to check if the att->Type == GL_TEXTURE for cbuf

[Mesa-dev] [PATCH 2/8] gallium: add helper to use as a replacement for the MIXED_COLOR cap

2013-10-04 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/gallium/auxiliary/util/u_format.c | 20 src/gallium/auxiliary/util/u_format.h | 11 +++ 2 files changed, 31 insertions(+) diff --git a/src/gallium/auxiliary/util/u_format.c b/src/gallium/auxiliary/util/u_format.c index a8aa571..8b9

[Mesa-dev] [PATCH 3/8] i915: hook up is_fb_format_supported

2013-10-04 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/gallium/drivers/i915/i915_screen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/i915/i915_screen.c b/src/gallium/drivers/i915/i915_screen.c index 77607d0..78800ac 100644 --- a/src/gallium/drivers/i915/i915_screen.c +++ b/src/gallium/d

[Mesa-dev] [PATCH 4/8] r300: hook up is_fb_format_supported

2013-10-04 Thread Ilia Mirkin
Only do this in the !r500 case, same as how PIPE_CAP_MIXED_COLORBUFFER_FORMATS is set. Signed-off-by: Ilia Mirkin --- src/gallium/drivers/r300/r300_screen.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/drivers/r300/r300_screen.c b/src/gallium/drivers/r300/r300_screen.c inde

[Mesa-dev] [PATCH 5/8] softpipe: hook up is_fb_format_supported

2013-10-04 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/gallium/drivers/softpipe/sp_screen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/softpipe/sp_screen.c b/src/gallium/drivers/softpipe/sp_screen.c index 47ef20e..5dca415 100644 --- a/src/gallium/drivers/softpipe/sp_screen.c +++ b/src/g

[Mesa-dev] [PATCH 6/8] nv30: hook up is_fb_format_supported

2013-10-04 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- src/gallium/drivers/nouveau/nv30/nv30_screen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c b/src/gallium/drivers/nouveau/nv30/nv30_screen.c index 807100e..c546c43 100644 --- a/src/gallium/drivers/nouveau/nv30/n

[Mesa-dev] [PATCH 7/8] gallium: remove PIPE_CAP_MIXED_COLORBUFFER_FORMATS

2013-10-04 Thread Ilia Mirkin
This is now completely handled by the new is_fb_format_supported callback. Signed-off-by: Ilia Mirkin --- src/gallium/docs/source/screen.rst | 2 -- src/gallium/drivers/freedreno/freedreno_screen.c | 1 - src/gallium/drivers/i915/i915_screen.c | 1 - src/gallium/driver

[Mesa-dev] [PATCH 8/8] nv30: make sure that cbufs and zsbuf have the same depth

2013-10-04 Thread Ilia Mirkin
This should finally fix RT_FORMAT errors due to weird fb configurations. Signed-off-by: Ilia Mirkin --- src/gallium/drivers/nouveau/nv30/nv30_screen.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c b/src/ga

[Mesa-dev] [Bug 70123] New: Freeze caused by 'winsys/radeon: remove cs_queue_empty' commit

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70123 Priority: medium Bug ID: 70123 Assignee: mesa-dev@lists.freedesktop.org Summary: Freeze caused by 'winsys/radeon: remove cs_queue_empty' commit Severity: normal Classific

[Mesa-dev] [PATCH 1/3] glx: Hide xGLXMakeCurrentReply inside SendMakeCurrentRequest

2013-10-04 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/glx/indirect_glx.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c index f8944a7..d0457fe 100644 --- a/src/glx/indirect_glx.c +++ b/src/glx/indirect_glx.c @@ -59,8 +59,9 @@ sta

[Mesa-dev] [PATCH 3/3] glx: Generate fewer errors in MakeContextCurrent

2013-10-04 Thread Adam Jackson
For a few reasons. 1: In the (current) common case, these conditionals are never true. All we're doing by checking them is slowing down MakeCurrent. The server does these checks already anyway. 2: GLX >= 3.0 contexts may legally be made current without a bound framebuffer. This does not fix pig

Re: [Mesa-dev] [PATCH 0/8] gallium: create a generic way to validate fb params

2013-10-04 Thread Marek Olšák
Wouldn't it be more flexible for you to just change the depth buffer format internally by reallocating the depth buffer in the driver whenever it's incompatible with the colorbuffer? I think you should be able to do that easily without changing the pipe_resource structure. If a depth buffer cannot

[Mesa-dev] [PATCH 2/3] glx: Propagate failures from SendMakeCurrentRequest where possible

2013-10-04 Thread Adam Jackson
Signed-off-by: Adam Jackson --- src/glx/indirect_glx.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c index d0457fe..d27b019 100644 --- a/src/glx/indirect_glx.c +++ b/src/glx/indirect_glx.c @@ -132,6 +132,7 @@ indirect_bi

Re: [Mesa-dev] [PATCH 1/3] glx: Hide xGLXMakeCurrentReply inside SendMakeCurrentRequest

2013-10-04 Thread Brian Paul
On 10/04/2013 07:30 AM, Adam Jackson wrote: Signed-off-by: Adam Jackson --- src/glx/indirect_glx.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c index f8944a7..d0457fe 100644 --- a/src/glx/indirect_glx.c ++

Re: [Mesa-dev] [PATCH 0/8] gallium: create a generic way to validate fb params

2013-10-04 Thread Ilia Mirkin
On Fri, Oct 4, 2013 at 10:02 AM, Marek Olšák wrote: > Wouldn't it be more flexible for you to just change the depth buffer > format internally by reallocating the depth buffer in the driver > whenever it's incompatible with the colorbuffer? I think you should be > able to do that easily without ch

Re: [Mesa-dev] [PATCH] glsl: Fix usage of the wrong union member in program_resource_visitor::recursion.

2013-10-04 Thread Ian Romanick
On 09/20/2013 07:28 PM, Francisco Jerez wrote: > In the array-of-struct case, recursion() takes the row_major flag for > each iteration from 't->fields.structure[i]', but 't' is not a record > type. Inherit the array declaration row_major flag instead. > > This mistake was found by running piglit

[Mesa-dev] [PATCH 1/6] haiku: Build Haiku's libGL from within Mesa

2013-10-04 Thread Alexander von Gluck IV
* This in essence means that Mesa would be taking control of Haiku's OpenGL kit. * This works by dispatching renderers from the OpenGL add-ons directory --- src/gallium/SConscript | 1 + src/gallium/targets/libgl-haiku/GLDispatcher.cpp | 72 +++ src/gallium/tar

[Mesa-dev] [PATCH 3/6] haiku: Correct Haiku softpipe library

2013-10-04 Thread Alexander von Gluck IV
* Use LoadableModule vs SharedLibrary --- src/gallium/targets/haiku-softpipe/SConscript | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/targets/haiku-softpipe/SConscript b/src/gallium/targets/haiku-softpipe/SConscript index 0a99976..6e12dc4 100644 --- a/src/gallium

[Mesa-dev] [PATCH 5/6] haiku: Clean up code, use target-helpers

2013-10-04 Thread Alexander von Gluck IV
* Thanks for the help xexaxo! --- src/gallium/targets/haiku-softpipe/GalliumContext.cpp | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/src/gallium/targets/haiku-softpipe/GalliumContext.cpp b/src/gallium/targets/haiku-softpipe/GalliumContext.cpp index 1c08fb1

[Mesa-dev] [PATCH 4/6] haiku: Drop haiku-softpipe.c; fix extern C

2013-10-04 Thread Alexander von Gluck IV
* It isn't needed any longer as we're moving in the code that called it. * The winsys code is C, so make sure we include the header in the extern C --- .../targets/haiku-softpipe/GalliumContext.cpp | 2 +- src/gallium/targets/haiku-softpipe/SConscript | 1 - .../targets/haiku-softp

[Mesa-dev] [PATCH 6/6] haiku: Ensure correct libraries are referenced.

2013-10-04 Thread Alexander von Gluck IV
--- scons/gallium.py | 2 ++ src/gallium/targets/haiku-softpipe/SConscript | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/scons/gallium.py b/scons/gallium.py index 2e341e9..5b20f3f 100755 --- a/scons/gallium.py +++ b/scons/gallium.py @@ -506,6 +50

[Mesa-dev] [PATCH 2/6] haiku: Add first Haiku renderer (softpipe)

2013-10-04 Thread Alexander von Gluck IV
* This shared library gets parsed by the system as a system "add-on" --- .../targets/haiku-softpipe/GalliumContext.cpp | 528 + .../targets/haiku-softpipe/GalliumContext.h| 83 .../targets/haiku-softpipe/GalliumFramebuffer.cpp | 116 + .../targets/haik

Re: [Mesa-dev] Documentation patch

2013-10-04 Thread Eric Anholt
"Rogovin, Kevin" writes: > Additionally, I really, really, want the text in doxygen/doxy_text to > eventually land in src/ and a policy put in place that as patches are > added the corresponding doxy_text files are updated to reflect the > patch. This will keep the documentation up to date and use

Re: [Mesa-dev] PATCH: R600/SI: Enable the verifier on most lit tests

2013-10-04 Thread Owen Anderson
On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote: > Unfortunately, we cannot enable the machine verifier on tests that have > branches, because the way IF and ELSE instructions are selected leaves us > with a copy instruction following the IF and ELSE terminators, which > violates one of the verif

Re: [Mesa-dev] PATCH: R600/SI: Enable the verifier on most lit tests

2013-10-04 Thread Tom Stellard
On Fri, Oct 04, 2013 at 10:01:32AM -0700, Owen Anderson wrote: > > On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote: > > > Unfortunately, we cannot enable the machine verifier on tests that have > > branches, because the way IF and ELSE instructions are selected leaves us > > with a copy instructi

[Mesa-dev] [PATCH] clover: fix building with llvm-3.4 since rev191922

2013-10-04 Thread Laurent Carlier
http://llvm.org/viewvc/llvm-project?view=revision&revision=191922 --- src/gallium/state_trackers/clover/llvm/invocation.cpp | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp b/src/gallium/state_trackers/clover/llvm/invoc

Re: [Mesa-dev] [PATCH] glsl: Fix usage of the wrong union member in program_resource_visitor::recursion.

2013-10-04 Thread Kenneth Graunke
On 10/04/2013 10:03 AM, Ian Romanick wrote: > On 09/20/2013 07:28 PM, Francisco Jerez wrote: >> In the array-of-struct case, recursion() takes the row_major flag for >> each iteration from 't->fields.structure[i]', but 't' is not a record >> type. Inherit the array declaration row_major flag inste

Re: [Mesa-dev] [PATCH 1/6] haiku: Build Haiku's libGL from within Mesa

2013-10-04 Thread Brian Paul
On 10/04/2013 11:36 AM, Alexander von Gluck IV wrote: * This in essence means that Mesa would be taking control of Haiku's OpenGL kit. * This works by dispatching renderers from the OpenGL add-ons directory --- src/gallium/SConscript | 1 + src/gallium/targe

[Mesa-dev] [Bug 70150] New: Mesa downsizes to i32 the types ulong and long

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70150 Priority: medium Bug ID: 70150 Assignee: mesa-dev@lists.freedesktop.org Summary: Mesa downsizes to i32 the types ulong and long Severity: normal Classification: Unclassified

[Mesa-dev] [Bug 31800] [r300g] src/mesa/state_tracker/st_mesa_to_tgsi.c:228:src_register: Assertion `index >= 0' failed.

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=31800 David "okias" Heidelberger changed: What|Removed |Added Status|NEW |RESOLVED Resolution

[Mesa-dev] [PATCH] doxygen: Generate Doxygen for i965

2013-10-04 Thread Chad Versace
Now, one can do the following to generate and read the i965 Doxygen: cd $MESA_TOP/doxygen make firefox i965/index.html CC: Frank Henigman Signed-off-by: Chad Versace --- doxygen/.gitignore | 1 + doxygen/Makefile | 3 ++- doxygen/i965.doxy | 50 ++

Re: [Mesa-dev] [PATCH 06/10] i965: Don't dead-code eliminate instructions that write to the accumulator.

2013-10-04 Thread Paul Berry
On 27 September 2013 18:17, Matt Turner wrote: > v2: Set destination register using brw_null_reg(). > Reviewed-by: Paul Berry > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 15 ++- > src/mesa/drivers/dri/i965/brw_vec4.cpp | 15 ++- > 2 files changed, 28 insertions(+

[Mesa-dev] [Bug 70150] Mesa downsizes to i32 the types ulong and long

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70150 --- Comment #1 from klondike --- I have been checking a bit around this issue, whilst modifying clc/clctypes.h to add "typedef unsigned __INT64_TYPE__ ulong;" works, doing the same with long doesn't, quite likely because it is a standard C type.

Re: [Mesa-dev] [PATCH 09/10] i965/fs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Paul Berry
On 3 October 2013 10:59, Matt Turner wrote: > v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. > v3: Store the bool emitted_addc_or_subb in the class, not static. > --- > src/mesa/drivers/dri/i965/brw_fs.h | 3 + > src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 104 >

Re: [Mesa-dev] [PATCH 1/2] r600g: texture offsets for non-TXF instructions

2013-10-04 Thread Marek Olšák
I've pushed this patch. I'll let somebody else review the other one. Marek On Thu, Oct 3, 2013 at 12:12 AM, Grigori Goronzy wrote: > All texture instructions can use offsets, not just TXF. Offsets into > the literals array were wrong, too. > --- > src/gallium/drivers/r600/r600_shader.c | 20 +++

Re: [Mesa-dev] [PATCH] egl: Flush the Wayland display queue after calling attach/damage/commit

2013-10-04 Thread Carl Worth
Giulio Camuffo writes: > eglSwapBuffers() should result in a call to wl_display_flush since it > may be called by a rendering thread, and it cannot rely on the main > loop to call it. ... > Cc: mesa-sta...@lists.freedesktop.org Hi Giulio, I'm the maintainer of Mesa's stable branches, so I'm goin

Re: [Mesa-dev] [PATCH 09/10] i965/fs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Paul Berry
On 4 October 2013 13:51, Paul Berry wrote: > On 3 October 2013 10:59, Matt Turner wrote: > >> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. >> v3: Store the bool emitted_addc_or_subb in the class, not static. >> --- >> src/mesa/drivers/dri/i965/brw_fs.h | 3 + >> src/mesa/dri

Re: [Mesa-dev] [PATCH 10/10] i965/vs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Paul Berry
On 3 October 2013 11:00, Matt Turner wrote: > v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. > v3: Store the bool emitted_addc_or_subb in the class, not static. > --- > src/mesa/drivers/dri/i965/brw_vec4.h | 3 + > src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 104 >

Re: [Mesa-dev] [PATCH 09/10] i965/fs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Matt Turner
On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry wrote: > On 4 October 2013 13:51, Paul Berry wrote: >> >> On 3 October 2013 10:59, Matt Turner wrote: >>> >>> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. >>> v3: Store the bool emitted_addc_or_subb in the class, not static. >>> --- >>> src/mes

Re: [Mesa-dev] [PATCH 09/10] i965/fs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Paul Berry
On 4 October 2013 14:24, Matt Turner wrote: > On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry > wrote: > > On 4 October 2013 13:51, Paul Berry wrote: > >> > >> On 3 October 2013 10:59, Matt Turner wrote: > >>> > >>> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. > >>> v3: Store the bool emitt

[Mesa-dev] i965: dynamic binding table size

2013-10-04 Thread Eric Anholt
Here's a series to pack our binding tables so that you (for example) don't waste BRW_MAX_DRAWBUFFERS-1 indices in your simple texturing shaders. I think it cleans up the code, though the net LOC change is approximately 0. Performance data for GLB2.7: IVB: No SSPD (n=754) gm45: -0.0323403% +/- 0.

[Mesa-dev] [PATCH 2/7] i965: Drop a couple of unused defines.

2013-10-04 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_context.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h index d4ace98..eb9d93c 100644 --- a/src/mesa/drivers/dri/i965/brw_context.h +++ b/src/mesa/drivers/dri/i965/brw_conte

[Mesa-dev] [PATCH 1/7] i965: Remove dead arguments from prog_data_compare.

2013-10-04 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_context.h | 3 +-- src/mesa/drivers/dri/i965/brw_state_cache.c | 4 +--- src/mesa/drivers/dri/i965/brw_vec4_gs.c | 3 +-- src/mesa/drivers/dri/i965/brw_vec4_gs.h | 3 +-- src/mesa/drivers/dri/i965/brw_vs.c | 3 +-- src/mesa/drivers/dri/i965/br

[Mesa-dev] [PATCH 3/7] i965: Always have the struct gl_program * in the backend visitor.

2013-10-04 Thread Eric Anholt
vec4 already had it, so put it in the FS, too. --- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 14 +++--- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 13 +++-- src/mesa/drivers/dri/i965/brw_shader.h | 1 + src/mesa/drivers/dri/i965/brw_vec4.h | 1 - 4 files ch

[Mesa-dev] [PATCH 5/7] i965/fs: Dynamically set up the WM binding table offsets.

2013-10-04 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_context.h| 41 -- src/mesa/drivers/dri/i965/brw_fs.cpp | 35 ++ src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2 +- 3 files changed, 35 insertions(+), 43 deletions(-) diff --git a/src/mesa/drivers/

[Mesa-dev] [PATCH 4/7] i965: Make a brw_stage_prog_data for storing the SURF_INDEX information.

2013-10-04 Thread Eric Anholt
It would be nice to be able to pack our binding table so that programs that use 1 render target don't upload an extra BRW_MAX_DRAW_BUFFERS - 1 binding table entries. To do that, we need the compiled program to have information on where its surfaces go. --- I consistently named the structure membe

[Mesa-dev] [PATCH 7/7] i965: Move the common binding table offset code to brw_shader.cpp.

2013-10-04 Thread Eric Anholt
Now that both vec4 and fs are dynamically assigning offsets, a lot of the code is the same. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 33 ++ src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 ++ src/mesa/drivers/dri/i965/brw_shader.cpp | 47 ++

[Mesa-dev] [PATCH 6/7] i965/vec4: Dynamically assign the VS/GS binding table offsets.

2013-10-04 Thread Eric Anholt
Note that the dropped comment in brw_context.h is mostly (better written) in brw_binding_table.c as well. --- src/mesa/drivers/dri/i965/brw_context.h | 51 src/mesa/drivers/dri/i965/brw_vec4.cpp | 29 +++--- src/mesa/drivers/dri/i965/brw_vec4_gen

Re: [Mesa-dev] [PATCH 1/6] haiku: Build Haiku's libGL from within Mesa

2013-10-04 Thread Alexander von Gluck IV
On Fri, 04 Oct 2013 13:31:11 -0600 Brian Paul wrote: > The series look OK to me. Do you want to add/update any docs/ files? > > Reviewed-by: Brian Paul > > Do you need me to push this for you? > Nah, I can push it. I'll see if I can add anything meaningful to the docs. Thanks! -- Alex

[Mesa-dev] [Bug 70150] Mesa downsizes to i32 the types ulong and long

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70150 --- Comment #2 from Tom Stellard --- The type sizes are supposed to be controlled by the TargetInfo class. TargetInfo::setForcedLangOptions() in clang/lib/basic/TargetInfo.cpp is where the type sizes are set for OpenCL. R600's TargetInfo is def

Re: [Mesa-dev] [PATCH 09/10] i965/fs: Add a peephole pass to combine ADD with ADDC/SUBB.

2013-10-04 Thread Eric Anholt
Matt Turner writes: > On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry wrote: >> On 4 October 2013 13:51, Paul Berry wrote: >>> >>> On 3 October 2013 10:59, Matt Turner wrote: v2: Check fixed_hw_reg.{file,nr} instead of dst.reg. v3: Store the bool emitted_addc_or_subb in the class, no

[Mesa-dev] [Bug 70150] Mesa downsizes to i32 the types ulong and long

2013-10-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=70150 klondike changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH 2/3] gallivm: handle explicit derivatives for cubemaps

2013-10-04 Thread sroland
From: Roland Scheidegger They need some special handling. Quite complicated. Additionally, use the same code for implicit derivatives too if no_rho_approx and no_quad_lod is set, because it seems while generally it should be ok to use per quad lod for implicit derivatives there's at least some te

[Mesa-dev] [PATCH 1/3] gallivm: ignore rho approximation for cube maps

2013-10-04 Thread sroland
From: Roland Scheidegger There's two reasons for this: 1) even when ignoring rho approximation for cube maps, the result is still not correct, but it's better as the max error at edges is now sqrt(2) instead of 2 (which was a full mip level), same as it is for ordinary 2d maps when doing rho appr

[Mesa-dev] [PATCH 3/3] gallivm: kill old per-quad face selection code

2013-10-04 Thread sroland
From: Roland Scheidegger Not used since ages, and it wouldn't work at all with explicit derivatives now (not that it did before as it ignored them but now the code would just use the derivs pre-projected which would be quite random numbers). v2: also get rid of 3 helper functions no longer used.

[Mesa-dev] Mesa 9.2.1

2013-10-04 Thread Carl Worth
Mesa 9.2.1 has been released. Mesa 9.2.1 is a bug fix release which fixes bugs fixed since the 9.2 release. The tag in the git repository for Mesa 9.2.1 is 'mesa-9.2.1'. Mesa 9.2.1 is available for download at ftp://freedesktop.org/pub/mesa/9.2.1/ md5sums: e6cdfa84dfddd86e3d36ec7ff4b6478a Mesa

[Mesa-dev] [PATCH 0/6] ARB_texture_query_levels for i965

2013-10-04 Thread Chris Forbes
This series adds support for ARB_texture_query_levels on i965 Gen6+. The fourth channel of the resinfo message (used for textureSize()) includes the correct value for textureQueryLevels(). ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http:/

[Mesa-dev] [PATCH 1/6] mesa: add plumbing for GL_ARB_texture_query_levels

2013-10-04 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/main/extensions.c | 1 + src/mesa/main/mtypes.h | 1 + 2 files changed, 2 insertions(+) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index c0f17c5..2507fdf 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/extensions.

[Mesa-dev] [PATCH 2/6] glsl: add plumbing for GL_ARB_texture_query_levels

2013-10-04 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/glsl/builtin_functions.cpp | 56 + src/glsl/glcpp/glcpp-parse.y| 3 ++ src/glsl/glsl_parser_extras.cpp | 1 + src/glsl/glsl_parser_extras.h | 2 ++ src/glsl/ir.cpp | 4 +-- src/g

[Mesa-dev] [PATCH 3/6] i965: ignore all texturing opcodes without a coordinate, for cubemap normalize

2013-10-04 Thread Chris Forbes
Previously we special-cased textureSize() but this is the more correct condition. Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_cubemap_normalize.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_cubemap_normalize.cpp b/src/mes

[Mesa-dev] [PATCH 4/6] i965/fs: implement ir_query_levels

2013-10-04 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp index a8ad659..6141009 100644 --- a/sr

[Mesa-dev] [PATCH 5/6] i965/vs: implement ir_query_levels

2013-10-04 Thread Chris Forbes
Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 15 ++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp index 9e6cc78..1112b23 100644 --- a/s

[Mesa-dev] [PATCH 6/6] i965: enable ARB_texture_query_levels on Gen6+

2013-10-04 Thread Chris Forbes
Theoretically would work on Gen5 as well but requires GLSL 1.30, which is not (yet) enabled by default there. Signed-off-by: Chris Forbes --- src/mesa/drivers/dri/i965/intel_extensions.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c b/src/mesa/

Re: [Mesa-dev] [PATCH 6/6] i965: enable ARB_texture_query_levels on Gen6+

2013-10-04 Thread Matt Turner
On Fri, Oct 4, 2013 at 9:17 PM, Chris Forbes wrote: > Theoretically would work on Gen5 as well but requires GLSL 1.30, which > is not (yet) enabled by default there. In that case, just do ctx->Extensions.ARB_texture_query_levels = ctx->Const.GLSLVersion >= 130; and put it in the if (brw->gen >=

[Mesa-dev] Mesa 9.1.7

2013-10-04 Thread Carl Worth
Mesa 9.1.7 has been released. Mesa 9.1.6 is a bug fix release which fixes bugs fixed since the 9.1.6 release. The tag in the git repository for Mesa 9.1.7 is 'mesa-9.1.7'. Note: This is the last anticipated release of the 9.1 series. Future effort for stable releases will be focused on the 9.2 se

Re: [Mesa-dev] [PATCH 0/6] ARB_texture_query_levels for i965

2013-10-04 Thread Matt Turner
On Fri, Oct 4, 2013 at 9:17 PM, Chris Forbes wrote: > This series adds support for ARB_texture_query_levels on i965 Gen6+. The > fourth channel of the resinfo message (used for textureSize()) includes the > correct value for textureQueryLevels(). The series looks good to me. With the one small