If there is actually hardware support, it's okay to add those formats.
See comments below.
On Sat, Nov 23, 2013 at 3:59 AM, Mark Mueller markkmuel...@gmail.com wrote:
How about let's forget the whole concept of GPU loading for a moment as that
is only clouding the issue. As you said: the mesa_*
https://bugs.freedesktop.org/show_bug.cgi?id=71944
Priority: medium
Bug ID: 71944
Assignee: mesa-dev@lists.freedesktop.org
Summary: vs-temp-array-mat3-index-col-rd test fails with SB
Severity: normal
Classification: Unclassified
https://bugs.freedesktop.org/show_bug.cgi?id=71944
--- Comment #1 from Martin Andersson g02ma...@gmail.com ---
Created attachment 89689
-- https://bugs.freedesktop.org/attachment.cgi?id=89689action=edit
Shader dump when test passes
--
You are receiving this mail because:
You are the assignee
Sounds good
On 22 Nov 2013 19:05, Ian Romanick i...@freedesktop.org wrote:
I've posted a wiki with some project suggestions for people wanting to
get into Mesa development:
http://wiki.freedesktop.org/dri/NewbieProjects/
These projects are a little bit larger than the one I previously
https://bugs.freedesktop.org/show_bug.cgi?id=71944
--- Comment #2 from Vadim Girlin pt...@yandex.ru ---
(In reply to comment #0)
If I revert that commit the test passes with SB, but I'm pretty sure that
commit only exposed a bug that weren't hit before.
Yes, I think you are right.
I looked
I have initial work on ARB_clear buffer_object for swrast there :
https://github.com/maxenceledore/mesa/commit/64e74a37118950d396392e38d6246d559049753f
Untested. And this is a ugly single patch implementation. Gonna split
it shortly.
AMD_shader_trinary_minmax :
I have played with
On 11/20/2013 02:29 PM, Paul Berry wrote:
From section 4.4.7 (Layered Framebuffers) of the GLSL 3.2 spec:
When the Clear or ClearBuffer* commands are used to clear a
layered framebuffer attachment, all layers of the attachment are
cleared.
This patch fixes meta clears to
https://bugs.freedesktop.org/show_bug.cgi?id=71944
--- Comment #3 from Martin Andersson g02ma...@gmail.com ---
Created attachment 89694
-- https://bugs.freedesktop.org/attachment.cgi?id=89694action=edit
Output with R600_DEBUG=vs,sbdump
--
You are receiving this mail because:
You are the
From: Ian Romanick ian.d.roman...@intel.com
libdrm 2.4.48 has been released.
This reverts commit bd4596efac2b783b789392a222da909efcd0fd3b.
---
src/mesa/drivers/dri/i965/brw_context.c | 8 +---
src/mesa/drivers/dri/i965/brw_reset.c | 7 ---
2 files changed, 5 insertions(+), 10
From: Ian Romanick ian.d.roman...@intel.com
drm_intel_get_reset_stats is only available in libdrm-2.4.48, and
libdrm-2.4.49 contains an important bug fix in that function.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Cc: 10.0 mesa-sta...@lists.freedesktop.org
---
configure.ac | 2 +-
1
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
Reported-by: Zhenyu Wang zhen...@linux.intel.com
Cc: 10.0 mesa-sta...@lists.freedesktop.org
---
src/mesa/drivers/dri/common/dri_util.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
From: Ian Romanick ian.d.roman...@intel.com
Rather than always advertising the extension but failing to create a
context with reset notifiction, just don't advertise it. I don't know
why it didn't occur to me to do it this way in the first place.
Signed-off-by: Ian Romanick
On Sat, Nov 23, 2013 at 2:26 AM, Marek Olšák mar...@gmail.com wrote:
If there is actually hardware support, it's okay to add those formats.
See comments below.
On Sat, Nov 23, 2013 at 3:59 AM, Mark Mueller markkmuel...@gmail.com
wrote:
How about let's forget the whole concept of GPU
Hello,
I have done the changes for ARB_map_buffer_alignment; will check I didn't
break anything and start sending the patches in tomorrow.
Kind regards,
Michal Navratil
I've posted a wiki with some project suggestions for people wanting to
get into Mesa development:
The fast tiled texture upload code does not compile with GCC 4.8's -Og
optimization flag.
memcpy() has the always_inline attribute set. This poses a problem,
since {x,y}tile_copy_faster calls it indirectly via {x,y}tile_copy,
and {x,y}tile_copy normally aren't inlined at -Og.
Using
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/builtin_functions.cpp | 12 ++--
src/glsl/ir.h | 3 +++
2 files changed, 13 insertions(+), 2 deletions(-)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index
Since the built-in functions rewrite, num_builtins_to_link is always either
0 or 1, so we don't need tho crazy loop starting at -1 with a special
case.
All we need to do is print the prototypes from the current shader, and
the single built-in function shader (if present).
Signed-off-by: Kenneth
There's no need to loop through the parameters list and remove every
element; move_nodes_to(parameters) already throws away all elements of
the destination list.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/ir.cpp | 6 --
1 file changed, 6 deletions(-)
diff --git
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/ast_function.cpp | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/glsl/ast_function.cpp b/src/glsl/ast_function.cpp
index 2707522..82456ad 100644
--- a/src/glsl/ast_function.cpp
+++
Previously, when we hit a no matching function error, it looked like:
0:0(0): error: no matching function for call to `cos()'
0:0(0): error: candidates are: float sin(float)
0:0(0): error:vec2 sin(vec2)
0:0(0): error:vec3 sin(vec3)
0:0(0): error:
Previously, we stored an array of up to 16 additional shaders to link,
as well as a count of how many each shader actually needed.
Since the built-in functions rewrite, all the built-ins are stored in a
single shader. So all we need is a boolean indicating whether a shader
needs to link against
On Sat, Nov 23, 2013 at 1:41 PM, Kenneth Graunke kenn...@whitecape.org wrote:
The fast tiled texture upload code does not compile with GCC 4.8's -Og
optimization flag.
memcpy() has the always_inline attribute set. This poses a problem,
since {x,y}tile_copy_faster calls it indirectly via
On 11/23/2013 01:17 PM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
libdrm 2.4.48 has been released.
This reverts commit bd4596efac2b783b789392a222da909efcd0fd3b.
This series looks okay to me.
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Third respin of the basic idea, with a complete-ish i965 backend now.
Works, except for a few things:
- Varying packing interaction is pretty bogus. Currently we look inside
the ir_swizzle that got generated, and don't correct for it, so this
only works for varyings which get packed into the
Will be used to implement interpolateAtCentroid(), interpolateAtOffset()
and interpolateAtSample() from ARB_gpu_shader5
Will be used to implement interpolateAtCentroid() from ARB_gpu_shader5.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/glsl/ir.cpp| 6 ++
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 99 +++-
1 file changed, 98 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
index
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_disasm.c | 16 +++-
1 file changed, 15 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
b/src/mesa/drivers/dri/i965/brw_disasm.c
index 128e717..d6c30c4 100644
---
V2: - Don't assume everyone wants interpolateAtSample() lowered to
interpolateAtOffset. It turns out this isn't what we want most
of the time for i965. Lowering can be added later in an ir pass
which drivers opt into, rather than bolting it straight into the
builtin
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_defines.h| 1 +
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 ++
src/mesa/drivers/dri/i965/brw_fs.h | 13 -
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 19
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_eu.h | 10 +++
src/mesa/drivers/dri/i965/brw_eu_emit.c | 50 +
2 files changed, 60 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_eu.h
The implementation is going to be nontrivial, so let's get it out of
visit().
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 29 ++--
2 files changed, 28 insertions(+), 2
The backend will have to do a message send, so we want to keep these in
one piece, just like texture ops.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
.../dri/i965/brw_fs_channel_expressions.cpp| 28 ++
1 file changed, 28 insertions(+)
diff --git
The new interpolateAt* builtins have strange restrictions on the
interpolant parameter.
- It must be a shader input, or an element of a shader input array.
- It must not include a swizzle.
V2: Don't abuse ir_var_mode_shader_in for this; make a new flag.
Signed-off-by: Chris Forbes
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_defines.h | 1 +
src/mesa/drivers/dri/i965/brw_structs.h | 21 +
2 files changed, 22 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
b/src/mesa/drivers/dri/i965/brw_defines.h
One would hope that it doesn't actually offer overloads of sin() when
you tried to call cos()?
Actual change looks good;
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sun, Nov 24, 2013 at 10:43 AM, Kenneth Graunke kenn...@whitecape.org wrote:
Previously, when we hit a no matching function
On 23 November 2013 12:58, Ian Romanick i...@freedesktop.org wrote:
On 11/20/2013 02:29 PM, Paul Berry wrote:
From section 4.4.7 (Layered Framebuffers) of the GLSL 3.2 spec:
When the Clear or ClearBuffer* commands are used to clear a
layered framebuffer attachment, all layers of
I believe that I have picked over all of the patches marked for the 10.0
release branch. I have included the full list far below. There are a
few patches that were sent to the mesa-stable mailing list that do not
appear to have landed on master yet. I have not picked these over.
These are:
Just to clarify unsized arrays cannot be assigned so var-max_array_access
will always be 0 as redecorations are already handled before this point by
get_variable_being_redeclared()
On Wed, 2013-11-20 at 22:41 +1100, Timothy Arceri wrote:
Left over from bug #34376.
Signed-off-by: Timothy
On 11/22/2013 10:30 AM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
On 11/22/2013 12:21 AM, Eric Anholt wrote:
The canary is basically just to give a better debugging message when you
ralloc_free() something that wasn't rallocated. Reduces maximum memory
usage of
The flags value is a bitfield so use the union's 'bf' field, not 'e'
(enum) field. There's no actual change in behavior here since both
fields of the union are the same size.
---
src/mesa/main/dlist.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/dlist.c
Now, sizeof(gl_dlist_node)==4 even on 64-bit systems. This can
halve the memory used by some display lists on 64-bit systems.
---
src/mesa/main/dlist.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index
This is a first step in reducing memory used by display lists on
64-bit systems. On 64-bit systems, the gl_dlist_node union type
is 8 bytes because of the 'data' and 'next' fields. This causes
every display list node/token to occupy 8 bytes instead of 4 as
originally designed. This basically
---
src/mesa/main/convolve.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/mesa/main/convolve.c b/src/mesa/main/convolve.c
index f44031a..b13b895 100644
--- a/src/mesa/main/convolve.c
+++ b/src/mesa/main/convolve.c
@@ -45,7 +45,7 @@
---
src/mesa/main/dlist.c | 21 +
1 file changed, 5 insertions(+), 16 deletions(-)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 138f272..d1e2035 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -555,9 +555,9 @@ union uint64_pair
/**
---
src/mesa/main/dlist.c |4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 74800da..83c56a7 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -6845,6 +6845,10 @@ save_error(struct gl_context *ctx, GLenum error, const
As is done for the old histogram functions.
---
src/mesa/main/colortab.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/mesa/main/colortab.c b/src/mesa/main/colortab.c
index 81e92d7..a8edb03 100644
--- a/src/mesa/main/colortab.c
+++
Trying to compile any of these functions into a display list
now just generates a GL_INVALID_OPERATION error.
---
src/mesa/main/dlist.c | 551 -
1 file changed, 551 deletions(-)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index
And simplify save_PixelMapfv() by using the memdup() function.
---
src/mesa/main/dlist.c | 29 -
1 file changed, 16 insertions(+), 13 deletions(-)
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 83c56a7..7f4ad17 100644
--- a/src/mesa/main/dlist.c
Patches 1 through 6 and 9 are
Reviewed-by: Ian Romanick ian.d.roman...@intel.com.
I actually wonder if in non-debug builds we should use function aliasing
(or similar) to use generic_noop for all these functions that just set
GL_INVALID_OPERATION. That might be a good newbie project. :)
I want
Would it be better to just make _mesa_glsl_get_builtin_function_shader a
friend?
On 11/23/2013 01:43 PM, Kenneth Graunke wrote:
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/builtin_functions.cpp | 12 ++--
src/glsl/ir.h | 3 +++
2 files
Patches 1 through 4 and 6 are
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
I had a question about 5.
On 11/23/2013 01:43 PM, Kenneth Graunke wrote:
There's no need to loop through the parameters list and remove every
element; move_nodes_to(parameters) already throws away all elements of
On Sat, Nov 23, 2013 at 10:23 PM, Mark Mueller markkmuel...@gmail.com wrote:
On Sat, Nov 23, 2013 at 2:26 AM, Marek Olšák mar...@gmail.com wrote:
If there is actually hardware support, it's okay to add those formats.
See comments below.
On Sat, Nov 23, 2013 at 3:59 AM, Mark Mueller
On 11/23/2013 03:21 PM, Paul Berry wrote:
On 23 November 2013 12:58, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
On 11/20/2013 02:29 PM, Paul Berry wrote:
From section 4.4.7 (Layered Framebuffers) of the GLSL 3.2 spec:
When the Clear or
On 11/23/2013 04:03 PM, Ian Romanick wrote:
Would it be better to just make _mesa_glsl_get_builtin_function_shader a
friend?
In this case, I don't think it makes much practical difference - the
builtin_builder class is entirely contained within the
builtin_functions.cpp file. So making this
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Mesa 10.0 release candidate 2 is now available for testing. The current
plan of record is to have the 10.0 release on Wedensday, November 27th.
The tag in the GIT repository for Mesa 10.0-rc2 is 'mesa-10.0-rc2'.
Mesa 10.0 release candidate 1 is
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
docs/envvars.html | 32
1 file changed, 32 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index 81e74e6..d831826 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -121,6 +121,38 @@ See the
These were removed back in 2012.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
docs/envvars.html | 4
1 file changed, 4 deletions(-)
diff --git a/docs/envvars.html b/docs/envvars.html
index 67d8e29..81e74e6 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -121,10 +121,6 @@ See
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/intel_debug.c | 1 -
src/mesa/drivers/dri/i965/intel_debug.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_debug.c
b/src/mesa/drivers/dri/i965/intel_debug.c
index 0c45c99..7180e18
DEBUG_IOCTL comes from i965, and is about to be removed. Both defines
have the same value (4).
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/radeon/radeon_ioctl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
---
docs/shading.html | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/shading.html b/docs/shading.html
index 19e20b1..4015cdf 100644
--- a/docs/shading.html
+++ b/docs/shading.html
@@ -67,7 +67,7 @@ Example: export MESA_GLSL=dump,nopt
h2 id=supportGLSL Version/h2
p
60 matches
Mail list logo