On Wed, Nov 07, 2012 at 09:24:13PM +, Andy Furniss wrote:
> Vincent Lejeune wrote:
> > git seems to have trouble sending my patch to ML atm, can you manually
> > apply it ? It fixes lock up here
> >
> > diff --git a/src/gallium/drivers/r600/r600_asm.c
> > b/src/gallium/drivers/r600/r600_asm.c
Fixes resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/drivers/svga/svga_resource_buffer.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/svga/svga_resource_buffer.c
b/src/gallium/drivers/svga/svga_resource_buffer.c
index 87a4a9b..3
https://bugs.freedesktop.org/show_bug.cgi?id=56685
--- Comment #8 from Thierry Reding ---
(In reply to comment #6)
> Thierry, I think your cross compiling patch caused this -- broken out of
> tree builds.
I suppose that had to happen to me, usually being very allergic to broken
out-of-tree build
https://bugs.freedesktop.org/show_bug.cgi?id=56685
--- Comment #7 from Thierry Reding ---
Comment on attachment 69667
--> https://bugs.freedesktop.org/attachment.cgi?id=69667
Possible Fix
Review of attachment 69667:
-
I came up w
hi, all:
I am looking at egl code recently about SandyBridge platform. But I
found one problem.
with the same r,g,b a byte numbers, i965 driver will return at least
two FB configs, one for single buffer, another form double buffer.
but the eglChooseConfig() function can't specify which
On 11/05/2012 01:14 PM, Tom Stellard wrote:
From: Tom Stellard
---
src/mesa/main/mtypes.h | 1 +
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 31 +-
2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/mtypes.h b/src/me
On 11/05/2012 01:14 PM, Tom Stellard wrote:
From: Tom Stellard
---
src/gallium/docs/source/screen.rst | 2 ++
src/gallium/drivers/r600/r600_pipe.c | 2 ++
src/gallium/drivers/radeonsi/radeonsi_pipe.c | 1 +
src/gallium/include/pipe/p_defines.h | 3 ++-
4 files cha
On 11/06/2012 03:16 PM, Dave Airlie wrote:
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: Da
On 11/05/2012 01:14 PM, Tom Stellard wrote:
From: Tom Stellard
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 12
src/gallium/auxiliary/tgsi/tgsi_ureg.h | 3 +++
2 files changed, 15 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_ureg.c
b/src/gallium/auxiliary/tgsi/t
Tom Stellard wrote:
On Wed, Nov 07, 2012 at 09:24:13PM +, Andy Furniss wrote:
Vincent Lejeune wrote:
git seems to have trouble sending my patch to ML atm, can you manually apply it
? It fixes lock up here
diff --git a/src/gallium/drivers/r600/r600_asm.c
b/src/gallium/drivers/r600/r600_as
Mike Lothian wrote:
Hi Andy
Try the LLVM ebuilds in the FireBurn overlay - they turn on intrinsics
without using the debug versions
Thanks, but I am using LFS.
I'm still seeing the issues when running some games but least it won't be a
debug build
OK, I'll try a non debug soon just to see
On 11/07/2012 01:51 PM, Dave Airlie wrote:
On Thu, Nov 8, 2012 at 1:00 AM, Brian Paul wrote:
On 11/06/2012 07:56 PM, Dave Airlie wrote:
From: Dave Airlie
fdo9833 piglit test caught this.
What exactly was the failure?
The pop attrib was failing with an error, probably due to the missing
Since cf438f5375e242, we store actual integers for the attribute data.
We just need to reinterpret the GLfloat array as a GLint/GLuint array
so we can read the proper data.
Fixes oglconform's glsl-vertex-attrib/basic.VertexAttribI[1234][u]i
subtests (after fixing an unrelated bug in those test cas
On 11/07/2012 05:32 PM, Chad Versace wrote:
> Patch 6 breaks the Android build. I've uploaded a branch 'hashtable'
> that contains your series without breaking the Android build. Please
> take a look at it.
Well, I should have said "I'll push the branch when fdo comes back up."
_
On 11/06/2012 11:18 PM, Eric Anholt wrote:
> 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
Vincent,
On Wednesday, November 07, 2012 08:32:03 Vincent Lejeune wrote:
> git seems to have trouble sending my patch to ML atm, can you manually apply
> it ? It fixes lock up here
>
> diff --git a/src/gallium/drivers/r600/r600_asm.c
> b/src/gallium/drivers/r600/r600_asm.c index 5f2548e..f06af44
On Wed, Nov 07, 2012 at 05:49:37PM +0100, Vincent Lejeune wrote:
> ---
Reviewed-by: Tom Stellard
> src/gallium/drivers/r600/r600_asm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/r600_asm.c
> b/src/gallium/drivers/r600/r600_asm.c
> index 5f
https://bugs.freedesktop.org/show_bug.cgi?id=56685
Matt Turner changed:
What|Removed |Added
CC||thierry.reding@avionic-desi
Hi Andy
Try the LLVM ebuilds in the FireBurn overlay - they turn on intrinsics
without using the debug versions
I'm still seeing the issues when running some games but least it won't be a
debug build
Cheers
Mike
On 7 November 2012 21:24, Andy Furniss wrote:
> Vincent Lejeune wrote:
>
>> git
Vincent Lejeune wrote:
git seems to have trouble sending my patch to ML atm, can you manually apply it
? It fixes lock up here
diff --git a/src/gallium/drivers/r600/r600_asm.c
b/src/gallium/drivers/r600/r600_asm.c
index 5f2548e..f06af44 100644
--- a/src/gallium/drivers/r600/r600_asm.c
+++ b/sr
On Thu, Nov 8, 2012 at 1:00 AM, Brian Paul wrote:
> On 11/06/2012 07:56 PM, Dave Airlie wrote:
>>
>> From: Dave Airlie
>>
>> fdo9833 piglit test caught this.
>
>
> What exactly was the failure?
The pop attrib was failing with an error, probably due to the missing
continue from below,
but you are
On 11/06/2012 11:18 PM, Eric Anholt wrote:
> ---
> 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
Hi,
+1 for more debugging ease, especially on remote targets.
-Ilyes
On Nov 7, 2012 9:06 PM, "Paul Berry" wrote:
> In situations where an OpenGL function has several equivalent names, Mesa
> currently does not follow a consistent standard for which name is used.
> For example, the function glDi
In situations where an OpenGL function has several equivalent names, Mesa
currently does not follow a consistent standard for which name is used.
For example, the function glDisablei() (introduced in GL 3.0) is equivalent
to glDisableIndexedEXT() (introduced in EXT_draw_buffers2). Mesa manages
the
https://bugs.freedesktop.org/show_bug.cgi?id=56685
Johannes Obermayr changed:
What|Removed |Added
Attachment #69491|0 |1
is obsolete|
On 11/06/2012 10:47 AM, Eric Anholt wrote:
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
On 11/06/2012 02: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/te
On 11/06/2012 02:16 PM, Dave Airlie wrote:
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
https://bugs.freedesktop.org/show_bug.cgi?id=56827
Paul Berry changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Reviewed-by: Alex Deucher
On Wed, Nov 7, 2012 at 11:49 AM, Vincent Lejeune
wrote:
> ---
> src/gallium/drivers/r600/r600_asm.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/r600_asm.c
> b/src/gallium/drivers/r600/r600_asm.c
> index 5f2548e..f0
On 11/06/2012 02:15 PM, Dave Airlie wrote:
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
On 11/06/2012 11:18 PM, Eric Anholt wrote:
---
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
@@
On 11/07/2012 10:18 AM, Eric Anholt wrote:
Brian Paul writes:
The glean glsl1 test was failing an assertion because FRAG_ATTRIB_FACE
(and FRAG_ATTRIB_PNTC) doesn't fit in a 12-bit field.
In the failing case we're using a fragment shader with fixed-function
vertex processing.
What about if t
---
src/gallium/drivers/r600/r600_asm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/r600_asm.c
b/src/gallium/drivers/r600/r600_asm.c
index 5f2548e..f06af44 100644
--- a/src/gallium/drivers/r600/r600_asm.c
+++ b/src/gallium/drivers/r600/r600_asm.c
@
On 11/07/2012 07:33 AM, Brian Paul wrote:
The glean glsl1 test was failing an assertion because FRAG_ATTRIB_FACE
(and FRAG_ATTRIB_PNTC) doesn't fit in a 12-bit field.
In the failing case we're using a fragment shader with fixed-function
vertex processing.
---
src/mesa/main/ffvertex_prog.c |
Brian Paul writes:
> The glean glsl1 test was failing an assertion because FRAG_ATTRIB_FACE
> (and FRAG_ATTRIB_PNTC) doesn't fit in a 12-bit field.
>
> In the failing case we're using a fragment shader with fixed-function
> vertex processing.
What about if they have an FS reading generic attribs
- Original Message -
> On 11/07/2012 08:26 AM, jfons...@vmware.com wrote:
> > From: José Fonseca
> >
> > Not fully tested.
> >
> > Based on diagrams from
> > http://msdn.microsoft.com/en-us/library/windows/desktop/bb205124.aspx#Primitive_Adjacency
> > ---
> > src/gallium/drivers/softpip
On 11/07/2012 09:35 AM, Roland Scheidegger wrote:
I think that probably needs a test to make sure the right vertices are
used, I can't quite tell if it's right. This is hit when adjacent
primitives are used but not geometry shader is active right?
Yeah, I had to look that up myself. Adjacency
I think that probably needs a test to make sure the right vertices are
used, I can't quite tell if it's right. This is hit when adjacent
primitives are used but not geometry shader is active right?
The rest of the series looks good to me.
Roland
Am 07.11.2012 16:26, schrieb jfons...@vmware.com:
git seems to have trouble sending my patch to ML atm, can you manually apply it
? It fixes lock up here
diff --git a/src/gallium/drivers/r600/r600_asm.c
b/src/gallium/drivers/r600/r600_asm.c
index 5f2548e..f06af44 100644
--- a/src/gallium/drivers/r600/r600_asm.c
+++ b/src/gallium/drivers/r600/r6
On 11/07/2012 08:26 AM, jfons...@vmware.com wrote:
From: José Fonseca
---
src/gallium/auxiliary/util/u_rect.h | 15 +++
1 file changed, 15 insertions(+)
Except for my other comments the series looks good.
Reviewed-by: Brian Paul
___
On 11/07/2012 08:26 AM, jfons...@vmware.com wrote:
From: José Fonseca
Alpha is also 1 for formats like R32G32_FLOAT.
---
src/gallium/auxiliary/util/u_format.c | 26 +-
1 file changed, 1 insertion(+), 25 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_format.c
On 11/07/2012 08:26 AM, jfons...@vmware.com wrote:
From: José Fonseca
We must multiply the factor against the destination, not the source.
---
src/gallium/drivers/softpipe/sp_quad_blend.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/softpipe
On 11/07/2012 08:26 AM, jfons...@vmware.com wrote:
From: José Fonseca
Not fully tested.
Based on diagrams from
http://msdn.microsoft.com/en-us/library/windows/desktop/bb205124.aspx#Primitive_Adjacency
---
src/gallium/drivers/softpipe/sp_prim_vbuf.c | 46 +++
1 file
On 11/07/2012 03:48 AM, Michel Dänzer wrote:
On Die, 2012-11-06 at 07:11 -0800, Brian Paul wrote:
Module: Mesa
Branch: master
Commit: 0d61f879a1f2e6bb37368731a29d5267e1c25195
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0d61f879a1f2e6bb37368731a29d5267e1c25195
Author: Brian Paul
Da
The glean glsl1 test was failing an assertion because FRAG_ATTRIB_FACE
(and FRAG_ATTRIB_PNTC) doesn't fit in a 12-bit field.
In the failing case we're using a fragment shader with fixed-function
vertex processing.
---
src/mesa/main/ffvertex_prog.c |6 +++---
1 files changed, 3 insertions(+),
From: José Fonseca
---
src/gallium/auxiliary/util/u_surface.c | 70
1 file changed, 44 insertions(+), 26 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_surface.c
b/src/gallium/auxiliary/util/u_surface.c
index 055e538..e99431e 100644
--- a/src/gallium/
From: José Fonseca
---
src/gallium/drivers/galahad/glhd_context.c | 192
1 file changed, 110 insertions(+), 82 deletions(-)
diff --git a/src/gallium/drivers/galahad/glhd_context.c
b/src/gallium/drivers/galahad/glhd_context.c
index 53e0b17..ac666d4 100644
--- a/src
From: José Fonseca
draw_delete_geometry_shader() seems to be the real one.
---
src/gallium/auxiliary/draw/draw_gs.c |8 ++--
src/gallium/auxiliary/draw/draw_gs.h |2 --
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/src/gallium/auxiliary/draw/draw_gs.c
b/src/gallium/
From: José Fonseca
---
src/gallium/drivers/trace/tr_context.c | 186
1 file changed, 71 insertions(+), 115 deletions(-)
diff --git a/src/gallium/drivers/trace/tr_context.c
b/src/gallium/drivers/trace/tr_context.c
index fed95c6..64d039c 100644
--- a/src/gallium
From: José Fonseca
util_pack_z_stencil was being unconditionally invoked for all formats,
causing an assertion failure for Z32_FLOAT_S8X24_UINT.
---
src/gallium/auxiliary/util/u_surface.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/src/gallium/auxili
From: José Fonseca
Alpha is also 1 for formats like R32G32_FLOAT.
---
src/gallium/auxiliary/util/u_format.c | 26 +-
1 file changed, 1 insertion(+), 25 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_format.c
b/src/gallium/auxiliary/util/u_format.c
index a41c46
From: José Fonseca
We must multiply the factor against the destination, not the source.
---
src/gallium/drivers/softpipe/sp_quad_blend.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/softpipe/sp_quad_blend.c
b/src/gallium/drivers/softpipe/sp_qu
From: José Fonseca
For drivers with native integer / SM4 support this is just an hindrance.
---
src/gallium/auxiliary/tgsi/tgsi_exec.c |1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c
b/src/gallium/auxiliary/tgsi/tgsi_exec.c
index 0b58f09..05bf070 100
From: José Fonseca
---
src/gallium/auxiliary/util/u_prim.h |2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_prim.h
b/src/gallium/auxiliary/util/u_prim.h
index 070df64..d62c636 100644
--- a/src/gallium/auxiliary/util/u_prim.h
+++ b/src/gallium/auxiliary/util/
From: José Fonseca
Not fully tested.
Based on diagrams from
http://msdn.microsoft.com/en-us/library/windows/desktop/bb205124.aspx#Primitive_Adjacency
---
src/gallium/drivers/softpipe/sp_prim_vbuf.c | 46 +++
1 file changed, 46 insertions(+)
diff --git a/src/gallium/dr
From: José Fonseca
---
src/gallium/auxiliary/util/u_rect.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_rect.h
b/src/gallium/auxiliary/util/u_rect.h
index 4cb90d3..8fccae8 100644
--- a/src/gallium/auxiliary/util/u_rect.h
+++ b/src/gallium/a
On 11/06/2012 07:56 PM, Dave Airlie wrote:
From: Dave Airlie
fdo9833 piglit test caught this.
What exactly was the failure?
BTW, that test should really be rewritten as non-bug test. I think
I'll do that.
Signed-off-by: Dave Airlie
---
src/mesa/main/attrib.c | 10 --
1 file
https://bugs.freedesktop.org/show_bug.cgi?id=56685
--- Comment #4 from Fabio Pedretti ---
> Created attachment 69491 [details] [review]
> Please try this patch.
I am still getting the same error:
https://launchpadlibrarian.net/122265792/buildlog_ubuntu-quantal-i386.mesa_9.1~git1211071345.65faed~
https://bugs.freedesktop.org/show_bug.cgi?id=44618
Thierry Reding changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=44618
--- Comment #17 from Ross Burton ---
I see the patches landed, can this bug be closed or is there still outstanding
work?
--
You are receiving this mail because:
You are the assignee for the bug.
___
On Die, 2012-11-06 at 07:11 -0800, Brian Paul wrote:
> Module: Mesa
> Branch: master
> Commit: 0d61f879a1f2e6bb37368731a29d5267e1c25195
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=0d61f879a1f2e6bb37368731a29d5267e1c25195
>
> Author: Brian Paul
> Date: Sun Nov 4 16:43:44 2012
https://bugs.freedesktop.org/show_bug.cgi?id=39393
libo changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--
You are receiving this mail because:
https://bugs.freedesktop.org/show_bug.cgi?id=39393
libo changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
64 matches
Mail list logo