On Don, 2011-12-29 at 10:03 +0800, Yuanhan Liu wrote:
On Wed, Dec 28, 2011 at 12:07:08PM -0800, Eric Anholt wrote:
On Wed, 28 Dec 2011 13:54:43 +0800, Yuanhan Liu
yuanhan@linux.intel.com wrote:
The current code would just calculate min/max_index for the first prim
unconditionally,
Hi guys,
was doing some piglit comparisons between softpipe and llvmpipe and
noticed the fbo-alphatest-formats test fails on llvmpipe for all the
I8 formats.
Now I looked at the code generated for
lp_tile_soa.c:lp_tile_i8_unorm_unswizzle_4ub and it references a[i+0]
and a[i+1], if I change it to
This commit should generates less ARL instructions when dealing with indirect
addressing.
v2: fix glsl-fs-uniform-array-4 piglit test
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 56 +---
1 files changed, 34 insertions(+), 22 deletions(-)
diff --git
- Original Message -
It is #ifdef'd out, and is already called unconditionnaly a couple
lines above.
---
src/gallium/drivers/llvmpipe/lp_context.c |5 -
1 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_context.c
On Don, 2011-12-29 at 14:57 +0100, Mathias Fröhlich wrote:
On Thursday, December 29, 2011 13:35:19 Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44151
Probably fixes:
Hey,
Op 26-12-11 14:00, Christian König schreef:
Buffers for shader based decoding can now be
released without its component still being around.
Signed-off-by: Christian König deathsim...@vodafone.de
Looks good to me.
Acked-by: Maarten Lankhorst m.b.lankho...@gmail.com
~Maarten
On i965 Gen6 and above, gl_PointSize is stored in component W of the
first VUE slot (which corresponds to VERT_RESULT_PSIZ in the VUE map).
Normally we store varying floats in component X of a VUE slot, so we
need special case logic for gl_PointSize.
For Gen6, we do this with a . swizzle in
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 453 ++--
1 files changed, 294 insertions(+), 159 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 59ecb52..3523159 100644
---
This code was generating the gcc warning:
variable ‘clearValue’ set but not used [-Wunused-but-set-variable]
---
src/mesa/swrast/s_depth.c |9 -
1 files changed, 0 insertions(+), 9 deletions(-)
diff --git a/src/mesa/swrast/s_depth.c b/src/mesa/swrast/s_depth.c
index
Hi,
These patches reworks sightly glsl-to-tgsi ir_dereference* visitor.
The first one is self explanatory.
The second one factor ir_dereference* in a separate visitor ; by itself,
this change does not bring anything to code generation, however it should
make easier to handle case where variable
Previously, on i965 Gen6 and above, we weren't allocating space for
gl_ClipVertex in the VUE, since the VS was automatically converting it
to clip distances. This prevented transform feedback from being able
to capture gl_ClipVertex.
This patch goes aheads and allocates space for gl_ClipVertex
---
src/glsl/linker.cpp |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 452d8b5..6def821 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -1420,7 +1420,7 @@ private:
/**
* The name of the variable,
https://bugs.freedesktop.org/show_bug.cgi?id=43332
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
AssignedTo|dri-devel@lists.freedesktop
From: Michel Dänzer michel.daen...@amd.com
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44151
Probably fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44007
https://bugs.freedesktop.org/show_bug.cgi?id=43993
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 77aa0d1..d337f9b 100644
--- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
+++
---
src/glsl/link_uniforms.cpp |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index c7de480..b331db7 100644
--- a/src/glsl/link_uniforms.cpp
+++ b/src/glsl/link_uniforms.cpp
@@ -365,9 +365,9 @@
Hi,
On Thursday, December 29, 2011 13:35:19 Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44151
Probably fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44007
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 37 ++--
1 files changed, 24 insertions(+), 13 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 77aa0d1..59ecb52 100644
---
Hi,
On Thursday, December 29, 2011 15:34:24 Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
A fence is a screen object and can outlive the context it was created from.
The previous code would access freed memory in that case, resulting in
various problems.
Fixes:
---
src/gallium/auxiliary/util/u_debug.h |6 --
1 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_debug.h
b/src/gallium/auxiliary/util/u_debug.h
index b5ea405..448d799 100644
--- a/src/gallium/auxiliary/util/u_debug.h
+++
---
src/mesa/drivers/dri/i965/brw_gs.c |9 +
src/mesa/drivers/dri/i965/brw_gs.h |7 +++
src/mesa/drivers/dri/i965/brw_gs_emit.c|2 +-
src/mesa/drivers/dri/i965/gen7_sol_state.c |2 ++
4 files changed, 19 insertions(+), 1 deletions(-)
diff --git
This commit should generates less ARL instructions when dealing with indirect
addressing.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 56 +---
1 files changed, 34 insertions(+), 22 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 455 ++--
1 files changed, 296 insertions(+), 159 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 59ecb52..becb774 100644
---
Ok, disregard this patch.
I see this is a bad idea and will fix the issue in another way in nvfx.
Thanks,
Lucas
Am Donnerstag, den 29.12.2011, 01:12 +0100 schrieb Marek Olšák:
HI Lucas,
The fallback will be slower on Radeons for these two reasons:
- Texture transfers are implemented using
Reviewed-by: Marek Olšák mar...@gmail.com
On Thu, Dec 29, 2011 at 6:16 PM, Paul Berry stereotype...@gmail.com wrote:
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
On 29 December 2011 09:16, Paul Berry stereotype...@gmail.com wrote:
Arguments in favor: (1) Because of transform feedback's intended use
and its position in the pipeline, the distinction between varyings and
other vertex shader outputs is irrelevant; in all likelihood the spec
writers
This patch series allows transform feedback to work properly on the
built-in vertex shader output variables gl_PointSize, gl_ClipVertex,
and gl_ClipDistance. gl_PointSize and gl_ClipVertex were broken due
to bugs in the i965 driver, and were trivial to fix--those are fixed
in patches 1 and 2.
Hey Christian,
Op 26-12-11 14:00, Christian König schreef:
Based on patches from Maarten Lankhorst m.b.lankho...@gmail.com
Signed-off-by: Christian König deathsim...@vodafone.de
diff --git a/src/gallium/include/pipe/p_context.h
b/src/gallium/include/pipe/p_context.h
index de79a9b..f7ee522
From: Michel Dänzer michel.daen...@amd.com
A fence is a screen object and can outlive the context it was created from.
The previous code would access freed memory in that case, resulting in
various problems.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44151
Probably fixes:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/28/2011 11:14 AM, Eric Anholt wrote:
We didn't consume these flags in any way that would produce a
functional difference, but we might have some day.
---
src/mesa/drivers/dri/intel/intel_buffer_objects.c |4 +++-
1 files changed, 3
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/28/2011 11:14 AM, Eric Anholt wrote:
The new assert in intelEmitCopyBlit() gets angry if we don't align to
dwords. Rather than make the assert have a special case for height ==
1 on the assumption that the hardware doesn't use it in that
https://bugs.freedesktop.org/show_bug.cgi?id=42645
--- Comment #3 from Stephane Marchesin marche...@icps.u-strasbg.fr 2011-12-29
12:21:52 PST ---
Can you retry with git mesa? It might be fixed by my recent changes.
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
Signed-off-by: Mathias Fröhlich mathias.froehl...@web.de
---
src/mesa/main/mtypes.h | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index 107371e..451d442 100644
--- a/src/mesa/main/mtypes.h
+++
Depending on the installed shader type, different arrays are used
from gl_array_object. Provide helper functions that compute
the bitmask of these arrays that are finally enabled for a given
shader type. The will be used in a followup change.
Signed-off-by: Mathias Fröhlich
Use a bitmask approach to compute gl_array_object::_MaxElement.
To make this work reliably depending on the shader type actually used,
make use of the newly introduced typed bitmaks getters.
With this change I gain about 5% draw time on some osgviewer examples.
Signed-off-by: Mathias Fröhlich
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 12/20/2011 12:31 PM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
This adds a new tests directory at the top-level and some extra build
infrastructure. The tests use the Google C++ Testing Framework, and
they will only be
https://bugs.freedesktop.org/show_bug.cgi?id=39017
--- Comment #6 from Peter Ward peteraw...@gmail.com 2011-12-29 16:33:45 PST
---
Created attachment 54965
-- https://bugs.freedesktop.org/attachment.cgi?id=54965
gdb session
I'm also getting this crash. I've attached a log from gdb showing the
https://bugs.freedesktop.org/show_bug.cgi?id=39017
Peter Ward peteraw...@gmail.com changed:
What|Removed |Added
CC||peteraw...@gmail.com
https://bugs.freedesktop.org/show_bug.cgi?id=44298
Vinson Lee v...@vmware.com changed:
What|Removed |Added
AssignedTo|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
https://bugs.freedesktop.org/show_bug.cgi?id=44298
--- Comment #2 from Stephane Marchesin marche...@icps.u-strasbg.fr 2011-12-29
17:29:28 PST ---
I think it's a similar problem to the one discussed in
http://lists.freedesktop.org/archives/mesa-dev/2011-September/011578.html
The bug will affect
https://bugs.freedesktop.org/show_bug.cgi?id=42645
--- Comment #4 from Christian Inci chris.pcguy.i...@gmail.com 2011-12-29
22:08:36 PST ---
I've already tested earlier the day if this bug still occurs.
It won't. :-)
But another bug still does:
With the classic driver (i915) the main menu is
https://bugs.freedesktop.org/show_bug.cgi?id=42645
Stephane Marchesin marche...@icps.u-strasbg.fr changed:
What|Removed |Added
Status|NEW |RESOLVED
42 matches
Mail list logo
