On 09/04/2013 04:46 PM, Timothy Arceri wrote:
Since the calling functions all should alias, all of this should
be removed too.
The functions have to do different validations on types they are not
exactly the same. If there is some way to signal this to be done
using an alias can you
On 09/04/2013 07:11 PM, Matt Turner wrote:
On Wed, Sep 4, 2013 at 3:22 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/ir.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
index
On 09/04/2013 03:11 PM, Timothy Arceri wrote:
Since you obviously didn't run 'make check', I will be reverting
this entire series later today.
YOU MUST RUN 'make check'.
Ok, well what can I say I didnt know (or I guess didnt check) that
Mesa had a Testsuite I will submit a patch to add a
On 09/04/2013 05:43 PM, Timothy Arceri wrote:
+/**
+ * Helper for _mesa_ObjectLabel() and _mesa_ObjectPtrLabel().
+ */
+static void
+set_label(struct gl_context *ctx, char **labelPtr, const char *label,
+ int length, const char *caller)
+{
+ if (*labelPtr) {
+ /* free
On 09/04/2013 07:11 PM, Matt Turner wrote:
On Wed, Sep 4, 2013 at 3:22 PM, Kenneth Graunke kenn...@whitecape.org wrote:
dotlike() uses ir_binop_mul for scalars, and ir_binop_dot for vectors.
When generating built-in functions, we often want to use regular
multiply for scalar signatures, and
On Wed, 2013-09-04 at 20:09 -0700, Ian Romanick wrote:
On 09/04/2013 05:43 PM, Timothy Arceri wrote:
+/**
+ * Helper for _mesa_ObjectLabel() and _mesa_ObjectPtrLabel().
+ */
+static void
+set_label(struct gl_context *ctx, char **labelPtr, const char *label,
+ int length,
sample_d is slower than the lowered version on gen7. For gen7, this improves
Xonotic benchmark with Ultimate effects by as much as 25%:
before the change: 40.06 fps
after the change: 51.10 fps
after the change with INTEL_DEBUG=no16: 44.46 fps
As
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
docs/devinfo.html | 23 +++
1 file changed, 23 insertions(+)
diff --git a/docs/devinfo.html b/docs/devinfo.html
index 4c1099c..d921e0d 100644
--- a/docs/devinfo.html
+++ b/docs/devinfo.html
@@ -155,6 +155,29 @@ of
A possible explanation for the perf change is that Xonotic uses
anisotropic filtering at this quality level. Lowering to txl defeats
it.
It would be worth doing an image quality comparison before and after the change.
-- Chris
On Thu, Sep 5, 2013 at 8:35 PM, Chia-I Wu olva...@gmail.com wrote:
On Die, 2013-09-03 at 15:23 +0200, Marek Olšák wrote:
The shader is responsible for writing to streamout buffers using
the TBUFFER_STORE_FORMAT_* instructions.
The locations of some input SGPRs and VGPRs are assigned dynamically, because
the input SGPRs controlling streamout are not declared
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/objectlabel.c | 124 +--
1 file changed, 62 insertions(+), 62 deletions(-)
diff --git a/src/mesa/main/objectlabel.c b/src/mesa/main/objectlabel.c
index 90d9e09..7e39c92 100644
---
Add this comment because other places in OpenGL include the NULL terminator in
the length.
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/objectlabel.c |5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/objectlabel.c b/src/mesa/main/objectlabel.c
On Thu, Sep 05, 2013 at 03:03:33AM +0200, Ondrej Riha wrote:
Hello,
I have i915: Atom N455 and have problem with OpenGL ES 2.0 test. On Windows I
have OpenGL 2, but on Linux I have only OpenGL 1.4 and therefore OpenGL ES
2.0 test crashed. For more info see:
On Wed, Sep 04, 2013 at 03:22:41PM -0700, Kenneth Graunke wrote:
This creates a new replacement for the existing built-in function code.
The new module lives in builtin_functions.cpp (not builtin_function.cpp)
and exists in parallel with the existing system. It isn't used yet.
The new
On Wed, Sep 04, 2013 at 03:22:41PM -0700, Kenneth Graunke wrote:
This creates a new replacement for the existing built-in function code.
The new module lives in builtin_functions.cpp (not builtin_function.cpp)
and exists in parallel with the existing system. It isn't used yet.
The new
On Thu, Sep 05, 2013 at 02:27:04PM +0300, Pohjolainen, Topi wrote:
On Wed, Sep 04, 2013 at 03:22:41PM -0700, Kenneth Graunke wrote:
This creates a new replacement for the existing built-in function code.
The new module lives in builtin_functions.cpp (not builtin_function.cpp)
and exists in
The warning is wrong and my gcc 4.7.3 doesn't show it. This code at
the beginning of the block where vdata is declared ensures vdata is
always initialized:
if (!num_comps || num_comps 4)
continue;
You might be seeing one of these bugs:
On Thu, Sep 5, 2013 at 5:12 PM, Chris Forbes chr...@ijw.co.nz wrote:
A possible explanation for the perf change is that Xonotic uses
anisotropic filtering at this quality level. Lowering to txl defeats
it.
I had a look at that. gl_sampler-MaxAnisotropy is never greater than
1.0 in
Original explanatory text:
Fix for throwing BadDrawable (invalid Pixmap or Window parameter)
by Xserver when calling XCreatePixmap or fill_bitmap from
DRI_glXUseXFont.
currentDrawable which came from __glXGetCurrentContext was not correct in
this case, we use
On Thu, Sep 5, 2013 at 6:25 PM, Maniak, Lukasz lukasz.man...@intel.com wrote:
Original explanatory text:
Fix for throwing BadDrawable (invalid Pixmap or Window parameter)
by Xserver when calling XCreatePixmap or fill_bitmap from
DRI_glXUseXFont.
currentDrawable which came
---
src/gallium/include/pipe/p_shader_tokens.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/include/pipe/p_shader_tokens.h
b/src/gallium/include/pipe/p_shader_tokens.h
index 872dfe9..1beec05 100644
--- a/src/gallium/include/pipe/p_shader_tokens.h
+++
start_instance doesn't affect gl_InstanceID.
There's no piglit test, but it's kinda obvious the code was wrong.
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
Hmm I don't think the math works out here actually, which may explain
why it's faster.
I believe the derivatives need to be transformed to cube coord system
and I don't see that being done here (this is actually something I
haven't figured out the math yet how to do with reasonable effort for
I was aware that this was only correct in one use case, but didn't know
which one. Probably should have added a comment about that.
Thanks for fixing it, both patches are: Reviewed-by: Christian König
christian.koe...@amd.com
Am 05.09.2013 16:41, schrieb Marek Olšák:
---
On Thu, Sep 5, 2013 at 6:40 PM, Alexander Monakov amona...@gmail.com wrote:
But it doesn't make sense to call glXUseXFont without a current context.
However ARB_create_context allows to make a context current without an
associated drawable.
___
On Mon, Sep 02, 2013 at 09:07:18PM +0200, Marek Olšák wrote:
For _XYZ, the type of VDATA is v4i32, because v3i32 doesn't exist.
The ADDR64 bit is not exposed. A simpler intrinsic that doesn't take
a resource descriptor might be nicer.
The maximum number of input SGPRs is bumped to 17.
On 09/05/2013 02:54 AM, Timothy Arceri wrote:
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
docs/devinfo.html | 23 +++
1 file changed, 23 insertions(+)
diff --git a/docs/devinfo.html b/docs/devinfo.html
index 4c1099c..d921e0d 100644
--- a/docs/devinfo.html
On 09/04/2013 09:09 PM, Ian Romanick wrote:
In the mean time, we should land Brian's patch to fix 'make check'.
Does that imply your R-b?
-Brian
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
On 09/04/2013 10:30 AM, Maniak, Lukasz wrote:
Fix for thro= wing BadDrawable (invalid Pixmap or Window parameter)
by Xserver w= hen calling XCreatePixmap or fill_bitmap from
DRI_glXUseXF= ont.
= /span
current= Drawable which came from __glXGetCurrentContext was not
On 09/05/2013 01:05 AM, Timothy Arceri wrote:
I also have one more question about working on Mesa. Is there a wiki
page or something where developers register who is working on what
extension to avoid double up?
There's no such page right now. People sometimes will post a message to
say
No, we use 11 user data SGPRs for the vertex shader, but there are
also 6 additional SGPRs loaded by the hw based on the VGT state (4
streamout offsets, streamout_enable, and streamout_write_index). The 6
SGPRs can be enabled by setting SPI_SHADER_PGM_RSRC2_VS.SO_* = 1.
Marek
On Thu, Sep 5, 2013
On 09/05/2013 03:57 AM, Timothy Arceri wrote:
Signed-off-by: Timothy Arceri t_arc...@yahoo.com.au
---
src/mesa/main/objectlabel.c | 124 +--
1 file changed, 62 insertions(+), 62 deletions(-)
diff --git a/src/mesa/main/objectlabel.c
On 4 September 2013 11:29, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
There is no GL_TEXTURE_CUBE_MAP_SEAMLESS in any version of OpenGL ES or
in any extension that applies to OpenGL ES. The same error check
already occurs for glTexParameteri.
On 09/01/2013 07:45 AM, Paul Berry wrote:
On 29 August 2013 21:31, Kenneth Graunke wrote:
[snip]
I definitely don't want to share portions of 3DSTATE_VS.
In an effort to help us make a more informed decision about this, I've
investigated the following questions:
1. Are there any historical
On 4 September 2013 11:29, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
The change is very small. Do seamless filtering if either the context
enable is set or the sampler enable is set.
The AMD_seamless_cubemap_per_texture says:
If
From: Tom Stellard thomas.stell...@amd.com
Also remove unused data fields from the DS_Load_Helper class.
---
lib/Target/R600/SIInstrInfo.td| 28
lib/Target/R600/SIInstructions.td | 6 +++---
2 files changed, 23 insertions(+), 11 deletions(-)
diff --git
From: Tom Stellard thomas.stell...@amd.com
---
lib/Target/R600/SIDefines.h| 3 ++-
lib/Target/R600/SIInstrFormats.td | 3 +++
lib/Target/R600/SIInstrInfo.cpp| 4
lib/Target/R600/SIInstrInfo.h | 1 +
lib/Target/R600/SILowerControlFlow.cpp | 12
On Thu, Sep 05, 2013 at 10:14:32PM +0800, Chen Jie wrote:
Hi all,
This thread is about
http://lists.freedesktop.org/archives/dri-devel/2013-April/037598.html.
We recently find some interesting thing about UVD based playback on
loongson 3a plaform, and also find a way to fix the problem.
According to tis
https://lists.gnu.org/archive/html/bug-tar/2005-02/msg1.html error 141
is 128+13 = SIGPIPE (broke pipe signal)
And this may be relevant
https://groups.google.com/forum/#!topic/golang-nuts/xjZ8jJx0IFw
Check whether you are using right yacc and bison?
Thanks for your
On Thu, Sep 05, 2013 at 03:29:52PM -0400, Jerome Glisse wrote:
On Thu, Sep 05, 2013 at 10:14:32PM +0800, Chen Jie wrote:
Hi all,
This thread is about
http://lists.freedesktop.org/archives/dri-devel/2013-April/037598.html.
We recently find some interesting thing about UVD based
From: Tom Stellard thomas.stell...@amd.com
This fixes some regressions in the piglit local memory store tests
introduced by recent commits which made the scheduler aware of the trans
slot.
It's not possible to test this using lit, because there is no way to
determine from the assembly dumps
Reviewed-by: Marek Olšák marek.ol...@amd.com
Though I'm not sure if 0x8000 is correct.
Marek
On Wed, Sep 4, 2013 at 11:55 PM, Alex Deucher alexdeuc...@gmail.com wrote:
IBs need to be a multiple of 4 dwords on r6xx asics
to avoid a hw bug.
Signed-off-by: Alex Deucher
On 09/04/2013 11:29 AM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
The change is very small. Do seamless filtering if either the context
enable is set or the sampler enable is set.
The AMD_seamless_cubemap_per_texture says:
If TEXTURE_CUBE_MAP_SEAMLESS_ARB is
From: Tom Stellard thomas.stell...@amd.com
---
configure.ac | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index b19ab18..702a58b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1387,7 +1387,10 @@ if test x$enable_opencl = xyes; then
fi
On Thu, Sep 5, 2013 at 4:27 PM, Tom Stellard t...@stellard.net wrote:
From: Tom Stellard thomas.stell...@amd.com
---
configure.ac | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index b19ab18..702a58b 100644
--- a/configure.ac
+++
Since BRW_MAX_WM_SURFACES is greater than BRW_MAX_VEC4_SURFACES, the
existing array isn't large enough to be used by the WM. Increasing it
will make it possible to share them.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_context.h | 2 +-
1 file
This gets the VS, GS, and PS all using the same data structure.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_context.h | 28 +---
src/mesa/drivers/dri/i965/brw_draw.c | 3 +-
src/mesa/drivers/dri/i965/brw_fs.cpp
These are largely based on the similar fields in brw-wm.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_context.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
On 09/05/2013 09:57 AM, Brian Paul wrote:
On 09/04/2013 09:09 PM, Ian Romanick wrote:
In the mean time, we should land Brian's patch to fix 'make check'.
Does that imply your R-b?
Yeah. I meant to go ahead and reply (again) to the patch. It was
getting late, and I forgot.
-Brian
On 09/05/2013 11:32 AM, Paul Berry wrote:
On 4 September 2013 11:29, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
mailto:ian.d.roman...@intel.com
There is no GL_TEXTURE_CUBE_MAP_SEAMLESS in any version of
From: Ian Romanick ian.d.roman...@intel.com
We return 0 for GL_NUM_SHADER_BINARY_FORMATS, so
GL_SHADER_BINARY_FORMATS should not write any data to the application
buffer.
Fixes piglit test 'arb_get_program_binary-overrun shader'.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
From: Tom Stellard thomas.stell...@amd.com
v2:
- Don't display an error message when the user doesn't ask for libclc.
---
configure.ac | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/configure.ac b/configure.ac
index b19ab18..fcfa4f7 100644
---
On 09/05/2013 04:20 AM, Pohjolainen, Topi wrote:
On Wed, Sep 04, 2013 at 03:22:41PM -0700, Kenneth Graunke wrote:
[snip]
+/**
+ * builtin_builder: A singleton object representing the core of the built-in
+ * function module.
+ *
+ * It has code to generate
+ * It generates IR for every
Matt noticed that this was missing. Nothing uses this currently.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/ir.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
index 8769c32..b654950 100644
--- a/src/glsl/ir.cpp
+++
On 09/04/2013 07:11 PM, Matt Turner wrote:
On Wed, Sep 4, 2013 at 3:22 PM, Kenneth Graunke kenn...@whitecape.org wrote:
[snip]
Name these arguments x/y/a to match GLSL mix()?
Also, add fma() maybe?
Good idea. Done.
___
mesa-dev mailing list
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Now that builtin_compiler is gone, nothing uses these.
Cc: Matt Turner matts...@gmail.com
---
configure.ac| 31 ---
m4/ax_prog_cc_for_build.m4 | 125
m4/ax_prog_cxx_for_build.m4 | 109 --
3
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
Sorry for annoying the mailing list but ...
irc_dri-devel
[Dienstag, 20. August 2013] [21:23:56] jobermayr calim: Would you accept
such a patch: https://github.com/jobermayr/mesa/commit/b859d1d
[Dienstag, 20. August 2013] [21:56:05] calim jobermayr: what's that good for
?
[Dienstag,
It looks like you have segfaults in /bin/sh. That reminds me why I am not
using my Fuloong anymore ;-)
Maybe you should consider cross-compiling Mesa on a stable system.
On Thu, Sep 5, 2013 at 1:14 PM, Christophe Jarry
christophe.ja...@ouvaton.org wrote:
According to tis
0x8000 is Type 2 NOP.
You could make it a little better/faster by inserting single multi-DWORD
Type 3 NOP
And pad to 8 DWORDs. CP fetches are 32 bytes each and R600 has requires
padding. Same with padding CP ring buffer updates to 32 bytes (pad to
32bytes before you update CP_RB_WPTR).
On
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
2013/9/6 Jerome Glisse j.gli...@gmail.com:
On Thu, Sep 05, 2013 at 03:29:52PM -0400, Jerome Glisse wrote:
On Thu, Sep 05, 2013 at 10:14:32PM +0800, Chen Jie wrote:
Hi all,
This thread is about
http://lists.freedesktop.org/archives/dri-devel/2013-April/037598.html.
We recently find
2013/9/6 Jerome Glisse j.gli...@gmail.com:
On Thu, Sep 05, 2013 at 10:14:32PM +0800, Chen Jie wrote:
Hi all,
This thread is about
http://lists.freedesktop.org/archives/dri-devel/2013-April/037598.html.
We recently find some interesting thing about UVD based playback on
loongson 3a plaform,
65 matches
Mail list logo