Re: [Mesa-dev] GL_ARB_debug_output from drivers

2013-02-23 Thread Jose Fonseca
- Original Message - > One of the features Valve asked for, that they expected after using other > drivers, is that they can easily get information about performance traps > (and other important stuff) from drivers. We wrote INTEL_DEBUG=perf this > summer as a quick fix, but it's not how

[Mesa-dev] [Bug 61333] New: Gallium drivers no longer link with shared libglsl

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61333 Priority: medium Bug ID: 61333 Assignee: mesa-dev@lists.freedesktop.org Summary: Gallium drivers no longer link with shared libglsl Severity: minor Classification: Unclassified

[Mesa-dev] [Bug 59187] [Steam] Implement GLSL 1.30 (for older chipsets than SandyBridge)

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=59187 matthewjohnruss...@gmail.com changed: What|Removed |Added CC||matthewjohnruss...@gmail.co

Re: [Mesa-dev] [PATCH v2] vl: Fix off-by-one error in device_name_length allocation.

2013-02-23 Thread Michel Dänzer
On Sam, 2013-02-23 at 05:37 +0100, Vinson Lee wrote: > Fixes out-of-bounds write reported by Coverity. > > Signed-off-by: Vinson Lee Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Debian, X

[Mesa-dev] [Bug 61333] Gallium drivers no longer link with shared libglsl

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61333 --- Comment #1 from Fabio Pedretti --- See: https://bugs.launchpad.net/ubuntu/precise/+source/mesa-lts-quantal/+bug/1117417 http://anonscm.debian.org/gitweb/?p=pkg-xorg/lib/mesa.git;a=tree;f=debian/patches;hb=refs/heads/ubuntu -- You are receiv

Re: [Mesa-dev] [PATCH 4/4] r600g: enable CP DMA on r6xx (v2)

2013-02-23 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Fri, Feb 22, 2013 at 8:38 PM, wrote: > From: Alex Deucher > > With the previous flushing changes this seems to work > reliably now. > > v2: add R600_CONTEXT_FLUSH_AND_INV > > Signed-off-by: Alex Deucher > --- > src/gallium/drivers/r600/r600

[Mesa-dev] [PATCH] util/dump: Use static assertion to detect string table size mismatches.

2013-02-23 Thread jfonseca
From: José Fonseca Suggested by Brian Paul. Could probably be extended to other enums. --- src/gallium/auxiliary/util/u_dump_defines.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util/u_dump_defines.c b/src/gallium/auxiliary/u

Re: [Mesa-dev] [PATCH 6/6] r600g: enable CP DMA on r6xx (v3)

2013-02-23 Thread Marek Olšák
For the series: Reviewed-by: Marek Olšák Marek On Fri, Feb 22, 2013 at 11:59 PM, wrote: > From: Alex Deucher > > With the previous flushing changes this seems to work > reliably now. > > v2: add R600_CONTEXT_FLUSH_AND_INV > v3: just enable CP DMA > > Signed-off-by: Alex Deucher > --- > src

Re: [Mesa-dev] [PATCH] util/dump: Use static assertion to detect string table size mismatches.

2013-02-23 Thread Brian Paul
On 02/23/2013 03:51 AM, jfons...@vmware.com wrote: From: José Fonseca Suggested by Brian Paul. Could probably be extended to other enums. --- src/gallium/auxiliary/util/u_dump_defines.c | 19 ++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxi

[Mesa-dev] [PATCH] util/u_blitter: Set pipe_sampler_state::normalized_coords correctly.

2013-02-23 Thread jfonseca
From: José Fonseca We might want to revisit the normalized_coords semantics, but this is the current expected behavior. Fixes fdo bug 61091. --- src/gallium/auxiliary/util/u_blitter.c | 26 ++ 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/src/gallium/a

[Mesa-dev] [PATCH 1/9] st/mesa: remove #include mfeatures.h

