Re: [Mesa-dev] [PATCH 1/2] r600g: set CB_COLOR_INFO to INVALID for disabled colorbuffers on r600-r700

2012-08-16 Thread Michel Dänzer
On Mit, 2012-08-15 at 19:48 +0200, Marek Olšák wrote: > --- > src/gallium/drivers/r600/r600_state.c |3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/gallium/drivers/r600/r600_state.c > b/src/gallium/drivers/r600/r600_state.c > index d2ef68e..e024907 100644 > --- a/src/gallium/

[Mesa-dev] [Bug 53572] New: scons: *** Two environments with different actions were specified for the same target: mesa/src/glsl/glcpp/pp.o

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53572 Bug #: 53572 Summary: scons: *** Two environments with different actions were specified for the same target: mesa/src/glsl/glcpp/pp.o Classification: Unclassified Produ

[Mesa-dev] [PATCH] build/glsl: fix android build

2012-08-16 Thread Tapani Pälli
Commit 77a3efc6b907943903190b385fdf107c4acfcdca broke android build that sets its own value for GLSL_SRCDIR. Patch sets the value in Makefile.sources only if it is not defined previously. Signed-off-by: Tapani Pälli --- src/glsl/Makefile.sources | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[Mesa-dev] [PATCH] radeonsi: Fix symbol conflicts with r600g.

2012-08-16 Thread Michel Dänzer
From: Michel Dänzer Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50389 Signed-off-by: Michel Dänzer --- src/gallium/drivers/radeonsi/r600.h| 10 +- src/gallium/drivers/radeonsi/r600_blit.c | 10 +- src/gallium/drivers/radeonsi/r600_buffer.c |8 +- src/ga

[Mesa-dev] [Bug 51598] llvmpipe crashes with wine-1.5.7

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51598 --- Comment #1 from Vasily Khoruzhick 2012-08-16 10:17:55 UTC --- Anyone can take a look? It crashes on any d3d app startup, both on git master and on 8.0.4 -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You

[Mesa-dev] [Bug 51598] llvmpipe crashes with wine-1.5.7

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51598 --- Comment #2 from Vasily Khoruzhick 2012-08-16 10:40:28 UTC --- Just rebuilt mesa against llvm-3.0 and it works -- Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- Y

[Mesa-dev] [PATCH] mesa/dlopen: use HAVE_DLOPEN instead of _GNU_SOURCE

2012-08-16 Thread Tapani Pälli
Patches changes mesa to use 'HAVE_DLOPEN' defined by configure and Android.mk instead of _GNU_SOURCE for detecting dlopen capability. This makes dlopen to work also on Android where _GNU_SOURCE is not defined. Signed-off-by: Tapani Pälli --- Android.common.mk | 4 +++- configure.ac

Re: [Mesa-dev] [PATCH] radeonsi: Fix symbol conflicts with r600g.

2012-08-16 Thread Alex Deucher
On Thu, Aug 16, 2012 at 6:04 AM, Michel Dänzer wrote: > From: Michel Dänzer > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50389 > > Signed-off-by: Michel Dänzer Reviewed-by: Alex Deucher BTW, any reason not to rename the files at this point? Alex

Re: [Mesa-dev] [PATCH] radeonsi: Fix symbol conflicts with r600g.

2012-08-16 Thread Michel Dänzer
On Don, 2012-08-16 at 09:26 -0400, Alex Deucher wrote: > On Thu, Aug 16, 2012 at 6:04 AM, Michel Dänzer wrote: > > From: Michel Dänzer > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50389 > > > > Signed-off-by: Michel Dänzer > > Reviewed-by: Alex Deucher Thanks, Alex. > BTW

Re: [Mesa-dev] [PATCH] Remove 3D registers from compute command stream

2012-08-16 Thread Tom Stellard
Hi, In the future, could you use git format-patch to send patches. Overall, I think this looks OK, I just need to test it out a little bit. -Tom On Mon, Aug 13, 2012 at 09:05:29PM +0100, archibald wrote: > diff --git a/src/gallium/drivers/r600/evergreen_compute.c > b/src/gallium/drivers/r600/e

Re: [Mesa-dev] [PATCH] Remove 3D registers from compute command stream

2012-08-16 Thread Tom Stellard
On Wed, Aug 15, 2012 at 12:30:38PM -0400, Matt Harvey wrote: > Hey, I'd be interested in testing this to make sure it works, but I have a > HD4200 running r600g, and I don't think that card has opencl support at the > moment. Do you know if that would still be useful for testing, and what I > would

