Re: [Mesa-dev] [PATCH 1/2] shared-glapi: implement _glapi_get_proc_name().

2012-10-23 Thread Kenneth Graunke
On 10/23/2012 12:37 PM, Paul Berry wrote: Previously this function was only implemented for non-shared-glapi builds. Since the function is only intended for debugging purposes we use a simple O(n) algorithm. --- src/mapi/mapi/mapi_glapi.c | 4 ++-- src/mapi/mapi/stub.c | 22 +++

Re: [Mesa-dev] [PATCH 4/4] i965: Make GL_TIME_ELAPSED only track time for this context.

2012-10-23 Thread Kenneth Graunke
I like this! For the series: Reviewed-by: Kenneth Graunke However, there's one thing you missed: since you made it emit_end_query for (potentially) both OQ -and- timestamp on batchbuffer end, you should reserve enough space for the extra PIPE_CONTROLs emitted by write_timestamp(). This mea

Re: [Mesa-dev] [PATCH] glsl_to_tgsi: set correct register type for array and structure elements

2012-10-23 Thread Marek Olšák
On Tue, Oct 23, 2012 at 11:52 PM, Bryan Cain wrote: > On 10/23/2012 04:50 PM, Brian Paul wrote: >> On 10/23/2012 10:58 AM, Bryan Cain wrote: >>> This fixes an issue where glsl_to_tgsi_visior::get_opcode() would >>> emit the >>> wrong opcode because the register type was GLSL_TYPE_ARRAY/STRUCT >>>

Re: [Mesa-dev] [PATCH] glsl_to_tgsi: set correct register type for array and structure elements

2012-10-23 Thread Bryan Cain
On 10/23/2012 04:50 PM, Brian Paul wrote: > On 10/23/2012 10:58 AM, Bryan Cain wrote: >> This fixes an issue where glsl_to_tgsi_visior::get_opcode() would >> emit the >> wrong opcode because the register type was GLSL_TYPE_ARRAY/STRUCT >> instead of >> GLSL_TYPE_FLOAT/INT/UINT/BOOL, so the function

Re: [Mesa-dev] [PATCH] glsl_to_tgsi: set correct register type for array and structure elements

2012-10-23 Thread Brian Paul
On 10/23/2012 10:58 AM, Bryan Cain wrote: This fixes an issue where glsl_to_tgsi_visior::get_opcode() would emit the wrong opcode because the register type was GLSL_TYPE_ARRAY/STRUCT instead of GLSL_TYPE_FLOAT/INT/UINT/BOOL, so the function would use the float opcodes for operations on integer or

Re: [Mesa-dev] [PATCH] gallium/docs: fix sphinx warning

2012-10-23 Thread Brian Paul
On 10/23/2012 12:36 PM, Andreas Boll wrote: src/gallium/docs/source/context.rst:495: WARNING: malformed hyperlink target. --- FYI I've pushed the generated gallium-docs to http://people.freedesktop.org/~ab/gallium-docs/ It would be cool if we can host this at http://dri.freedesktop.org/gallium-

[Mesa-dev] [PATCH 4/4] i965: Make GL_TIME_ELAPSED only track time for this context.

2012-10-23 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_queryobj.c | 49 -- 1 file changed, 33 insertions(+), 16 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c b/src/mesa/drivers/dri/i965/brw_queryobj.c index 36caf1e..a698b5b 100644 --- a/src/mesa/drivers/dri/i965/brw

[Mesa-dev] [PATCH 1/4] i965: Rename misleading "active" field of brw->query.

2012-10-23 Thread Eric Anholt
"Active" is an already-used term for the query being between glBeginQuery() and glEndQuery(), while this is tracking whether the start of the packet pair for emitting state has been inserted into the current batchbuffer. --- src/mesa/drivers/dri/i965/brw_context.h |2 +- src/mesa/drivers/dri/

[Mesa-dev] [PATCH 2/4] i965: Merge brw_prepare_query_begin() and brw_emit_query_begin().

2012-10-23 Thread Eric Anholt
This is a leftover from when we had to split those two functions due to the separate BO validation step. --- src/mesa/drivers/dri/i965/brw_context.h |1 - src/mesa/drivers/dri/i965/brw_draw_upload.c |5 + src/mesa/drivers/dri/i965/brw_queryobj.c| 23 ++- 3

[Mesa-dev] [PATCH 3/4] i965: Refactor in preparation for fixing GL_TIME_ELAPSED across batchbuffers.

2012-10-23 Thread Eric Anholt
--- src/mesa/drivers/dri/i965/brw_context.h | 15 +- src/mesa/drivers/dri/i965/brw_queryobj.c | 87 ++ 2 files changed, 66 insertions(+), 36 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_context.h b/src/mesa/drivers/dri/i965/brw_context.h index 9232

[Mesa-dev] [PATCH 2/2] dispatch_sanity: print names of functions that shouldnt be in dispatch table.

2012-10-23 Thread Paul Berry
Previously we just printed the dispatch table index and the user had to convert it to a function name. That was a pain because when FEATURE_remap_table is defined, the assignment of functions to dispatch table entries is done at run time. --- src/mesa/main/tests/dispatch_sanity.cpp | 10 +

[Mesa-dev] [PATCH 1/2] shared-glapi: implement _glapi_get_proc_name().

2012-10-23 Thread Paul Berry
Previously this function was only implemented for non-shared-glapi builds. Since the function is only intended for debugging purposes we use a simple O(n) algorithm. --- src/mapi/mapi/mapi_glapi.c | 4 ++-- src/mapi/mapi/stub.c | 22 ++ src/mapi/mapi/stub.h | 3 +

[Mesa-dev] [PATCH] gallium/docs: fix sphinx warning

