[Mesa-dev] [PATCH] scons: Require drm to build gallium/state_trackers/egl/x11/x11_screen.c.

2012-12-03 Thread Vinson Lee
x11_screen.c includes xf86drm.h, which comes from libdrm-dev. This patch fixes this build error. Compiling src/gallium/state_trackers/egl/x11/x11_screen.c ... src/gallium/state_trackers/egl/x11/x11_screen.c:30:21: fatal error: xf86drm.h: No such file or directory Signed-off-by: Vinson Lee --

[Mesa-dev] [Bug 57866] New: src/gallium/auxiliary/os/os_time.c:77: error: ‘CLOCK_REALTIME’ undeclared (first use in this function)

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57866 Priority: medium Bug ID: 57866 CC: jfons...@vmware.com Assignee: mesa-dev@lists.freedesktop.org Summary: src/gallium/auxiliary/os/os_time.c:77: error: ‘CLOCK_REALTIME’

[Mesa-dev] [Bug 57863] New: [llvmpipe] SIGSEGV src/gallium/drivers/llvmpipe/lp_rast.c:407

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57863 Priority: medium Bug ID: 57863 Keywords: regression CC: jfons...@vmware.com Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] SIGSEGV src/gallium/d

Re: [Mesa-dev] [PATCH 5/5] driconf: Fix German translations by removing a couple of bogus backslashes

2012-12-03 Thread Ian Romanick
On 12/03/2012 03:43 PM, Carl Worth wrote: As can be seen, many other translation strings already include a single apostrophe just fine without any escaping. This strangely-escaped apostrophe was causing a build failure ("invalid escape sequence") resulting in no "de" translations in the final opt

Re: [Mesa-dev] [PATCH 2/2] mesa: Use the new hash table for the variable refcount visitor.

2012-12-03 Thread Ian Romanick
On 12/03/2012 02:52 PM, Jordan Justen wrote: From: Eric Anholt Were you able to measure a performance improvement on any of the problematic cases? Reviewed-by: Jordan Justen [jordan.l.jus...@intel.com: open_hash_table => hash_table] Signed-off-by: Jordan Justen --- src/glsl/Makefile.a

[Mesa-dev] [PATCH 2/3] mesa: Don't allow RGB interger formats in OpenGL ES 3.0

2012-12-03 Thread Ian Romanick
From: Ian Romanick Signed-off-by: Ian Romanick Cc: Matt Turner Cc: Marek Olšák --- src/mesa/main/fbobject.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c index ab53bac..281cdd0 100644 --- a/src/mesa/main/fbobject.

[Mesa-dev] [PATCH 3/3] intel: Fix copy-and-paste bug setting gl_constants::MaxSamples

2012-12-03 Thread Ian Romanick
From: Ian Romanick gl_constants::MaxSamples is an integer, so setting it to 1.0 is just silly. Signed-off-by: Ian Romanick --- src/mesa/drivers/dri/intel/intel_context.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/d

[Mesa-dev] [PATCH 1/3] mesa: Disallow deprecated SNORM formats for renderbuffers

2012-12-03 Thread Ian Romanick
From: Ian Romanick The OpenGL 3.2 core profile spec says: "The following base internal formats from table 3.11 are color-renderable: RED, RG, RGB, and RGBA. The sized internal formats from table 3.12 that have a color-renderable base internal format are also color-renderable. No

[Mesa-dev] [Bug 57852] swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 --- Comment #5 from Alexandre Demers --- (In reply to comment #4) > does > > http://lists.freedesktop.org/archives/mesa-dev/2012-December/030988.html > > help either case? > > we may need to fix the patch posted her to use CLOCK_LIB. Patch fr

[Mesa-dev] [Bug 57852] swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 --- Comment #4 from Dave Airlie --- does http://lists.freedesktop.org/archives/mesa-dev/2012-December/030988.html help either case? we may need to fix the patch posted her to use CLOCK_LIB. -- You are receiving this mail because: You are the

[Mesa-dev] [Bug 57852] swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 Matt Turner changed: What|Removed |Added CC||alexandre.f.dem...@gmail.co

[Mesa-dev] [Bug 57859] [bisected] os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 Matt Turner changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH automake] Add -I$(top_srcdir)/include to GALLIUM_CFLAGS.

