Re: [Mesa-dev] glxgears is faster but 3D render is so slow

2013-03-20 Thread jupiter
Hi Brian, On 3/19/13, Brian Paul bri...@vmware.com wrote: It is fair to say, if running llvm driver in my local machine (a 32-bit CentOS 6.2 without VNC connection), it was indeed faster than the xlib driver. Seems to me that the llvm driver broken the xlib VNC connection which could be

[Mesa-dev] [PATCH] radeonsi: Emit pixel shader state even when only the vertex shader changed

2013-03-20 Thread Michel Dänzer
From: Michel Dänzer michel.daen...@amd.com Fixes random failures with piglit glsl-max-varyings. NOTE: This is a candidate for the 9.1 branch. Signed-off-by: Michel Dänzer michel.daen...@amd.com --- src/gallium/drivers/radeonsi/si_state_draw.c | 5 + 1 file changed, 5 insertions(+) diff

Re: [Mesa-dev] [PATCH] radeonsi: Emit pixel shader state even when only the vertex shader changed

2013-03-20 Thread Christian König
Am 20.03.2013 11:43, schrieb Michel Dänzer: From: Michel Dänzer michel.daen...@amd.com Fixes random failures with piglit glsl-max-varyings. NOTE: This is a candidate for the 9.1 branch. Signed-off-by: Michel Dänzer michel.daen...@amd.com Reviewed-by: Christian König christian.koe...@amd.com

Re: [Mesa-dev] glxgears is faster but 3D render is so slow

2013-03-20 Thread Brian Paul
On 03/20/2013 04:07 AM, jupiter wrote: Hi Brian, On 3/19/13, Brian Paulbri...@vmware.com wrote: It is fair to say, if running llvm driver in my local machine (a 32-bit CentOS 6.2 without VNC connection), it was indeed faster than the xlib driver. Seems to me that the llvm driver broken the

Re: [Mesa-dev] Error while compiling the MAPI directory

2013-03-20 Thread Jose Fonseca
You're building with scons right? Jose - Original Message - Hi, I used the latest mesa and I am still receiving the same errors. It works perfectly fine in Ubuntu though. Can somebody please tell in the file mapi_tmp.h how does the following constant included? #include

[Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Christoph Bumiller
Sorry, this has become longer than I anticipated ... I've been toying with adding support for TGSI_FILE_INPUT/OUTPUT arrays because, since I cannot allocate varyings in the same order that the register index specifies, I need it: === EXAMPLE: OUT[0], CLIPDIST[1], must be allocated at address

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

2013-03-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61364 --- Comment #6 from Jerome Glisse gli...@freedesktop.org --- Yeah i saw same issue on radeon -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list

Re: [Mesa-dev] Error while compiling the MAPI directory

2013-03-20 Thread Ritvik_Sharma
I am using Visual Studio. I found that all these missing constants like MAPI_TABLE_NUM_STATIC are getting there values in mapi_abi.py. Since I am building it in UEFI I am making [.inf] files and using them to generate the makefilesand not the makefiles given in the mesa kit. Could this be a

Re: [Mesa-dev] [PATCH] st/egl: Fix build after changes in src/egl/wayland/

2013-03-20 Thread Kristian Høgsberg
I pushed a different fix for this. The gallium egl code doesn't have support for buffer sharing via fd passing so we can't just ask the protocol code to advertise that, even if the kernel has the DRM_CAP_PRIME features. Instead we just pass 0 for the flags argument. thanks, Kristian On Tue,

Re: [Mesa-dev] [PATCH] i965: Add a driconf option to disable flush throttling.

2013-03-20 Thread Paul Berry
On 19 March 2013 17:10, Eric Anholt e...@anholt.net wrote: Paul Berry stereotype...@gmail.com writes: Normally when submitting the first batch buffer after a flush, we check whether the GPU has completed processing of the first batch buffer of the previous frame. If it hasn't, we wait

[Mesa-dev] [PATCH] meta: fix incorrect slice, r coordinate computation

2013-03-20 Thread Brian Paul
The arithmetic to convert a 3D texture slice to an R coordinate was incorrect. Found when MSVC warned of a divide by zero. Note that we don't actually ever hit this path. We don't decompress slices of 3D textures and we don't support 3D mipmap generation yet. --- src/mesa/drivers/common/meta.c

Re: [Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Roland Scheidegger
Am 20.03.2013 15:41, schrieb Christoph Bumiller: Sorry, this has become longer than I anticipated ... I've been toying with adding support for TGSI_FILE_INPUT/OUTPUT arrays because, since I cannot allocate varyings in the same order that the register index specifies, I need it: ===

Re: [Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Christoph Bumiller
On 20.03.2013 17:05, Roland Scheidegger wrote: Am 20.03.2013 15:41, schrieb Christoph Bumiller: Sorry, this has become longer than I anticipated ... I've been toying with adding support for TGSI_FILE_INPUT/OUTPUT arrays because, since I cannot allocate varyings in the same order that the

Re: [Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Roland Scheidegger
Am 20.03.2013 17:46, schrieb Christoph Bumiller: On 20.03.2013 17:05, Roland Scheidegger wrote: Am 20.03.2013 15:41, schrieb Christoph Bumiller: Sorry, this has become longer than I anticipated ... I've been toying with adding support for TGSI_FILE_INPUT/OUTPUT arrays because, since I cannot

Re: [Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Christoph Bumiller
On 20.03.2013 18:30, Roland Scheidegger wrote: Am 20.03.2013 17:46, schrieb Christoph Bumiller: On 20.03.2013 17:05, Roland Scheidegger wrote: Not sure I fully understand this, but I'm thinking whenever in doubt, use something close to what dx10 does since that's likely going to work

Re: [Mesa-dev] Error while compiling the MAPI directory

2013-03-20 Thread Jose Fonseca
SCons builds via Visual Studio compilers. So I assume by I am using Visual Studio you mean, no, I'm not using SCons... I'd strongly recommend using scons instead of replicating Mesa build with MSVC project, as Mesa build is extremely complex (a lot of code generation). If you are determined

[Mesa-dev] [Bug 62571] Mesa 9.0 uses missing nouveau library

2013-03-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=62571 Jesus Cortez jesus.corte...@gmail.com changed: What|Removed |Added Assignee|nouveau@lists.freedesktop.o

Re: [Mesa-dev] RFC: TGSI scalar arrays

2013-03-20 Thread Roland Scheidegger
Am 20.03.2013 19:09, schrieb Christoph Bumiller: On 20.03.2013 18:30, Roland Scheidegger wrote: Am 20.03.2013 17:46, schrieb Christoph Bumiller: On 20.03.2013 17:05, Roland Scheidegger wrote: Not sure I fully understand this, but I'm thinking whenever in doubt, use something close to what

[Mesa-dev] [Bug 62571] Mesa 9.0 uses missing nouveau library

2013-03-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=62571 Maarten Lankhorst m.b.lankho...@gmail.com changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [Bug 62571] Mesa 9.0 uses missing nouveau library

2013-03-20 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=62571 --- Comment #3 from Jesus Cortez jesus.corte...@gmail.com --- (In reply to comment #2) First of all, this is a bug with scientific linux' packaging probably. Second, are you sure that it doesn't exist? ldd /usr/lib/dri/nouveau_dri.so Yes,

[Mesa-dev] [PATCH 1/4] i965: Move brw_vs_prog_data::outputs_written into VUE map.

2013-03-20 Thread Paul Berry
Future patches will allow for there to be separate VUE maps when both a geometry shader and a vertex shader are in use. When this happens, we will want to have correspondingly separate outputs_written bitfields. Moving outputs_written into the VUE map will make this easy. For consistency with

[Mesa-dev] [PATCH 2/4] i965: Create a pointer in brw_context to the geometry output VUE map.

2013-03-20 Thread Paul Berry
Currently, the GPU pipeline has one active VUE map in effect at any given time--the one representing the layout of vertex data coming from the vertex shader. However, when geometry shaders are added, they will have their own independent VUE map. Later pipeline stages (clip, sf, fs) will need to

[Mesa-dev] [PATCH 3/4] i965: Use brw.vue_map_geom_out instead of VS output VUE map where appropriate.

2013-03-20 Thread Paul Berry
This patch modifies post-GS pipeline stages (transform feedback, clip, sf, fs) to refer to the VUE map through brw-vue_map_geom_out rather than brw-vs.prog_data-vue_map. This ensures that when geometry shader support is added, these pipeline stages will consult the geometry shader output VUE map

[Mesa-dev] [PATCH 4/4] i965/fs: Rename vp_outputs_written to input_slots_valid.

2013-03-20 Thread Paul Berry
With the introduction of geometry shaders, fragment inputs will no longer come exclusively from the vertex shader; sometimes they come from the geometry shader. So the name vp_outputs_written will become a misnomer. This patch renames vp_outputs_written to input_slots_valid, to reflect the true

Re: [Mesa-dev] Mismatch between Mesas dispatch table and the one used by the X server

2013-03-20 Thread Stefan Brüns
On Tuesday 12 March 2013 09:25:07 Ian Romanick wrote: On 03/10/2013 11:24 AM, Stefan Brüns wrote: Hi everyone, I have run into a problem leading to a crashing X server for a bunch of indirect GLX calls. This problem affects any OpenGL clients using indirect rendering and calling

Re: [Mesa-dev] [PATCH 1/2] build libgallium shared by default.

2013-03-20 Thread Johannes Obermayr
Am Dienstag, 19. März 2013, 21:36:47 schrieb Andreas Boll: 2013/3/19 Johannes Obermayr johannesoberm...@gmx.de: Am Montag, 18. März 2013, 15:38:31 schrieb Maarten Lankhorst: This is one of the 2 patches used in ubuntu for decreasing size of mesa build. The other one is more hacky, and

Re: [Mesa-dev] Mismatch between Mesas dispatch table and the one used by the X server

2013-03-20 Thread Ian Romanick
On 03/20/2013 02:43 PM, Stefan Brüns wrote: On Tuesday 12 March 2013 09:25:07 Ian Romanick wrote: On 03/10/2013 11:24 AM, Stefan Brüns wrote: Hi everyone, I have run into a problem leading to a crashing X server for a bunch of indirect GLX calls. This problem affects any OpenGL clients using

Re: [Mesa-dev] Mismatch between Mesas dispatch table and the one used by the X server

2013-03-20 Thread Stefan Brüns
On Wednesday 20 March 2013 15:47:24 you wrote: On 03/20/2013 02:43 PM, Stefan Brüns wrote: On Tuesday 12 March 2013 09:25:07 Ian Romanick wrote: On 03/10/2013 11:24 AM, Stefan Brüns wrote: Hi everyone, I have run into a problem leading to a crashing X server for a bunch of indirect

Re: [Mesa-dev] Mismatch between Mesas dispatch table and the one used by the X server

2013-03-20 Thread Stefan Brüns
On Wednesday 20 March 2013 15:47:24 you wrote: On 03/20/2013 02:43 PM, Stefan Brüns wrote: On Tuesday 12 March 2013 09:25:07 Ian Romanick wrote: On 03/10/2013 11:24 AM, Stefan Brüns wrote: Hi everyone, I have run into a problem leading to a crashing X server for a bunch of indirect

Re: [Mesa-dev] [PATCH v2 03/15] glsl: parse in/out types for interface blocks

2013-03-20 Thread Eric Anholt
Jordan Justen jordan.l.jus...@intel.com writes: Previously only 'uniform' was allowed for uniform blocks. Now, in/out can be parsed, but it will only be allowed for GLSL = 150. basic_interface_block: - UNIFORM NEW_IDENTIFIER '{' member_list '}' instance_name_opt ';' +

Re: [Mesa-dev] [PATCH v2 05/15] glsl parser: on desktop GL require GLSL 150 for instance names

2013-03-20 Thread Eric Anholt
Jordan Justen jordan.l.jus...@intel.com writes: Interface blocks in GLSL 150 allow an instance name to be used. Signed-off-by: Jordan Justen jordan.l.jus...@intel.com --- src/glsl/glsl_parser.yy | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git

Re: [Mesa-dev] [PATCH v2 06/15] glsl parser: allow in out for interface block members

2013-03-20 Thread Eric Anholt
Jordan Justen jordan.l.jus...@intel.com writes: Previously uniform blocks allowed for the 'uniform' keyword to be used with members of a uniform blocks. With interface blocks 'in' can be used on 'in' interface block members and 'out' can be used on 'out' interface block members. The

[Mesa-dev] i965 varying-index uniforms improvement

2013-03-20 Thread Eric Anholt
https://bugs.freedesktop.org/show_bug.cgi?id=61554 It's had more me toos than I would have expected, so I've marked all but 2 incidental patches as a candidate for 9.1. It's also fairly invasive, so I'm quite uncomfortable doing so. I've tested on gm45, snb, and ivb so far, and it seems to be

[Mesa-dev] [PATCH 01/13] i965/fs: Allow constant propagation into MACH.

2013-03-20 Thread Eric Anholt
This happens quite a bit with varying-index uniform loads. We could also do better by avoiding the MACH entirely, but there's no reason not to at least take this step. --- src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff

[Mesa-dev] [PATCH 02/13] i965/fs: Remove creation of a MOV instruction that's never used.

2013-03-20 Thread Eric Anholt
We weren't inserting it into the list, so it did nothing. This line was replaced by the MOV/MUL block above. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs.cpp |1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp

[Mesa-dev] [PATCH 03/13] i965/fs: Move varying uniform offset compuation into the helper func.

2013-03-20 Thread Eric Anholt
I'm going to want to change the math for gen7 using sampler LD instructions in a way that gets CSE to occur like we'd hope. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 16 +--- src/mesa/drivers/dri/i965/brw_fs.h |3

[Mesa-dev] [PATCH 06/13] i965/fs: Avoid inappropriate optimization with regs_written 1.

2013-03-20 Thread Eric Anholt
Right now we don't have anything with regs_written() 1 and !inst-mlen, but that's about to change. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs.cpp |6 ++ 1 file changed, 6 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp

[Mesa-dev] [PATCH 05/13] i965: Make the fragment shader pull constants index by dwords, not vec4s.

2013-03-20 Thread Eric Anholt
We want to load vec4s, since loading a vec4 instead of a dword is basically no increased latency. But for variable indexed access, the previous requirement of aligned vec4s for a sampler LD was hard to implement. Note that this change only affects those messages that use the surface format, like

[Mesa-dev] [PATCH 09/13] i965/fs: Clean up the setup of gen4 simd16 message destinations.

2013-03-20 Thread Eric Anholt
I think this makes it much more obvious what's going on here. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp |9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp

[Mesa-dev] [PATCH 08/13] i965/fs: Do CSE on gen7's varying-index pull constant loads.

2013-03-20 Thread Eric Anholt
This is our first CSE on a regs_written() 1 instruction, so it takes a bit of extra fixup. Reduces the number of loads on kwin's Lanczos shader from 12 to 2. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=61554 NOTE: This is a candidate for the 9.1 branch. ---

[Mesa-dev] [PATCH 11/13] i965/fs: Don't double-emit SEND dependency workarounds at control flow.

2013-03-20 Thread Eric Anholt
We weren't setting needs_dep[i] in the loops, so we'd continue on to potentially add the same workaround MOVs to the later basic block boundaries, too. We can either set needs_dep[i] to exit through the normal path, or we can just return since we know we're done. ---

[Mesa-dev] [PATCH 07/13] i965/fs: Improve performance of varying-index uniform loads on IVB.

2013-03-20 Thread Eric Anholt
Like we have done for the VS and for constant-index uniform loads, we use the sampler engine to get caching in front of the L3 to avoid tickling the IVB L3 bug. This is also a bit of a functional change, as we're now loading a vec4 instead of a single dword, though we're not taking advantage of

[Mesa-dev] [PATCH 10/13] i965/fs: Bake regs_written into the IR instead of recomputing it later.

2013-03-20 Thread Eric Anholt
For sampler messages, it depends on the target gen, and on gen4 SIMD16-sampler-on-SIMD8-execution we were returning 4 instead of 8 like we should. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs.cpp | 29 +++-

[Mesa-dev] [PATCH 12/13] i965/fs: Use LD messages for pre-gen7 varying-index uniform loads

2013-03-20 Thread Eric Anholt
This comes at a minor performance cost at the moment (-3.2% +/- 0.2%, n=14 on my GM45 forced to load all uniforms through the varying-index path), but we get a whole vec4 at a time to reuse in the next commit. NOTE: This is a candidate for the 9.1 branch. --- src/mesa/drivers/dri/i965/brw_fs.cpp

[Mesa-dev] [PATCH 13/13] i965/fs: Allow CSE on pre-gen7 varying-index uniform loads

2013-03-20 Thread Eric Anholt
All the other expression types allowed here have inst-mlen == 0, and this one has implied MRF writes for all of its payload, so nothing else in the implementation should need to change. Reduces SEND messages for loading from pull constants in kwin's Lanczos shader from 16 to 6. (Due to a