https://bugs.freedesktop.org/show_bug.cgi?id=107878
coolo...@gmail.com changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
From: Gert Wollny
Since blending will be disabled later for integer formats we have to
consider that in the case of a mixed set of integer/non-integer format
buffers blending must be handled on a per buffer basis.
Fixes on r600:
dEQP-GLES31.functional.draw_buffers_indexed.random.
max_req
Fixed pack_uint_Z_FLOAT32 by casting row data to float instead uint.
Remove code duplicate function pack_uint_Z_FLOAT32_X24S8.
Edited case in "_mesa_get_pack_uint_z_func".
Now it looks like "_mesa_get_pack_float_z_func".
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91433
Signed-off-by: I
https://bugs.freedesktop.org/show_bug.cgi?id=107822
--- Comment #3 from Alexander ---
Hello
Thanks for the answer. Old? but officially there are no new stable versions out
and i like to wait for it that the new versions come to the stable repo.
But nevermind in the past i tested the game with U
Hi,
On 11.09.2018 19:24, Nicholas Kazlauskas wrote:
Applications that don't present at a predictable rate (ie. not games)
shouldn't have adapative sync enabled. This list covers some of the
common desktop compositors and some web browsers.
Why videos (in video players like vlc & mpv) wouldn't
On 2018-09-13 12:01 p.m., Eero Tamminen wrote:
> On 11.09.2018 19:24, Nicholas Kazlauskas wrote:
>> Applications that don't present at a predictable rate (ie. not games)
>> shouldn't have adapative sync enabled. This list covers some of the
>> common desktop compositors and some web browsers.
>
>
Hello,
thanks for looking on that.
Will update after any answer on
https://github.com/KhronosGroup/OpenGL-API/issues/41
Hope that is correct place for such things.
On Thu, Sep 13, 2018 at 4:57 AM, Timothy Arceri
wrote:
> On 13/9/18 12:44 am, Brian Paul wrote:
>
>> On 09/12/2018 07:30 AM, Sergii
From: Tomeu Vizoso
Check if server supports version negotation by sending a PING_PROTOCOL_VERSION
message right before a dummy RESOURCE_BUSY_WAIT. If we don't get a reply
for the first, we know the server doesn't support it.
If it does support it, we can query the max protocol version supported
From: Chris Wilson
We stream from a tiled and aligned source into an unaligned user buffer,
so we need to use _mm_storeu_si128.
Fixes: d21c086d819d78fb3f6abcbb14aa492970f442aa (i965/tiled_memcpy: inline
movntdqa loads in tiled_to_linear)
Reviewed-by: Kenneth Graunke
---
src/mesa/drivers/dri/i
It shouldn't be needed to emit the initial graphics or compute
state when beginning a new command buffer. Emitting them in
the preamble should be enough and this will reduce IB sizes.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 29 --
src/amd/vulkan/
This series brings back tiled memcpy changes reverted earlier
because of build breakage. I've fixed Android build and introduced a
conditional check for sse41 in meson build.
These have been reviewed expect my android build system changes that
add libmesa_intel_tiled_memcpy + the meson changes.
/
From: Scott D Phillips
Rename the (un)map_gtt functions to (un)map_map (map by
returning a map) and add new functions (un)map_tiled_memcpy that
return a shadow buffer populated with the intel_tiled_memcpy
functions.
Tiling/detiling with the cpu will be the only way to handle Yf/Ys
tiling, when s
On 09/07/2018 03:25 PM, andrey simiklit wrote:
Hi all,
Could somebody run it on CI to confirm that this patch fixes one test
and not add regressions or maybe even take a look at this patch)
CI reports failures in following conformance test:
KHR-GL46.enhanced_layouts.varying_structure_locat
https://bugs.freedesktop.org/show_bug.cgi?id=100960
--- Comment #8 from Sergii Romantsov ---
Created attachment 141542
--> https://bugs.freedesktop.org/attachment.cgi?id=141542&action=edit
trimmed and modified trace
--
You are receiving this mail because:
You are the assignee for the bug.
You
Fixed pack_uint_Z_FLOAT32 by casting row data to float instead uint.
Remove code duplicate function pack_uint_Z_FLOAT32_X24S8.
Edited case in "_mesa_get_pack_uint_z_func".
Now it looks like "_mesa_get_pack_float_z_func".
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91433
Signed-off-by: I
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #3 from robusgrobu...@gmail.com ---
Created attachment 141546
--> https://bugs.freedesktop.org/attachment.cgi?id=141546&action=edit
stderr.log
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #4 from robusgrobu...@gmail.com ---
Created attachment 141547
--> https://bugs.freedesktop.org/attachment.cgi?id=141547&action=edit
dxvk_dxgi.log
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #5 from robusgrobu...@gmail.com ---
Created attachment 141548
--> https://bugs.freedesktop.org/attachment.cgi?id=141548&action=edit
dxvk_d3d11.log
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #7 from robusgrobu...@gmail.com ---
My hardware is:
Motherboard - MSI X470 Gaming Pro AM4
APU - AMD Ryzen 5 2400G with Radeon Vega Graphics (Raven Ridge)
RAM - 2x DDR4 Goodram Iridium 8Gb 2800Mhz (Total: 16 Gb Dual channel 2800M
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #6 from robusgrobu...@gmail.com ---
Created attachment 141549
--> https://bugs.freedesktop.org/attachment.cgi?id=141549&action=edit
stdout.log
--
You are receiving this mail because:
You are the assignee for the bug.
You are the Q
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #8 from robusgrobu...@gmail.com ---
Created attachment 141550
--> https://bugs.freedesktop.org/attachment.cgi?id=141550&action=edit
BeastBattleSimulator_gamecrash.tar.gz
Crash log, created by game.
--
You are receiving this mail
https://bugs.freedesktop.org/show_bug.cgi?id=107914
--- Comment #9 from robusgrobu...@gmail.com ---
(In reply to RobusGrobus95 from comment #7)
> $ pacman -Qi lib32-mesa-git | grep -i version
> Version : 18.3.0_devel.104757.1a263b377c-1
> Description : an open-source implementation of
That removes two special cases for clip/cull distances.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 8b881e8f06
This is just for consistency because LLVM can detect and
remove unused loads.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index af34c548c15..fd7d
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 47 ++-
1 file changed, 34 insertions(+), 13 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index d3885646182..037c5679e3b 100644
--- a/src/amd/vul
Instead of having holes. The other ring parameters like
offset and stride can be updated later.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 28
1 file changed, 12 insertions(+), 16 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.
According to RadeonSI, it's unnecessary to multiply by
the stride. That field seems to always be 64.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
i
Same code is generated because LLVM ends up by using bfe, but
that seems cleaner to me.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 21 ++---
1 file changed, 6 insertions(+), 15 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vul
Tapani Pälli writes:
> From: D Scott Phillips
>
> The reference for MOVNTDQA says:
>
> For WC memory type, the nontemporal hint may be implemented by
> loading a temporary internal buffer with the equivalent of an
> aligned cache line without filling this data to the cache.
> [..
On 2018-09-13 2:40 a.m., Marek Olšák wrote:
> From: Marek Olšák
>
> [...]
>
> static void
> -util_init_cache_number(void)
> +util_init_thread_pinning(void)
> {
> /* Get a semi-random number. */
> int64_t t = os_time_get_nano();
> L3_cache_number = (t ^ (t >> 8) ^ (t >> 16));
> +
>
Quoting D Scott Phillips (2018-09-13 08:28:29)
> Tapani Pälli writes:
>
> > From: D Scott Phillips
> >
> > The reference for MOVNTDQA says:
> >
> > For WC memory type, the nontemporal hint may be implemented by
> > loading a temporary internal buffer with the equivalent of an
> > ali
https://bugs.freedesktop.org/show_bug.cgi?id=107923
Bug ID: 107923
Summary: build_id.c:126: multiple definition of
`build_id_length'
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
Thanks Tapani!
On Wed, Sep 12, 2018 at 9:47 PM Tapani Pälli wrote:
>
> Reviewed-by: Tapani Pälli
>
> On 09/13/2018 02:14 AM, Kristian H. Kristensen wrote:
> > Signed-off-by: Kristian H. Kristensen
> > ---
> > src/egl/drivers/dri2/platform_android.c | 2 +-
> > 1 file changed, 1 insertion(+),
https://bugs.freedesktop.org/show_bug.cgi?id=106404
--- Comment #8 from Todd ---
Thanks!
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
mesa-dev mailing list
mesa-dev@lists.freedesktop.
Rather than trying to encode all of the rules in a header, lets just put
them in the build system where they belong. This fixes the build on
FreeBSD, which does have pthraed_setaffinity_np, but it's in a
pthread_np.h, not behind _GNU_SOURCE. FreeBSD also implements cpu_set
slightly differently, so
Reviewed-by: Marek Olšák
Marek
On Thu, Sep 13, 2018 at 2:41 PM, Dylan Baker wrote:
> Rather than trying to encode all of the rules in a header, lets just put
> them in the build system where they belong. This fixes the build on
> FreeBSD, which does have pthraed_setaffinity_np, but it's in a
>
On Thu, Sep 13, 2018 at 11:48 AM, Michel Dänzer wrote:
> On 2018-09-13 2:40 a.m., Marek Olšák wrote:
>> From: Marek Olšák
>>
>> [...]
>>
>> static void
>> -util_init_cache_number(void)
>> +util_init_thread_pinning(void)
>> {
>> /* Get a semi-random number. */
>> int64_t t = os_time_get_
Ping.
On 9/6/18 12:37 PM, Sagar Ghuge wrote:
> v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin)
>
> v3: Handle case where genxml is empty (Lionel Landwerlin)
>
> Signed-off-by: Sagar Ghuge
> ---
> src/intel/common/gen_decoder.c | 18 +-
> 1 file changed, 13 i
Reviewed-by: Lionel Landwerlin
On 06/09/2018 20:37, Sagar Ghuge wrote:
v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin)
v3: Handle case where genxml is empty (Lionel Landwerlin)
Signed-off-by: Sagar Ghuge
---
src/intel/common/gen_decoder.c | 18 +-
1 fil
It's reasonably well-known that you can replace an unsigned integer
division by a constant with a sequence of multiply (by constant), adds, and
shifts. This little series implements such an optimization for NIR. As
can be seen from the last patch, there really aren't that many shaders
affected by
---
src/compiler/nir/nir_constant_expressions.py | 1 +
src/compiler/nir/nir_opcodes.py | 43 ++--
2 files changed, 40 insertions(+), 4 deletions(-)
diff --git a/src/compiler/nir/nir_constant_expressions.py
b/src/compiler/nir/nir_constant_expressions.py
index 118af9
It's a reasonably well-known fact in the world of compilers that integer
divisions by constants can be replaced by a multiply, an add, and some
shifts. This commit adds such an optimization to NIR for easiest case
of udiv. Other division operations will be added in following commits.
In order to
Shader-db results on Sky Lake:
total instructions in shared programs: 15105795 -> 15111403 (0.04%)
instructions in affected programs: 72774 -> 78382 (7.71%)
helped: 0
HURT: 265
Note that hurt here actually means helped because we're getting rid of
integer quotient operations (whic
---
src/compiler/nir/nir.h | 1 +
src/compiler/nir/nir_lower_int64.c | 54 ++
2 files changed, 55 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index bf4bd916d27..c68134b2baf 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compil
This is useful for compiler optimizations that want to convert integer
division by constants into mul+add+shift. It's put in util because it
may also be useful for other things and because having it split out of
the compiler pass itself makes it much easier to test.
---
configure.ac
---
src/util/Makefile.sources | 1 +
src/util/bigmath.h| 95 +++
src/util/meson.build | 1 +
3 files changed, 97 insertions(+)
create mode 100644 src/util/bigmath.h
diff --git a/src/util/Makefile.sources b/src/util/Makefile.sources
index b562d6c
Reviewed-by: Bas Nieuwenhuizen
for the series
On Wed, Sep 12, 2018 at 3:40 PM Samuel Pitoiset
wrote:
>
> Indirect descriptors only need one entry, we don't have to
> emit a location for every descriptors.
>
> Fixes GPU hangs with new CTS:
> dEQP-VK.binding_model.descriptorset_random.*
>
> CC: 18
Fixes: da997ebec929421939553 "vulkan: Add KHR_display extension using DRM [v10]"
Cc: Keith Packard
Signed-off-by: Eric Engestrom
---
src/vulkan/wsi/wsi_common_display.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/vulkan/wsi/wsi_common_display.c
b/src/vulkan/wsi/wsi_common_display.
Signed-off-by: Denis Pauk
CC: Marek Olšák
CC: Rhys Perry
CC: Bruce Cherniak
CC: Matt Turner
---
docs/features.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/features.txt b/docs/features.txt
index 9ccf803d47..ce6595320c 100644
--- a/docs/features.txt
+++ b/docs/fe
Reuse Code from:
f69bc797e1 gallium/auxiliary: Add helper support for bptc format
compress/decompress
Signed-off-by: Denis Pauk
Reviewed-by: Bruce Cherniak
CC: Marek Olšák
CC: Tim Rowley
---
src/gallium/drivers/swr/swr_screen.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff
I also got myself some benchmark data! I wrote a python script to which
generates a shader_test file with 5000 back-to-back random integer division
operations. The compiler takes a long time to compile the shader but,
running it with shader_time, I see about a 2x improvement with my pass.
On Thu
On 09/13/2018 12:40 PM, Jason Ekstrand wrote:
> Shader-db results on Sky Lake:
>
> total instructions in shared programs: 15105795 -> 15111403 (0.04%)
> instructions in affected programs: 72774 -> 78382 (7.71%)
> helped: 0
> HURT: 265
>
> Note that hurt here actually means helped
Reuse Code from:
f69bc797e1 gallium/auxiliary: Add helper support for bptc format
compress/decompress
Signed-off-by: Denis Pauk
Reviewed-by: Bruce Cherniak
CC: Marek Olšák
CC: Tim Rowley
---
src/gallium/drivers/swr/swr_screen.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff
Hi!
Could you please review changes? And could you also merge changes if it is ok.
First patch is already reviewed by Bruce Cherniak
and does not have any changes.
Second patch contains updates for docs/features.txt for status bptc support.
[PATCH v2 1/2] gallium/swr: Enable support bptc forma
Signed-off-by: Denis Pauk
CC: Marek Olšák
CC: Rhys Perry
CC: Bruce Cherniak
CC: Matt Turner
---
docs/features.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/features.txt b/docs/features.txt
index 9ccf803d47..ce6595320c 100644
--- a/docs/features.txt
+++ b/docs/fe
Reviewed-by: Bas Nieuwenhuizen
On Thu, Sep 13, 2018 at 4:36 PM Samuel Pitoiset
wrote:
>
> Same code is generated because LLVM ends up by using bfe, but
> that seems cleaner to me.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_nir_to_llvm.c | 21 ++---
> 1 file c
On Thu, Sep 13, 2018 at 1:24 PM Samuel Pitoiset
wrote:
>
> It shouldn't be needed to emit the initial graphics or compute
> state when beginning a new command buffer. Emitting them in
> the preamble should be enough and this will reduce IB sizes.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/am
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Wed, Sep 12, 2018 at 11:19 PM Samuel Pitoiset
wrote:
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/si_cmd_buffer.c | 14 ++
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/src/amd/vulkan/si_cmd_buffer.
From: Ian Romanick
No shader-db changes on any Intel platform... which probably explains
why no bugs have been bisected to this problem since it landed in Mesa
18.1. :( The commit mentioned below is in 18.2, so 18.1 would need a
slightly different fix (due to code refactoring).
Signed-off-by: Ia
On Thu, Sep 13, 2018 at 3:06 PM Eric Engestrom
wrote:
> Fixes: da997ebec929421939553 "vulkan: Add KHR_display extension using DRM
> [v10]"
> Cc: Keith Packard
> Signed-off-by: Eric Engestrom
> ---
> src/vulkan/wsi/wsi_common_display.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/
huh, this took me way too long why this works on VI. But
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Thu, Sep 13, 2018 at 3:58 PM Samuel Pitoiset
wrote:
>
> According to RadeonSI, it's unnecessary to multiply by
> the stride. That field seems to always be 64.
>
> Signed-off-by: Samuel Pi
61 matches
Mail list logo