2012-12-03 Thread Johannes Obermayr
--- src/gallium/Automake.inc |1 + 1 Datei geändert, 1 Zeile hinzugefügt(+) diff --git a/src/gallium/Automake.inc b/src/gallium/Automake.inc index ccdb7e0..fabc2af 100644 --- a/src/gallium/Automake.inc +++ b/src/gallium/Automake.inc @@ -1,4 +1,5 @@ GALLIUM_CFLAGS = \ + -I$(top_srcdir)/

[Mesa-dev] [PATCH automake after merge] llvmpipe: Fix build after *_soa removal.

2012-12-03 Thread Johannes Obermayr
--- src/gallium/drivers/llvmpipe/Makefile.am | 10 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 9 Zeilen entfernt(-) diff --git a/src/gallium/drivers/llvmpipe/Makefile.am b/src/gallium/drivers/llvmpipe/Makefile.am index df40186..36d8c22 100644 --- a/src/gallium/drivers/llvmpipe/Makefil

[Mesa-dev] [Bug 57859] [bisected] os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 --- Comment #3 from Alexandre Demers --- Missing "-lrt" in the linker command. -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-dev@lists.freede

Re: [Mesa-dev] [PATCH 2/2] intel: Always enable GL_ARB_framebuffer_object

2012-12-03 Thread Ian Romanick
On 12/03/2012 11:05 AM, Chad Versace wrote: On 12/01/2012 11:10 AM, Ian Romanick wrote: From: Ian Romanick Now that _mesa_BindFramebuffer does the right thing in ES contexts when the gl_extensions::ARB_framebuffer_object bit is set, the Intel driver doesn't need this hack. No piglit or GLES2

Re: [Mesa-dev] [PATCH] mesa: fix uint64 printing in syncobj.c

2012-12-03 Thread Ian Romanick
On 12/03/2012 10:05 AM, Brian Paul wrote: From: Brian Paul To silence printf format warnings. v2: insert "0x" prefix Reviewed-by: Ian Romanick --- src/mesa/main/syncobj.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/syncobj.c b/src/mesa/main

[Mesa-dev] [Bug 57859] [bisected] os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 --- Comment #2 from Alexandre Demers --- (In reply to comment #1) > Introduced by 16f0d70ffe6d42d22b9e6b927b297e75a199aa78 llvmpipe: Implement PIPE_QUERY_TIMESTAMP and PIPE_QUERY_TIME_ELAPSED. José Fonseca James Benton 12/3/12 2:00 AM gallivm:

[Mesa-dev] [Bug 57859] [bisected] os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 Alexandre Demers changed: What|Removed |Added Summary|os_time.c:77: undefined |[bisected] os_time.c:77:

[Mesa-dev] [Bug 57859] os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 --- Comment #1 from Alexandre Demers --- Introduced by 16f0d70ffe6d42d22b9e6b927b297e75a199aa78 -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa

