On 14 August 2013 18:55, Kenneth Graunke wrote:
> Currently, i965 uploads a single SAMPLER_STATE table shared across all
> shader stages (VS, FS). This series splits it out, uploading a unique
> table for each stage.
>
> I think this may actually fix some bugs with vertex texturing:
> Piglit's f
On 14 August 2013 18:55, Kenneth Graunke wrote:
> Now that we have the number of samplers available, we don't need to
> iterate over all 16. This should be particularly helpful for vertex
> shaders.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
> src/m
Am 15.08.2013 14:54, schrieb Emil Velikov:
On 15/08/13 13:41, Rico Schüller wrote:
On 15.08.2013 02:10, Emil Velikov wrote:
Fix a typo introduced with commit d1ba1055d9 -
vl: Add support for max level query v2
Cc: Rico Schüller
Cc: Christian König
Bugzilla: https://bugs.freedesktop.org/show_
Hello,
one day I bought Mass Effect 3, but currently I'm not really able to play
it on my Linux laptop featuring
3.0 Mesa 9.1.4.
Gallium 0.4 on AMD CEDAR
I tried the fglrx driver, but it gives more problems than it solves (yet
the fps is there).
With the radeon driver I manage to get to an ALMOS
Hello,
one day I bought Mass Effect 3, but currently I'm not really able to play
it on my Linux laptop featuring
3.0 Mesa 9.1.4.
Gallium 0.4 on AMD CEDAR
I tried the fglrx driver, but it gives more problems than it solves (yet
the fps is there).
With the radeon driver I manage to get to an ALMOS
On 14 August 2013 17:09, Kenneth Graunke wrote:
> Previously, INTEL_DEBUG=bat would dump messages like:
>
> intel_mipmap_tree.c:1643: Batchbuffer flush with 456b used
>
> This only reported the space used for command packets, and didn't
> report any information on the space used for indirect stat
On 14 August 2013 12:07, Kenneth Graunke wrote:
> Consistently using a "The ___ driver hook." line at the the top of each
> function's comment block makes it easy to see at a glance what function
> is being implemented.
>
> Signed-off-by: Kenneth Graunke
>
Series is:
Reviewed-by: Paul Berry
_
On Thursday, August 15, 2013 11:27:14 AM Matt Turner wrote:
> This same message is printed in the validate_matrix_layout_for_type
> function.
> ---
> src/glsl/ast_to_hir.cpp | 13 +
> 1 file changed, 1 insertion(+), 12 deletions(-)
>
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/a
https://bugs.freedesktop.org/show_bug.cgi?id=67925
Kenneth Graunke changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|mesa-dev@l
On 08/12/2013 01:18 PM, Matt Turner wrote:
Necessary to avoid combining a bitcast and a modifier into a single
operation. Otherwise if safe, the MOV should be removed by
copy-propagation or register coalescing.
Has that been verified with shaderdb?
---
src/mesa/drivers/dri/i965/brw_fs_visit
v2: use resource slots 16..31 for FMASK textures
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 121 -
1 file changed, 116 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
inde
v2: bind FMASK textures to shader resource slots 16..31
---
This depends on the patch:
"radeonsi: add flexible shader descriptor management and use it for sampler
views"
src/gallium/drivers/radeonsi/r600_resource.h | 1 +
src/gallium/drivers/radeonsi/r600_texture.c | 1 +
src/gallium/driver
From: Roland Scheidegger
There's a new debug value used to disable per-quad lod optimizations
in fragment shader (ignored for vs/gs as the results are just too wrong
typically). Also trying to detect if a supplied lod value is really a
scalar (if it's coming from immediate or constant file) in wh
Previously, copy propagation would cause bitcast_f2u(abs(float)) to
be performed in a single step, but the application of source modifiers
(abs, neg) happens after type conversion, leading to incorrect results.
That is, for bitcast_f2u(abs(float)) we would in fact generate code to
do abs(bitcast_f
On 12 August 2013 13:18, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp| 13 +
> src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 3 +++
> src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 10 ++
> 3 files changed, 22 ins
On 12 August 2013 13:18, Matt Turner wrote:
> Necessary to avoid combining a bitcast and a modifier into a single
> operation. Otherwise if safe, the MOV should be removed by
> copy-propagation or register coalescing.
> ---
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 4 ++--
> src/mesa/dr
On 08/13/2013 05:57 PM, Kenneth Graunke wrote:
On 08/13/2013 05:49 PM, Ian Romanick wrote:
On 08/13/2013 04:47 PM, Kenneth Graunke wrote:
Sandybridge is the only platform that supports an IF instruction
with an embedded comparison. In this case, we need to emit a CMP
to go along with the SEL.
On 16.8.2013 0:20, Ian Romanick wrote:
On 08/15/2013 06:55 AM, Armin K. wrote:
Can we have Bison 3 fixes in 9.2 branch so they make it into 9.2 release?
eb7c8c7fb6e49a04f3fe84a6d438160dc4a14ac0
f043381334a0760ec118d07b6fb7785b5692572a
de917b4c4c4dfc949d5f8e3d9eb2dd48b63a3de5
6d2a9220b832d9a0c0c
On Tue, Aug 13, 2013 at 5:57 PM, Kenneth Graunke wrote:
> On 08/13/2013 05:49 PM, Ian Romanick wrote:
>>
>> On 08/13/2013 04:47 PM, Kenneth Graunke wrote:
>>>
>>> Sandybridge is the only platform that supports an IF instruction
>>> with an embedded comparison. In this case, we need to emit a CMP
On 12 August 2013 13:11, Kenneth Graunke wrote:
> Hello,
>
> This surprisingly large series fixes bugs in the data flow algorithm
> for copy propagation. As far as I can tell, there were a myriad of
> issues. The updated algorithm seems to follow the textbook and
> online materials much more cl
On 12 August 2013 13:11, Kenneth Graunke wrote:
> Since we start with an overestimation of livein (0x), successive
> steps may should actually take away values. This means we can't simply
>
Is "may should" a Southernism?
In any case,
Reviewed-by: Paul Berry
> OR in new liveout valu
On Tue, Aug 13, 2013 at 4:47 PM, Kenneth Graunke wrote:
> Sandybridge is the only platform that supports an IF instruction
> with an embedded comparison. In this case, we need to emit a CMP
> to go along with the SEL.
>
> Fixes regressions in Piglit's glsl-fs-atan-3, fs-unpackHalf2x16,
> fs-facef
On 08/15/2013 06:55 AM, Armin K. wrote:
Can we have Bison 3 fixes in 9.2 branch so they make it into 9.2 release?
eb7c8c7fb6e49a04f3fe84a6d438160dc4a14ac0
f043381334a0760ec118d07b6fb7785b5692572a
de917b4c4c4dfc949d5f8e3d9eb2dd48b63a3de5
6d2a9220b832d9a0c0cf35fcc5b9de1542af267d
5ffa28df4e4cc22481
On 12 August 2013 13:11, Kenneth Graunke wrote:
> This is the "COPY" set from Muchnick's textbook, which is necessary
> to do the dataflow algorithm correctly.
>
I believe this can be done more easily. The only ACP entries that are in
the hastable on exit from opt_copy_propagate_local() are the
On Thu, Aug 15, 2013 at 07:50:10PM +0200, Michel Dänzer wrote:
> On Don, 2013-08-15 at 09:16 -0700, Tom Stellard wrote:
> > On Thu, Aug 15, 2013 at 08:22:39AM -0700, Tom Stellard wrote:
> > > On Thu, Aug 15, 2013 at 11:55:36AM +0200, Michel Dänzer wrote:
> > > > On Fre, 2013-08-02 at 17:58 +0200, M
> We do have the set_caching ioctl. It's enough to flip the PTEs to UC and
> let MOCS manage things. I actually did a few experiments on my IVB. I
> made all Mesa's buffers UC via PTEs by patching libdrm to change the
> cache mode of each bo after allocation. Then I fiddled with the MOCS
> LLC bits
Am 15.08.2013 20:27, schrieb Zack Rusin:
>> I realize this function isn't used but it looks unnecessarily
>> complicated - two constants one AND plus one comparison when you
>> could simply do a single comparison (compare x with x with
>> unordered not equal). This is actually doubly bad with AVX
On Thu, Aug 15, 2013 at 11:27 AM, Matt Turner wrote:
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59648
> ---
> src/glsl/ast_to_hir.cpp | 9 +
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
> index c2fdbd5..
> I realize this function isn't used but it looks unnecessarily
> complicated - two constants one AND plus one comparison when you could
> simply do a single comparison (compare x with x with unordered not
> equal). This is actually doubly bad with AVX because the int comparison
> is going to use 4
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59648
---
src/glsl/ast_to_hir.cpp | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index c2fdbd5..f35c11f 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_
No longer used.
---
src/glsl/ast.h | 6 --
src/glsl/ast_to_hir.cpp | 5 ++---
src/glsl/glsl_parser.yy | 1 -
src/glsl/glsl_parser_extras.cpp | 1 -
4 files changed, 2 insertions(+), 11 deletions(-)
diff --git a/src/glsl/ast.h b/src/glsl/ast.h
index b0446b5..33
This same message is printed in the validate_matrix_layout_for_type
function.
---
src/glsl/ast_to_hir.cpp | 13 +
1 file changed, 1 insertion(+), 12 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 9bc713d..c9ffa30 100644
--- a/src/glsl/ast_to_hir.cpp
The variable means that UBO qualifiers are allowed in a particular
context (e.g., not allowed in a struct field declaration), rather than a
particular set of UBO qualifiers are valid.
---
src/glsl/ast.h | 2 +-
src/glsl/ast_to_hir.cpp | 6 +++---
src/glsl/glsl_parser.yy
Am 15.08.2013 19:12, schrieb Zack Rusin:
> If clipdistance for one of the vertices is nan (or inf) then the
> entire primitive should be discarded.
>
> Signed-off-by: Zack Rusin
> ---
> src/gallium/auxiliary/draw/draw_cliptest_tmp.h |2 +-
> src/gallium/auxiliary/draw/draw_llvm.c |
On Don, 2013-08-15 at 09:16 -0700, Tom Stellard wrote:
> On Thu, Aug 15, 2013 at 08:22:39AM -0700, Tom Stellard wrote:
> > On Thu, Aug 15, 2013 at 11:55:36AM +0200, Michel Dänzer wrote:
> > > On Fre, 2013-08-02 at 17:58 +0200, Michel Dänzer wrote:
> > > > On Mit, 2013-07-31 at 08:42 -0700, Tom Stel
Am 15.08.2013 19:01, schrieb Marek Olšák:
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It supports partial resource updates and it can also unbind resources
(required for FMASK texturing).
The buffer contains all sampler
If clipdistance for one of the vertices is nan (or inf) then the
entire primitive should be discarded.
Signed-off-by: Zack Rusin
---
src/gallium/auxiliary/draw/draw_cliptest_tmp.h |2 +-
src/gallium/auxiliary/draw/draw_llvm.c |3 ++
src/gallium/auxiliary/draw/draw_pipe_clip.c
On 12 August 2013 13:11, Kenneth Graunke wrote:
> fs_copy_prop_dataflow::setup_kills() walks through each basic block's
> instructions, looking for instructions which overwrite registers used
> in ACP entries.
>
> This would be fine, except that it didn't exclude the instructions
> which generate
Hello list
Feeling inspired by the automake work done in mesa, I felt like
finishing a few things that did not receive too much attention
* use Makefile.sources wherever possible
* cleanup the duplicated C{,PP,XX}FLAGS and factor out the the common
ones into Automake.inc
If anyone is interested
On 08/15/2013 09:11 AM, Ville Syrjälä wrote:
On Thu, Aug 15, 2013 at 08:08:12AM -0700, Chad Versace wrote:
I would like such a cache-control ioctl, as long the ioctl can also
be used to change the object's cacheing policy in addition to
setting it at object creation. This would be
needed when
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It supports partial resource updates and it can also unbind resources
(required for FMASK texturing).
The buffer contains all sampler view descriptors for one shader stage,
repr
On Thu, Aug 15, 2013 at 08:22:39AM -0700, Tom Stellard wrote:
> On Thu, Aug 15, 2013 at 11:55:36AM +0200, Michel Dänzer wrote:
> > On Fre, 2013-08-02 at 17:58 +0200, Michel Dänzer wrote:
> > > On Mit, 2013-07-31 at 08:42 -0700, Tom Stellard wrote:
> > > > On Wed, Jul 31, 2013 at 01:04:01PM +0200, M
On Thu, Aug 15, 2013 at 08:08:12AM -0700, Chad Versace wrote:
> On 08/14/2013 12:50 AM, Ville Syrjälä wrote:
> > On Wed, Aug 14, 2013 at 10:45:23AM +0300, Ville Syrjälä wrote:
> >> On Tue, Aug 13, 2013 at 05:46:55PM -0700, Chad Versace wrote:
> >>> On 08/13/2013 03:31 PM, Vedran Rodic wrote:
>
On Thu, Aug 15, 2013 at 7:19 AM, Armin K wrote:
> ---
> src/mesa/drivers/osmesa/Makefile.am | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/mesa/drivers/osmesa/Makefile.am
> b/src/mesa/drivers/osmesa/Makefile.am
> index a1c0bb5..d625faa 100644
> --- a/src/mesa/drivers/osmesa/Makefil
On Wed, Aug 14, 2013 at 09:46:44 -0700, Tom Stellard wrote:
> On Wed, Aug 14, 2013 at 09:08:55AM +0200, Maarten Lankhorst wrote:
> > Op 14-08-13 04:37, Tom Stellard schreef:
> > > On Tue, Aug 13, 2013 at 05:53:52PM +0200, Maarten Lankhorst wrote:
> > >> Fixes a build failure of 9.2 on ubuntu, beca
Chad Versace writes:
> On 08/08/2013 01:43 PM, Eric Anholt wrote:
>> 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 ma
- Original Message -
> From: Roland Scheidegger
>
> This is a very well hidden bug found by accident (only the fixed glean
> tstencil2 test so far seems to hit it).
> We must use new mask with combined s_pass values and orig_mask values
> for zpass/zfail stencil ops, otherwise both the sf
On Thu, Aug 15, 2013 at 11:55:36AM +0200, Michel Dänzer wrote:
> On Fre, 2013-08-02 at 17:58 +0200, Michel Dänzer wrote:
> > On Mit, 2013-07-31 at 08:42 -0700, Tom Stellard wrote:
> > > On Wed, Jul 31, 2013 at 01:04:01PM +0200, Michel Dänzer wrote:
> > > >
> > > > LLVM revision 187139 ('Allocate l
https://bugs.freedesktop.org/show_bug.cgi?id=67962
U. Artie Eoff changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=67962
U. Artie Eoff changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 08/14/2013 12:50 AM, Ville Syrjälä wrote:
On Wed, Aug 14, 2013 at 10:45:23AM +0300, Ville Syrjälä wrote:
On Tue, Aug 13, 2013 at 05:46:55PM -0700, Chad Versace wrote:
On 08/13/2013 03:31 PM, Vedran Rodic wrote:
On Mon, Aug 12, 2013 at 3:07 PM, wrote:
From: Ville Syrjälä
For L3 cache
On 08/14/2013 10:04 PM, Chia-I Wu wrote:
On Thu, Aug 15, 2013 at 10:03 AM, Kenneth Graunke wrote:
On 08/08/2013 03:13 PM, Chad Versace wrote:
[snip]
By the way, I talked to krh today, and he suggested that we delete egl_glx
rather than allow it to bitrot.
I'm in favor, but I don't know who
On Don, 2013-08-15 at 12:36 +0200, Marek Olšák wrote:
> On Thu, Aug 15, 2013 at 9:33 AM, Michel Dänzer wrote:
> > On Don, 2013-08-15 at 05:25 +0200, Marek Olšák wrote:
> >> (This should be applied before MSAA, which will need to be rebased.)
> >>
> >> It moves all sampler view descriptors to a buf
From: Roland Scheidegger
This is a very well hidden bug found by accident (only the fixed glean
tstencil2 test so far seems to hit it).
We must use new mask with combined s_pass values and orig_mask values
for zpass/zfail stencil ops, otherwise both the sfail op and one of
zpass/zfail op are appl
---
src/mesa/drivers/osmesa/Makefile.am | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/osmesa/Makefile.am
b/src/mesa/drivers/osmesa/Makefile.am
index a1c0bb5..d625faa 100644
--- a/src/mesa/drivers/osmesa/Makefile.am
+++ b/src/mesa/drivers/osmesa/Makefile.am
@@ -56,6 +56,7 @@
On 08/15/2013 03:55 PM, Armin K. wrote:
> Can we have Bison 3 fixes in 9.2 branch so they make it into 9.2 release?
>
> eb7c8c7fb6e49a04f3fe84a6d438160dc4a14ac0
> f043381334a0760ec118d07b6fb7785b5692572a
> de917b4c4c4dfc949d5f8e3d9eb2dd48b63a3de5
> 6d2a9220b832d9a0c0cf35fcc5b9de1542af267d
> 5ffa28
Can we have Bison 3 fixes in 9.2 branch so they make it into 9.2 release?
eb7c8c7fb6e49a04f3fe84a6d438160dc4a14ac0
f043381334a0760ec118d07b6fb7785b5692572a
de917b4c4c4dfc949d5f8e3d9eb2dd48b63a3de5
6d2a9220b832d9a0c0cf35fcc5b9de1542af267d
5ffa28df4e4cc22481b4ed41c78632f35765f41d
___
On Wed, Aug 14, 2013 at 11:25 PM, Marek Olšák wrote:
> (This should be applied before MSAA, which will need to be rebased.)
>
> It moves all sampler view descriptors to a buffer.
> It supports partial resource updates and it can also unbind resources
> (required for FMASK texturing).
>
> The buffe
On 15/08/13 13:41, Rico Schüller wrote:
> On 15.08.2013 02:10, Emil Velikov wrote:
>> Fix a typo introduced with commit d1ba1055d9 -
>> vl: Add support for max level query v2
>>
>> Cc: Rico Schüller
>> Cc: Christian König
>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68126
>> Signed-o
Am 15.08.2013 10:16, schrieb Michel Dänzer:
> On Mit, 2013-08-14 at 18:35 +0200, srol...@vmware.com wrote:
>> From: Roland Scheidegger
>>
>> This makes things a bit nicer, and more importantly it fixes an issue
>> where a "downgraded" array texture (due to view reduced to 1 layer and
>> addressed
On 15.08.2013 02:10, Emil Velikov wrote:
Fix a typo introduced with commit d1ba1055d9 -
vl: Add support for max level query v2
Cc: Rico Schüller
Cc: Christian König
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=68126
Signed-off-by: Emil Velikov
---
FWIW the original patch could have
Am 15.08.2013 12:54, schrieb Marek Olšák:
On Thu, Aug 15, 2013 at 10:27 AM, Christian König
wrote:
Am 15.08.2013 05:25, schrieb Marek Olšák:
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It supports partial resource up
Signed-off-by: Axel Davy
---
include/GL/internal/dri_interface.h | 1 +
src/gallium/drivers/i915/i915_resource.c| 8 ++--
src/gallium/drivers/ilo/ilo_resource.c | 2 +-
src/gallium/drivers/nv50/nv50_miptree.c | 3 +++
src/gallium/drivers/nvc0/nvc0_miptree.c | 3 +++
s
On Thu, Aug 15, 2013 at 10:27 AM, Christian König
wrote:
> Am 15.08.2013 05:25, schrieb Marek Olšák:
>
>> (This should be applied before MSAA, which will need to be rebased.)
>>
>> It moves all sampler view descriptors to a buffer.
>> It supports partial resource updates and it can also unbind res
Signed-off-by: Axel Davy
---
include/GL/internal/dri_interface.h | 1 +
src/gallium/drivers/i915/i915_resource.c| 8 ++--
src/gallium/drivers/ilo/ilo_resource.c | 2 +-
src/gallium/drivers/nv50/nv50_miptree.c | 3 +++
src/gallium/drivers/nvc0/nvc0_miptree.c | 3 +++
s
On Thu, Aug 15, 2013 at 9:33 AM, Michel Dänzer wrote:
> On Don, 2013-08-15 at 05:25 +0200, Marek Olšák wrote:
>> (This should be applied before MSAA, which will need to be rebased.)
>>
>> It moves all sampler view descriptors to a buffer.
>> It supports partial resource updates and it can also unb
On Fre, 2013-08-02 at 17:58 +0200, Michel Dänzer wrote:
> On Mit, 2013-07-31 at 08:42 -0700, Tom Stellard wrote:
> > On Wed, Jul 31, 2013 at 01:04:01PM +0200, Michel Dänzer wrote:
> > >
> > > LLVM revision 187139 ('Allocate local registers in order for optimal
> > > coloring.') broke some derivati
Am 15.08.2013 05:25, schrieb Marek Olšák:
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It supports partial resource updates and it can also unbind resources
(required for FMASK texturing).
The buffer contains all sampler
On Mit, 2013-08-14 at 18:35 +0200, srol...@vmware.com wrote:
> From: Roland Scheidegger
>
> This makes things a bit nicer, and more importantly it fixes an issue
> where a "downgraded" array texture (due to view reduced to 1 layer and
> addressed with (non-array) samplec instruction) would use th
Am 15.08.2013 09:33, schrieb Michel Dänzer:
On Don, 2013-08-15 at 05:25 +0200, Marek Olšák wrote:
(This should be applied before MSAA, which will need to be rebased.)
It moves all sampler view descriptors to a buffer.
It supports partial resource updates and it can also unbind resources
(requir
On Don, 2013-08-15 at 05:25 +0200, Marek Olšák wrote:
> (This should be applied before MSAA, which will need to be rebased.)
>
> It moves all sampler view descriptors to a buffer.
> It supports partial resource updates and it can also unbind resources
> (required for FMASK texturing).
>
> The buf
71 matches
Mail list logo
