[Mesa-dev] [PATCH] i965: fix time query

2011-07-13 Thread Zou Nan hai
PIPE_CONTROL reported time stamp is a 64 bits value, toggles every 80ns. Signed-off-by: Zou Nan hai --- src/mesa/drivers/dri/i965/brw_queryobj.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c b/src/mesa/drivers/

[Mesa-dev] [Bug 39209] [bisected] Wrong display after "prefer native texture formats when possible" commit - part2

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39209 Alex Deucher changed: What|Removed |Added Attachment #49066|application/octet-stream|text/plain mime type|

[Mesa-dev] [Bug 39209] [bisected] Wrong display after "prefer native texture formats when possible" commit - part2

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39209 --- Comment #3 from Stephane Marchesin 2011-07-13 17:40:39 PDT --- Created an attachment (id=49066) --> (https://bugs.freedesktop.org/attachment.cgi?id=49066) Patch Does the attached patch help? -- Configure bugmail: https://bugs.freedesktop

Re: [Mesa-dev] I need some help about OGL extensions (Was Re: About merging pipe-video to master)

2011-07-13 Thread Marek Olšák
IIRC, the OpenGL spec says that alpha-test and blending should be done in the precision of a target colorbuffer. For example, if the colorbuffer is UNORM, fragment shader outputs are supposed to be clamped before blending (likely also before alpha-test). I am not sure how R600 is flexible, but it

Re: [Mesa-dev] [PATCH 01/12] intel: Use the post-execution batchbuffer contents for dumping.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I'm witholding rb's and acks' on patches 1, 9, 10, 11. I don't understand them well enough for my tags to hold weight. - -- Chad Versace c...@chad-versace.us -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with M

Re: [Mesa-dev] [PATCH 07/12] i965: Remove i915 paths from brw_update_draw_buffers().

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/i965/brw_vtbl.c | 48 > -- > 1 files changed, 11 insertions(+), 37 deletions(-) Reviewed-by: Chad Versace - -- Chad Versace c...@chad-versac

Re: [Mesa-dev] [PATCH 12/12] i915: Add support for HW rendering with no color draw buffer.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > This is useful for shadow map generation. Tested with glsl-bug-22603, > which rendered the depth textures with fallbacks before. > --- > src/mesa/drivers/dri/i915/i915_vtbl.c | 98 > +---

Re: [Mesa-dev] [PATCH 08/12] i915: Remove i965 paths from i915_update_drawbuffer() and i830's too.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/i915/i830_vtbl.c | 46 +++- > src/mesa/drivers/dri/i915/i915_vtbl.c | 47 > +++-- > 2 files changed, 20 insertions(+),

Re: [Mesa-dev] [PATCH 06/12] i965: Remove unused region calculations in brw_update_draw_buffer().

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/i965/brw_vtbl.c | 62 + > 1 files changed, 2 insertions(+), 60 deletions(-) Reviewed-by: Chad Versace - -- Chad Versace c...@chad-versace.us

Re: [Mesa-dev] [PATCH 05/12] i965: Remove empty brw_set_draw_region.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/i965/brw_vtbl.c | 14 -- > 1 files changed, 0 insertions(+), 14 deletions(-) Reviewed-by: Chad Versace - -- Chad Versace c...@chad-versace.us -BEGIN PGP SIGN

Re: [Mesa-dev] [PATCH 04/12] i965: Remove FALLBACK() from brw_update_draw_region().

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > The 965 driver doesn't use these for deciding on fallbacks. > --- > src/mesa/drivers/dri/i965/brw_vtbl.c | 16 > 1 files changed, 0 insertions(+), 16 deletions(-) Reviwed-by: Chad Versac

Re: [Mesa-dev] [PATCH 03/12] intel: Move intel_draw_buffers() code into each driver.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > The illusion of shared code here wasn't fooling anybody. It was > tempting to keep i830 and i915 still shared, but I think I actually > want to make them diverge shortly. > --- > src/mesa/drivers/dri/i915/i

