On Fri, May 25, 2012 at 01:13:13PM -0700, Eric Anholt wrote:
> On Fri, 25 May 2012 16:16:30 +0300, Oliver McFadden
> wrote:
> > This bug was discovered while running the Khronos conformance test suite
> > on i965 classic driver and receiving "implementation error: meta program
> > compile failed.
Can someone please review and commit this? A lot of the piglit
tests fail because of this.
Kurt
On Fri, May 11, 2012 at 12:19:42AM +0200, Kurt Roeckx wrote:
> commit 87f12bb2d95236c7b025d1a8be56b5ab1683d702 tried to fix rb->mt
> being NULL, but change this case wrong.
>
> Note: this is a candi
On 05/24/2012 06:29 PM, Stéphane Marchesin wrote:
From: Antoine Labour
When we have multiple shared contexts, and one of them is
long-running, this will lead to never freeing those resources
since they are shared. Instead, free them right away on context
destruction since we know the other conte
On 05/24/2012 06:29 PM, Stéphane Marchesin wrote:
Works around crashes when X connections break.
Reviewed-by: Ian Romanick
This should also be tagged with:
NOTE: This is a candidate for the 8.0 branch.
---
src/glx/glxext.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
di
On 05/24/2012 06:29 PM, Stéphane Marchesin wrote:
From: Antoine Labour
When a value was replaced, the new key was strdup'd and leaked.
To fix this, we modify the hash table implementation to return
whether the value was replaced and free() the (now useless)
duplicate string.
This looks good, b
On 05/24/2012 06:29 PM, Stéphane Marchesin wrote:
From: Antoine Labour
This is an array of uniforms, not a single one.
Reviewed-by: Ian Romanick
This should also be tagged with:
NOTE: This is a candidate for the 8.0 branch.
---
src/mesa/main/shaderobj.c |3 ++-
1 files changed, 2 i
On 05/25/2012 01:13 PM, Eric Anholt wrote:
On Fri, 25 May 2012 16:16:30 +0300, Oliver
McFadden wrote:
This bug was discovered while running the Khronos conformance test suite
on i965 classic driver and receiving "implementation error: meta program
compile failed."
This bug was recently introd
On 05/23/2012 12:45 PM, Eric Anholt wrote:
Mesa already always depends on python to build. The checked in
changes are not reviewed (because any trivial change rewrites the
world). We also have been pushing commits between xml change and
regen where at-build-time xml-generated code disagrees wit
On Fri, 25 May 2012 16:16:30 +0300, Oliver McFadden
wrote:
> This bug was discovered while running the Khronos conformance test suite
> on i965 classic driver and receiving "implementation error: meta program
> compile failed."
>
> This bug was recently introduced by the i965 clear patch set and
On Fri, 25 May 2012 10:51:31 -0700, Han Shen(沈涵) wrote:
> Hi I am working on chromium, and while I was compiling mesa trunk
> using gcc 4.7, I had to add explicit type cast for narrower conversion
> inside the initialization list.
>
> Could you take a look at the patch and upstream it if it is ok
Pushed, thanks.
Marek
On Fri, May 25, 2012 at 4:27 PM, Kai Wasserbäch
wrote:
> Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50318
>
> Signed-off-by: Kai Wasserbäch
> ---
> Verified with a full Piglit run on my HW (see the bug).
> Thanks to Marek for pointing out b2d63860 to me!
>
> In
https://bugs.freedesktop.org/show_bug.cgi?id=50318
Kai changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
Hi I am working on chromium, and while I was compiling mesa trunk
using gcc 4.7, I had to add explicit type cast for narrower conversion
inside the initialization list.
Could you take a look at the patch and upstream it if it is ok to you? Thanks,
-Han
diff --git a/src/mesa/main/ff_fragment_shade
On May 25, 2012 7:23 AM, "Brian Paul" wrote:
>
> On 05/24/2012 05:45 PM, Eric Anholt wrote:
>>
>> It still has the flaw that changes to source lists and Makefile.ams
>> don't rebuild Makefile as often as they should. This is because our
>> top-level Makefile isn't in automake, and has just a stub
Replace draw_set_index_buffer() and draw_set_mapped_index_buffer() with
draw_set_indexes() which simply takes a pointer and an index size.
---
src/gallium/auxiliary/draw/draw_context.c | 30 ++
src/gallium/auxiliary/draw/draw_context.h |7 +
src/gallium/au
Am 25.05.2012 13:27, schrieb Christoph Bumiller:
> On 25.05.2012 13:24, Christoph Bumiller wrote:
>> v2: use a define for the maximum sample count
>> v3: also test odd sample counts (r300 supports MS3)
>
> I'd only checked r500 docs which really doesn't support 3 samples (so it
> seemed really no
On 05/23/2012 10:55 AM, Thomas Gstädtner wrote:
Without passing the -ldflags parameter before $(LDFLAGS) in some cases
flags will be passed to MKLIB which it does not understand.
This might be -m64, -m32 or similar.
Signed-off-by: Thomas Gstädtner
---
src/gallium/targets/Makefile.xorg |2 +
https://bugs.freedesktop.org/show_bug.cgi?id=50318
--- Comment #2 from Kai 2012-05-25 07:30:16 PDT
---
(In reply to comment #1)
> I don't have the open driver installed right now and I don't have much time to
> write code either. Just wanted to let you know, this could be easily fixed by
> backp
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50318
Signed-off-by: Kai Wasserbäch
---
Verified with a full Piglit run on my HW (see the bug).
Thanks to Marek for pointing out b2d63860 to me!
In case you accept this patch: please commit it for me, I don't have access.
src/gallium/driver
On 05/24/2012 05:45 PM, Eric Anholt wrote:
It still has the flaw that changes to source lists and Makefile.ams
don't rebuild Makefile as often as they should. This is because our
top-level Makefile isn't in automake, and has just a stub am-refresh
target. I have a series to convert that, but it
From: Tom Stellard
v2:
-Separate IR type and LLVM triple
-Do the OpenCL C->LLVM IR and linking steps for all PIPE_SHADER_IR
types.
v3:
- Coding style fixes
- Removed compatibility code for LLVM < 3.1
- Split build_module_llvm() into three functions:
compile(), link(), and build_
From: Tom Stellard
This structure is used to pass non-TGSI programs to the drivers. clover
will use this for LLVM bytecode programs.
---
src/gallium/include/pipe/p_state.h |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/
From: Tom Stellard
---
.../state_trackers/clover/core/compiler.hpp|3 +--
src/gallium/state_trackers/clover/core/program.cpp |2 +-
.../state_trackers/clover/tgsi/compiler.cpp|3 +--
3 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trac
A device now has two function for getting information about the IR
it needs to return.
ir_format() => returns the preferred IR
ir_target() => returns the triple for the target that is understood by
clang/llvm.
v2:
- renamed ir_target() to ir_format()
- renamed llvm_triple() t
From: Francisco Jerez
v2: Tom Stellard
- Update CAP description
---
src/gallium/docs/source/screen.rst |5 +
src/gallium/include/pipe/p_defines.h |1 +
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/gallium/docs/source/screen.rst
b/src/gallium/docs/source/scr
On 05/25/2012 05:24 AM, Christoph Bumiller wrote:
v2: use a define for the maximum sample count
v3: also test odd sample counts (r300 supports MS3)
While multisample renderbuffers are supported by mesa, MS visuals
are not, so we need a way to tell dri/st not to advertise them even
if the gallium
Mesa 8.1-devel implementation error: meta program compile failed:
��
Signed-off-by: Oliver McFadden
---
src/mesa/drivers/common/meta.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 95336fc..316d7b
This bug was discovered while running the Khronos conformance test suite
on i965 classic driver and receiving "implementation error: meta program
compile failed."
This bug was recently introduced by the i965 clear patch set and would
not be visible while using the GL API (only ES2 API) and only on
On Fri, May 25, 2012 at 01:26:01PM +0400, Vadim Girlin wrote:
> Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50315
>
> Tested-by: Kai Wasserbäch
> Signed-off-by: Vadim Girlin
For the series:
Reviewed-by: Tom Stellard
> ---
> src/gallium/drivers/r600/r600_asm.c |2 ++
> 1 file chan
-Original Message-
From: mesa-dev-requ...@lists.freedesktop.org
Sent: 25 May 2012 09:32:33 GMT
To: mesa-dev@lists.freedesktop.org
Subject: mesa-dev Digest, Vol 26, Issue 165
Send mesa-dev mailing list submissions to
mesa-dev@lists.freedesktop.org
To subscribe or unsubscribe via
On 25.05.2012 13:24, Christoph Bumiller wrote:
> v2: use a define for the maximum sample count
> v3: also test odd sample counts (r300 supports MS3)
I'd only checked r500 docs which really doesn't support 3 samples (so it
seemed really no hw supported an odd number), but r300 seems to after
all, s
v2: use a define for the maximum sample count
v3: also test odd sample counts (r300 supports MS3)
While multisample renderbuffers are supported by mesa, MS visuals
are not, so we need a way to tell dri/st not to advertise them even
if the gallium driver does support multisampled surfaces.
Otherwi
v2: use a define for the maximum sample count
While multisample renderbuffers are supported by mesa, MS visuals
are not, so we need a way to tell dri/st not to advertise them even
if the gallium driver does support multisampled surfaces.
Otherwise applications selecting these non-functional visua
This reverts commit 60bf0f05b472e66bf1175fcec7a274dab6f7e2a3.
It seems round_mode behaves differently in some cases depending on the
instruction/slot. Reverting it for now.
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50232
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/evergr
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/radeon/R600Instructions.td | 20
1 file changed, 20 insertions(+)
diff --git a/src/gallium/drivers/radeon/R600Instructions.td
b/src/gallium/drivers/radeon/R600Instructions.td
index 04f49cd..b6b9fe0 100644
--- a/src/galli
Use TRUNC before FLT_TO_INT on evergreen/cayman.
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/radeon/R600Instructions.td | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/R600Instructions.td
b/src/gallium/drivers/radeon/R600Instr
Sampler index isn't a second source operand for some tgsi texture
instructions.
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50230
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/r600_llvm.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/gallium/
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50312
Tested-by: Kai Wasserbäch
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/radeon/R600Instructions.td |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/R600Instructions.td
b/src/gallium/
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/radeon/loader.cpp |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/loader.cpp
b/src/gallium/drivers/radeon/loader.cpp
index 79348d5..3ea8cd8 100644
--- a/src/gallium/drivers/radeon/loader.cp
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=50315
Tested-by: Kai Wasserbäch
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/r600_asm.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/r600/r600_asm.c
b/src/gallium/drivers/r600/r600_asm.c
index b73bbb
On Fri, May 25, 2012 at 11:18:07AM +0300, Oliver McFadden wrote:
> Discovered by Eric Anholt's recent changes to the i965 driver GL clear
> logic while executing some of the Khronos conformance tests.
Oops. I didn't fix this completely; please ignore while I roll a new
patch.
>
> Signed-off-by:
Discovered by Eric Anholt's recent changes to the i965 driver GL clear
logic while executing some of the Khronos conformance tests.
Signed-off-by: Oliver McFadden
---
src/mesa/drivers/common/meta.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/common
42 matches
Mail list logo