Re: [Mesa-dev] [PATCH] st/mesa: index can be negative in the PROGRAM_CONSTANT case

2012-08-16 Thread Brian Paul
On 08/12/2012 10:35 AM, Niels Ole Salscheider wrote: --- src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp index 39717b6..9f58312 100644 --- a/s

[Mesa-dev] [PATCH] mesa: silence coverity warning in _mesa_get_attachment()

2012-08-16 Thread Brian Paul
The warning is kind of bogus since the i >= ctx->Const.MaxColorAttachments will prevent out of bounds array accesses, but it's easily silenced. See http://bugs.freedesktop.org/show_bug.cgi?id=53426 --- src/mesa/main/fbobject.c |5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff -

Re: [Mesa-dev] [PATCH] i965/fs: Don't try 16-wide if 8-wide uses more than half the registers.

2012-08-16 Thread Paul Berry
On 15 August 2012 16:37, Kenneth Graunke wrote: > 16-wide programs use roughly twice as many registers as 8-wide, and we > don't support spilling in 16-wide. So if an 8-wide program uses more > than half the available GRFs, the 16-wide one almost certainly will fail > to compile during register

Re: [Mesa-dev] [PATCH] wayland-drm: close fd after the display is uninitialized

2012-08-16 Thread Kristian Høgsberg
On Wed, Aug 15, 2012 at 06:14:44PM +0200, Philipp Brüschweiler wrote: > This fixes a "kernel rejected pushbuf: Bad file descriptor" error on > wl_drm display destruction. Thanks, that looks good, applied. Kristian > --- > src/gallium/state_trackers/egl/wayland/native_drm.c | 5 +++-- > 1 Datei

Re: [Mesa-dev] [PATCH] mesa: do more teximage error checking for generic compressed formats

2012-08-16 Thread Brian Paul
On 08/15/2012 02:31 PM, Anuj Phogat wrote: On Tue, May 1, 2012 at 2:07 PM, Brian Paul mailto:bri...@vmware.com>> wrote: When glTexImage or glCopyTexImage is called with internalFormat being a generic compressed format (like GL_COMPRESSED_RGB) we need to do the same error chec

Re: [Mesa-dev] [PATCH] glsl: Fix array overflow.

2012-08-16 Thread Brian Paul
On 08/14/2012 07:40 PM, Stéphane Marchesin wrote: Otherwise we run past the end of the array and crash. NOTE: This is a candidate for the 8.0 branch. Signed-off-by: Stéphane Marchesin --- src/glsl/link_uniforms.cpp |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src

Re: [Mesa-dev] [PATCH] r600g-llvm: Crude fix for a race in initialization of the llvm backend

2012-08-16 Thread Mathias Fröhlich
Hi, > This is likely caused by the fact that the same LLVM Context is used > for all threads. Take a look at the comment starting at > src/gallium/auxilary/gallivm/lp_bld_init.c:314 > > You should be able to reproduce this error on llvmpipe as well. > Creating a new LLVM Context for each thread

Re: [Mesa-dev] [PATCH] mesa: silence coverity warning in _mesa_get_attachment()

2012-08-16 Thread Jose Fonseca
Given this sort of warning will pop-up all over the place, it would be nice to educate coverity without introducing any behaviour change or making logic statements unnecessarily more complex. For example, I wonder if adding: assert(ctx->Const.MaxColorAttachments <= Elements(fb->Attachment));

Re: [Mesa-dev] [PATCH 1/3] build: Require X11 pkg-config files

2012-08-16 Thread Matt Turner
On Wed, Aug 15, 2012 at 2:21 AM, Andreas Boll wrote: >> @@ -975,38 +941,19 @@ xyesno) >> fi >> >> # find the DRI deps for libGL >> -if test "$x11_pkgconfig" = yes; then >> -PKG_CHECK_MODULES([XCB],[x11-xcb xcb-glx >= 1.8.1]) > > Don't we need to check against xcb-glx >= 1.8.1

[Mesa-dev] [Bug 53572] scons: *** Two environments with different actions were specified for the same target: mesa/src/glsl/glcpp/pp.o

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

Re: [Mesa-dev] [PATCH] mesa/dlopen: use HAVE_DLOPEN instead of _GNU_SOURCE