[Mesa-dev] [Bug 39211] New: if VA enabled, dependency not checked

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39211 Summary: if VA enabled, dependency not checked Product: Mesa Version: git Platform: Other OS/Version: All Status: NEW Severity: normal Priority: medium Com

Re: [Mesa-dev] [PATCH 02/12] intel: Clarify the depthRb == stencilRb logic.

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/12/2011 03:22 PM, Eric Anholt wrote: > --- > src/mesa/drivers/dri/intel/intel_fbo.c | 31 +++ > 1 files changed, 15 insertions(+), 16 deletions(-) > > diff --git a/src/mesa/drivers/dri/intel/intel_fbo.c > b/src/me

[Mesa-dev] [Bug 39210] New: incompatible types when assigning to type ‘struct VADriverVTable *’ from type ‘struct VADriverVTable’

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39210 Summary: incompatible types when assigning to type ‘struct VADriverVTable *’ from type ‘struct VADriverVTable’ Product: Mesa Version: git Platform: All OS/Version: All

Re: [Mesa-dev] [PATCH] linker: Only over-ride built-ins when a prototype has been seen

2011-07-13 Thread Chad Versace
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 07/11/2011 10:50 AM, Ian Romanick wrote: > From: Ian Romanick > > The GLSL spec says: > > "If a built-in function is redeclared in a shader (i.e., a > prototype is visible) before a call to it, then the linker will > only attempt to r

Re: [Mesa-dev] [PATCH] utils: Add missing parentheses

2011-07-13 Thread Brian Paul
On 07/13/2011 04:19 PM, Emil Velikov wrote: Signed-off-by: Emil Velikov --- src/gallium/auxiliary/util/u_video.h |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/gallium/auxiliary/util/u_video.h b/src/gallium/auxiliary/util/u_video.h index 78cceb6..46da136 10064

[Mesa-dev] [Bug 39209] [bisected] Wrong display after "prefer native texture formats when possible" commit - part2

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39209 --- Comment #2 from Victor Tseng 2011-07-13 15:35:47 PDT --- Created an attachment (id=49065) --> (https://bugs.freedesktop.org/attachment.cgi?id=49065) window only redraw part of itself on maximize -- Configure bugmail: https://bugs.freedeskt

[Mesa-dev] [Bug 39209] [bisected] Wrong display after "prefer native texture formats when possible" commit - part2

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39209 --- Comment #1 from Victor Tseng 2011-07-13 15:34:57 PDT --- Created an attachment (id=49064) --> (https://bugs.freedesktop.org/attachment.cgi?id=49064) window only repaint on resize -- Configure bugmail: https://bugs.freedesktop.org/userprefs

[Mesa-dev] [Bug 39209] New: [bisected] Wrong display after "prefer native texture formats when possible" commit - part2

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39209 Summary: [bisected] Wrong display after "prefer native texture formats when possible" commit - part2 Product: Mesa Version: git Platform: x86-64 (AMD64) OS/Version: Linux (All)

[Mesa-dev] [PATCH 2/2] glsl: Ensure that sampler declarations are always uniform or "in" parameters.

2011-07-13 Thread Paul Berry
This brings us into compliance with page 17 (page 22 of the PDF) of the GLSL 1.20 spec: "[Sampler types] can only be declared as function parameters or uniform variables (see Section 4.3.5 "Uniform"). ... [Samplers] cannot be used as out or inout function parameters." The spec isn't e

[Mesa-dev] [PATCH 1/2] glsl: Move type_contains_sampler() into glsl_type for later reuse.

2011-07-13 Thread Paul Berry
The new location, as a member function of glsl_type, is more consistent with queries like is_sampler(), is_boolean(), is_float(), etc. Placing the function inside glsl_type also makes it available to any code that uses glsl_types. --- src/glsl/glsl_types.cpp | 16 src/glsl/glsl

[Mesa-dev] [PATCH 2/2] gallium: don't use enum bitfields in p_video_state.h

2011-07-13 Thread Brian Paul
Silences many warnings about "type of bit-field ‘field_select’ is a GCC extension". --- src/gallium/include/pipe/p_video_state.h |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/include/pipe/p_video_state.h b/src/gallium/include/pipe/p_video_state.h inde