2013-02-23 Thread Brian Paul
None of these were needed. --- src/mesa/state_tracker/st_cb_bitmap.c |1 - src/mesa/state_tracker/st_cb_bitmap.h |1 - src/mesa/state_tracker/st_cb_blit.c |1 - src/mesa/state_tracker/st_cb_blit.h |1 - src/mesa/state_tracker/st_cb_drawpixels.c |1 - src/mes

[Mesa-dev] [PATCH 2/9] mesa: remove #ifdef FEATURE_ES2, add some comments instead

2013-02-23 Thread Brian Paul
--- src/mesa/main/shaderapi.c | 11 +-- 1 files changed, 9 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c index be69467..8b160bc 100644 --- a/src/mesa/main/shaderapi.c +++ b/src/mesa/main/shaderapi.c @@ -1451,8 +1451,10 @@ _mesa_Validate

[Mesa-dev] [PATCH 3/9] mesa: remove old comment about FEATURE_GL

2013-02-23 Thread Brian Paul
--- src/mesa/main/get.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c index da1e01c..524418e 100644 --- a/src/mesa/main/get.c +++ b/src/mesa/main/get.c @@ -394,8 +394,7 @@ extra_NV_read_buffer_api_gl[] = { * remaining combina

[Mesa-dev] [PATCH 4/9] st/mesa: remove #if FEATURE_GL/ES tests

2013-02-23 Thread Brian Paul
--- src/mesa/state_tracker/st_manager.c |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/src/mesa/state_tracker/st_manager.c b/src/mesa/state_tracker/st_manager.c index a3a6771..cf96cba 100644 --- a/src/mesa/state_tracker/st_manager.c +++ b/src/mesa/state_tracker/st_

[Mesa-dev] [PATCH 5/9] glapi: remove FEATURE_remap_table test (it's always defined)

2013-02-23 Thread Brian Paul
--- src/mapi/glapi/gen/gl_table.py | 15 --- 1 files changed, 0 insertions(+), 15 deletions(-) diff --git a/src/mapi/glapi/gen/gl_table.py b/src/mapi/glapi/gen/gl_table.py index 382eaaf..99957f6 100644 --- a/src/mapi/glapi/gen/gl_table.py +++ b/src/mapi/glapi/gen/gl_table.py @@ -147

[Mesa-dev] [PATCH 6/9] mesa: remove FEATURE_remap_table from remap.[ch]

2013-02-23 Thread Brian Paul
It was always defined. --- src/mesa/main/remap.c |6 -- src/mesa/main/remap.h | 34 -- 2 files changed, 0 insertions(+), 40 deletions(-) diff --git a/src/mesa/main/remap.c b/src/mesa/main/remap.c index a098705..bc4ac9b 100644 --- a/src/mesa/main/remap.c +

[Mesa-dev] [PATCH 7/9] mesa: remove #include "mfeatures.h"

2013-02-23 Thread Brian Paul
None of the remaining FEATURE_x symbols in mfeatures.h are used anymore. --- src/mesa/drivers/dri/intel/intel_buffer_objects.c |1 - src/mesa/drivers/dri/intel/intel_extensions.c |1 - src/mesa/drivers/dri/intel/intel_fbo.c|1 - src/mesa/drivers/dri/intel/intel_screen.c

[Mesa-dev] [PATCH 8/9] glapi: no longer emit #include "mfeatures.h" in generated files

2013-02-23 Thread Brian Paul
None of the symbols in mfeatures.h are used anymore. --- src/mapi/glapi/gen/gl_enums.py |1 - src/mapi/glapi/gen/gl_genexec.py |1 - src/mapi/glapi/gen/gl_table.py |6 -- 3 files changed, 0 insertions(+), 8 deletions(-) diff --git a/src/mapi/glapi/gen/gl_enums.py b/src/mapi/gl

[Mesa-dev] [PATCH 9/9] mesa: remove mfeatures.h

2013-02-23 Thread Brian Paul
No longer needed. --- src/mesa/main/mfeatures.h | 53 - 1 files changed, 0 insertions(+), 53 deletions(-) delete mode 100644 src/mesa/main/mfeatures.h diff --git a/src/mesa/main/mfeatures.h b/src/mesa/main/mfeatures.h deleted file mode 100644 index 5

[Mesa-dev] [PATCH 0/9] remove mfeatures.h file

2013-02-23 Thread Brian Paul
This series removes the dependencies on the mfeatures.h file and the file itself. I'd appreciated someone doing a test build of this series to double-check my work. The last thing left to do is remove the -DFEATURE_GL/ES1/ES2 stuff from the autoconf and scons files. -Brian

Re: [Mesa-dev] [PATCH] util/u_blitter: Set pipe_sampler_state::normalized_coords correctly.

2013-02-23 Thread Brian Paul
On 02/23/2013 06:50 AM, jfons...@vmware.com wrote: From: José Fonseca We might want to revisit the normalized_coords semantics, but this is the current expected behavior. Fixes fdo bug 61091. --- src/gallium/auxiliary/util/u_blitter.c | 26 ++ 1 file changed, 22 ins

Re: [Mesa-dev] [PATCH] llvmpipe: Fix creation of shared and scanout textures.

2013-02-23 Thread John Kåre Alsaker
I'd still like to have this applied. On Tue, Dec 18, 2012 at 3:16 PM, John Kåre Alsaker wrote: > On Tue, Dec 18, 2012 at 11:20 AM, Jose Fonseca wrote: >> Looks fine. >> >> I'm curious, what does this fix in practice? > It fixes Weston crashing when running with llvmpipe. > >> >> Jose >> >> -

[Mesa-dev] [PATCH 1/3] dri: Add another duplicateImage to DRIimageExtension which allows you to create a sRGB view of a DRI image

2013-02-23 Thread John Kåre Alsaker
duplicateImage will allow you to create a linear or sRGB view into a DRIimage you have access to. This is useful because compositors may want a specific view which doesn't correspond to the one used by applications. --- include/GL/internal/dri_interface.h | 21 - 1 file chang

[Mesa-dev] [PATCH 2/3] gallium: Implement DRIimageExtension.duplicateImage

2013-02-23 Thread John Kåre Alsaker
--- src/gallium/include/state_tracker/st_api.h | 1 + src/gallium/state_trackers/dri/common/dri_screen.c | 1 + src/gallium/state_trackers/dri/common/dri_screen.h | 1 + src/gallium/state_trackers/dri/drm/dri2.c | 39 +- src/mesa/state_tracker/st_manager.c

[Mesa-dev] [PATCH 3/3] egl: Add MESA_image_sRGB extension.

2013-02-23 Thread John Kåre Alsaker
This gives applications access to use DRIimage.duplicateImage to create sRGB and linear views from EGL images. --- docs/MESA_image_sRGB.spec | 132 include/EGL/eglmesaext.h| 7 ++ src/egl/drivers/dri2/egl_dri2.c | 115 ++

[Mesa-dev] [Bug 61361] New: Version mismatch error. This is libtool 2.4.2, but the definition of this LT_INIT comes from libtool 2.2.8.

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61361 Priority: medium Bug ID: 61361 Assignee: mesa-dev@lists.freedesktop.org Summary: Version mismatch error. This is libtool 2.4.2, but the definition of this LT_INIT comes from libtool

[Mesa-dev] [Bug 61364] New: LLVM assertion when starting X11

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61364 Priority: medium Bug ID: 61364 Assignee: mesa-dev@lists.freedesktop.org Summary: LLVM assertion when starting X11 Severity: major Classification: Unclassified OS: Linux (All)

Re: [Mesa-dev] [PATCH 04/14] mesa: Fix _mesa_problem() on context destroy after application debug output

2013-02-23 Thread Jordan Justen
On Fri, Feb 22, 2013 at 7:52 PM, Eric Anholt wrote: > This was apparently not noticed because we don't have any testing of > application-generated debug output. However, as I'm changing the > GL-generated debug output to use the same path as > application/middleware-generated debug output, this o

Re: [Mesa-dev] [PATCH] llvmpipe: Fix creation of shared and scanout textures.

2013-02-23 Thread Jose Fonseca
Pushed now. - Original Message - > I'd still like to have this applied. > > On Tue, Dec 18, 2012 at 3:16 PM, John Kåre Alsaker > wrote: > > On Tue, Dec 18, 2012 at 11:20 AM, Jose Fonseca wrote: > >> Looks fine. > >> > >> I'm curious, what does this fix in practice? > > It fixes Weston c

Re: [Mesa-dev] [PATCH 03/14] mesa: Move debug type/severity enums to mesa core.

2013-02-23 Thread Jordan Justen
On Fri, Feb 22, 2013 at 7:52 PM, Eric Anholt wrote: > +static enum mesa_debug_type > +gl_enum_to_debug_type(GLenum e) > +{ > + int i; > + > + for (i = 0; i < Elements(debug_type_enums); i++) { I guess we have both Elements and ARRAY_SIZE. Should we try to merge these? Elements appears to be

[Mesa-dev] [Bug 61366] New: oprofilejit should be included in the list of LLVM components required

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61366 Priority: medium Bug ID: 61366 Assignee: mesa-dev@lists.freedesktop.org Summary: oprofilejit should be included in the list of LLVM components required Severity: minor Cl

Re: [Mesa-dev] [PATCH 12/14] mesa: Remove unused gl_winsys_error enum.

2013-02-23 Thread Jordan Justen
On Fri, Feb 22, 2013 at 7:52 PM, Eric Anholt wrote: > --- > src/mesa/main/mtypes.h |8 > 1 file changed, 8 deletions(-) > > diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h > index 11e1f9e..65c823d 100644 > --- a/src/mesa/main/mtypes.h > +++ b/src/mesa/main/mtypes.h > @@

Re: [Mesa-dev] GL_ARB_debug_output from drivers

2013-02-23 Thread Jordan Justen
Series Reviewed-by: Jordan Justen On Fri, Feb 22, 2013 at 7:52 PM, Eric Anholt wrote: > One of the features Valve asked for, that they expected after using other > drivers, is that they can easily get information about performance traps > (and other important stuff) from drivers. We wrote INTEL

Re: [Mesa-dev] [PATCH] util/u_blitter: Set pipe_sampler_state::normalized_coords correctly.

2013-02-23 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Sat, Feb 23, 2013 at 2:50 PM, wrote: > From: José Fonseca > > We might want to revisit the normalized_coords semantics, but this is > the current expected behavior. > > Fixes fdo bug 61091. > --- > src/gallium/auxiliary/util/u_blitter.c | 26 ++

[Mesa-dev] [Bug 61364] LLVM assertion when starting X11

2013-02-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61364 --- Comment #1 from John Kåre Alsaker --- Backtrace #0 0x7f40f703d2c5 in raise () from /usr/lib/libc.so.6 #1 0x7f40f703e748 in abort () from /usr/lib/libc.so.6 #2 0x7f40f7036312 in __assert_fail_base () from /usr/lib/libc.so.6 #3

[Mesa-dev] [PATCH] st/vega: Fix memory leak in combine_shaders.

2013-02-23 Thread Vinson Lee
Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee --- src/gallium/state_trackers/vega/shaders_cache.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/vega/shaders_cache.c b/src/gallium/state_trackers/vega/shaders_cache.c

[Mesa-dev] [PATCH] radeonsi: Fix memory leak in si_set_constant_buffer.

2013-02-23 Thread Vinson Lee
Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee --- src/gallium/drivers/radeonsi/si_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index 769ba0c..a395ec4 100644 --- a/src/ga