On 11/06/2012 11:43 PM, Vinson Lee wrote:
Fixes side effect in assertion defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/
Fixes side effect in assertion defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/mesa/drivers/dri/i965/brw_eu_emit.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c
b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 3d95b92..
The previous 1023-entry chaining hash table never resized, so it was very
inefficient when there were many objects live. While one could have an even
more efficient implementation than this (keep an array for genned names with
packed IDs, or take advantage of the fact that key == hash or key ==
*(
---
src/mesa/drivers/dri/i915/Makefile.am |5 +--
src/mesa/drivers/dri/i915/Makefile.sources |2 +-
src/mesa/drivers/dri/i965/Android.mk |4 +-
src/mesa/drivers/dri/i965/Makefile.am |4 +-
src/mesa/drivers/dri/i965/Makefile.sources | 61 +--
sparc/clip.c got moved to sparc/sparc-clip.c to avoid doing this workaround in
the parent directory.
---
src/mesa/libdricore/Makefile.am | 23 +++
1 file changed, 7 insertions(+), 16 deletions(-)
diff --git a/src/mesa/libdricore/Makefile.am b/src/mesa/libdricore/Makefile.am
---
src/glsl/Android.mk |4 +---
src/glsl/Makefile.am|1 -
src/glsl/Makefile.sources |9 -
src/glsl/SConscript |2 +-
src/mesa/libdricore/Makefile.am |1 -
5 files changed, 6 insertions(+), 11 deletions(-)
diff --git a/src/glsl
Chad Versace writes:
> I look forward to seeing this improved hash table land.
>
> Great that it includes tests. I failed to find a test that artificially
> constructed collisions and verified that they were handled correctly.
> Maybe I'll submit a test for that.
>
> Comments below. Everything lo
While simplifying mesa/Makefile.am, the more important feature of this commit
is allowing a file with the same name to appear in both main/ and program/.
---
This only does one of the subdirectories, one of the two I had to
change to make main/hash_table.c possible. If people are OK with this
gen
They were always used with the corresponding *_FILES variables now that
automake handles rule generation.
---
src/mesa/Android.mk |4
src/mesa/Makefile.am|2 --
src/mesa/libdricore/Makefile.am |1 -
src/mesa/sources.mak| 27 +++
---
src/mesa/sources.mak | 17 -
1 file changed, 17 deletions(-)
diff --git a/src/mesa/sources.mak b/src/mesa/sources.mak
index b4b58db..e2499f5 100644
--- a/src/mesa/sources.mak
+++ b/src/mesa/sources.mak
@@ -350,23 +350,6 @@ ALL_FILES = \
$(MESA_ASM_FILES) \
---
src/glsl/Makefile.sources |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/Makefile.sources b/src/glsl/Makefile.sources
index aafb53e..0da6958 100644
--- a/src/glsl/Makefile.sources
+++ b/src/glsl/Makefile.sources
@@ -95,7 +95,7 @@ GLSL_COMPILER_CXX_FILES = \
On Gen6-7, we don't compact clip planes, and nr_userclip_plane_consts
is the last bit set, so iterating from i = 0..nr_userclip_plane_consts
covers all active clip planes and is the right thing to do.
works and is the right thing to do.
However, that doesn't work at all on Gen4-5. Since we don't
There's no compaction, so we can drop that code and simply use 'i'.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
Really tempting to squash this.
diff --git a/src/mesa/drivers/dri/i965/brw_vec4
Since Gen4-5 compacts clip planes and Gen6-7 doesn't, it makes sense to
split them into separate code paths. This patch simply copies the code
to both halves; the next commits will simplify it.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 60 ++
>
> So I've been implementing this in between other things for a while now,
> and I think it's in reasonable shape now, with softpipe and r600g (evergreen)
> fully supported.
>
> I've got too do a few more piglit runs to confirm I've no regressions on r600g
> but softpipe seems to be fine (will con
From: Dave Airlie
fdo9833 piglit test caught this.
Signed-off-by: Dave Airlie
---
src/mesa/main/attrib.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 05db35b..92d6a97 100644
--- a/src/mesa/main/attrib.c
+++
https://bugs.freedesktop.org/show_bug.cgi?id=56826
Fredrik Hultin changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=55856
Fredrik Höglund changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
On Wed, Nov 7, 2012 at 9:50 AM, Brian Paul wrote:
> On 11/06/2012 04:45 PM, Dave Airlie wrote:
+
+ /*
+* Use the proxy texture driver hook to see if the size/level/etc
are
+* legal.
+*/
>>>
>>>
>>>
>>> That comment seems to be in the wrong place. T
On 11/06/2012 04:45 PM, Dave Airlie wrote:
+
+ /*
+* Use the proxy texture driver hook to see if the size/level/etc are
+* legal.
+*/
That comment seems to be in the wrong place. The code after it is concerned
with format/type checking.
must be a bit of cut-n-paste will clean
>> +
>> + /*
>> +* Use the proxy texture driver hook to see if the size/level/etc are
>> +* legal.
>> +*/
>
>
> That comment seems to be in the wrong place. The code after it is concerned
> with format/type checking.
must be a bit of cut-n-paste will clean it up.
>
> I think you mi
On Tuesday 06 November 2012, Ian Romanick wrote:
> On 11/05/2012 09:57 AM, Fredrik Höglund wrote:
> > This extension is functionally the same as GL_MESA_pack_invert,
> > but is also available in GLES.
>
> Do we have any piglit tests for this?
Not yet.
> If we're going to have the two pieces of s
On 11/06/2012 03:15 PM, Dave Airlie wrote:
This adds the mesa core + texture + fbo support for the
texture cube map array extension.
Signed-off-by: Dave Airlie
---
src/mesa/main/fbobject.c | 9 -
src/mesa/main/mtypes.h | 2 ++
src/mesa/main/shared.c | 1 +
src/mesa/main/tex
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
This series lives on my etc2-v9 branch (https://github.com/aphogat/mesa).
This series enables 8 out of 10 ETC2 texture formats for all Intel
hardware by simply decoding the ETC2 data into RGBX data at the time
of glCompressedTexImage2D. These patches ca
https://bugs.freedesktop.org/show_bug.cgi?id=56827
Priority: medium
Bug ID: 56827
CC: stereotype...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: mesa/main/api_exec.c:940: undefined reference to
`_es_Alpha
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
I think all of the patches that enable the texture formats for the Intel
driver should get moved to the end and squashed together.
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c |5 ++
src/mesa/drivers/dri/i
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
---
src/mesa/main/texcompress_etc.c | 108 +++
src/mesa/main/texcompress_etc.h |8 +++
2 files changed, 116 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/texcompress_etc
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
---
src/mesa/main/texcompress_etc.c | 96 +--
src/mesa/main/texcompress_etc.h |7 +++
2 files changed, 99 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/texcompress_etc.
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
---
src/mesa/main/format_unpack.c |6 ++
src/mesa/main/formats.c | 13 +
src/mesa/main/formats.h |1 +
src/mesa/main/glformats.c |2 ++
src/mesa/main/texcompress.c |9 +
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
It seems like this patch should just be squashed into patch 3.
---
src/mesa/main/texcompress_etc.c | 17 +
src/mesa/main/texcompress_etc.h |3 ++-
2 files changed, 11 insertions(+), 9 deletions(-)
On 10/19/2012 04:28 PM, Anuj Phogat wrote:
Signed-off-by: Anuj Phogat
---
src/mesa/main/texcompress_etc.c | 460 ++-
src/mesa/main/texcompress_etc.h | 15 +-
2 files changed, 466 insertions(+), 9 deletions(-)
diff --git a/src/mesa/main/texcompress_etc.
https://bugs.freedesktop.org/show_bug.cgi?id=56826
Priority: medium
Bug ID: 56826
Assignee: mesa-dev@lists.freedesktop.org
Summary: Not specifying user data for gluTessVertex causes NULL
pointer to vertex callback.
Severity:
This adds support to the softpipe texture sampler and tgsi exec.
In order to handle the extra input to the texture sampling,
I've had to expand the interfaces to take a c1 value for storing
the texture compare value for the TEX2 case.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tg
This contains the evergreen support.
Support is possible on rv670 upwards and the code in here
should work, but it doesn't and I haven't debugged it to
figure out why.
Beyond just adding support for the cube map array sampling,
r600 resinfo isn't conformant with the GL specification,
which states
This adds cube array support to the blitter.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/util/u_blitter.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_blitter.c
b/src/gallium/auxiliary/util/u_blitter.c
index d10ce23..5ea6aca 1006
This adds mesa state tracker support for the new extension,
along with glsl->tgsi conversion to use the new opcodes
where appropriate.
Signed-off-by: Dave Airlie
---
src/mesa/state_tracker/st_cb_texture.c | 4 ++
src/mesa/state_tracker/st_extensions.c | 3 +-
src/mesa/state_tracker/st_
This just adds the texture target and capability along
with 3 new opcodes required to support this extension.
As this extension requires some texture opcodes with samp + 5 args,
we need to use another src register, this is only required
for TEX, TXL and TXB opcodes to implement this spec.
TEX2 is
This adds all the new builtins + the new sampler types,
and hooks them up if the extension is supported.
Signed-off-by: Dave Airlie
---
src/glsl/builtin_types.h | 15 +++
.../profiles/ARB_texture_cube_map_array.glsl | 13 ++
src/glsl/builtins/tools
Signed-off-by: Dave Airlie
---
src/mesa/main/get.c | 2 ++
src/mesa/main/get_hash_params.py | 3 +++
2 files changed, 5 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index afae0893..39709b8 100644
--- a/src/mesa/main/get.c
+++ b/src/mesa/main/get.c
@@ -335,6 +
This adds the mesa core + texture + fbo support for the
texture cube map array extension.
Signed-off-by: Dave Airlie
---
src/mesa/main/fbobject.c | 9 -
src/mesa/main/mtypes.h | 2 ++
src/mesa/main/shared.c | 1 +
src/mesa/main/teximage.c | 85 +
This just adds the bit + extension name.
Signed-off-by: Dave Airlie
---
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 f8688cd..d5a7886 100644
--- a/src/mesa/main/ext
This adds the ARB_texture_cube_map_array enums.
Signed-off-by: Dave Airlie
---
src/mapi/glapi/gen/ARB_texture_cube_map_array.xml | 18 ++
src/mapi/glapi/gen/gl_API.xml | 2 +-
2 files changed, 19 insertions(+), 1 deletion(-)
create mode 100644 src/mapi/glapi
Hi,
So I've been implementing this in between other things for a while now,
and I think it's in reasonable shape now, with softpipe and r600g (evergreen)
fully supported.
I've got too do a few more piglit runs to confirm I've no regressions on r600g
but softpipe seems to be fine (will confirm aga
From: Dave Airlie
While developing cube map array support I found that we didn't
support this properly, also piglit didn't test for it at all.
I've submitted a test to piglit to check for this, and this
fixes explicit lod and lod bias with cube maps.
NOTE: This is a candidate for the 9.0 branch
From: Dave Airlie
For cube map arrays I'll need another driver private constant
buffer, and looking forward to UBOs. So clean up with some
defines, that can be modified when adding cube map array and ubos
later.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/r600_pipe.c | 2 +-
Nevermind, I didn't see the beginning of your initial mail. In fact this is a
bug, I'm investigating it
- Mail original -
> De : Andy Furniss
> À : Mathias Fröhlich
> Cc : Vincent Lejeune ; "mesa-dev@lists.freedesktop.org"
>
> Envoyé le : Mardi 6 novembre 2012 22h20
> Objet : Re: [Me
Mathias Fröhlich wrote:
Hi Vincent,
On Tuesday, November 06, 2012 12:14:52 Vincent Lejeune wrote:
Do you use the master branch ?
Yes, e3213f01f7764af573ed641a7bc98dde5824e321 is my current head in llvm.
This commit generates some llvm.R600.store.pixel.color/stencil/depth
intrinsics, which w
https://bugs.freedesktop.org/show_bug.cgi?id=55856
--- Comment #6 from wojtek ---
it works too :)
thanks for fix
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http:/
On 4 November 2012 09:55, Eric Anholt wrote:
> Kenneth Graunke writes:
>
> > On 11/02/2012 05:13 PM, Ian Romanick wrote:
> >> On 11/02/2012 03:01 PM, Paul Berry wrote:
> >>> On 2 November 2012 14:11, Ian Romanick >>> Oh yuck! Why not just fix the names of the Mesa functions? That
> >>>
On 11/06/2012 11:40 AM, Matt Turner wrote:
Previously we were accepting garbage after #else and #endif tokens when
the previous preprocessor conditional had evaluated to false (eg, #if 0)
When the preprocessor hits a false conditional, it switches the lexer
into the SKIP state, in which it ignor
---
lib/Target/AMDGPU/AMDILISelDAGToDAG.cpp | 11 --
lib/Target/AMDGPU/R600AllocateMemoryRegs.cpp | 23 ++--
lib/Target/AMDGPU/R600ISelLowering.cpp | 53 +++-
lib/Target/AMDGPU/R600ISelLowering.h | 2 ++
lib/Target/AMDGPU/R600Instructions.td
Hi Vincent,
On Tuesday, November 06, 2012 12:14:52 Vincent Lejeune wrote:
> Do you use the master branch ?
Yes, e3213f01f7764af573ed641a7bc98dde5824e321 is my current head in llvm.
> This commit generates some llvm.R600.store.pixel.color/stencil/depth
> intrinsics, which were introduced by this
On 11/05/2012 11:33 AM, Kenneth Graunke wrote:
These were only used for geometry shader support back in the days before
the new GLSL compiler. Future geometry shader support will not use
these.
Reviewed-by: Ian Romanick
---
src/mesa/program/prog_instruction.c | 2 --
src/mesa/progra
Hi,
Do you use the master branch ?
This commit generates some llvm.R600.store.pixel.color/stencil/depth
intrinsics, which were introduced by this commit :
http://cgit.freedesktop.org/~tstellar/llvm/commit/?id=761b5e4d7d62f7dc7ac5561f355a1eb0d7dd766e
I think the "unreachable" error means that ll
https://bugs.freedesktop.org/show_bug.cgi?id=55856
--- Comment #5 from Ian Romanick ---
Comment on attachment 69634
--> https://bugs.freedesktop.org/attachment.cgi?id=69634
egl_dri2/x11: Fix eglPostSubBufferNV()
Review of attachment 69634:
--
On 11/06/2012 12:58 PM, Ian Romanick wrote:
On 11/05/2012 09:57 AM, Fredrik Höglund wrote:
This extension is functionally the same as GL_MESA_pack_invert,
but is also available in GLES.
Do we have any piglit tests for this?
No. But there's a Mesa demo (src/tests/invert.c) that might be
re-
On 11/05/2012 09:57 AM, Fredrik Höglund wrote:
This extension is functionally the same as GL_MESA_pack_invert,
but is also available in GLES.
Do we have any piglit tests for this?
---
docs/relnotes-9.1.html |1 +
src/mapi/glapi/gen/es_EXT.xml|7 +++
src/mesa/main
On 11/06/2012 12:24 PM, Ian Romanick wrote:
On 11/04/2012 03:43 PM, Brian Paul wrote:
---
src/mesa/main/get.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index c34d873..fccad67 100644
--- a/src/mesa/main/get.c
+++ b/src/mesa/mai
On 11/06/2012 12:25 PM, Ian Romanick wrote:
On 11/04/2012 03:43 PM, Brian Paul wrote:
---
src/mesa/main/fbobject.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index fc5681c..d54f6a9 100644
--- a/src/mesa/main/fbobje
On 11/06/2012 12:26 PM, Ian Romanick wrote:
On 11/04/2012 03:44 PM, Brian Paul wrote:
---
src/mesa/main/transformfeedback.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/transformfeedback.c
b/src/mesa/main/transformfeedback.c
index 1afc0dc..1e3a7ea 100644
Previously we were accepting garbage after #else and #endif tokens when
the previous preprocessor conditional had evaluated to false (eg, #if 0)
When the preprocessor hits a false conditional, it switches the lexer
into the SKIP state, in which it ignores non-control tokens. The parser
pops the SK
Brian Paul writes:
> On 10/25/2012 10:13 AM, Eric Anholt wrote:
>> +struct hash_entry *
>> +_mesa_hash_table_random_entry(struct hash_table *ht,
>> + bool (*predicate)(struct hash_entry *entry))
>
> Just curious, what is this function used for? Just testing?
We have
On 11/04/2012 03:44 PM, Brian Paul wrote:
---
src/mesa/main/transformfeedback.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/transformfeedback.c
b/src/mesa/main/transformfeedback.c
index 1afc0dc..1e3a7ea 100644
--- a/src/mesa/main/transformfeedback
On 11/04/2012 03:43 PM, Brian Paul wrote:
---
src/mesa/main/fbobject.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index fc5681c..d54f6a9 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@
On 11/04/2012 03:43 PM, Brian Paul wrote:
---
src/mesa/main/get.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index c34d873..fccad67 100644
--- a/src/mesa/main/get.c
+++ b/src/mesa/main/get.c
@@ -1480,7 +1480,7 @@ _mesa_Ge
Hi,
On Monday, November 05, 2012 20:38:24 Tom Stellard wrote:
> I think you will have better luck narrowing down the issue if you bisect
> Mesa, rather than LLVM.
I see something that looks for the first cut at least quiet similar.
Reverting
commit 84b437213294ff4e1a3bcae2f9cbb36a9b4955c4
Auth
We were successfully freeing our compile data at context destroy, but until
then we were allocating a new store every compile without freeing it.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=56019
---
src/mesa/drivers/dri/i965/brw_context.h |1 -
src/mesa/drivers/dri/i965/brw_vtbl.c
https://bugs.freedesktop.org/show_bug.cgi?id=55856
--- Comment #4 from Fredrik Höglund ---
Created attachment 69634
--> https://bugs.freedesktop.org/attachment.cgi?id=69634&action=edit
egl_dri2/x11: Fix eglPostSubBufferNV()
This patch should fix the problem.
--
You are receiving this mail be
https://bugs.freedesktop.org/show_bug.cgi?id=55856
--- Comment #3 from Fredrik Höglund ---
KWin uses EGL_NV_post_sub_buffer to do partial updates. But it looks like
dri2_swap_buffers_region(), which this commit changed, is called by both
eglSwapBuffersRegionNOK() and eglPostSubBufferNV(). So this
---
src/gallium/drivers/r600/r600_shader.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/r600/r600_shader.c
b/src/gallium/drivers/r600/r600_shader.c
index 4634c30..1dc0cc2 100644
--- a/src/gallium/drivers/r600/r600_shader.c
+++ b/src/gallium/drivers/r600/r600_shade
Vector indexing on matrixes generates several copy of the
constant matrix, for instance vec=mat4[i][j] generates :
vec=mat4[i].x;
vec=(j==1)?mat4[i].y;
vec=(j==2)?mat4[i].z;
vec=(j==3)?mat4[i].w;
In the case of constant matrixes, the mat4[i] expression generates
copy of the 16 elements of the matri
On 11/05/2012 11:30 PM, Vinson Lee wrote:
Fixes a uninitialized pointer read defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/drivers/svga/svga_swtnl_draw.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_swtnl_draw.c
b/
Series looks good AFAICT
Jose
- Original Message -
> ---
> src/glsl/ir_builder.cpp |2 +-
> src/glsl/ir_builder.h |2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/ir_builder.cpp b/src/glsl/ir_builder.cpp
> index d96e25c..c62f0b1 100644
> ---
On 11/05/2012 06:19 PM, Roland Scheidegger wrote:
> Am 05.11.2012 18:45, schrieb Adhemerval Zanella:
>> Hi all,
>>
>> This patch adds PPC Altivec support for pack/unpack operations using Altivec
>> supported vector type (8xi8, 16xi16, 4xi32, 4xf32). I focused my work on the
>> lp_test_conv testcase
https://bugs.freedesktop.org/show_bug.cgi?id=55856
Andrea Scarpino changed:
What|Removed |Added
CC||and...@archlinux.org
--
You are recei
https://bugs.freedesktop.org/show_bug.cgi?id=55856
ValdikSS changed:
What|Removed |Added
CC||i...@valdikss.org.ru
--
You are receiving th
https://bugs.freedesktop.org/show_bug.cgi?id=55856
weigert.ste...@googlemail.com changed:
What|Removed |Added
CC||weigert.stefan@googlemail.
77 matches
Mail list logo