[Mesa-dev] [PATCH 1/2] gallium: put video-related enums in separate header

2011-07-13 Thread Brian Paul
The forward references to video enum types in p_context.h causes a massive number of compiler warnings (ISO C forbids forward references to ‘enum’ types). By putting the new video enums in a separate header that can be included by p_context.h and p_screen.h we can avoid this. --- src/gallium/auxi

[Mesa-dev] [PATCH] utils: Add missing parentheses

2011-07-13 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/auxiliary/util/u_video.h |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/gallium/auxiliary/util/u_video.h b/src/gallium/auxiliary/util/u_video.h index 78cceb6..46da136 100644 --- a/src/gallium/auxiliary/util/u_video.h ++

Re: [Mesa-dev] [PATCH 0/10] fs shader debugging

2011-07-13 Thread Ben Widawsky
On Wed, Jul 13, 2011 at 01:51:42PM -0700, Ben Widawsky wrote: > > > Version 2 of the patch series is pretty much the same as version 1. 2 of the > patches have already been picked up by the kernel and mesa so they are > gone. > > The only major change is in mesa where I no longer load a binary b

Re: [Mesa-dev] [PATCH 04/10] i965: setup system routine

2011-07-13 Thread Chris Wilson
On Wed, 13 Jul 2011 13:51:46 -0700, Ben Widawsky wrote: > Upload the system routine as part of the invariant state if debugging. > > Remove SIP setting if not debugging to make it more friendly for others > that may be debugging shaders or media kernels. > > Signed-off-by: Ben Widawsky > --- >

[Mesa-dev] [PATCH 06/10] i965: attach to a listening debugger

2011-07-13 Thread Ben Widawsky
Use unix domain sockets to connect to a debugger and send the information the debugger will use to properly handle debug events. Signed-off-by: Ben Widawsky --- src/mesa/drivers/dri/i965/brw_wm_debug.c | 85 - 1 files changed, 82 insertions(+), 3 deletions(-) diff

[Mesa-dev] [PATCH 05/10] i965: emit breakpoints

2011-07-13 Thread Ben Widawsky
Provide a function to allow emitting breakpoints in the instruction oword. Use breakpoints when debugging. Signed-off-by: Ben Widawsky --- src/mesa/drivers/dri/i965/brw_eu.c |6 ++ src/mesa/drivers/dri/i965/brw_eu.h |1 + src/mesa/drivers/dri/i965/brw_wm.c |3 +++ 3 files changed

[Mesa-dev] [PATCH 04/10] i965: setup system routine

2011-07-13 Thread Ben Widawsky
Upload the system routine as part of the invariant state if debugging. Remove SIP setting if not debugging to make it more friendly for others that may be debugging shaders or media kernels. Signed-off-by: Ben Widawsky --- src/mesa/drivers/dri/i965/brw_misc_state.c | 13 + 1 files

[Mesa-dev] [PATCH 03/10] i965: Reserve scratch space for debugger communication

2011-07-13 Thread Ben Widawsky
Since the debug system routine will share scratch space with threads doing register spilling, we must offset the registers to accommodate. This is more easily accomplished (and less bug prone) in Mesa. The debugger will only work with the new fragment shader backend. Signed-off-by: Ben Widawsky

[Mesa-dev] [PATCH 0/10] fs shader debugging

2011-07-13 Thread Ben Widawsky
Version 2 of the patch series is pretty much the same as version 1. 2 of the patches have already been picked up by the kernel and mesa so they are gone. The only major change is in mesa where I no longer load a binary blob from an environment variable, but instead compile the bytes directly in

[Mesa-dev] Follow up on potential inlining bug (was: [PATCH 06/11] glsl: Refactor logic for determining whether to lower return statements.)

2011-07-13 Thread Paul Berry
On 7 July 2011 10:53, Kenneth Graunke wrote: > Furthermore, I think can_inline probably ought to be altered.  Right now > it returns true if there's only one "return".  But it doesn't seem to be > checking that it's the _canonical_ return. > > Without lower_jumps, I think the following code would

