[Mesa-dev] [PATCH] draw: Change slot from unsigned to int.

2013-08-04 Thread Vinson Lee
unfilled_stage::face_slot is of type int. Fixes Unsigned compared against 0 defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/draw/draw_pipe_unfilled.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH] r600g/sb: Dump correct value for CND.

2013-08-04 Thread Vinson Lee
Fixes Copy-paste error reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/sb/sb_bc_dump.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/sb/sb_bc_dump.cpp b/src/gallium/drivers/r600/sb/sb_bc_dump.cpp

[Mesa-dev] [PATCH] nvc0: Initialize ptr for unexpected sample_count on release builds.

2013-08-04 Thread Vinson Lee
Fixes Uninitialized pointer read defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nvc0/nvc0_context.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nvc0/nvc0_context.c b/src/gallium/drivers/nvc0/nvc0_context.c index

[Mesa-dev] [RFC PATCH 11/10] i965/clip: Support clip distances for line clipping

2013-08-04 Thread Chris Forbes
This does the same thing as we do for triangle clipping -- select the appropriate source (either dot(hpos,fixed plane) or a clipdistance slot). Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_clip_line.c | 66 ++- 1 file changed, 47

[Mesa-dev] [PATCH] llvmpipe: Do not need to free anything if there is no geometry shader.

2013-08-04 Thread Vinson Lee
If gs is null, then freeing state-shader.tokens would result in a null dereference. Fixes Dereference after null check defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/llvmpipe/lp_state_gs.c | 7 +-- 1 file changed, 5 insertions(+), 2

Re: [Mesa-dev] [RFC PATCH 03/10] i965/fs: Gen4/5: Fix inconsistency in the VUE map.

2013-08-04 Thread Chris Forbes
Turns out this patch is rubbish, and only fixes the layout for the interpolation tests at the cost of breaking a bunch of other stuff. -- Chris ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] r600g/sb: Dump correct value for CND.

2013-08-04 Thread Vadim Girlin
On 08/04/2013 11:02 AM, Vinson Lee wrote: Fixes Copy-paste error reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/sb/sb_bc_dump.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/sb/sb_bc_dump.cpp

[Mesa-dev] [Bug 67740] New: render2.c: In function '__indirect_glMap1d'

2013-08-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=67740 Priority: medium Bug ID: 67740 Assignee: mesa-dev@lists.freedesktop.org Summary: render2.c: In function '__indirect_glMap1d' Severity: blocker Classification: Unclassified

Re: [Mesa-dev] [PATCH] i965: Don't allocate curbe buffers on Gen6+.

2013-08-04 Thread Eric Anholt
Kenneth Graunke kenn...@whitecape.org writes: These are only used on Gen4-5. Why waste the 8kB of space? Signed-off-by: Kenneth Graunke kenn...@whitecape.org Reviewed-by: Eric Anholt e...@anholt.net pgp9qi9CVZ_mA.pgp Description: PGP signature

Re: [Mesa-dev] [PATCH 1/2] i965: Add a new debug mode for the VUE map

2013-08-04 Thread Kenneth Graunke
On 08/03/2013 07:41 PM, Chris Forbes wrote: INTEL_DEBUG=vue now emits a listing of each slot in the VUE map, and the corresponding interpolation mode. Signed-off-by: Chris Forbes chr...@ijw.co.nz --- src/mesa/drivers/dri/i965/brw_interpolation_map.c | 28 +++