2012-08-16 Thread Matt Turner
On Thu, Aug 16, 2012 at 3:59 AM, Tapani Pälli wrote: > Patches changes mesa to use 'HAVE_DLOPEN' defined by configure and Android.mk > instead of _GNU_SOURCE for detecting dlopen capability. This makes dlopen to > work also on Android where _GNU_SOURCE is not defined. > > Signed-off-by: Tapani Päl

[Mesa-dev] [Bug 51598] llvmpipe crashes with wine-1.5.7

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51598 --- Comment #3 from José Fonseca 2012-08-16 18:16:53 UTC --- I don't know why some many issues when running WINE. Unfortunately I don't have the time to chace them. I known that at one point the problem was not WINE per se, but the fact that WI

[Mesa-dev] [Bug 53608] New: lp_test_conv crashes on Sandy Bridge

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53608 Bug #: 53608 Summary: lp_test_conv crashes on Sandy Bridge Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: All Status: NEW

Re: [Mesa-dev] [PATCH] r600g-llvm: Crude fix for a race in initialization of the llvm backend

2012-08-16 Thread Tom Stellard
On Thu, Aug 16, 2012 at 07:44:53PM +0200, Mathias Fröhlich wrote: > > Hi, > > > This is likely caused by the fact that the same LLVM Context is used > > for all threads. Take a look at the comment starting at > > src/gallium/auxilary/gallivm/lp_bld_init.c:314 > > > > You should be able to reprod

Re: [Mesa-dev] [PATCH] r600g-llvm: Crude fix for a race in initialization of the llvm backend

2012-08-16 Thread Tom Stellard
On Thu, Aug 16, 2012 at 02:39:48PM -0400, Tom Stellard wrote: > On Thu, Aug 16, 2012 at 07:44:53PM +0200, Mathias Fröhlich wrote: > > > > Hi, > > > > > This is likely caused by the fact that the same LLVM Context is used > > > for all threads. Take a look at the comment starting at > > > src/gal

Re: [Mesa-dev] [PATCH] mesa: silence coverity warning in _mesa_get_attachment()

2012-08-16 Thread Brian Paul
Yeah, this could happen a lot. Vinson, can you try the assertion that Jose suggests below? -Brian On 08/16/2012 11:51 AM, Jose Fonseca wrote: Given this sort of warning will pop-up all over the place, it would be nice to educate coverity without introducing any behaviour change or making logi

Re: [Mesa-dev] [PATCH] mesa: silence coverity warning in _mesa_get_attachment()

2012-08-16 Thread Brian Paul
Actually, in this particular case, the assertion would have to be: assert(BUFFER_COLOR0 + ctx->Const.MaxColorAttachments <= Elements(fb->Attachment)); I'm not sure that Coverity will figure this out. -Brian On 08/16/2012 12:57 PM, Brian Paul wrote: Yeah, this could happen a lot. Vinson, c

[Mesa-dev] [Bug 53426] out-of-bounds access src/mesa/main/fbobject:222

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53426 --- Comment #1 from Brian Paul 2012-08-16 19:02:18 UTC --- This warning is kind of bogus. Jose suggested adding an assertion like this: assert(BUFFER_COLOR0 + ctx->Const.MaxColorAttachments <= Elements(fb->Attachment)); -- Configure bugma

Re: [Mesa-dev] R600 VDPAU 422 regression since r600g: make sure copying of all texture formats is accelerated

2012-08-16 Thread Marek Olšák
I have fixed this master. You were right - tiling on 422 was broken. Marek On Wed, Aug 8, 2012 at 1:04 AM, Andy Furniss wrote: > Marek Olšák wrote: >> >> Do you have any idea what could be wrong with the patch? Also could >> please tell me how to setup VDPAU and where to download the tests, so >

[Mesa-dev] OSMesa and Shared GLAPI

2012-08-16 Thread Kevin H. Hobbs
With shared GLAPI can Mesa be built so that I link the same application to GL and OSMesa and get onscreen rendering (or even accelerated rendering) rendering when I have DISPLAY set and OSMesa rendering when I don't? signature.asc Description: OpenPGP digital signature __

[Mesa-dev] [Bug 51598] llvmpipe crashes with wine-1.5.7

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=51598 --- Comment #4 from Vasily Khoruzhick 2012-08-16 20:51:40 UTC --- I'm using 32-bit system, so all apps are 32bit. Issue is reproducible only with wine (and looks like only with DirectX-apps, have no issue with openarena which is OpenGL I suppose

Re: [Mesa-dev] [PATCH 1/3] i965: Move hiz resolve to after renderbuffer resizing