Re: [Mesa-dev] I need some help about OGL extensions (Was Re: About merging pipe-video to master)

2011-07-13 Thread Christoph Bumiller
On 13.07.2011 12:29, Christian König wrote: > Hi Henri and everybody else on the list, > > Am Dienstag, den 12.07.2011, 19:01 +0200 schrieb Henri Verbeet: >> I guess my point was mostly that there's not much of a point in doing >> the clamping both through BLEND_CLAMP and the fragment shader. Also,

Re: [Mesa-dev] [PATCH] mesa/program: split reference program into hot/cold paths.

2011-07-13 Thread Dave Airlie
On Wed, Jul 13, 2011 at 4:53 PM, Brian Paul wrote: > On Fri, Jun 24, 2011 at 7:20 AM, Brian Paul wrote: >> On 06/23/2011 11:22 PM, Dave Airlie wrote: >>> >>> From: Dave Airlie >>> >>> inline the hotpath of the reference remaining the same, this shouldn't >>> penalise the slow path at all but impr

[Mesa-dev] [PATCH 2/2] i915: Simplify intel_wpos_* with a helper function.

2011-07-13 Thread Eric Anholt
--- src/mesa/drivers/dri/i915/i915_fragprog.c |9 +--- src/mesa/drivers/dri/i915/intel_tris.c | 64 ++- src/mesa/drivers/dri/intel/intel_context.h |1 - 3 files changed, 27 insertions(+), 47 deletions(-) diff --git a/src/mesa/drivers/dri/i915/i915_fragprog.c

[Mesa-dev] [PATCH 1/2] i915: Include gl_FragCoord.w data, not just xyz.

2011-07-13 Thread Eric Anholt
Fixes piglit fragcoord_w test. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=34323 --- src/mesa/drivers/dri/i915/i915_fragprog.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/dri/i915/i915_fragprog.c b/src/mesa/drivers/dri/i915/i915_fragpr

Re: [Mesa-dev] [PATCH] mesa/program: split reference program into hot/cold paths.

2011-07-13 Thread Dave Airlie
On Wed, Jul 13, 2011 at 7:26 PM, Brian Paul wrote: > On Wed, Jul 13, 2011 at 12:04 PM, Dave Airlie wrote: >> On Wed, Jul 13, 2011 at 4:53 PM, Brian Paul wrote: > [...] >>> Dave, were you going to commit this patch? >>> >> >> I'm on holidays/moving around the world, so its sitting on my laptop >>

[Mesa-dev] [PATCH 2/2] configure: fix gcc version check

2011-07-13 Thread Marcin Baczyński
--- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index ef9f4b2..28c8e2f 100644 --- a/configure.ac +++ b/configure.ac @@ -89,7 +89,7 @@ if test "x$GCC" = xyes -a "x$CLANG" = xno; then GCC_VERSION=`$CC -dumpversion` if t

[Mesa-dev] [PATCH 1/2] configure: allow C{,XX}FLAGS override

2011-07-13 Thread Marcin Baczyński
--- Without this patch CFLAGS=-fstrict-aliasing ./autogen.sh ... has no effect, as the flag is followed by -fno-strict-aliasing added by configure. configure.ac |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/configure.ac b/configure.ac index eace790..ef9f4b2 10

Re: [Mesa-dev] [PATCH] mesa/program: split reference program into hot/cold paths.

2011-07-13 Thread Brian Paul
On Wed, Jul 13, 2011 at 12:04 PM, Dave Airlie wrote: > On Wed, Jul 13, 2011 at 4:53 PM, Brian Paul wrote: [...] >> Dave, were you going to commit this patch? >> > > I'm on holidays/moving around the world, so its sitting on my laptop > here, probably get to it once I start back to work. OK, if y

Re: [Mesa-dev] I need some help about OGL extensions (Was Re: About merging pipe-video to master)