[Mesa-dev] [Bug 57859] New: os_time.c:77: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57859 Priority: medium Bug ID: 57859 Assignee: mesa-dev@lists.freedesktop.org Summary: os_time.c:77: undefined reference to `clock_gettime' Severity: normal Classification: Unclassified

[Mesa-dev] [PATCHv2 automake] gallium/auxiliary: Add possibility to build libgallium shared.

2012-12-03 Thread Johannes Obermayr
--- configure.ac |7 +++ src/gallium/auxiliary/Makefile.am |8 src/gallium/targets/opencl/Makefile.am |2 +- 3 Dateien geändert, 16 Zeilen hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/configure.ac b/configure.ac index 02aaa64..316d200 10

[Mesa-dev] [Bug 57852] swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 --- Comment #2 from Aaron Watry --- Tested Laurent's patch successfully with r600g with mesa git version 21694b8eac. -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-

[Mesa-dev] [PATCHv2] mesa: Return INVALID_ENUM for glReadPixels(..., GL_DEPTH_*, ...) on ES 3

2012-12-03 Thread Matt Turner
I'm not sure if this is the correct fix. The _mesa_es_error_check_format_and_type function (used above in the ES 1 and 2 cases) was originally added for glTexImage checking and allows GL_DEPTH_STENCIL/GL_UNSIGNED_INT_24_8 combinations. Using it in ES 3 causes other tests to regress. Fixes es3confo

[Mesa-dev] [PATCH 10/10] st/dri: implement MSAA for GLX/DRI2 framebuffers

2012-12-03 Thread Marek Olšák
All MSAA buffers are allocated privately and resolved into the DRI-provided back and front buffers. If an MSAA visual is chosen, the buffers st/mesa receives are all multi-sample. st/mesa doesn't have access to the single-sample buffers in that case. --- .../state_trackers/dri/common/dri_drawable

[Mesa-dev] [PATCH 09/10] gallium: pass the current context to the flush_front state tracker function

2012-12-03 Thread Marek Olšák
I will later use the context to resolve an MSAA front buffer. --- src/gallium/include/state_tracker/st_api.h |5 - src/gallium/state_trackers/dri/common/dri_drawable.c |6 -- src/gallium/state_trackers/dri/common/dri_drawable.h |3 ++- src/gallium/state_trackers/dri/d

[Mesa-dev] [PATCH 08/10] st/dri: don't expose MSAA configs with accumulation buffer

2012-12-03 Thread Marek Olšák
--- src/gallium/state_trackers/dri/common/dri_screen.c | 14 +- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/dri/common/dri_screen.c b/src/gallium/state_trackers/dri/common/dri_screen.c index 96efa1f..df2cd3f 100644 --- a/src/gallium/state

[Mesa-dev] [PATCH 07/10] st/dri: refactor dri_fill_in_modes

2012-12-03 Thread Marek Olšák
- We can use a single loop for adding new configs. - The useless parameter depth_bits is removed. - The maximum number of samples is bumped to 32. - We can support Z16_UNORM and Z32_UNORM unconditionally since the zbuffers are private. --- src/gallium/state_trackers/dri/common/dri_screen.c | 14

[Mesa-dev] [PATCH 06/10] st/dri: always allocate private depth-stencil buffers

2012-12-03 Thread Marek Olšák
This disables DRI2 sharing of zbuffers. The window zbuffer is allocated just like any other texture - through resource_create. The idea of allocating a zbuffer through DRI2 isn't very useful with MSAA, where a single-sample zbuffer is useless. IIRC, the Intel driver does the same thing. --- src/

[Mesa-dev] [PATCH 05/10] st/mesa: implement CopyTexSubImage for MSAA framebuffers

2012-12-03 Thread Marek Olšák
Just use pipe->blit, which can do resolve, flipping, and format conversions. The util_blit_pixels codepath is still there for the cases where we have to force alpha to 1. This also turns on acceleration for copying GL_DEPTH_STENCIL. --- src/mesa/state_tracker/st_cb_texture.c | 179 ++

[Mesa-dev] [PATCH 04/10] gallium/u_blitter: fix conflict with u_memory.h

2012-12-03 Thread Marek Olšák
--- src/gallium/auxiliary/util/u_blitter.h |6 +- src/gallium/drivers/r600/r600_pipe.c |1 + src/gallium/drivers/r600/r600_state_common.c |1 + 3 files changed, 7 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_blitter.h b/src/gallium/auxiliar

[Mesa-dev] [PATCH 03/10] r600g: transfers of MSAA color textures should do the resolve

2012-12-03 Thread Marek Olšák
so that ReadPixels and various fallbacks work. --- src/gallium/drivers/r600/r600_texture.c | 37 ++- 1 file changed, 32 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/r600/r600_texture.c b/src/gallium/drivers/r600/r600_texture.c index 0054c5b..d6d1b3

[Mesa-dev] [PATCH 02/10] trace: dump pipe_resource::nr_samples

2012-12-03 Thread Marek Olšák
--- src/gallium/drivers/trace/tr_dump_state.c |1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/trace/tr_dump_state.c b/src/gallium/drivers/trace/tr_dump_state.c index c7e1e8f..4cc25d2 100644 --- a/src/gallium/drivers/trace/tr_dump_state.c +++ b/src/gallium/drivers/trace/

Re: [Mesa-dev] [PATCH] mesa: fix uint64 printing in syncobj.c

2012-12-03 Thread Chad Versace
On 12/03/2012 10:05 AM, Brian Paul wrote: > From: Brian Paul > > To silence printf format warnings. > > v2: insert "0x" prefix > --- Reviewed-by: Chad Versace ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mai

[Mesa-dev] [PATCH 5/5] driconf: Fix German translations by removing a couple of bogus backslashes

2012-12-03 Thread Carl Worth
As can be seen, many other translation strings already include a single apostrophe just fine without any escaping. This strangely-escaped apostrophe was causing a build failure ("invalid escape sequence") resulting in no "de" translations in the final options.h file. --- src/mesa/drivers/dri/commo

[Mesa-dev] [PATCH 3/5] driconf: Make the gen_xmlpool.py script executable.

2012-12-03 Thread Carl Worth
So that there's some chance that "make options.h" can even work. --- 0 files changed mode change 100644 => 100755 src/mesa/drivers/dri/common/xmlpool/gen_xmlpool.py diff --git a/src/mesa/drivers/dri/common/xmlpool/gen_xmlpool.py b/src/mesa/drivers/dri/common/xmlpool/gen_xmlpool.py old mode 1006

[Mesa-dev] [PATCH 2/5] driconf: Add some translations which have been available, but were not compiled

2012-12-03 Thread Carl Worth
These translations have existed in the de.po file, but were not in the generated options.h file. This was fixed by simply running "make options.h". --- src/mesa/drivers/dri/common/xmlpool/options.h | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/mesa/driv

[Mesa-dev] [PATCH 1/5] driconf: Add option definitions to source file, not generated target

2012-12-03 Thread Carl Worth
For the last two most-recently-added driconf options, their definition was manually added to options.h, a file which is intended to be automatically generated, (as part of support for translated driconf option descriptions). This means that these options would be eliminated if the generation step w

[Mesa-dev] [PATCH] automake/gallium: attempt to fix -lrt

2012-12-03 Thread Dave Airlie
From: Dave Airlie I don't think this will fix all the cases, but it should be correct to start with. Signed-off-by: Dave Airlie --- configure.ac | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 803c256..a133902 100644 --- a/configur

[Mesa-dev] [PATCH 2/2] mesa: Use the new hash table for the variable refcount visitor.

2012-12-03 Thread Jordan Justen
From: Eric Anholt Reviewed-by: Jordan Justen [jordan.l.jus...@intel.com: open_hash_table => hash_table] Signed-off-by: Jordan Justen --- src/glsl/Makefile.am |1 + src/glsl/builtin_compiler/Makefile.am |1 + src/glsl/ir_variable_refcount.cpp | 47

[Mesa-dev] [PATCH 1/2] program/hash_table.c: rename to program/prog_hash_table.c

2012-12-03 Thread Jordan Justen
Removes a collision of the object file name for main/hash_table and program/hash_table. Signed-off-by: Jordan Justen --- src/glsl/Makefile.am |4 +- src/glsl/SConscript|2 +- src/glsl/builtin_compiler/Makefile.am |2 +- src/glsl/glcpp/Makefile.a

[Mesa-dev] [PATCH automake] gallium/auxiliary: Add possibility to build libgallium shared.

2012-12-03 Thread Johannes Obermayr
--- configure.ac |7 +++ src/gallium/auxiliary/Makefile.am |8 2 Dateien geändert, 15 Zeilen hinzugefügt(+) diff --git a/configure.ac b/configure.ac index 241b480..e4021b7 100644 --- a/configure.ac +++ b/configure.ac @@ -729,6 +729,13 @@ fi AC_SUBST([SHA

[Mesa-dev] [Bug 57852] swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 --- Comment #1 from Laurent carlier --- Created attachment 70978 --> https://bugs.freedesktop.org/attachment.cgi?id=70978&action=edit Add librt linking -- You are receiving this mail because: You are the assignee for the bug.

Re: [Mesa-dev] [PATCH] i965: Allow INTEL_DEBUG=fs as a synonym for INTEL_DEBUG=wm.

2012-12-03 Thread Eric Anholt
Kenneth Graunke writes: > I keep accidentally trying to use it. "fs" is a sensible name for > fragment shader debugging, and "wm" is...not. It's also more symmetric > with "vs". > > Leave INTEL_DEBUG=wm because old habits die hard. Reviewed-by: Eric Anholt pgpwnGeQyMO6R.pgp Description: PGP

Re: [Mesa-dev] [PATCH V3 7/7] mesa: expose ARB_texture_cube_map_array in core contexts as well

2012-12-03 Thread Eric Anholt
Chris Forbes writes: > Signed-off-by: Chris Forbes > Reviewed-by: Dave Airlie Patch 4,6,7 are Reviewed-by: Eric Anholt pgpLMjdi47pIc.pgp Description: PGP signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.o

Re: [Mesa-dev] [PATCH V3 3/7] i965: fs: fix gen6 math operands in one place

2012-12-03 Thread Eric Anholt
Chris Forbes writes: > Signed-off-by: Chris Forbes > --- > src/mesa/drivers/dri/i965/brw_fs.cpp | 48 > +--- > 1 file changed, 23 insertions(+), 25 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp > b/src/mesa/drivers/dri/i965/brw_fs.cpp > ind

Re: [Mesa-dev] [PATCH V3 1/7] i965: Add various plumbing for cubemap arrays

2012-12-03 Thread Eric Anholt
Chris Forbes writes: > diff --git a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c > b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c > index fb9cb83..69669b3 100644 > --- a/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c > +++ b/src/mesa/drivers/dri/i965/brw_wm_sampler_state.c > @@ -248,7

Re: [Mesa-dev] [PATCH V3 5/7] i965: fs: Add fixup for textureSize on Gen6/7

2012-12-03 Thread Eric Anholt
Chris Forbes writes: > V2: Moved up into emit(ir_texture *) to avoid duplication and fix > ordering for Gen7; Gen6 math quirks moved into previous patches. > > Tested on Gen6 only; passes all the cube_map_array piglits. > > V3: Fixed weird whitespace > > Signed-off-by: Chris Forbes > --- > src/

[Mesa-dev] [PATCH automake] gallium/auxiliary: Add -fno-rtti to CXXFLAGS on LLVM >= 3.2.

2012-12-03 Thread Johannes Obermayr
Contrary to old build system automake fails without LLVM_CXXFLAGS. --- configure.ac |7 ++- src/gallium/auxiliary/Makefile.am |6 ++ 2 Dateien geändert, 12 Zeilen hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/configure.ac b/configure.ac index e64f643..241b4

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-12-03 Thread Andreas Boll
2012/12/3 Emil Velikov : > On 03/12/12 01:42, Marek Olšák wrote: >> On Mon, Dec 3, 2012 at 2:30 AM, Emil Velikov >> wrote: >>> On 03/12/12 00:15, Marek Olšák wrote: On Sun, Dec 2, 2012 at 9:48 PM, Emil Velikov wrote: snip > Hi Andreas > > I've been testing the branch for

Re: [Mesa-dev] [PATCH] mesa: Return INVALID_ENUM for glReadPixels(..., GL_DEPTH_*, ...) on ES 3

2012-12-03 Thread Anuj Phogat
On Fri, Nov 30, 2012 at 3:13 PM, Matt Turner wrote: > I'm not sure if this is the correct fix. The > _mesa_es_error_check_format_and_type function (used above in the ES 1 > and 2 cases) was originally added for glTexImage checking and allows > GL_DEPTH_STENCIL/GL_UNSIGNED_INT_24_8 combinations. Us

Re: [Mesa-dev] [PATCH] i965: Allow INTEL_DEBUG=fs as a synonym for INTEL_DEBUG=wm.

2012-12-03 Thread Chad Versace
On 12/03/2012 11:53 AM, Kenneth Graunke wrote: > I keep accidentally trying to use it. "fs" is a sensible name for > fragment shader debugging, and "wm" is...not. It's also more symmetric > with "vs". > > Leave INTEL_DEBUG=wm because old habits die hard. This has tripped me too. Reviewed-by: C

Re: [Mesa-dev] [PATCH] android: fix bison warning of conflicting outputs to file

2012-12-03 Thread Chad Versace
On 11/30/2012 09:58 AM, Ian Romanick wrote: > On 11/29/2012 11:52 PM, gro...@gmail.com wrote: >> From: Adrian Marius Negreanu >> >> Bison -o parameter expects a .c file. >> The corresponding .h filename is obtained >> by removing the extension of the initial .c. >> >> This was breaking compilation

[Mesa-dev] [PATCH] i965: Allow INTEL_DEBUG=fs as a synonym for INTEL_DEBUG=wm.

2012-12-03 Thread Kenneth Graunke
I keep accidentally trying to use it. "fs" is a sensible name for fragment shader debugging, and "wm" is...not. It's also more symmetric with "vs". Leave INTEL_DEBUG=wm because old habits die hard. --- src/mesa/drivers/dri/intel/intel_context.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [Mesa-dev] [PATCH 1/2 v2] mesa: Only require Gen'ed name for glBind{Framebuffer, Renderbuffer} on desktop

2012-12-03 Thread Matt Turner
On Mon, Dec 3, 2012 at 9:26 AM, Ian Romanick wrote: > From: Ian Romanick > > Desktop OpenGL implementations that support either > GL_ARB_framebuffer_object or OpenGL 3.0 must require names from > glGenFramebuffers for glBindFramebuffer. We have enforced this rule for > quite some time. However,

Re: [Mesa-dev] [PATCH 1/2 v2] mesa: Only require Gen'ed name for glBind{Framebuffer, Renderbuffer} on desktop

2012-12-03 Thread Kenneth Graunke
On 12/03/2012 09:26 AM, Ian Romanick wrote: From: Ian Romanick Desktop OpenGL implementations that support either GL_ARB_framebuffer_object or OpenGL 3.0 must require names from glGenFramebuffers for glBindFramebuffer. We have enforced this rule for quite some time. However, OpenGL ES 1.0, 2.

Re: [Mesa-dev] [PATCH 1/2] mesa: Only require Gen'ed name for glBindFramebuffer on desktop

2012-12-03 Thread Chad Versace
On 12/03/2012 09:15 AM, Ian Romanick wrote: > On 12/01/2012 12:41 PM, Kenneth Graunke wrote: >> On 12/01/2012 11:10 AM, Ian Romanick wrote: >>> From: Ian Romanick >>> >>> Desktop OpenGL implementations that support either >>> GL_ARB_framebuffer_object >>> or OpenGL 3.0 must require names from glGe

Re: [Mesa-dev] [PATCH 2/2] intel: Always enable GL_ARB_framebuffer_object

2012-12-03 Thread Chad Versace
On 12/01/2012 11:10 AM, Ian Romanick wrote: > From: Ian Romanick > > Now that _mesa_BindFramebuffer does the right thing in ES contexts when the > gl_extensions::ARB_framebuffer_object bit is set, the Intel driver doesn't > need this hack. > > No piglit or GLES2 conformance regressions observed

Re: [Mesa-dev] numeric_limits::infinity() on android

2012-12-03 Thread Chad Versace
On 12/01/2012 08:08 PM, Liu wrote: > Hi, Chad, > > I notice that you have submitted a fix android build for > glsl.(5497cc428fa7c6670d252d34f4a67c9498ae3895). as you pointed out, you > included "external/astl" and plan to submit the second half fix to upstream. > what's going on for the second ha

[Mesa-dev] [Bug 57852] New: swrast_dri.so.tmp: undefined reference to `clock_gettime'

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57852 Priority: medium Bug ID: 57852 CC: jfons...@vmware.com Assignee: mesa-dev@lists.freedesktop.org Summary: swrast_dri.so.tmp: undefined reference to `clock_gettime'

Re: [Mesa-dev] [PATCH 1/3] mesa: pass context parameter to gl_renderbuffer::Delete()

2012-12-03 Thread Brian Paul
On 12/03/2012 11:05 AM, Matt Turner wrote: On Fri, Nov 30, 2012 at 10:19 AM, Brian Paul wrote: We sometimes need a rendering context when deleting renderbuffers. Pass it explicitly instead of trying to grab a current context (which might be NULL). The next patch will make use of this. Note: t

Re: [Mesa-dev] [PATCH 1/3] mesa: pass context parameter to gl_renderbuffer::Delete()

2012-12-03 Thread Matt Turner
On Fri, Nov 30, 2012 at 10:19 AM, Brian Paul wrote: > We sometimes need a rendering context when deleting renderbuffers. > Pass it explicitly instead of trying to grab a current context > (which might be NULL). The next patch will make use of this. > > Note: this is a candidate for the stable bra

[Mesa-dev] [PATCH] mesa: fix uint64 printing in syncobj.c

2012-12-03 Thread Brian Paul
From: Brian Paul To silence printf format warnings. v2: insert "0x" prefix --- src/mesa/main/syncobj.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/syncobj.c b/src/mesa/main/syncobj.c index 745e982..cc541c8 100644 --- a/src/mesa/main/syncobj.c +++ b/

Re: [Mesa-dev] [PATCH] mesa: fix uint64 printing in syncobj.c

2012-12-03 Thread Ian Romanick
On 12/03/2012 07:37 AM, Brian Paul wrote: From: Brian Paul To silence printf format warnings. --- src/mesa/main/syncobj.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/syncobj.c b/src/mesa/main/syncobj.c index 745e982..5ecbbd9 100644 --- a/src/mesa

Re: [Mesa-dev] [PATCH] mesa: Disable GL_NV_primitive_restart extension in core contexts.

2012-12-03 Thread Ian Romanick
On 12/03/2012 12:34 AM, Kenneth Graunke wrote: The NV formulation of primitive restart is turned on/off with glEnableClientState/glDisableClientState. These two functions don't exist in core contexts, which mean that GL_NV_primitive_restart is essentially useless. I'd go one further... this ex

Re: [Mesa-dev] [PATCH] glcpp: Rewrite line-continuation support to act globally.

2012-12-03 Thread Carl Worth
Kenneth Graunke writes: > I'm pretty tempted by ralloc_rewrite_tail & ralloc_strn_rewrite_tail. I still don't like the lack of symmetry here. Maybe bite the bullet and make the first one ralloc_str_rewrite_tail? But that "rewrite_tail" keeps feeling so awkward. I think the real problem is that i

Re: [Mesa-dev] [PATCH] llvmpipe: Implement PIPE_QUERY_TIMESTAMP and PIPE_QUERY_TIME_ELAPSED.

2012-12-03 Thread Jose Fonseca
- Original Message - > On Mon, Dec 3, 2012 at 7:33 PM, Dave Airlie > wrote: > >> diff --git a/src/gallium/auxiliary/os/os_time.c > >> b/src/gallium/auxiliary/os/os_time.c > >> index 3e9d50a..4055125 100644 > >> --- a/src/gallium/auxiliary/os/os_time.c > >> +++ b/src/gallium/auxiliary/os/

[Mesa-dev] [PATCH 1/2 v2] mesa: Only require Gen'ed name for glBind{Framebuffer, Renderbuffer} on desktop

2012-12-03 Thread Ian Romanick
From: Ian Romanick Desktop OpenGL implementations that support either GL_ARB_framebuffer_object or OpenGL 3.0 must require names from glGenFramebuffers for glBindFramebuffer. We have enforced this rule for quite some time. However, OpenGL ES 1.0, 2.0, and 3.0 implementations are required to all

Re: [Mesa-dev] [PATCH 1/2] mesa: Only require Gen'ed name for glBindFramebuffer on desktop

2012-12-03 Thread Ian Romanick
On 12/01/2012 12:41 PM, Kenneth Graunke wrote: On 12/01/2012 11:10 AM, Ian Romanick wrote: From: Ian Romanick Desktop OpenGL implementations that support either GL_ARB_framebuffer_object or OpenGL 3.0 must require names from glGenFramebuffers for glBindFramebuffer. We have enforced this rule

Re: [Mesa-dev] [PATCH] mesa: Disable GL_NV_primitive_restart extension in core contexts.

2012-12-03 Thread Eric Anholt
Kenneth Graunke writes: > The NV formulation of primitive restart is turned on/off with > glEnableClientState/glDisableClientState. These two functions don't > exist in core contexts, which mean that GL_NV_primitive_restart is > essentially useless. > > However, leaving it on causes oglconform's

[Mesa-dev] [PATCH] mesa: fix uint64 printing in syncobj.c

2012-12-03 Thread Brian Paul
From: Brian Paul To silence printf format warnings. --- src/mesa/main/syncobj.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/syncobj.c b/src/mesa/main/syncobj.c index 745e982..5ecbbd9 100644 --- a/src/mesa/main/syncobj.c +++ b/src/mesa/main/syncobj.c

[Mesa-dev] [PATCH] st/mesa: add null pointer check in st_renderbuffer_delete()

2012-12-03 Thread Brian Paul
From: Brian Paul In my testing I haven't found any cases where we get a null context pointer, but it might still be possible. Check for null just to be safe. Note: This is a candidate for the stable branches. --- src/mesa/state_tracker/st_cb_fbo.c |8 1 files changed, 4 insertions

[Mesa-dev] [Bug 57802] including GL/gl.h and another gl header causes redefinition warnings

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57802 --- Comment #2 from Brian Paul --- Just curious: why are you including both OpenGL's gl.h and other Khronos (ES?) headers in the same file? The changes look OK to me and I can commit them later. -- You are receiving this mail because: You are

[Mesa-dev] [Bug 57755] [llvmpipe] src/gallium/auxiliary/gallivm/lp_bld_arit.c:1051:lp_build_lerp: Assertion `!type.sign' failed.

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57755 José Fonseca changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|mesa-dev@list

