On Thu, Aug 8, 2013 at 2:19 PM, Eric Anholt wrote:
> Mark Mueller writes:
> > Signed-off-by: Mark Mueller
> > ---
> > src/mesa/drivers/dri/i965/brw_draw.c | 16 ++--
> > 1 file changed, 6 insertions(+), 10 deletions(-)
> >
> > diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
> b/s
Cayman and trinity systems still seem to suffer from
stability problems with GPUVM. This also fixes compute
on these asics. It can still be enabled for testing
by setting env var RADEON_VA=true.
Fixes:
https://bugs.freedesktop.org/show_bug.cgi?id=65958
Signed-off-by: Alex Deucher
CC: "9.2"
CC
On Thu, Aug 8, 2013 at 4:54 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Fixes piglit tests const-inout-parameter.frag and
> const-out-parameter.frag.
>
> Signed-off-by: Ian Romanick
> Cc: "9.2" mesa-sta...@lists.freedesktop.org
> Cc: Matt Turner
> ---
> This currently causes a regression i
https://bugs.freedesktop.org/show_bug.cgi?id=67928
Priority: medium
Bug ID: 67928
Keywords: regression
CC: za...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit tesselation quads wireframe
> Am 09.08.2013 00:40, schrieb Zack Rusin:
> > > Series looks good though I'm unsure why the pipeline stage doesn't work.
> >> Where does that decomposition happen? Is that something like GS
> >> outputting multiple prims in the same topology which all need the same id?
> >
> > No, it's because t
https://bugs.freedesktop.org/show_bug.cgi?id=67925
--- Comment #1 from Fredrik Höglund ---
I have yet a third version in my arb-vertex-attrib-binding branch:
http://cgit.freedesktop.org/~fredrik/mesa/commit/?h=arb-vertex-attrib-binding&id=df98f9f8
--
You are receiving this mail because:
You ar
From: Ian Romanick
Fixes piglit tests const-inout-parameter.frag and
const-out-parameter.frag.
Signed-off-by: Ian Romanick
Cc: "9.2" mesa-sta...@lists.freedesktop.org
Cc: Matt Turner
---
This currently causes a regression in
relaxed-parameter-qualifier-ordering.vert. I believe that test is
in
https://bugs.freedesktop.org/show_bug.cgi?id=67926
Priority: medium
Bug ID: 67926
Keywords: have-backtrace, regression
CC: za...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit tesselation polygon w
https://bugs.freedesktop.org/show_bug.cgi?id=67921
Roland Scheidegger changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=67925
Priority: medium
Bug ID: 67925
CC: co...@octayn.net, fred...@kde.org,
i...@freedesktop.org, kenn...@whitecape.org
Assignee: mesa-dev@lists.freedesktop.org
Blocks: 6722
Am 09.08.2013 01:05, schrieb Dieter Nützel:
> After git pull 8. Aug 19:18 compilation fails in
> src/gallium/auxiliary/util/u_cpu_detect.c
>
> gmake[3]: Entering directory `/opt/mesa/src/gallium/auxiliary'
> CC util/u_cpu_detect.lo
> util/u_cpu_detect.c: In function 'sse2_has_daz':
> util/
From: Ian Romanick
The functional change is that now invalidate_framebuffer is called if
the texture is actually detached from one of the currently bound FBOs.
Previously this was only done for renderbuffers.
The remaining changes make the texture delete path look more similar to
the renderbuffe
From: Ian Romanick
Also add a return value indicating whether any work was done.
This will be used by the next patch.
Signed-off-by: Ian Romanick
Cc: "9.2" mesa-sta...@lists.freedesktop.org
---
src/mesa/main/fbobject.c | 42 +-
src/mesa/main/fbobject.h
From: Ian Romanick
Fixes failures in oglconform fbo mipmap.manual.color,
mipmap.manual.colorAndDepth, mipmap.automatic, and
mipmap.manualIterateTexTargets subtests.
Signed-off-by: Ian Romanick
Cc: "9.2" mesa-sta...@lists.freedesktop.org
---
src/mesa/drivers/common/meta.c | 11 ---
1 fi
https://bugs.freedesktop.org/show_bug.cgi?id=67924
Priority: medium
Bug ID: 67924
Keywords: regression
CC: za...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit tesselation quad_strip wireframe
On 08/06/2013 02:13 PM, Siarhei Siamashka wrote:
Hello,
Some months ago, the commit "configure.ac: Allow OpenGL ES1 and ES2 only
with enabled OpenGL" dropped support for the OpenGL-free configuration.
http://lists.freedesktop.org/archives/mesa-dev/2013-February/033909.html
http://list
On 07/30/2013 09:57 AM, Ian Romanick wrote:
On 07/23/2013 03:50 PM, Matt Turner wrote:
On Mon, Jul 15, 2013 at 6:37 AM, Fredrik Höglund wrote:
On Monday 15 July 2013, Kenneth Graunke wrote:
Fixes Piglit's ARB_vertex_attrib_bgra/api-errors test.
Signed-off-by: Kenneth Graunke
---
src/mesa/
Am 09.08.2013 00:40, schrieb Zack Rusin:
> > Series looks good though I'm unsure why the pipeline stage doesn't work.
>> Where does that decomposition happen? Is that something like GS
>> outputting multiple prims in the same topology which all need the same id?
>
> No, it's because the pipeline
> Series looks good though I'm unsure why the pipeline stage doesn't work.
> Where does that decomposition happen? Is that something like GS
> outputting multiple prims in the same topology which all need the same id?
No, it's because the pipeline stage is ran on the decomposed primitives. The
i
Am 08.08.2013 21:46, schrieb Zack Rusin:
> We can't be injecting the primitive id's in the pipeline because
> by that time the primitives have already been decomposed. To
> properly number the primitives we need to handle the adjacency
> primitives by hand. This patch moves the prim id injection in
https://bugs.freedesktop.org/show_bug.cgi?id=67921
Alexandre Demers changed:
What|Removed |Added
CC||srol...@vmware.com
--
You are receiv
https://bugs.freedesktop.org/show_bug.cgi?id=67921
Priority: medium
Bug ID: 67921
Assignee: mesa-dev@lists.freedesktop.org
Summary: [bisected commit 883987] crosscompiling fails with
util/u_cpu_detect.c:247:4: error: 'asm' undeclared
On 08/06/2013 05:30 PM, Jakob Bornecrantz wrote:
On Wed, Aug 7, 2013 at 2:00 AM, Chad Versace
wrote:
On 08/06/2013 03:02 PM, Jakob Bornecrantz wrote:
On Wed, Aug 7, 2013 at 12:00 AM, Jakob Bornecrantz
wrote:
On Tue, Aug 6, 2013 at 9:21 PM, Chad Versace <
chad.vers...@linux.intel.com
w
Mark Mueller writes:
> Signed-off-by: Mark Mueller
> ---
> src/mesa/drivers/dri/i965/brw_draw.c | 16 ++--
> 1 file changed, 6 insertions(+), 10 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
> b/src/mesa/drivers/dri/i965/brw_draw.c
> index 6170d07..1b5ed55 10064
On 08/08/2013 03:59 AM, Eric Anholt wrote:
Previously, we were asserting that each driver specified an NConfigOptions
exactly equal to the number of options they supplied, leading to frequent
bugs when people would forget to adjust the value when adjusting driver
options. Instead, just overalloc
We can't just smash it on globally due to (probably resolvable) issues
with the asm in glapi. And we don't want to penalize developers with
longer build times for their normal debug environment.
Due to libdricore making almost all of our symbols public, the effect is
very small -- cairo-gl with I
Signed-off-by: Mark Mueller
---
src/mesa/drivers/dri/i965/brw_draw.c | 16 ++--
1 file changed, 6 insertions(+), 10 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
b/src/mesa/drivers/dri/i965/brw_draw.c
index 6170d07..1b5ed55 100644
--- a/src/mesa/drivers/dri/i965/brw
Looks good.
Reviewed-by: Zack Rusin
- Original Message -
> From: Roland Scheidegger
>
> My previous attempt at doing so double-failed miserably (minification of
> zero still gives one, and even if it would not the value was never written
> anyway).
> While here also rename the confusin
Series looks good to me.
Reviewed-by: Zack Rusin
- Original Message -
> From: Roland Scheidegger
>
> d3d10 has no notion of distinct array resources neither at the resource nor
> sampler view level. However, shader dcl of resources certainly has, and
> d3d10 expects resinfo to return t
Don't worry about this one too much. The next patch removes draw_pipe_ia.c
anyway...
- Original Message -
> Without reseting the vertex id, with primitives where the same
> vertex is used with different primitives (e.g. tri/lines strips)
> our vbuf module won't re-emit those vertices with
We can't be injecting the primitive id's in the pipeline because
by that time the primitives have already been decomposed. To
properly number the primitives we need to handle the adjacency
primitives by hand. This patch moves the prim id injection into
the original primitive assembler and completel
Without reseting the vertex id, with primitives where the same
vertex is used with different primitives (e.g. tri/lines strips)
our vbuf module won't re-emit those vertices with the changed
primitive id. So lets reset the vertex id whenever injecting
new primitive id to make sure that the vertex da
Before inserting new front face and prim id outputs cleanup
the old extra outputs, otherwise our cache will use previous
output slots which will break as soon as outputs of the current
shader don't match the last.
Signed-off-by: Zack Rusin
---
src/gallium/auxiliary/draw/draw_context.c |1 +
On Thu, Aug 8, 2013 at 1:34 PM, Marek Olšák wrote:
> On Thu, Aug 8, 2013 at 6:57 PM, Christian König
> wrote:
>> Am 08.08.2013 16:33, schrieb Marek Olšák:
>>>
>>> On Thu, Aug 8, 2013 at 3:09 PM, Christian König
>>> wrote:
Am 08.08.2013 14:38, schrieb Marek Olšák:
> .On Thu, A
Ian Romanick writes:
> On 08/08/2013 03:59 AM, Eric Anholt wrote:
>> Previously, we were asserting that each driver specified an NConfigOptions
>> exactly equal to the number of options they supplied, leading to frequent
>> bugs when people would forget to adjust the value when adjusting driver
>
On Tue, Aug 6, 2013 at 12:31 PM, Ian Romanick wrote:
> On 08/06/2013 12:10 PM, mmueller wrote:Signed-off-by: mmueller <
> markkmuel...@gmail.com>
>
>> ---
>> src/mesa/drivers/dri/i965/brw_**draw.c | 18 +-
>> 1 file changed, 9 insertions(+), 9 deletions(-)
>>
>> diff --git a/sr
On 08/08/2013 03:59 AM, Eric Anholt wrote:
Previously, we were asserting that each driver specified an NConfigOptions
exactly equal to the number of options they supplied, leading to frequent
bugs when people would forget to adjust the value when adjusting driver
options. Instead, just overalloc
https://bugs.freedesktop.org/show_bug.cgi?id=67740
Ian Romanick changed:
What|Removed |Added
Status|NEW |NEEDINFO
CC|
On Thu, Aug 08, 2013 at 05:36:09PM +0200, Michel Dänzer wrote:
> On Don, 2013-08-08 at 08:00 -0700, Tom Stellard wrote:
> > On Thu, Aug 08, 2013 at 02:20:54AM +0200, Marek Olšák wrote:
> > >
> > > diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
> > > b/src/gallium/drivers/radeonsi/si_st
On 08/06/2013 02:36 PM, Kenneth Graunke wrote:
Otherwise, blits to the window system buffer may cause crashes,
since dst_irb->mt may be NULL.
This code is lifted straight out of brw_blorp_framebuffer()'s
try_blorp_blit() helper.
Fixes crashes in Piglit's fbo-sys-blit on systems without BLORP.
On 08/06/2013 02:36 PM, Kenneth Graunke wrote:
Otherwise, blits to the window system buffer may cause crashes,
since dst_irb->mt may be NULL.
This code is lifted straight out of brw_blorp_framebuffer()'s
try_blorp_blit() helper.
Fixes crashes in Piglit's fbo-sys-blit on systems without BLORP.
https://bugs.freedesktop.org/show_bug.cgi?id=67672
Roland Scheidegger changed:
What|Removed |Added
CC||lordhea...@gmail.com
--- Comment #1
https://bugs.freedesktop.org/show_bug.cgi?id=67910
Roland Scheidegger changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Thu, Aug 8, 2013 at 6:57 PM, Christian König wrote:
> Am 08.08.2013 16:33, schrieb Marek Olšák:
>>
>> On Thu, Aug 8, 2013 at 3:09 PM, Christian König
>> wrote:
>>>
>>> Am 08.08.2013 14:38, schrieb Marek Olšák:
>>>
.On Thu, Aug 8, 2013 at 9:47 AM, Christian König
wrote:
>
> A
https://bugs.freedesktop.org/show_bug.cgi?id=67910
Priority: medium
Bug ID: 67910
Assignee: mesa-dev@lists.freedesktop.org
Summary: make check fail on lp_test_arit test
Severity: normal
Classification: Unclassified
OS: Linux
Am 08.08.2013 16:33, schrieb Marek Olšák:
On Thu, Aug 8, 2013 at 3:09 PM, Christian König wrote:
Am 08.08.2013 14:38, schrieb Marek Olšák:
.On Thu, Aug 8, 2013 at 9:47 AM, Christian König
wrote:
Am 08.08.2013 02:20, schrieb Marek Olšák:
FMASK is bound as a separate texture. For every text
On Don, 2013-08-08 at 08:00 -0700, Tom Stellard wrote:
> On Thu, Aug 08, 2013 at 02:20:54AM +0200, Marek Olšák wrote:
> >
> > diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
> > b/src/gallium/drivers/radeonsi/si_state_draw.c
> > index 746ace6..4208fa7 100644
> > --- a/src/gallium/driver
From: Roland Scheidegger
My previous attempt at doing so double-failed miserably (minification of
zero still gives one, and even if it would not the value was never written
anyway).
While here also rename the confusingly named int_vec bld as we have int vecs
of different sizes, and rename need_nr
On Don, 2013-08-08 at 08:02 -0700, Tom Stellard wrote:
> On Thu, Aug 08, 2013 at 01:51:39PM +0200, Marek Olšák wrote:
> > Interleaving might not be a good idea, but they could be in the same
> > array, like this:
> >
> > 0..15: textures
> > 16..31: FMASK textures
> >
> > I'll test LLVM master, bu
On Thu, Aug 08, 2013 at 02:20:54AM +0200, Marek Olšák wrote:
> ---
> src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 ++--
> src/gallium/drivers/radeonsi/radeonsi_shader.h | 58
> ++
> src/gallium/drivers/radeonsi/si_state_draw.c | 1 +
> 3 files changed, 36 inserti
On Thu, Aug 08, 2013 at 02:20:54AM +0200, Marek Olšák wrote:
> ---
> src/gallium/drivers/radeonsi/radeonsi_shader.c | 7 ++--
> src/gallium/drivers/radeonsi/radeonsi_shader.h | 58
> ++
> src/gallium/drivers/radeonsi/si_state_draw.c | 1 +
> 3 files changed, 36 inserti
On Thu, Aug 08, 2013 at 01:51:39PM +0200, Marek Olšák wrote:
> Interleaving might not be a good idea, but they could be in the same
> array, like this:
>
> 0..15: textures
> 16..31: FMASK textures
>
> I'll test LLVM master, but we should probably bump the LLVM version
> requirement in configure.a
On Thu, Aug 8, 2013 at 3:09 PM, Christian König wrote:
> Am 08.08.2013 14:38, schrieb Marek Olšák:
>
>> .On Thu, Aug 8, 2013 at 9:47 AM, Christian König
>> wrote:
>>>
>>> Am 08.08.2013 02:20, schrieb Marek Olšák:
>>>
FMASK is bound as a separate texture. For every texture, there can be
On Don, 2013-08-08 at 13:51 +0200, Marek Olšák wrote:
> I'll test LLVM master, but we should probably bump the LLVM version
> requirement in configure.ac to prevent users from using LLVM 3.3.
That might be a good idea for the Mesa release after 9.2 anyway, but
another possibility might be to only
On Don, 2013-08-08 at 13:44 +0200, Marek Olšák wrote:
> Do you mean the surface_sync pm4 or the one in si_state?
I mean this:
diff --git a/src/gallium/drivers/radeonsi/r600_hw_context.c
b/src/gallium/drivers/radeonsi/r600_hw_context.c
index 25c972b..d9fc151 100644
--- a/src/gallium/drivers/radeo
On Thu, Aug 8, 2013 at 2:55 PM, Christian König wrote:
> Am 08.08.2013 14:02, schrieb Marek Olšák:
>
>> Why is the order so important? Note that cache flushes shouldn't be
>> treated like states and neither should the draw packets and any state
>> that comes from pipe_draw_info. The only things le
Am 08.08.2013 14:38, schrieb Marek Olšák:
.On Thu, Aug 8, 2013 at 9:47 AM, Christian König
wrote:
Am 08.08.2013 02:20, schrieb Marek Olšák:
FMASK is bound as a separate texture. For every texture, there can be
an FMASK. Therefore a separate array of resource slots has to be added.
This adds
Am 08.08.2013 14:02, schrieb Marek Olšák:
Why is the order so important? Note that cache flushes shouldn't be
treated like states and neither should the draw packets and any state
that comes from pipe_draw_info. The only things left are the register
updates and resource updates where the order do
.On Thu, Aug 8, 2013 at 9:47 AM, Christian König
wrote:
> Am 08.08.2013 02:20, schrieb Marek Olšák:
>
>> FMASK is bound as a separate texture. For every texture, there can be
>> an FMASK. Therefore a separate array of resource slots has to be added.
>>
>> This adds a new mechanism for emitting res
Why is the order so important? Note that cache flushes shouldn't be
treated like states and neither should the draw packets and any state
that comes from pipe_draw_info. The only things left are the register
updates and resource updates where the order doesn't matter, does it?
Anyway, the order is
Interleaving might not be a good idea, but they could be in the same
array, like this:
0..15: textures
16..31: FMASK textures
I'll test LLVM master, but we should probably bump the LLVM version
requirement in configure.ac to prevent users from using LLVM 3.3.
Marek
On Thu, Aug 8, 2013 at 10:37
Do you mean the surface_sync pm4 or the one in si_state?
Marek
On Thu, Aug 8, 2013 at 10:31 AM, Michel Dänzer wrote:
> On Don, 2013-08-08 at 02:20 +0200, Marek Olšák wrote:
>>
>> diff --git a/src/gallium/drivers/radeonsi/r600_hw_context.c
>> b/src/gallium/drivers/radeonsi/r600_hw_context.c
>> i
Previously, we were asserting that each driver specified an NConfigOptions
exactly equal to the number of options they supplied, leading to frequent
bugs when people would forget to adjust the value when adjusting driver
options. Instead, just overallocate the table by a bit and leave sanity
check
On Don, 2013-08-08 at 02:21 +0200, Marek Olšák wrote:
>
> This is MSAA support for the radeonsi driver. It implements MSAA
> rendering, texturing, and colorbuffer compression. The only missing
> feature is the MSAA fast color clear; other than that, it operates at
> maximum performance the hardwar
Am 08.08.2013 10:30, schrieb Michel Dänzer:
On Don, 2013-08-08 at 02:20 +0200, Marek Olšák wrote:
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index 746ace6..4208fa7 100644
--- a/src/gallium/drivers/radeonsi/si_state_draw.c
+++ b/src/g
Am 08.08.2013 02:20, schrieb Marek Olšák:
It's the same as in r600g. Look how simple it is.
That concept has the problem that we don't necessary know in which order
the state is emitted.
Why not just add an "emit" callback to si_pm4_state for the short term
instead?
For the long term we s
On Don, 2013-08-08 at 02:20 +0200, Marek Olšák wrote:
>
> diff --git a/src/gallium/drivers/radeonsi/r600_hw_context.c
> b/src/gallium/drivers/radeonsi/r600_hw_context.c
> index 25c972b..382382b 100644
> --- a/src/gallium/drivers/radeonsi/r600_hw_context.c
> +++ b/src/gallium/drivers/radeonsi/r600
On Don, 2013-08-08 at 02:20 +0200, Marek Olšák wrote:
>
> diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
> b/src/gallium/drivers/radeonsi/si_state_draw.c
> index 746ace6..4208fa7 100644
> --- a/src/gallium/drivers/radeonsi/si_state_draw.c
> +++ b/src/gallium/drivers/radeonsi/si_state_d
Am 08.08.2013 02:20, schrieb Marek Olšák:
This started as an attempt to add support for MSAA texture transfers and
MSAA depth-stencil decompression for the DB->CB copy path.
It has gotten a bit out of control, but it's for the greater good.
Some changes do not make much sense, they are there jus
Am 08.08.2013 02:20, schrieb Marek Olšák:
FMASK is bound as a separate texture. For every texture, there can be
an FMASK. Therefore a separate array of resource slots has to be added.
This adds a new mechanism for emitting resource descriptors, its features are:
- resource descriptors are stored
70 matches
Mail list logo