2012-08-16 Thread Paul Berry
On 14 August 2012 16:58, Chad Versace wrote: > Do all pre-draw hiz resolves *after* the renderbuffers are resized by > intel_prepare_render. Otherwise, we may resolve buffers that are > immediately discarded afterwards. > > Fixes the assertion failure below when resizing windows in KDE and under

Re: [Mesa-dev] [PATCH] mesa: do more teximage error checking for generic compressed formats

2012-08-16 Thread Anuj Phogat
On Thu, Aug 16, 2012 at 10:23 AM, Brian Paul wrote: > On 08/15/2012 02:31 PM, Anuj Phogat wrote: >> >> On Tue, May 1, 2012 at 2:07 PM, Brian Paul > > wrote: >> >> When glTexImage or glCopyTexImage is called with internalFormat >> being a >> generic compressed

Re: [Mesa-dev] [PATCH] build/glsl: fix android build

2012-08-16 Thread Matt Turner
On Thu, Aug 16, 2012 at 2:13 AM, Tapani Pälli wrote: > Commit 77a3efc6b907943903190b385fdf107c4acfcdca broke android > build that sets its own value for GLSL_SRCDIR. Patch sets the value > in Makefile.sources only if it is not defined previously. > > Signed-off-by: Tapani Pälli > --- > src/glsl/

[Mesa-dev] [PATCH mesa] add EGL_TEXTURE_EXTERNAL_WL to WL_bind_wayland_display spec

2012-08-16 Thread Rob Clark
From: Rob Clark Signed-off-by: Rob Clark --- docs/WL_bind_wayland_display.spec |5 + include/EGL/eglmesaext.h |1 + 2 files changed, 6 insertions(+) diff --git a/docs/WL_bind_wayland_display.spec b/docs/WL_bind_wayland_display.spec index 02bd6ea..ce52e2d 100644 --- a/docs

[Mesa-dev] [PATCH weston] compositor: add support for OES_EGL_image_external

2012-08-16 Thread Rob Clark
From: Rob Clark In cases where the GPU can natively handle certain YUV formats, eglQueryWaylandBufferWL() can return the value EGL_TEXTURE_EXTERNAL_WL and the compositor will treat the buffer as a single egl-image-external. See: http://www.khronos.org/registry/gles/extensions/OES/OES_EGL_image_e

Re: [Mesa-dev] R600 VDPAU 422 regression since r600g: make sure copying of all texture formats is accelerated

2012-08-16 Thread Andy Furniss
Marek Olšák wrote: I have fixed this master. You were right - tiling on 422 was broken. Ok, thanks, working now. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [Bug 53426] out-of-bounds access src/mesa/main/fbobject:222

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53426 --- Comment #2 from Brian Paul 2012-08-16 23:06:25 UTC --- Alternately, can you try this patch, Vinson? diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c index 792a92d..03094cc 100644 --- a/src/mesa/main/fbobject.c +++ b/src/mesa

[Mesa-dev] [Bug 53616] New: [softpipe] sp_state_derived.c:308:update_polygon_stipple_enable: Assertion `unit >= softpipe->num_samplers[1]' failed.

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53616 Bug #: 53616 Summary: [softpipe] sp_state_derived.c:308:update_polygon_stipple_enable: Assertion `unit >= softpipe->num_samplers[1]' failed. Classification: Unclassified

[Mesa-dev] [Bug 53617] New: [llvmpipe] piglit fbo-depthtex regression

2012-08-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53617 Bug #: 53617 Summary: [llvmpipe] piglit fbo-depthtex regression Classification: Unclassified Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: Linux (All)

[Mesa-dev] X.Org Book Sprint 2012 (guide to writing graphics drivers)

2012-08-16 Thread Matt Dew
(Cross post. Yes, I know this is short notice and a short window to reply. My apologies on that.) X.Org Book Sprint 2012 Monday Sept 17 & Tuesday Sept 18. Nürnberg (Nuremberg), Germany. The X.Org Consortium will hold a book sprint on the Monday and Tuesday before the Developers Conference

Re: [Mesa-dev] [PATCH] build/glsl: fix android build

2012-08-16 Thread Tapani Pälli
On 08/17/2012 01:05 AM, Matt Turner wrote: > On Thu, Aug 16, 2012 at 2:13 AM, Tapani Pälli wrote: >> Commit 77a3efc6b907943903190b385fdf107c4acfcdca broke android >> build that sets its own value for GLSL_SRCDIR. Patch sets the value >> in Makefile.sources only if it is not defined previously. >>