[Mesa-dev] [Bug 57699] lp_test_format regression

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57699 --- Comment #3 from José Fonseca --- *** Bug 57756 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list m

[Mesa-dev] [Bug 57756] [llvmpipe] piglit arb_sampler_objects-srgb-decode regression

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57756 José Fonseca changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 57699] lp_test_format regression

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=57699 José Fonseca changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH] gallivm: fix srgb format fetch

2012-12-03 Thread Jose Fonseca
Looks good to me too. Thanks Roland. Jose - Original Message - > From: Roland Scheidegger > > we need to rely on util code for fetching those, just like before > 9f06061d50f90bf425a5337cea1b0adb94a46d25. > Fixes bugs 57699 and 57756. > --- > src/gallium/auxiliary/gallivm/lp_bld_format_

[Mesa-dev] [Bug 56710] src/mapi/glapi/glapitemp.h:1640:1: warning: no previous prototype for ‘glReadBufferNV’ [-Wmissing-prototypes]

2012-12-03 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=56710 Alan Hourihane changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|mesa-dev@li

Re: [Mesa-dev] Please test the automake-gallium4 branch

2012-12-03 Thread Emil Velikov
On 03/12/12 01:42, Marek Olšák wrote: > On Mon, Dec 3, 2012 at 2:30 AM, Emil Velikov wrote: >> On 03/12/12 00:15, Marek Olšák wrote: >>> On Sun, Dec 2, 2012 at 9:48 PM, Emil Velikov >>> wrote: On 23/11/12 21:23, Andreas Boll wrote: > Hi mesa-list, > > I'm happy to announce that

