Fixes out-of-bounds write reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/auxiliary/vl/vl_winsys_dri.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c
b/src/gallium/auxiliary/vl/vl_winsys_dri.c
index 560c914..59c02bc 1
Fixes resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/state_trackers/vdpau/output.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/vdpau/output.c
b/src/gallium/state_trackers/vdpau/output.c
index a835126..8237eac 100644
--- a/
Fixes resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/state_trackers/vdpau/bitmap.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/vdpau/bitmap.c
b/src/gallium/state_trackers/vdpau/bitmap.c
index ecc5b15..469f3e8 100644
--- a/
Fixes uninitialized scalar variable defect reported by Coverity.
Signed-off-by: Vinson Lee
---
src/gallium/state_trackers/xvmc/tests/xvmc_bench.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/xvmc/tests/xvmc_bench.c
b/src/gallium/state_trackers/xvmc/tests/xv
https://bugs.freedesktop.org/show_bug.cgi?id=46376
--- Comment #4 from ye.tian ---
The issue still exists with the follow commits:
-
Kernel_version: 3.8
Libdrm: 2.4.42
Mesa: (9.1)9.1-rc2
Xserver:(server-1.13-branch)xorg-ser
https://bugs.freedesktop.org/show_bug.cgi?id=61052
--- Comment #3 from Vinson Lee ---
Comment on attachment 75266
--> https://bugs.freedesktop.org/attachment.cgi?id=75266
possible fix
Review of attachment 75266:
-
Tested-by: Vins
https://bugs.freedesktop.org/show_bug.cgi?id=61199
--- Comment #4 from Vinson Lee ---
Comment on attachment 75234
--> https://bugs.freedesktop.org/attachment.cgi?id=75234
fix without typos included
Review of attachment 75234:
-
T
On 02/19/2013 08:39 PM, Andy Furniss wrote:
Vadim Girlin wrote:
Could you please test glxgears and other simple mesa demos? It's easier
to spot the problems with small apps that don't use a lot of complex
shaders. If some of them don't work correctly, please send me the dumps
with "R600_DUMP_SH
On 02/22/2013 04:23 AM, Alex Deucher wrote:
On Thu, Feb 21, 2013 at 6:52 PM, Vadim Girlin wrote:
v4: implement exact computation taking into account wavefront size
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/r600_asm.c| 44 +--
src/gallium/drivers/r600/r600_asm.h
On Thu, Feb 21, 2013 at 6:52 PM, Vadim Girlin wrote:
> v4: implement exact computation taking into account wavefront size
>
> Signed-off-by: Vadim Girlin
> ---
> src/gallium/drivers/r600/r600_asm.c| 44 +--
> src/gallium/drivers/r600/r600_asm.h| 24 --
> src/gallium/drivers
v4: implement exact computation taking into account wavefront size
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/r600_asm.c| 44 +--
src/gallium/drivers/r600/r600_asm.h| 24 --
src/gallium/drivers/r600/r600_shader.c | 131 ++---
3 file
From: José Fonseca
Weird format, which Mesa state tracker recently started using.
Fixes segfault in
./bin/texture-packed-formats -auto
because swizzle[foo] was 0xff for padding channel (X). It still fails though.
I believe the only reason this doesn't BGRX doesn't crash is that we have
spec
Series is
Reviewed-by: Jose Fonseca
- Original Message -
> ---
> src/gallium/drivers/llvmpipe/lp_state_setup.c | 15 +--
> src/gallium/drivers/llvmpipe/lp_state_setup.h |4 ++--
> 2 files changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/src/gallium/drivers/l
On 02/20/2013 05:44 PM, Eric Anholt wrote:
> Chad Versace writes:
>
>> -BEGIN PGP SIGNED MESSAGE-
>> Hash: SHA1
>>
>> I'm seeking feedback on an EGL extension that I'm drafting. The ideas have
>> already been discussed at Khronos meetings to a good reception, but I want
>> feedback from M
https://bugs.freedesktop.org/show_bug.cgi?id=61052
--- Comment #2 from Andreas Boll ---
Created attachment 75266
--> https://bugs.freedesktop.org/attachment.cgi?id=75266&action=edit
possible fix
Does this patch help?
--
You are receiving this mail because:
You are the assignee for the bug.
_
On Thu, Feb 21, 2013 at 10:06 AM, Tom Stellard wrote:
> On Thu, Feb 21, 2013 at 10:02:34AM -0600, Aaron Watry wrote:
> > On Thu, Feb 21, 2013 at 8:33 AM, Tom Stellard wrote:
> >
> > > On Thu, Feb 21, 2013 at 08:25:20AM -0600, Aaron Watry wrote:
> > > > Hi Tom,
> > > >
> > > > Mesa+Clover does in
From: Maarten Lankhorst
v2: Andreas Boll
- Fix formatting - use one CFLAG per line
NOTE: This is a candidate for the 9.1 branch.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59238
Reviewed-by: Andreas Boll
---
src/gallium/auxiliary/Makefile.am |7 +--
src/galliu
- Original Message -
> From: Roland Scheidegger
>
> This clarifies some things and gets rid of some old stuff.
> The most significant one is probably that buffers cannot have formats
> (nearly all drivers completely ignored format and used width0 as byte size
> already in any case). There
On Thu, Feb 21, 2013 at 8:41 AM, Andreas Boll
wrote:
> NOTE: This is a candidate for the 9.1 branch.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=61200
> ---
> src/gallium/targets/xa-vmwgfx/Makefile.am |2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/gallium/targets
Ian Romanick writes:
> On 02/15/2013 10:46 PM, Eric Anholt wrote:
>> The desktop spec asks for gl_PointCoord to be defined only when
>> GL_POINT_SPRITE is enabled, and it's undefined otherwise (why?!). The
>> ES spec doesn't have GL_POINT_SPRITE and gl_PointCoord is always
>> defined. So just m
NOTE: This is a candidate for the 9.1 branch.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=61200
---
src/gallium/targets/xa-vmwgfx/Makefile.am |2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/targets/xa-vmwgfx/Makefile.am
b/src/gallium/targets/xa-vmwgfx/Makefile.am
From: Roland Scheidegger
Now with buffer formats clarification don't need all that logic any longer.
(Note that it never would have worked in any case, because blockwidth and
blockheight were swapped any allocation with multi-byte format would have
had zero size.)
---
src/gallium/drivers/llvmpip
From: Roland Scheidegger
This clarifies some things and gets rid of some old stuff.
The most significant one is probably that buffers cannot have formats
(nearly all drivers completely ignored format and used width0 as byte size
already in any case). There seems to be no use case for "structured"
Mesa 9.0.3 has been released. Mesa 9.0.3 is a bug fix release which
fixes bugs found since the 9.0.2 release.
The tag in the GIT repository for Mesa 9.0.3 is 'mesa-9.0.3'.
Mesa 9.0.3 is available for download at
ftp://freedesktop.org/pub/mesa/9.0.3/
md5sums:
168384ac0101f4600a15edd3561acdc7 M
On Thu, Feb 21, 2013 at 10:02:34AM -0600, Aaron Watry wrote:
> On Thu, Feb 21, 2013 at 8:33 AM, Tom Stellard wrote:
>
> > On Thu, Feb 21, 2013 at 08:25:20AM -0600, Aaron Watry wrote:
> > > Hi Tom,
> > >
> > > Mesa+Clover does indeed build against master llvm/clang, but I'm having
> > > trouble bu
On Thu, Feb 21, 2013 at 8:33 AM, Tom Stellard wrote:
> On Thu, Feb 21, 2013 at 08:25:20AM -0600, Aaron Watry wrote:
> > Hi Tom,
> >
> > Mesa+Clover does indeed build against master llvm/clang, but I'm having
> > trouble building against it when I try to do a clean build of Piglit.
> >
> > Error r
Hi all,
I am currently wondering why I have a big difference in performance between
2 Mesa drivers: egl_glx and egl_gallium.
I have already posted a mesasge 2 weeks ago on the mesa user list, but I
did not receive any answer. Please find below my previous mail
I am currently working on an ARM pro
---
src/gallium/drivers/llvmpipe/lp_state_setup.c | 15 +--
src/gallium/drivers/llvmpipe/lp_state_setup.h |4 ++--
2 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_state_setup.c
b/src/gallium/drivers/llvmpipe/lp_state_setup.c
index
The llvm pipeline handles regular filled triangle offsets, but it
doesn't handle offsets for triangles drawn in point or line mode.
Fixes failures found with new piglit polygon-mode-offset test.
Note: This is a candidate for the stable branches.
---
src/gallium/drivers/llvmpipe/lp_state_rasteriz
There were several issues. We weren't handling different front/back
polygon fill modes. We weren't checking whether the offset applied to
fill mode vs. line mode vs. point mode.
Fixes problems found with the Visualization Toolkit (VTK) test suite.
Note: This is a candidate for the stable branch
The conditionals were testing the wrong state. Just get rid of the
conditionals since they really don't buy us anything.
Note: This is a candidate for the stable branches.
---
src/mesa/state_tracker/st_atom_rasterizer.c | 17 +
1 files changed, 5 insertions(+), 12 deletions(-)
On Mon, Feb 18, 2013 at 05:27:28PM +0100, Vincent Lejeune wrote:
> ---
This fixes the for-loop.cl piglit test, so I've pushed this patch and
backported it to the stable branch.
-Tom
> lib/Target/R600/R600Instructions.td | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Target/R600/
On 16 February 2013 15:16, Paul Berry wrote:
> On 16 February 2013 13:52, Kenneth Graunke wrote:
>
>> On 02/16/2013 07:29 AM, Paul Berry wrote:
>>
>>> From the Ivy Bridge PRM, p268 (3DSTATE_SBE), in the description of
>>> "Point Sprite Texture Coordinate Enable":
>>>
>>> "This field must b
On Thu, Feb 21, 2013 at 08:25:20AM -0600, Aaron Watry wrote:
> Hi Tom,
>
> Mesa+Clover does indeed build against master llvm/clang, but I'm having
> trouble building against it when I try to do a clean build of Piglit.
>
> Error received:
>
> [ 18%] Built target piglitutil_cl
> Linking C executa
Hi Tom,
Mesa+Clover does indeed build against master llvm/clang, but I'm having
trouble building against it when I try to do a clean build of Piglit.
Error received:
[ 18%] Built target piglitutil_cl
Linking C executable ../../../../../bin/cl-custom-run-simple-kernel
/usr/local/lib/libOpenCL.so:
For the series:
Reviewed-by: Tom Stellard
On Wed, Feb 20, 2013 at 06:46:47PM +0100, Christian König wrote:
> From: Christian König
>
> Those two files got mixed up.
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstrFormats.td | 500 +--
> lib
On Wed, Feb 20, 2013 at 06:47:00PM +0100, Christian König wrote:
> From: Christian König
>
> Signed-off-by: Christian König
> ---
> lib/Target/R600/SIInstructions.td | 14 +-
> 1 file changed, 5 insertions(+), 9 deletions(-)
>
> diff --git a/lib/Target/R600/SIInstructions.td
> b
On Thu, Feb 21, 2013 at 09:06:36AM -0500, Tom Stellard wrote:
> With the changes Michel suggested, this series is:
>
> Reviewed-by: Tom Stellard
>
> Are any of these candidates for the stable branch?
>
Please ignore this, I overlooked v2 of this series. I will review the
new version now.
>
With the changes Michel suggested, this series is:
Reviewed-by: Tom Stellard
Are any of these candidates for the stable branch?
-Tom
On Tue, Feb 19, 2013 at 02:54:23PM +0100, Christian König wrote:
> From: Christian König
>
> Those two files got mixed up.
>
> Signed-off-by: Christian König
https://bugs.freedesktop.org/show_bug.cgi?id=61200
--- Comment #2 from Rafał Mużyło ---
(In reply to comment #1)
> Created attachment 75240 [details] [review]
> possible fix
>
> Does this patch help?
Well, it should help - that's exactly the solution I've proposed.
On the other hand, I'm not th
Tom Stellard writes:
> From: Tom Stellard
>
Acked-by: Francisco Jerez
> ---
> .../state_trackers/clover/llvm/invocation.cpp | 47 ---
> 1 files changed, 39 insertions(+), 8 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
> b/src/gall
Looks perfect. Thanks Roland.
Jose
- Original Message -
> From: Roland Scheidegger
>
> Some parts calculated key size by using shader information, others by using
> the pipe_vertex_element information. Since it is perfectly valid to have more
> vertex_elements set than the vertex shade
On Don, 2013-02-21 at 10:38 +0100, Christian König wrote:
> Am 21.02.2013 09:39, schrieb Michel Dänzer:
> > On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
> >> diff --git a/lib/Target/R600/SIInstructions.td
> >> b/lib/Target/R600/SIInstructions.td
> >> index 700b8f8..866c7cb 100644
> >
Am 21.02.2013 09:39, schrieb Michel Dänzer:
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index 700b8f8..866c7cb 100644
--- a/lib/Target/R600/SIInstructions.td
+++ b/lib/Target/R600/SIInstructions.td
On Mit, 2013-02-20 at 17:56 +0100, Michel Dänzer wrote:
> On Don, 2013-02-14 at 11:32 +0100, Christian König wrote:
> > Am 13.02.2013 18:22, schrieb Michel Dänzer:
> > > On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote:
> > >> Am 13.02.2013 18:11, schrieb Michel Dänzer:
> > >>> On Mit, 20
Reviewed-by: Marek Olšák
Marek
On Thu, Feb 21, 2013 at 8:43 AM, Vinson Lee wrote:
> Signed-off-by: Vinson Lee
> Reported-by: Michel Dänzer
> ---
> src/gallium/drivers/r600/r600_state_common.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/r600/r600_state_common.
Reviewed-by: Marek Olšák
Marek
On Wed, Feb 20, 2013 at 10:20 PM, wrote:
> From: Alex Deucher
>
> Can cause lockups in certain cases when
> zfunc/zenable/zwrite change without a flush
> in between.
>
> Fixes:
> https://bugs.freedesktop.org/show_bug.cgi?id=60969
>
> This is a candidate for the
https://bugs.freedesktop.org/show_bug.cgi?id=61200
--- Comment #1 from Andreas Boll ---
Created attachment 75240
--> https://bugs.freedesktop.org/attachment.cgi?id=75240&action=edit
possible fix
Does this patch help?
--
You are receiving this mail because:
You are the assignee for the bug.
_
On Don, 2013-02-21 at 09:44 +0100, Christian König wrote:
> Am 21.02.2013 09:26, schrieb Michel Dänzer:
> > On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
> >>
> >> diff --git a/lib/Target/R600/SIInstructions.td
> >> b/lib/Target/R600/SIInstructions.td
> >> index 833e6cd..7c157e9 10
Am 21.02.2013 09:26, schrieb Michel Dänzer:
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
diff --git a/lib/Target/R600/SIInstructions.td b/lib/Target/R600/SIInstructions.td
index 833e6cd..7c157e9 100644
--- a/lib/Target/R600/SIInstructions.td
+++ b/lib/Target/R600/SIInstructions.t
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
>
> diff --git a/lib/Target/R600/SIInstructions.td
> b/lib/Target/R600/SIInstructions.td
> index 700b8f8..866c7cb 100644
> --- a/lib/Target/R600/SIInstructions.td
> +++ b/lib/Target/R600/SIInstructions.td
> @@ -620,7 +620,7 @@ def V_INTER
On Mit, 2013-02-20 at 18:46 +0100, Christian König wrote:
>
> diff --git a/lib/Target/R600/SIInstructions.td
> b/lib/Target/R600/SIInstructions.td
> index 833e6cd..7c157e9 100644
> --- a/lib/Target/R600/SIInstructions.td
> +++ b/lib/Target/R600/SIInstructions.td
> @@ -1232,6 +1232,24 @@ def : P
52 matches
Mail list logo