2011-07-13 Thread Christian König
Am Mittwoch, den 13.07.2011, 18:29 +0200 schrieb Christoph Bumiller: > > As the name of this (well documented) hardware flag already implies: It > > just disabled clamping the output of the fragment shader BEFORE blending > > to a normalized buffer. > >From ARB_color_buffer_float: > > 3. How does

Re: [Mesa-dev] [PATCH] st/mesa: flush bitmap cache on query and conditional render boundaries

2011-07-13 Thread Brian Paul
On 07/12/2011 11:01 PM, Vadim Girlin wrote: Bitmap caching shouldn't affect the results of the queries and conditional render. Signed-off-by: Vadim Girlin Pushed, thanks. -Brian ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.

Re: [Mesa-dev] [PATCH] mesa/program: split reference program into hot/cold paths.

2011-07-13 Thread Brian Paul
On Fri, Jun 24, 2011 at 7:20 AM, Brian Paul wrote: > On 06/23/2011 11:22 PM, Dave Airlie wrote: >> >> From: Dave Airlie >> >> inline the hotpath of the reference remaining the same, this shouldn't >> penalise the slow path at all but improve the hot path so we don't have >> to jump to the function

[Mesa-dev] [Bug 38629] openarena textures blacked out

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38629 --- Comment #1 from Emil Velikov 2011-07-13 07:57:36 PDT --- This issue seems to be nouveau/nvfx related as openarena works perfectly on my system - both swrast and nouveau/nv50. Can you reproduce the issue with the swrast/llvm driver [1]? Che

[Mesa-dev] [Bug 39147] egl_gallium.so: errors when linking

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=39147 Chia-I Wu changed: What|Removed |Added Status|NEW |RESOLVED Resolution|

Re: [Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)

2011-07-13 Thread Brian Paul
On 07/13/2011 07:55 AM, Jose Fonseca wrote: - Original Message - On Mon, Jul 11, 2011 at 3:20 PM, Jose Fonseca wrote: - Original Message - On 07/09/2011 07:03 AM, Jose Fonseca wrote: I heard no concerns so I went ahead and made a branch where: - I removed GLUT - derived Mesa

[Mesa-dev] [Bug 38629] openarena textures blacked out

2011-07-13 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=38629 Andrew Randrianasulu changed: What|Removed |Added CC||rand...@mail.ru -- Configure bug

Re: [Mesa-dev] Merging glsl-to-tgsi to master

2011-07-13 Thread Bryan Cain
On Tue, Jul 12, 2011 at 12:00 PM, Ian Romanick wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 07/11/2011 09:53 AM, Egon Ashrafinia wrote: > > Hello guys. > > > > 1 month ago, we talked about merging glsl-to-tgsi to master but it still > > not happend. What about now? I could comp

Re: [Mesa-dev] robust-tarballs branch (Was: Error building on Windows with SCons)

2011-07-13 Thread Jose Fonseca
- Original Message - > On Mon, Jul 11, 2011 at 3:20 PM, Jose Fonseca > wrote: > > - Original Message - > >> On 07/09/2011 07:03 AM, Jose Fonseca wrote: > >> > I heard no concerns so I went ahead and made a branch where: > >> > - I removed GLUT > >> > - derived Mesa tarballs' file

[Mesa-dev] [PATCH] st/mesa: flush bitmap cache on query and conditional render boundaries

2011-07-13 Thread Vadim Girlin
Bitmap caching shouldn't affect the results of the queries and conditional render. Signed-off-by: Vadim Girlin --- src/mesa/state_tracker/st_cb_condrender.c |5 + src/mesa/state_tracker/st_cb_queryobj.c |5 + 2 files changed, 10 insertions(+), 0 deletions(-) diff --git a/src/m

[Mesa-dev] I need some help about OGL extensions (Was Re: About merging pipe-video to master)

2011-07-13 Thread Christian König
Hi Henri and everybody else on the list, Am Dienstag, den 12.07.2011, 19:01 +0200 schrieb Henri Verbeet: > I guess my point was mostly that there's not much of a point in doing > the clamping both through BLEND_CLAMP and the fragment shader. Also, I > guess we need this for EG+ as well. Oh, I got