Re: [Mesa-dev] [PATCH] llvmpipe: Implement PIPE_QUERY_TIMESTAMP and PIPE_QUERY_TIME_ELAPSED.

2012-12-03 Thread Dave Airlie
On Mon, Dec 3, 2012 at 7:33 PM, Dave Airlie wrote: >> diff --git a/src/gallium/auxiliary/os/os_time.c >> b/src/gallium/auxiliary/os/os_time.c >> index 3e9d50a..4055125 100644 >> --- a/src/gallium/auxiliary/os/os_time.c >> +++ b/src/gallium/auxiliary/os/os_time.c >> @@ -36,6 +36,7 @@ >> #include

Re: [Mesa-dev] [PATCH] llvmpipe: Implement PIPE_QUERY_TIMESTAMP and PIPE_QUERY_TIME_ELAPSED.

2012-12-03 Thread Dave Airlie
> diff --git a/src/gallium/auxiliary/os/os_time.c > b/src/gallium/auxiliary/os/os_time.c > index 3e9d50a..4055125 100644 > --- a/src/gallium/auxiliary/os/os_time.c > +++ b/src/gallium/auxiliary/os/os_time.c > @@ -36,6 +36,7 @@ > #include "pipe/p_config.h" > > #if defined(PIPE_OS_UNIX) > +# incl

[Mesa-dev] [PATCH] mesa: Disable GL_NV_primitive_restart extension in core contexts.

2012-12-03 Thread Kenneth Graunke
The NV formulation of primitive restart is turned on/off with glEnableClientState/glDisableClientState. These two functions don't exist in core contexts, which mean that GL_NV_primitive_restart is essentially useless. However, leaving it on causes oglconform's primitive-restart-nv tests to run in