https://bugs.freedesktop.org/show_bug.cgi?id=89018
--- Comment #12 from Tapani Pälli ---
The 'terrain section not rendered' does not feel like regression, I can
reproduce them with trace (comment #6) with Mesa 10.3.7 on Haswell.
--
You are receiving this mail because:
You are the QA Contact for
https://bugs.freedesktop.org/show_bug.cgi?id=89018
--- Comment #11 from Ilia Mirkin ---
(In reply to Tapani Pälli from comment #10)
> (In reply to Sami Liedes from comment #9)
> > Ok, interesting results. I managed to get an apitrace from an fglrx-enabled
> > machine. From it I suspect that what'
https://bugs.freedesktop.org/show_bug.cgi?id=89018
--- Comment #10 from Tapani Pälli ---
(In reply to Sami Liedes from comment #9)
> Ok, interesting results. I managed to get an apitrace from an fglrx-enabled
> machine. From it I suspect that what's missing is at least something that
> CivBE uses
On Sun, Apr 12, 2015 at 6:38 PM, Jason Ekstrand wrote:
>
> On Apr 12, 2015 3:24 PM, "Thomas Helland" wrote:
>>
>> Hi,
>>
>> This looks correct as far as I can tell.
>> I have some comments inline, but I don't feel strongly about
>> either of them, so do as you please.
>>
>> Also, maybe this is a
On Apr 12, 2015 3:24 PM, "Thomas Helland" wrote:
>
> Hi,
>
> This looks correct as far as I can tell.
> I have some comments inline, but I don't feel strongly about
> either of them, so do as you please.
>
> Also, maybe this is a candidate for /src/util ?
Maybe. If so I'm OK leaving it here until
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 63 -
1 file changed, 54 insertions(+), 9 deletions(-)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index 610e5d9..bdbbe16 1
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index c9f9e03..a2f15ed 100644
--- a/src/gallium/auxiliary/nir/
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index a2f15ed..610e5d9 100644
--- a/src/gal
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 30 +-
1 file changed, 29 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index 9d988b06..c9f9e03 1006
Mesa 10.5.3 is now available. This release addresses issues in the OpenCL
state-tracker (clover) and nearly every dri driver. Unlike mesa 10.5.2, the
build process should finally be python/mako free :-)
Dave Airlie (1):
st_glsl_to_tgsi: only do mov copy propagation on temps (v2)
Emil Velik
Hi,
This looks correct as far as I can tell.
I have some comments inline, but I don't feel strongly about
either of them, so do as you please.
Also, maybe this is a candidate for /src/util ?
2015-04-11 2:48 GMT+02:00 Jason Ekstrand :
> ---
> src/glsl/nir/nir_array.h | 96
>
https://bugs.freedesktop.org/show_bug.cgi?id=89018
--- Comment #9 from Sami Liedes ---
Ok, interesting results. I managed to get an apitrace from an fglrx-enabled
machine. From it I suspect that what's missing is at least something that CivBE
uses to color the terrain. Perhaps that being missing
Add a test to exercise a worst case collision scenario
that may cause us to not be able to find an empty
slot in the table even though it is not full.
This hits the bug in my last revision of the series
converting the hash table to quadratic probing.
V2: Feedback from Emil Velikov
-Don't inclu
https://bugs.freedesktop.org/show_bug.cgi?id=89905
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=89905
--- Comment #6 from Emil Velikov ---
Alexander
I've just pushed an alternative patch which adds the missing files to the
tarball and confirmed that the Linux Scons build works like a charm. Mesa
10.5.3 (coming out in a second) will feature the f
12. apr. 2015 18.33 skrev "Emil Velikov" :
>
> Hi Thomas,
>
> On 11/04/15 23:25, Thomas Helland wrote:
> > Add a test to exercise a worst case collision scenario
> > that may cause us to not be able to find an empty
> > slot in the table even though it is not full.
> > This hits the bug in my last
On Sunday, April 12, 2015 07:08:58 PM mathias.froehl...@gmx.net wrote:
> From: Mathias Froehlich
>
> Hi all,
>
> the attached patch fixes timer queries as noticed with osgviewer.
> Please review!
>
> Greetings and thanks
>
> Mathias
>
>
>
> This change fixes a regression with timer querie
On Sun, Apr 12, 2015 at 10:08 AM, wrote:
> From: Mathias Froehlich
>
> Hi all,
>
> the attached patch fixes timer queries as noticed with osgviewer.
> Please review!
>
> Greetings and thanks
>
> Mathias
>
In the future, instead of putting this in the commit message itself,
use git send-email
From: Mathias Froehlich
Hi all,
the attached patch fixes timer queries as noticed with osgviewer.
Please review!
Greetings and thanks
Mathias
This change fixes a regression with timer queries introduced with
commit 3eb6258. There the pending batchbuffer is flushed
only if glEndQuery is ex
Hi Thomas,
On 11/04/15 23:25, Thomas Helland wrote:
> Add a test to exercise a worst case collision scenario
> that may cause us to not be able to find an empty
> slot in the table even though it is not full.
> This hits the bug in my last revision of the series
> converting the hash table to quad
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mapi/glapi/gen/ARB_direct_state_access.xml | 15
> src/mesa/main/blit.c | 50
> ++
> src/mesa/main/blit.h | 6
> src/mesa/main/tests/dispatch_sa
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/drivers/dri/i915/i830_vtbl.c | 2 +-
> src/mesa/drivers/dri/i915/i915_vtbl.c | 2 +-
> src/mesa/drivers/dri/r200/r200_state.c | 2 +-
> src/mesa/drivers/dri/radeon/radeon_common.c | 2
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> This wasn't neccessary for ARB_direct_state_access, but felt like a good idea
> for the sake of completeness.
> ---
> src/mesa/main/blend.c | 5 +++--
> src/mesa/main/blend.h | 3 ++-
> src/mesa/main/readpix.c |
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/main/blend.c | 19 ++-
> src/mesa/main/blend.h | 6 --
> src/mesa/main/framebuffer.c | 2 +-
> src/mesa/main/get.c | 8
> sr
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/main/blend.c | 13 -
> src/mesa/main/blend.h | 7 +--
> src/mesa/main/framebuffer.c | 2 +-
> 3 files changed, 14 insertions(+), 8 deletions(-)
>
> diff --git a/src/mesa
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> _mesa_update_framebuffer now operates on arbitrary read and draw framebuffers.
> This allows BlitNamedFramebuffer to update the state of its arbitrary read and
> draw framebuffers.
> ---
> src/mesa/drivers/dri/i915/
This patch is:
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/main/blit.c | 36 ++--
> 1 file changed, 18 insertions(+), 18 deletions(-)
>
> diff --git a/src/mesa/main/blit.c b/src/mesa/main/blit.c
> index 2898723
There should be two blank lines between the function defintions,
but with that nitpick fixed:
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/main/blit.c | 119
> ---
> src/mesa/main/blit.h | 7 ++
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mapi/glapi/gen/ARB_direct_state_access.xml | 7 ++
> src/mesa/main/fbobject.c | 131
> -
> src/mesa/main/fbobject.h | 10 ++
> src/mesa/main/tests/dispatch_sanity.c
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mapi/glapi/gen/ARB_direct_state_access.xml | 6 +++
> src/mesa/main/fbobject.c | 73
> --
> src/mesa/main/fbobject.h | 7 +++
> src/mesa/main/tests/dispatch_sanity.c
This patch is:
Reviewed-by: Fredrik Höglund
On Wednesday 04 March 2015, Laura Ekstrand wrote:
> ---
> src/mesa/main/fbobject.c | 82
>
> 1 file changed, 41 insertions(+), 41 deletions(-)
>
> diff --git a/src/mesa/main/fbobject.c b/src/mesa/main
This looks like a very nice cleanup indeed!
One thing I noticed though is that fbobject.c seems to use two blank
lines between function definitions, although it's not consistent about it.
The functions introduced in this patch only have one blank line between
them.
Some more comments (mostly nitp
https://bugs.freedesktop.org/show_bug.cgi?id=89978
--- Comment #3 from Emil Velikov ---
(In reply to David Heidelberg (okias) from comment #2)
> Created attachment 115012 [details] [review]
> 0001-st-nine-require-gcc-4.6.patch
>
> This is clearly GCC bug[1], should be fixed since GCC 4.6 which e
On Sun, Apr 12, 2015 at 10:02:03AM +0300, Pohjolainen, Topi wrote:
> On Fri, Apr 10, 2015 at 12:52:04PM -0700, Ben Widawsky wrote:
> > Certain platforms support the ability to sample from a texture, and write
> > it out
> > to the file RT - thus saving a costly send instructions (note that this is
Sure. I have just realized the patch doesn't work, because autoconf
replaces [[ ]] with [ ].
Marek
On Sun, Apr 12, 2015 at 2:48 PM, Emil Velikov wrote:
> Hi Marek,
>
> On 11 April 2015 at 20:11, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> ---
>> configure.ac | 4 +++-
>> 1 file changed, 3 i
Hi Marek,
On 11 April 2015 at 20:11, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> configure.ac | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/configure.ac b/configure.ac
> index 4ed4b74..113fb49 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -421,7 +421,9
https://bugs.freedesktop.org/show_bug.cgi?id=89018
--- Comment #8 from Sami Liedes ---
(In reply to Jason Ekstrand from comment #7)
> Running it on proprietary drivers on Linux would probably be useful if you
> have easy access to it.
Access, yes, but easy, not sure. I'll try to find a way, perh
In commit 4ebeb71573ad44f7657810dc5dd2c9030e3e63db, I deleted the
emit_shader_time_end() call in emit_urb_writes(). But I failed to add
it to run_vs(), as I intended. So no data was recorded at all.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 3 +++
1 file changed
On Fri, Apr 10, 2015 at 12:52:04PM -0700, Ben Widawsky wrote:
> Certain platforms support the ability to sample from a texture, and write it
> out
> to the file RT - thus saving a costly send instructions (note that this is a
> potnential win if one wanted to backport to a tag that didn't have the
40 matches
Mail list logo
