https://bugs.freedesktop.org/show_bug.cgi?id=64091
Michel Dänzer mic...@daenzer.net changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop.
Do not propagate a copy if source and destination are identical.
Otherwise code like
MOV TEMP[0].xyzw, TEMP[0].wzyx
MOV TEMP[1].xyzw, TEMP[0].xyzw
is changed to
MOV TEMP[0].xyzw, TEMP[0].wzyx
MOV TEMP[1].xyzw, TEMP[0].wzyx
This fixes Piglit test shaders/glsl-copy-propagation-self-2 for
Do not propagate a copy if source and destination are identical.
Otherwise code like
MOV TEMP[0].xyzw, TEMP[0].wzyx
MOV TEMP[1].xyzw, TEMP[0].xyzw
is changed to
MOV TEMP[0].xyzw, TEMP[0].wzyx
MOV TEMP[1].xyzw, TEMP[0].wzyx
This fixes Piglit test shaders/glsl-copy-propagation-self-2 for
https://bugs.freedesktop.org/show_bug.cgi?id=63404
Bug 63404 depends on bug 54111, which changed state.
Bug 54111 Summary: [IVB]I-G-T prime test/module_reload fail with *ERROR*
“Memory manager not clean. Delaying takedown”
https://bugs.freedesktop.org/show_bug.cgi?id=54111
What
- Original Message -
MSVC doesn't like pointer arithmetic with void * so use GLubyte *.
---
src/mesa/swrast/s_texfetch_tmp.h |8
src/mesa/swrast/s_texfilter.c|2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/swrast/s_texfetch_tmp.h
https://bugs.freedesktop.org/show_bug.cgi?id=34645
José Fonseca jfons...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=34644
José Fonseca jfons...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=49519
José Fonseca jfons...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=35168
José Fonseca jfons...@vmware.com changed:
What|Removed |Added
Status|NEW |RESOLVED
- Original Message -
What is IA?
Input Assembly, a D3D10 term that roughly matches pipe_vertex_elements /
pipe_vertex_buffer state.
BTW, I think that Chris Forbes makes a good point -- a GS might choose to
not emit any vertices (e.g., that does fancy culling) -- so
Tom, Jose,
On Tuesday, April 30, 2013 16:56:56 Tom Stellard wrote:
I took the linker script from your email and took at shot at creating
libMesaLLVM.so within Mesa. I've pushed my initial code here:
http://cgit.freedesktop.org/~tstellar/mesa/log/?h=libmesallvm
Thank you very much and sorry
https://bugs.freedesktop.org/show_bug.cgi?id=64091
Jerome Glisse gli...@freedesktop.org changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop
Primitive culling in the geometry shader can be faster if emitting
lots of primitives is a bottleneck, for example rendering to multiple
viewports. The geometry shader execution is not slow per se. It's the
stuff around it that makes it slow. (the primitives might need to be
stored in video memory
- Original Message -
Primitive culling in the geometry shader can be faster if emitting
lots of primitives is a bottleneck, for example rendering to multiple
viewports. The geometry shader execution is not slow per se. It's the
stuff around it that makes it slow. (the primitives might
Without this patch, radeon_uvd failed to find the libdrm includes:
In file included from radeon_uvd.c:48:
../../winsys/radeon/drm/radeon_winsys.h:44:35: error:
libdrm/radeon_surface.h: No such file or directory
Signed-off-by: Lauri Kasanen c...@gmx.com
---
src/gallium/drivers/radeon/Makefile.am
Just like radeon/uvd, r600/sb fails to find the libdrm includes.
Signed-off-by: Lauri Kasanen c...@gmx.com
---
src/gallium/drivers/r600/Makefile.am |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/r600/Makefile.am
b/src/gallium/drivers/r600/Makefile.am
Hi list
The recently added r600 sb backend fails to build on GCC 4.3, since
it uses binary constants (0b0101).
Is the GCC version dependency intentional, or should the constants be
changed to int/hex?
- Lauri
___
mesa-dev mailing list
This is a funny subject. Originally, we only used SURFACE_SYNC on
Evergreen, which is what the hw guys recommend using, but then Jerome
came and rewrote it with no reasonable argument to back it up (what he
was trying to fix by his cache-flush rework is not fixed to this day),
such that we now
This should fix build issues with GCC 4.3
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
---
cc: Lauri Kasanen c...@gmx.com
Lauri, please test to make sure that I didn't miss anything.
src/gallium/drivers/r600/r600_shader.c | 6 +++---
src/gallium/drivers/r600/sb/sb_bc.h |
On 02/25/2013 11:55 AM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
The shadow comparitor needs to be loaded into the Z component of the
last DWord.
Fixes es3conform's shadow_execution_vert and oglconform's
shadow-grad advanced.textureGrad.1D tests on Haswell.
NOTE: This
On 02/25/2013 11:51 AM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
GLSL provides gradients for the 'r' coordinate (face ID), while our
hardware apparently ignores them. Sadly, this means that sample_d and
sample_d_c appear to be unsuitable for OpenGL, and need to be
ping
2013/4/19 Andreas Boll andreas.boll@gmail.com
Not used anymore.
---
Note: Untested
src/mesa/SConscript |1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/SConscript b/src/mesa/SConscript
index 3cd0f87..b87e40d 100644
--- a/src/mesa/SConscript
+++
On 04/21/2013 04:08 PM, Todd Previte wrote:
Initial work on the implementation of GL_ARB_shading_language_420pack.
The patch adds the functionality from this extension which allows
for C-style array initialization for GLSL. The extension enable bits
and extension definition are also included in
On Wed, 1 May 2013 21:35:16 +0400
Vadim Girlin vadimgir...@gmail.com wrote:
This should fix build issues with GCC 4.3
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
---
cc: Lauri Kasanen c...@gmx.com
Lauri, please test to make sure that I didn't miss anything.
This patch fixes the
On Wed, May 1, 2013 at 11:54 AM, Andreas Boll
andreas.boll@gmail.com wrote:
ping
2013/4/19 Andreas Boll andreas.boll@gmail.com
Not used anymore.
---
Note: Untested
src/mesa/SConscript |1 -
1 file changed, 1 deletion(-)
diff --git a/src/mesa/SConscript
Without this, the X lib path was not properly passed for tests/:
/usr/bin/ld: cannot find -lXvMCW
/usr/bin/ld: cannot find -lXvMC
/usr/bin/ld: cannot find -lXv
/usr/bin/ld: cannot find -lX11
collect2: ld returned 1 exit status
Signed-off-by: Lauri Kasanen c...@gmx.com
---
Hi
Running R600_DEBUG=sb glxgears on a RV710 gives wrong output:
http://i40.tinypic.com/t7gx09.png
This is on current master, git-8eef6ad.
Let me know what you need to debug this.
- Lauri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
On 05/01/2013 11:42 PM, Lauri Kasanen wrote:
Hi
Running R600_DEBUG=sb glxgears on a RV710 gives wrong output:
http://i40.tinypic.com/t7gx09.png
This is on current master, git-8eef6ad.
Let me know what you need to debug this.
Please send me the output with R600_DEBUG=sb,ps,vs
Vadim
On 05/01/2013 11:36 PM, Lauri Kasanen wrote:
Hi Vadim
Now that it built, I could test your optimizations in my own apps.
These are on current master 8eef6ad, on a RV710 (HD 4350 pci-e).
In one of my private apps, using R600_DEBUG=sb caused regressions: FPS
went from 28 to 7, the SSAO shader
On 04/19/2013 12:35 PM, Jordan Justen wrote:
Convert interface blocks with instance names into flat
interface blocks without an instance name.
Signed-off-by: Jordan Justen jordan.l.jus...@intel.com
---
src/glsl/Makefile.sources |1 +
src/glsl/ir_optimization.h
I've run these tests only on my implementation of
EXT_framebuffer_multisample_blit_scaled
extension on i965 drivers. NVIDIA, AMD machines I've don't support this
extension.
Could somebody try running on NVIDIA drivers?
Anuj Phogat (4):
Add functionality to create a FBO with multisample
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
tests/spec/ext_framebuffer_multisample/common.cpp | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/tests/spec/ext_framebuffer_multisample/common.cpp
b/tests/spec/ext_framebuffer_multisample/common.cpp
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
tests/all.tests| 6 +
.../ext_framebuffer_multisample/CMakeLists.gl.txt | 1 +
.../blit-scaled-glsl.cpp | 268 +
3 files changed, 275 insertions(+)
create
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
tests/all.tests| 1 +
.../ext_framebuffer_multisample/CMakeLists.gl.txt | 1 +
.../negative-blit-scaled.cpp | 111 +
3 files changed, 113 insertions(+)
create
Anuj Phogat (3):
mesa: Implement ext_framebuffer_multisample_blit_scaled extension
intel: Add multisample scaled blitting in blorp engine
i965: Enable ext_framebuffer_multisample_blit_scaled on intel h/w
src/mesa/drivers/dri/i965/brw_blorp.h | 16 +-
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/fbobject.c | 17 ++---
src/mesa/main/mtypes.h | 1 +
3 files changed, 16 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
In traditional multisampled framebuffer rendering, color samples must be
explicitly
resolved via BlitFramebuffer before doing the scaled blitting of the
framebuffer.
So, scaled blitting of a multisample framebuffer takes two separate calls to
BlitFramebuffer.
This patch implements the
This patch enables ext_framebuffer_multisample_blit_scaled extension
on intel h/w = gen6.
Note: Patches for piglit tests to verify this functionality are out
for review on piglit mailing list. Tests pass for all of the scaling
factors except 1.3 and 1.8. I'm still investigating what's so special
On Wed, May 1, 2013 at 2:10 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
@@ -1063,22 +1176,21 @@ brw_blorp_blit_program::translate_tiling(bool
old_tiled_w, bool new_tiled_w)
* X' = (X ~0b1011) 1 | (Y 0b1) 2 | X 0b1 (4)
* Y' = (Y ~0b1) 1 | (X 0b1000) 2 | (X
On Wed, May 1, 2013 at 1:58 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
I've run these tests only on my implementation of
EXT_framebuffer_multisample_blit_scaled
extension on i965 drivers. NVIDIA, AMD machines I've don't support this
extension.
Could somebody try running on NVIDIA
On 04/21/2013 04:08 PM, Todd Previte wrote:
Initial work on the implementation of GL_ARB_shading_language_420pack.
The patch adds the functionality from this extension which allows
for C-style array initialization for GLSL. The extension enable bits
and extension definition are also included in
On 05/01/2013 03:10 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogatanuj.pho...@gmail.com
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/fbobject.c | 17 ++---
src/mesa/main/mtypes.h | 1 +
3 files changed, 16 insertions(+), 3 deletions(-)
diff --git
On Wed, May 1, 2013 at 2:44 PM, Matt Turner matts...@gmail.com wrote:
On Wed, May 1, 2013 at 2:10 PM, Anuj Phogat anuj.pho...@gmail.com wrote:
@@ -1063,22 +1176,21 @@ brw_blorp_blit_program::translate_tiling(bool
old_tiled_w, bool new_tiled_w)
* X' = (X ~0b1011) 1 | (Y 0b1) 2 |
The GPU apparently goes looking for constants even though there are no
shader stages enabled, and gets stuck because we haven't told it there are
no constants to collect. If any other user of the 3D pipeline had run
(even the Render accel of the X server!) since power on, then the in-GPU
constant
On 05/01/2013 04:28 PM, Eric Anholt wrote:
The GPU apparently goes looking for constants even though there are no
shader stages enabled, and gets stuck because we haven't told it there are
no constants to collect. If any other user of the 3D pipeline had run
(even the Render accel of the X
On Wed, May 01, 2013 at 04:11:51PM +0200, Mathias Fröhlich wrote:
Tom, Jose,
On Tuesday, April 30, 2013 16:56:56 Tom Stellard wrote:
I took the linker script from your email and took at shot at creating
libMesaLLVM.so within Mesa. I've pushed my initial code here:
It's valid. Some shaders do the negation on unsigned and then
use the results in opcodes taking signed integers.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/gallivm/lp_bld_tgsi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
instead of crashing just fill zeros at the input slots that don't
match, that's the mandated behavior and it avoids debug asserts.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_gs.c | 93 --
1 file changed, 54 insertions(+), 39
Am 02.05.2013 03:13, schrieb Zack Rusin:
It's valid. Some shaders do the negation on unsigned and then
use the results in opcodes taking signed integers.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/gallivm/lp_bld_tgsi.c |2 +-
1 file changed, 1 insertion(+), 1
- Original Message -
Am 02.05.2013 03:13, schrieb Zack Rusin:
It's valid. Some shaders do the negation on unsigned and then
use the results in opcodes taking signed integers.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/gallivm/lp_bld_tgsi.c |2 +-
The GPU apparently goes looking for constants even though there are no
shader stages enabled, and gets stuck because we haven't told it there are
no constants to collect. If any other user of the 3D pipeline had run
(even the Render accel of the X server!) since power on, then the in-GPU
On Tue, Apr 30, 2013 at 10:01 AM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 9:57 AM, Ian Romanick i...@freedesktop.org wrote:
On 04/27/2013 04:32 PM, Jordan Justen wrote:
This GLSL extension requires that AMD_vertex_shader_layer be
enabled by the driver.
Most (all?)
Shaders are unified on most hardware (= same limits in all stages).
No idea what the assertion was good for.
---
src/mesa/main/config.h |6 ++
src/mesa/main/context.c|6 ++
src/mesa/state_tracker/st_extensions.c |2 +-
3 files changed, 5
Const.MaxTextureImageUnits - Const.FragmentProgram.MaxTextureImageUnits
Const.MaxVertexTextureImageUnits - Const.VertexProgram.MaxTextureImageUnits
etc.
---
src/glsl/glsl_parser_extras.cpp |4 ++--
src/glsl/linker.cpp |8
The limits should not be different and OpenGL requires both to be at least 32,
which is also the maximum limit on radeon.
---
src/mesa/main/config.h |2 --
src/mesa/main/context.c|2 --
src/mesa/main/get_hash_params.py |4 ++--
---
src/mesa/main/config.h |3 ---
src/mesa/main/context.c |1 -
src/mesa/main/mtypes.h |2 --
3 files changed, 6 deletions(-)
diff --git a/src/mesa/main/config.h b/src/mesa/main/config.h
index c48ffdc..ea87b75 100644
--- a/src/mesa/main/config.h
+++ b/src/mesa/main/config.h
@@
---
src/mesa/main/enable.c |2 --
src/mesa/main/mtypes.h |3 ---
src/mesa/main/state.c |1 -
src/mesa/main/varray.c |4
4 files changed, 10 deletions(-)
diff --git a/src/mesa/main/enable.c b/src/mesa/main/enable.c
index 33853e3..016e432 100644
--- a/src/mesa/main/enable.c
If Const.CheckArrayBounds is false, the only code using _MaxElement is
glDrawRangeElements, so I changed it and explained in the code why
_MaxElement is not very useful there.
BTW, the big magic number was copied to the letter
from _mesa_update_array_max_element.
---
src/mesa/main/state.c
NOTE: This is a candidate for the 9.1 branch.
---
src/glsl/builtin_variables.cpp |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/builtin_variables.cpp b/src/glsl/builtin_variables.cpp
index b0c7a20..15f5bf3 100644
--- a/src/glsl/builtin_variables.cpp
+++
Also simplify UBO support checking.
NOTE: This is a candidate for the 9.1 branch.
---
src/mesa/state_tracker/st_extensions.c | 64 +---
1 file changed, 43 insertions(+), 21 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
---
src/mesa/state_tracker/st_extensions.c | 26 +++---
1 file changed, 11 insertions(+), 15 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index d32f50a..0aac086 100644
--- a/src/mesa/state_tracker/st_extensions.c
---
src/gallium/docs/source/screen.rst |2 ++
src/gallium/drivers/llvmpipe/lp_screen.c |2 ++
src/gallium/drivers/nv30/nv30_screen.c |1 +
src/gallium/drivers/nv50/nv50_screen.c |2 ++
src/gallium/drivers/nvc0/nvc0_screen.c |2 ++
NOTE: This is a candidate for the stable branches.
---
src/mesa/state_tracker/st_extensions.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index 43fba01..b64d363 100644
---
Version 2 that shouldn't crash.
---
src/mesa/state_tracker/st_cb_bufferobjects.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/mesa/state_tracker/st_cb_bufferobjects.c
b/src/mesa/state_tracker/st_cb_bufferobjects.c
index d166fe6..c868d5e 100644
---
- don't reference a buffer for a local variable
(that's never useful unless it can be the only reference to the buffer)
- check if the buffer is not NULL
- set buffer_size as specified with BindBufferRange
NOTE: This is a candidate for the 9.1 branch.
Reviewed-by: Fredrik Höglund
Probably copied from i965. However st/mesa has its flags ST_NEW_xxx_PROGRAM.
---
src/mesa/state_tracker/st_atom_constbuf.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_constbuf.c
b/src/mesa/state_tracker/st_atom_constbuf.c
index
_NEW_BUFFER_OBJECT means glBufferData was called. We can just set our own
flag in BufferData.
---
src/mesa/state_tracker/st_atom_array.c |2 +-
src/mesa/state_tracker/st_cb_bufferobjects.c |4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
MaxUniformComponents contains only the limit for the default uniform buffer,
but the linker also adds all uniforms blocks to the uniform usage stats,
causing bogus linker failures.
NOTE: This is a candidate for the 9.1 branch.
---
src/glsl/linker.cpp |6 +++---
1 file changed, 3
and add assertions to prevent buffer overflow. This fixes corruption
of the r600_shader struct.
NOTE: This is a candidate for the stable branches.
---
src/gallium/drivers/r600/r600_shader.c |2 ++
src/gallium/drivers/r600/r600_shader.h |4 ++--
2 files changed, 4 insertions(+), 2
---
src/gallium/drivers/r600/r600_pipe.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/r600_pipe.c
b/src/gallium/drivers/r600/r600_pipe.c
index 4e92698..038514b 100644
--- a/src/gallium/drivers/r600/r600_pipe.c
+++
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
b/src/gallium/drivers/radeonsi/radeonsi_pipe.c
index c923c67..3b9be54 100644
--- a/src/gallium/drivers/radeonsi/radeonsi_pipe.c
+++
On Wed, May 1, 2013 at 8:41 PM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 10:01 AM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 9:57 AM, Ian Romanick i...@freedesktop.org wrote:
On 04/27/2013 04:32 PM, Jordan Justen wrote:
This GLSL extension requires
On Wed, May 1, 2013 at 9:13 PM, Matt Turner matts...@gmail.com wrote:
On Wed, May 1, 2013 at 8:41 PM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 10:01 AM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 9:57 AM, Ian Romanick i...@freedesktop.org wrote:
On
On 05/01/2013 09:25 PM, Jordan Justen wrote:
On Wed, May 1, 2013 at 9:13 PM, Matt Turner matts...@gmail.com wrote:
On Wed, May 1, 2013 at 8:41 PM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 10:01 AM, Jordan Justen jljus...@gmail.com wrote:
On Tue, Apr 30, 2013 at 9:57 AM,
74 matches
Mail list logo
