This series adds support for the ARB_texture_storage_multisample extension.
Only minor changes from V1:
* Added missing error cases in TexStorage*Multisample. Thanks Eric for
pointing out that this was a bit lacking.
* Dropped spurious special case for TEXTURE_MAX_LEVELS. This wasn't in
the
This is about to be used in teximagemultisample() when immutable=true.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/main/texstorage.c | 37 -
src/mesa/main/texstorage.h | 3 +++
2 files changed, 23 insertions(+), 17 deletions(-)
diff --git
The new entrypoints will come later, but this adds the actual logic for
supporting immutable multisample textures:
- The immutability flag is set as desired.
- Attempting to modify an immutable multisample texture produces
INVALID_OPERATION.
Note: The extension spec does not mention adding
Adds XML for the extension, dispatch_sanity enabling, and the two new
entrypoints. These are both implemented by calling the shared
teximagemultisample() with immutable=GL_TRUE.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Eric Anholt e...@anholt.net
---
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Eric Anholt e...@anholt.net
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index e90a296..004fc8e 100644
---
Now that there are 4 variants, just pass the function name into
teximagemultisample rather than reconstructing it.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Eric Anholt e...@anholt.net
---
src/mesa/main/teximage.c | 30 ++
1 file changed, 14
ARB_texture_storage_multisample allows texture parameters to be
queried for TEXTURE_2D_MULTISAMPLE and TEXTURE_2D_MULTISAMPLE_ARRAY
targets.
Some parameters may also be set, with the following exceptions:
- TEXTURE_BASE_LEVEL may not be set to a nonzero value; generates
INVALID_OPERATION
-
This can be enabled everywhere that ARB_texture_multisample is
supported -- ARB_texture_storage is supported on everything.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Eric Anholt e...@anholt.net
---
src/mesa/drivers/dri/intel/intel_extensions.c | 1 +
1 file changed, 1
Signed-off-by: Chris Forbes chr...@ijw.co.nz
Reviewed-by: Eric Anholt e...@anholt.net
---
docs/GL3.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 640dec2..898d921 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -151,7 +151,7 @@
I've got this working now (in the simplest cases) on Ivybridge.
If anyone is curious, the (messy) git trees are at
http://github.com/chrisforbes/mesa.git arb_texture_gather
http://github.com/chrisforbes/piglit.git texture_gather
Nowhere near ready for review, but it's a start.
Remaining things
Am 28.03.2013 20:34, schrieb Vadim Girlin:
On 03/28/2013 01:01 PM, � wrote:
Am 27.03.2013 20:37, schrieb Vadim Girlin:
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
---
src/gallium/drivers/r600/r600_shader.c | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
- Original Message -
We do support so with multiple primitives.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_so_emit_tmp.h |1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/auxiliary/draw/draw_so_emit_tmp.h
Thanks for the cleanup.
Reviewed-by: Jose Fonseca jfons...@vmware.com
Jose
- Original Message -
Instead of void pointers use a base interface.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_llvm.c | 77
---
Looks good to me Brian.
Just a couple of comments.
- Original Message -
Let's try to update this when we find other broken applications...
docs/ is growing a lot of stuff, with disparate target audiences. Maybe we
could establish some sort of directory hierachary there:
On 29.03.2013 10:56, Christian König wrote:
Am 28.03.2013 20:34, schrieb Vadim Girlin:
On 03/28/2013 01:01 PM, � wrote:
Am 27.03.2013 20:37, schrieb Vadim Girlin:
Signed-off-by: Vadim Girlin vadimgir...@gmail.com
---
src/gallium/drivers/r600/r600_shader.c | 19 +++
1 file
Hi
This include is also in
src/gallium/state_trackers/clover/llvm/invocation.cpp
diff -Naur a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
--- a/src/gallium/state_trackers/clover/llvm/invocation.cpp
2013-03-29 11:15:52.851581526
---
src/gallium/docs/source/screen.rst |2 ++
src/gallium/drivers/freedreno/freedreno_screen.c |1 +
src/gallium/drivers/i915/i915_screen.c |1 +
src/gallium/drivers/llvmpipe/lp_screen.c |2 ++
src/gallium/drivers/nv30/nv30_screen.c |1
Sorry that patch doesn't fix the build
diff -Naur a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
--- a/src/gallium/state_trackers/clover/llvm/invocation.cpp 2013-03-29
12:14:25.514504748 +
+++
Also, renamed pixels-rendered to samples-passed because the
occlusion counter increments even if colour and depth writes are
disabled, or (on some implementations) for killed that passed the
depth test when early_fragment_tests has been set for the PS.
---
src/gallium/auxiliary/hud/hud_context.c
---
src/gallium/docs/source/context.rst |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/gallium/docs/source/context.rst
b/src/gallium/docs/source/context.rst
index 9e57930..2cc1848 100644
--- a/src/gallium/docs/source/context.rst
+++
On 03/29/2013 04:50 AM, Jose Fonseca wrote:
Looks good to me Brian.
Just a couple of comments.
- Original Message -
Let's try to update this when we find other broken applications...
docs/ is growing a lot of stuff, with disparate target audiences. Maybe we
could establish some
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Fri, Mar 29, 2013 at 1:05 PM, Christoph Bumiller
e0425...@student.tuwien.ac.at wrote:
---
src/gallium/docs/source/screen.rst |2 ++
src/gallium/drivers/freedreno/freedreno_screen.c |1 +
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Fri, Mar 29, 2013 at 2:33 PM, Christoph Bumiller
e0425...@student.tuwien.ac.at wrote:
---
src/gallium/docs/source/context.rst |8 +---
1 files changed, 5 insertions(+), 3 deletions(-)
diff --git
On 03/29/2013 02:30 AM, Chris Forbes wrote:
This series adds support for the ARB_texture_storage_multisample extension.
Only minor changes from V1:
* Added missing error cases in TexStorage*Multisample. Thanks Eric for
pointing out that this was a bit lacking.
* Dropped spurious special
On 03/28/2013 03:24 PM, Marek Olšák wrote:
---
src/mesa/main/blend.c | 28 +++-
src/mesa/main/blend.h |8
src/mesa/main/fbobject.c| 10 ++
src/mesa/main/framebuffer.c |1 +
src/mesa/main/mtypes.h |4 +++-
On 03/28/2013 03:24 PM, Marek Olšák wrote:
It has 2 dependencies: glClampColor and the framebuffer, we might just as well
do the update where those two are changed.
---
src/mesa/main/blend.c | 28
src/mesa/main/blend.h |6 ++
On 03/28/2013 03:24 PM, Marek Olšák wrote:
---
src/mesa/state_tracker/st_extensions.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index 11db9d3..2d8b9ef 100644
---
From: Roland Scheidegger srol...@vmware.com
This replaces the existing float-to-half implementation.
There are definitely a couple of differences - the old implementation
had unspecified(?) rounding behavior, and could at least in theory
construct Inf values out of NaNs. NaNs and Infs should now
From: Roland Scheidegger srol...@vmware.com
Conceptually the same as previously done in float_to_half.
Should cut down number of instructions from 14 to 10 or so, but
will promote some NaNs to Infs, so it's disabled.
It gets a bit tricky though handling all the cases correctly...
Passes basic
We are intentionally not allocating a slot for gl_ClipVertex. But by
leaving the bit set in the slots_valid, the fragment shader's computation
of where varyings are in urb entry coming out of the SF would be off by
one. Fixes rendering in Freespace 2 SCP.
Bugzilla:
On Fri, Mar 29, 2013 at 4:21 PM, Brian Paul bri...@vmware.com wrote:
On 03/28/2013 03:24 PM, Marek Olšák wrote:
---
src/mesa/state_tracker/st_**extensions.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/mesa/state_tracker/st_**extensions.c
On 29 March 2013 10:20, Eric Anholt e...@anholt.net wrote:
We are intentionally not allocating a slot for gl_ClipVertex. But by
leaving the bit set in the slots_valid, the fragment shader's computation
of where varyings are in urb entry coming out of the SF would be off by
one. Fixes
On 03/29/2013 10:40 AM, Eric Anholt wrote:
Alessandro Pignotti noted when I added this code in commit
0e723b135bfd59868c92c3ae243f1adaedaec3a5 that it's in the else block for
if (busy), so this debug print couldn't happen.
---
src/mesa/drivers/dri/intel/intel_buffer_objects.c |5 -
1
On 03/29/2013 07:30 AM, Brian Paul wrote:
On 03/29/2013 04:50 AM, Jose Fonseca wrote:
Looks good to me Brian.
Just a couple of comments.
- Original Message -
Let's try to update this when we find other broken applications...
docs/ is growing a lot of stuff, with disparate target
On 03/29/2013 10:20 AM, Eric Anholt wrote:
We are intentionally not allocating a slot for gl_ClipVertex. But by
leaving the bit set in the slots_valid, the fragment shader's computation
of where varyings are in urb entry coming out of the SF would be off by
one. Fixes rendering in Freespace 2
Mesa constant-folds built-in functions by using a miniature GLSL
interpreter (see
ir_function_signature::constant_expression_evaluate_expression_list()).
This interpreter had a bug in its handling of array indexing, which
caused expressions like m[i][j] (where m is a matrix) to be handled
On 27 March 2013 09:30, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Search and replace:
][0] - ].x
][1] - ].y
][2] - ].z
][3] - ].w
Fixes piglit tests inverse-mat[234].{vert,frag}. These tests call the
inverse function with
A slightly modified version of this (drop the target parameter) should
be a candidate for the 9.1 branch.
On Fri, Mar 29, 2013 at 5:23 PM, Kenneth Graunke kenn...@whitecape.org wrote:
On 03/28/2013 08:28 PM, Chris Forbes wrote:
Previously at least i915 failed to provide an implementation, but
We do support so with multiple primitives.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_so_emit_tmp.h |1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/auxiliary/draw/draw_so_emit_tmp.h
b/src/gallium/auxiliary/draw/draw_so_emit_tmp.h
index
We were largely ignoring primitive id.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_gs.c| 20 +++-
src/gallium/auxiliary/draw/draw_gs.h|1 +
src/gallium/auxiliary/draw/draw_llvm.c | 15 +--
It's quite helpful during the rendering when we know
exactly the count of the vertices available in the
buffer.
Signed-off-by: Zack Rusin za...@vmware.com
---
src/gallium/auxiliary/draw/draw_context.h|1 +
src/gallium/auxiliary/draw/draw_pt.c | 18 +++---
The original goal of pre-register allocation scheduling was to reduce
live ranges so we'd use fewer registers and hopefully fit into 16-wide.
In shader-db, this change causes us to lose 30 16-wide programs, but we
gain 29... so it's a toss-up. At least by choosing instructions in a
better order
---
src/mesa/drivers/dri/common/drirc |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/common/drirc
b/src/mesa/drivers/dri/common/drirc
index a13941f..556d1b5 100644
--- a/src/mesa/drivers/dri/common/drirc
+++ b/src/mesa/drivers/dri/common/drirc
On 03/29/2013 12:49 PM, Kenneth Graunke wrote:
On 03/29/2013 07:30 AM, Brian Paul wrote:
On 03/29/2013 04:50 AM, Jose Fonseca wrote:
BTW, do users need to manually configure this, or will it be
automatically set based on process name?
Manually. Though the DRI conf tool can be used to set
https://bugs.freedesktop.org/show_bug.cgi?id=62919
Priority: medium
Bug ID: 62919
Keywords: regression
CC: mar...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: piglit getteximage-targets S3TC 2D_ARRAY regression
If the active uniform is an array, then the length of the uniform name should
include the three extra characters for the [0] suffix, which is required by
the GL 4.2 spec to be appended to the uniform name in glGetActiveUniform().
This avoids the situation where the output buffer does not have
https://bugs.freedesktop.org/show_bug.cgi?id=62920
Priority: medium
Bug ID: 62920
Keywords: regression
CC: mar...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] getteximage-targets S3TC CUBE_ARRAY
I found an issue with the shader compiler for Cayman when I looked
into why the ext_transform_feedback/order test case caused a GPU stall.
It turned out the stall was an infinite loop that was the result of broken
calculation in the shader function. The issue is that Cayman uses the
tgsi_umad
The tgsi_umad function does not work for Cayman since it does not
populate the y, z and w slots for UMUL that Cayman requires.
---
src/gallium/drivers/r600/r600_shader.c | 47 +-
1 file changed, 46 insertions(+), 1 deletion(-)
diff --git
https://bugs.freedesktop.org/show_bug.cgi?id=62919
--- Comment #1 from Marek Olšák mar...@gmail.com ---
The commit turned off the decompression with a blit for software rasterizers,
so we're hitting a code path we never used for years. I think this uncovers an
existing old bug instead of
https://bugs.freedesktop.org/show_bug.cgi?id=62920
Vinson Lee v...@freedesktop.org changed:
What|Removed |Added
Summary|[softpipe] |[softpipe] piglit
https://bugs.freedesktop.org/show_bug.cgi?id=62921
Priority: medium
Bug ID: 62921
Keywords: regression
CC: srol...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [llvmpipe] piglit arb_color_buffer_float-drawpixels
https://bugs.freedesktop.org/show_bug.cgi?id=62922
Priority: medium
Bug ID: 62922
Keywords: regression
CC: jfons...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit arb_seamless_cubemap regression
53 matches
Mail list logo