On Mit, 2013-02-20 at 08:20 +0100, Vinson Lee wrote:
Fixes resource leak defect reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
src/gallium/drivers/radeonsi/si_state.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
Why did glsl implement this really as x * (1 - a) + y * a?
The usual way for lerp would be (y - x) * a + x, i.e. two ops for most
gpus (sub+mad, or sub+mul+add). But I'm wondering if that sacrifices
precision
Yes.
Following changes improve gles2 conformance on android. There are
additional failures related to sampling but those will be dealt later.
Tapani Pälli (2):
mesa: add missing case in _mesa_GetTexParameterfv()
mesa/es: NULL check in EGLImageTargetTexture2DOES
src/mesa/main/teximage.c | 6
missing case GL_REQUIRED_TEXTURE_IMAGE_UNITS_OES is required
by OES_EGL_image_external extension.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/main/texparam.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c
index
check that pointer passed is valid and return error if not.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
---
src/mesa/main/teximage.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index f8f517a..f0de0fb 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=61149
Antony Riakiotakis kal...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
From: Michel Dänzer michel.daen...@amd.com
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
Not sure why the lack of this didn't seem to cause problems in my testing
last week...
lib/Target/R600/SIInstructions.td | 3
From: Michel Dänzer michel.daen...@amd.com
Only compile tested, but should fix at least some piglit fbo-blending tests.
NOTE: This is a candidate for the stable branches.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
src/gallium/drivers/r600/evergreen_state.c | 13 +
1
From: Michel Dänzer michel.daen...@amd.com
11 more little piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
---
v2: Marek pointed out there actually is a hardware bit for this as of
Cayman. :)
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Wed, Feb 20, 2013 at 1:16 PM, Michel Dänzer mic...@daenzer.net wrote:
From: Michel Dänzer michel.daen...@amd.com
11 more little piglits.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
Reviewed-by: Marek Olšák mar...@gmail.com
Marek
On Wed, Feb 20, 2013 at 1:16 PM, Michel Dänzer mic...@daenzer.net wrote:
From: Michel Dänzer michel.daen...@amd.com
Only compile tested, but should fix at least some piglit fbo-blending tests.
NOTE: This is a candidate for the stable branches.
Looks good to me.
Jose
- Original Message -
We sometimes convert GL_QUAD_STRIP prims into GL_TRIANGLE_STRIP, but
that changes the results of the u_trim_pipe_prim() call. We need to
pass the original primitive type to the trim function.
Note that OpenGL's GL_x prim type values
Good catch Roland.
Reviewed-by: Jose Fonseca jfons...@vmware.com
Jose
- Original Message -
From: Roland Scheidegger srol...@vmware.com
For constant and temporary register fetches, the bitcasts weren't done
correctly for the indirect case, leading to crashes due to type mismatches.
When we destroy an ARB vp/fp whose ID was gen'd but not otherwise used we
get a pointer to the dummy/placeholder program. We can't destroy that one
so just skip it. This only failed during context tear-down because
glDeleteProgramsARB() was already aware of dummy programs.
Fixes
https://bugs.freedesktop.org/show_bug.cgi?id=59187
--- Comment #3 from Lukas M l...@slyon.de ---
In #60481 Ian Romanick said Source-Games would depend on GL3..
I demonstrated that the Source-Games don't need full GL3, but just depend on
GLSL 1.30 [0]. I run CS:S using Mesa-9.0's softpipe driver
https://bugs.freedesktop.org/show_bug.cgi?id=59187
Lukas M l...@slyon.de changed:
What|Removed |Added
Summary|[Steam] Black screen but|[Steam] Implement GLSL 1.30
Am 20.02.2013 11:39, schrieb Aras Pranckevicius:
Why did glsl implement this really as x * (1 - a) + y * a?
The usual way for lerp would be (y - x) * a + x, i.e. two ops for most
gpus (sub+mad, or sub+mul+add). But I'm wondering if that sacrifices
precision
Yes.
https://bugs.freedesktop.org/show_bug.cgi?id=59187
--- Comment #4 from Paul Berry stereotype...@gmail.com ---
(In reply to comment #3)
In #60481 Ian Romanick said Source-Games would depend on GL3..
I demonstrated that the Source-Games don't need full GL3, but just depend on
GLSL 1.30 [0]. I
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, 2013-02-13 at 11:34 -0500, Tom Stellard wrote:
There's just the one cleanup
https://bugs.freedesktop.org/show_bug.cgi?id=59187
--- Comment #5 from Paul Berry stereotype...@gmail.com ---
(In reply to comment #4)
(In reply to comment #3)
In #60481 Ian Romanick said Source-Games would depend on GL3..
I demonstrated that the Source-Games don't need full GL3, but just
https://bugs.freedesktop.org/show_bug.cgi?id=59187
seves985 seves...@gmail.com changed:
What|Removed |Added
Summary|[Steam] Implement GLSL 1.30 |[Steam] Implement GLSL 1.30
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R600/SIInstrInfo.td
index
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 52 ++--
1 file changed, 34 insertions(+), 18 deletions(-)
diff --git a/lib/Target/R600/SIInstrInfo.td
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td |7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/lib/Target/R600/SIInstrInfo.td b/lib/Target/R600/SIInstrInfo.td
index
From: Christian König christian.koe...@amd.com
Fixing asm operation names.
v2: use ZERO constant, also add asm operands
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 38 +-
1 file changed, 21 insertions(+), 17
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUInstructions.td | 15 +++
lib/Target/R600/SIInstructions.td | 18 ++
2 files changed, 33 insertions(+)
diff --git
From: Christian König christian.koe...@amd.com
Fixing asm operation names.
v2: fix name of the e64 encoding, also add asm operands
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/AMDGPUInstructions.td |5 +
lib/Target/R600/SIInstrInfo.td| 23 +-
From: Christian König christian.koe...@amd.com
Order the classes and add asm operands.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrInfo.td | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/InstPrinter/AMDGPUInstPrinter.cpp |4 ++
lib/Target/R600/SIInstructions.td | 45 +++--
2 files changed, 28 insertions(+), 21
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstructions.td |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
index
From: Christian König christian.koe...@amd.com
Instead of using custom inserters, it's simpler and
should make DAG folding easier.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIISelLowering.cpp | 36
From: Christian König christian.koe...@amd.com
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstructions.td | 14 +-
1 file changed, 5 insertions(+), 9 deletions(-)
diff --git a/lib/Target/R600/SIInstructions.td
b/lib/Target/R600/SIInstructions.td
https://bugs.freedesktop.org/show_bug.cgi?id=59187
--- Comment #6 from seves985 seves...@gmail.com ---
This isn't only an IronLake-issue.
This issue is related to any chipsets older then SandyBridge.
--
You are receiving this mail because:
You are the assignee for the bug.
From: Christian König christian.koe...@amd.com
Those two files got mixed up.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIInstrFormats.td | 500 +--
lib/Target/R600/SIInstrInfo.td| 515 -
From: Christian König christian.koe...@amd.com
Fixing asm operation names.
v2: use ZERO constant, also add asm operands
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIISelLowering.cpp |3 ---
lib/Target/R600/SIInstrInfo.td | 37
From: Christian König christian.koe...@amd.com
It actually fixes quite a bunch of piglit tests.
This is a candidate for the mesa-stable branch.
Signed-off-by: Christian König christian.koe...@amd.com
---
lib/Target/R600/SIISelLowering.cpp | 22 --
On Wed, Feb 20, 2013 at 12:13:43PM +0100, Michel Dänzer wrote:
From: Michel Dänzer michel.daen...@amd.com
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer michel.daen...@amd.com
Reviewed-by: Tom Stellard thomas.stell...@amd.com
---
Not sure why the lack
On 02/18/2013 07:20 AM, Brian Paul wrote:
On 02/18/2013 12:12 AM, Tapani Pälli wrote:
This patch implements a stub for GL_EXT_discard_framebuffer with
required checks listed by the extension specification. This extension
is required by GLBenchmark 2.5 when compiled with OpenGL ES 2.0
as the
https://bugs.freedesktop.org/show_bug.cgi?id=59187
Matt Turner matts...@gmail.com changed:
What|Removed |Added
Summary|[Steam] Implement GLSL 1.30 |[Steam] Implement GLSL
This fixes a bug introduced in commit 258453716f001eab1288d99765213 and
triggered whenever rb is NULL. This uses the same condition as similar
code in brw_cc.c (used for older hardware).
Fixes bug #59445:
[SNB/IVB/HSW Bisected]Oglc draw-buffers2(advanced.blending.none)
segfault
Well, I really wonder why it doesn't work with TF2. We have a lot of
piglit tests for DISCARD_RANGE (AKA INVALIDATE_RANGE in GL) and the
copy-buffer functionality and they all pass. There are actually 3
different implementations of copy-buffer: streamout, CP DMA, and async
DMA. It's highly
Matt Turner matts...@gmail.com writes:
Is already checked 20 lines below.
This series is:
Reviewed-by: Eric Anholt e...@anholt.net
The last 2 patches are a bit weird-looking: in changing the gen7
behavior, they change a test from = 6 to 7 instead of == 6.
This is actually OK because the 6
Matt Turner matts...@gmail.com writes:
From: Kenneth Graunke kenn...@whitecape.org
Previously, we had separate constructors for one, two, and four operand
expressions. This patch consolidates them into a single constructor
which uses NULL default parameters.
The unary and binary operator
On 20 February 2013 12:25, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
From: Kenneth Graunke kenn...@whitecape.org
Previously, we had separate constructors for one, two, and four operand
expressions. This patch consolidates them into a single constructor
On 02/11/2013 04:31 PM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
From: Ian Romanick ian.d.roman...@intel.com
Three parts to the fix:
1. If the array dereference is a constant index, figure out the real
write mask.
2. If the array dereference not is a constant index,
From: Alex Deucher alexander.deuc...@amd.com
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 9.1 branch.
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
From: Tom Stellard thomas.stell...@amd.com
---
.../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
On Wed, Feb 20, 2013 at 04:20:30PM -0500, alexdeuc...@gmail.com wrote:
From: Alex Deucher alexander.deuc...@amd.com
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
The recent change for GL core broke the older setup, which broke gl_PointCoord
on pre-gen6 (where gl_PointCoord is undefined if point sprites are disabled).
Fixes the new piglit GLES-2.0/glsl-fs-pointcoord test.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=32429
Note: This is a
When you didn't have a texcoord array bound (or a non-1 current w
attrib), we were telling the fragment shader that it could just use 1
instead of doing expensive pre-gen6 math to invert it. If you drew the
point with a non-1 W value, then you'd get the right size (since all the
vertex
Paul Berry stereotype...@gmail.com writes:
On 20 February 2013 12:25, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
From: Kenneth Graunke kenn...@whitecape.org
Previously, we had separate constructors for one, two, and four operand
expressions. This patch
I've managed to capture a trace that loads TF2 to the menu and reproduces
some of the flickering. I haven't managed to capture any gameplay yet due
to an error in CD Key authentication due to how I'm launching the game.
URL:
http://www.watrys.net/tf2_menu.trace.xz
--Aaron
On Wed, Feb 20, 2013
Ian Romanick i...@freedesktop.org writes:
On 02/19/2013 04:27 PM, Carl Worth wrote:
This fixes a bug introduced in commit 258453716f001eab1288d99765213 and
triggered whenever rb is NULL.
Fixes bug #59445:
[SNB/IVB/HSW Bisected]Oglc draw-buffers2(advanced.blending.none)
segfault
Chad Versace chad.vers...@linux.intel.com 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 Mesa developers too.
Summary
https://bugs.freedesktop.org/show_bug.cgi?id=59331
fangxun xunx.f...@intel.com changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--- Comment #5
https://bugs.freedesktop.org/show_bug.cgi?id=59238
--- Comment #9 from Rafał Mużyło galtge...@o2.pl ---
Just a minor thing: looking at the content of GALLIUM_CFLAGS shouldn't they be
added to AM_CPPFLAGS instead of AM_CFLAGS ?
Also, given that $(DEFINES) variable is technically deprecated, that
https://bugs.freedesktop.org/show_bug.cgi?id=61199
Priority: medium
Bug ID: 61199
Keywords: regression
CC: lem...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: mapi/glapi/glprocs.h:2573:5: error:
In a debug build this led to assertion failures, but on a non-debug
build the hardware would just reference the whole vec8 instead of the
same channel 8 times.
Fixes the new piglit glsl-1.40/uniform-buffer/fs-exp2.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=57121
Note: This is a
https://bugs.freedesktop.org/show_bug.cgi?id=61200
Priority: medium
Bug ID: 61200
Assignee: mesa-dev@lists.freedesktop.org
Summary: insufficient linking of libxatracker.so
Severity: normal
Classification: Unclassified
OS:
On 02/20/2013 06:39 PM, Eric Anholt wrote:
In a debug build this led to assertion failures, but on a non-debug
build the hardware would just reference the whole vec8 instead of the
same channel 8 times.
Fixes the new piglit glsl-1.40/uniform-buffer/fs-exp2.
Bugzilla:
On 02/20/2013 04:58 PM, Eric Anholt wrote:
The recent change for GL core broke the older setup, which broke gl_PointCoord
on pre-gen6 (where gl_PointCoord is undefined if point sprites are disabled).
Fixes the new piglit GLES-2.0/glsl-fs-pointcoord test.
Bugzilla:
https://bugs.freedesktop.org/show_bug.cgi?id=61199
--- Comment #1 from Tapani Pälli lem...@gmail.com ---
Sorry for the regression, I don't know much about scons but I'm able to
reproduce the error and will try to fix this.
--
You are receiving this mail because:
You are the assignee for the
Signed-off-by: Vinson Lee v...@freedesktop.org
Reported-by: Michel Dänzer mic...@daenzer.net
---
src/gallium/drivers/r600/r600_state_common.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/r600/r600_state_common.c
b/src/gallium/drivers/r600/r600_state_common.c
index
https://bugs.freedesktop.org/show_bug.cgi?id=61199
--- Comment #2 from Tapani Pälli lem...@gmail.com ---
Created attachment 75232
-- https://bugs.freedesktop.org/attachment.cgi?id=75232action=edit
fix to the build
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=61199
Tapani Pälli lem...@gmail.com changed:
What|Removed |Added
Attachment #75232|0 |1
is obsolete|
65 matches
Mail list logo