2012-10-23 Thread Andreas Boll
src/gallium/docs/source/context.rst:495: WARNING: malformed hyperlink target. --- FYI I've pushed the generated gallium-docs to http://people.freedesktop.org/~ab/gallium-docs/ It would be cool if we can host this at http://dri.freedesktop.org/gallium-docs and then we can add a link at mesa3d.org

[Mesa-dev] [PATCH] build: Ship install-sh in the tarball

2012-10-23 Thread Matt Turner
Fixes the problem where configure from the tarball would report missing files: $ ./configure configure: error: cannot find install-sh, install.sh, or shtool in bin NOTE: This is a candidate for the 9.0 branch. --- Makefile.am |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

Re: [Mesa-dev] [PATCH v2 0/6] Minor fixes to the code that populates the dispatch table.

2012-10-23 Thread Matt Turner
On Mon, Oct 22, 2012 at 5:22 PM, Paul Berry wrote: > Changes based on comments from Matt and Brian. Patch 3/6 has been > added. Patch 6/6 has been updated. > > [PATCH v2 1/6] main: Fix warning ('struct gl_context' declared inside > parameter list). > [PATCH v2 2/6] glapi: Alias ClampColor and C

Re: [Mesa-dev] [PATCH 00/27] intel: Add ETC2 compressed textures support

2012-10-23 Thread Ian Romanick
On 10/19/2012 11:32 PM, Philipp Klaus Krause wrote: On 20.10.2012 01:28, Anuj Phogat wrote: This series lives on my etc2-v9 branch (https://github.com/aphogat/mesa). This series enables 8 out of 10 ETC2 texture formats for all Intel hardware by simply decoding the ETC2 data into RGBX data at th

Re: [Mesa-dev] [PATCH v2 0/6] Minor fixes to the code that populates the dispatch table.

2012-10-23 Thread Ian Romanick
On 10/22/2012 05:22 PM, Paul Berry wrote: Changes based on comments from Matt and Brian. Patch 3/6 has been added. Patch 6/6 has been updated. [PATCH v2 1/6] main: Fix warning ('struct gl_context' declared inside parameter list). [PATCH v2 2/6] glapi: Alias ClampColor and ClampColorARB. [PATC

[Mesa-dev] [PATCH] glsl_to_tgsi: set correct register type for array and structure elements

2012-10-23 Thread Bryan Cain
This fixes an issue where glsl_to_tgsi_visior::get_opcode() would emit the wrong opcode because the register type was GLSL_TYPE_ARRAY/STRUCT instead of GLSL_TYPE_FLOAT/INT/UINT/BOOL, so the function would use the float opcodes for operations on integer or boolean values dereferenced from an array o

Re: [Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Brian Paul
On 10/23/2012 07:24 AM, Michel Dänzer wrote: On Die, 2012-10-23 at 15:21 +0200, Marek Olšák wrote: On Tue, Oct 23, 2012 at 11:24 AM, Michel Dänzer wrote: From: Michel Dänzer The layer dimension of array textures is not subject to mipmap minification. OTOH we were missing an assertion for the

Re: [Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Michel Dänzer
On Die, 2012-10-23 at 15:27 +0200, Marek Olšák wrote: > On Tue, Oct 23, 2012 at 3:24 PM, Michel Dänzer wrote: > > On Die, 2012-10-23 at 15:21 +0200, Marek Olšák wrote: > >> On Tue, Oct 23, 2012 at 11:24 AM, Michel Dänzer wrote: > >> > From: Michel Dänzer > >> > > >> > The layer dimension of arr

Re: [Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Marek Olšák
On Tue, Oct 23, 2012 at 3:24 PM, Michel Dänzer wrote: > On Die, 2012-10-23 at 15:21 +0200, Marek Olšák wrote: >> On Tue, Oct 23, 2012 at 11:24 AM, Michel Dänzer wrote: >> > From: Michel Dänzer >> > >> > The layer dimension of array textures is not subject to mipmap >> > minification. >> > OTOH

Re: [Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Michel Dänzer
On Die, 2012-10-23 at 15:21 +0200, Marek Olšák wrote: > On Tue, Oct 23, 2012 at 11:24 AM, Michel Dänzer wrote: > > From: Michel Dänzer > > > > The layer dimension of array textures is not subject to mipmap minification. > > OTOH we were missing an assertion for the depth dimension. > > > > Fixes

Re: [Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Marek Olšák
On Tue, Oct 23, 2012 at 11:24 AM, Michel Dänzer wrote: > From: Michel Dänzer > > The layer dimension of array textures is not subject to mipmap minification. > OTOH we were missing an assertion for the depth dimension. > > Fixes assertion failures with piglit {f,v}s-textureSize-sampler1DArrayShad

[Mesa-dev] [PATCH] st/mesa: Fix assertions for copying texture image to finalized miptree.

2012-10-23 Thread Michel Dänzer
From: Michel Dänzer The layer dimension of array textures is not subject to mipmap minification. OTOH we were missing an assertion for the depth dimension. Fixes assertion failures with piglit {f,v}s-textureSize-sampler1DArrayShadow. For some reason, they only resulted in piglit 'warn' results f

[Mesa-dev] [Bug 55998] Pretty huge slowdown in mesa 9.0

2012-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=55998 Rohan Garg changed: What|Removed |Added CC||rohan16g...@gmail.com -- You are receiving

Re: [Mesa-dev] OSMesa broken on ubuntu with version 9 push?

2012-10-23 Thread Rafi Witten
Thanks Jaime. I wrote a note on the bug report page but it is seemingly unrelated. On Sat, Oct 20, 2012 at 12:50 PM, Jaime Rave wrote: > 2012/10/19 Rafi Witten : > > Hi guys, > > > > First time posting so I'm probably just doing something stupid! > > > > I am using the libosmesa6 package on Ubun