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
> > wrote:
> > > The current code would just calculate min/max_index for the first prim
> > > unconditionally, which is wrong i
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
When using transform feedback, there are three circumstances in which
it is useful for Mesa to instruct a driver to stream out just a
portion of a varying slot (rather than the whole vec4):
(a) When a varying is smaller than a vec4, Mesa needs to instruct the
driver to stream out just the first on
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 a/src/mesa/state
- 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
> b/src/ga
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
> >
> > Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44151
> >
> > Probably fixes: https://bugs.freedesktop.org/show_bug.cgi?id=44007
> >
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
>
Looks good to me.
Acked-by: Maarten Lankhorst
~Maarten
___
me
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
--- a/src/mesa/state_tracker
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 f87adaa..53f21c
On drivers that set gl_shader_compiler_options::LowerClipDistance (for
example i965), references to gl_ClipDistance (a float[8] array) will
be converted to references to gl_ClipDistanceMESA (a vec4[2] array).
This patch modifies the linker so that requests for transform feedback
of gl_ClipDistance
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 lo
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 in
---
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, pars
https://bugs.freedesktop.org/show_bug.cgi?id=43332
Michel Dänzer changed:
What|Removed |Added
AssignedTo|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
From: Michel Dänzer
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
---
This introduces a potential race condition with
---
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
+++ b/sr
---
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 @@ link_assign_uniform
Hi,
On Thursday, December 29, 2011 13:35:19 Michel Dänzer wrote:
> From: Michel Dänzer
>
> 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
>
>
---
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
--- a/src/mesa/state_tracker/s
Hi,
On Thursday, December 29, 2011 15:34:24 Michel Dänzer wrote:
> From: Michel Dänzer
>
> 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.freedeskto
---
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
+++ b/src/gallium/auxiliar
---
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 a/sr
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
b/src/mesa/stat
---
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
--- a/src/mesa/state_tracker
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 usi
Reviewed-by: Marek Olšák
On Thu, Dec 29, 2011 at 6:16 PM, Paul Berry 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
> b/src/mesa/state_tracker/st_glsl_to_tgsi.c
On 29 December 2011 09:16, Paul Berry 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 intended for it to work on
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.
gl_
Hey Christian,
Op 26-12-11 14:00, Christian König schreef:
> Based on patches from Maarten Lankhorst
>
> Signed-off-by: Christian König
>
> diff --git a/src/gallium/include/pipe/p_context.h
> b/src/gallium/include/pipe/p_context.h
> index de79a9b..f7ee522 100644
> --- a/src/gallium/include/pipe
From: Michel Dänzer
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: https://bugs.freedesktop.org/show_bu
-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 c
https://bugs.freedesktop.org/show_bug.cgi?id=42645
--- Comment #3 from Stephane Marchesin 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
--- You are receiving this m
Hi,
Following a short series of cleanup following the gl_array_object change.
The most important change is the use ot an ffs based loop to compute
gl_array_object::_MaxElement. This change provides a noticable performance
improovement for my average workloads.
To implement this, a small set of
Signed-off-by: Mathias Fröhlich
---
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
+++ b/src/mesa/main/mtypes.h
@@ -199,19 +199,19 @
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
---
src/mesa/main/arr
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
>
> 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 built if configure c
https://bugs.freedesktop.org/show_bug.cgi?id=39017
--- Comment #6 from Peter Ward 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 immediate
cause of
https://bugs.freedesktop.org/show_bug.cgi?id=39017
Peter Ward changed:
What|Removed |Added
CC||peteraw...@gmail.com
--
Configure bugmail:
https://bugs.freedesktop.org/show_bug.cgi?id=44298
Vinson Lee 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 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 all gallium drivers, and req
https://bugs.freedesktop.org/show_bug.cgi?id=42645
--- Comment #4 from Christian Inci 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 checkered, but not with the
ga
https://bugs.freedesktop.org/show_bug.cgi?id=42645
Stephane Marchesin changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
45 matches
Mail list logo
