On 13/2/19 8:28 am, Dieter Nützel wrote:
Hello Marek, Timo, Nicolai,
Timo SOLVED this long-standing NIR corruption on Polaris with his 'nir:
rewrite varying component packing' commit.
It was triggered with
commit 86b52d42368ac496fe24bc6674e754c323381635
Author: Marek Olšák
Date: Fri Jul
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 24
1 file changed, 24 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index bad1c2a990e..f6ad1aa7e77 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/c
v2: remove 16-bit additions and rebase
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index f6ad1aa7e77..defbfdf4297 100644
--- a/src/amd/common/ac_nir_to_llvm.
Signed-off-by: Rhys Perry
---
src/compiler/nir/nir_opcodes.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/compiler/nir/nir_opcodes.py b/src/compiler/nir/nir_opcodes.py
index dc4cd9ac63d..0f40bd6c548 100644
--- a/src/compiler/nir/nir_opcodes.py
+++ b/src/compiler/ni
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 68 +++
src/amd/common/ac_shader_abi.h| 1 +
src/amd/vulkan/radv_nir_to_llvm.c | 3 ++
3 files changed, 72 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llv
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/pan_context.c | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/panfrost/pan_context.c
b/src/gallium/drivers/panfrost/pan_context.c
index 97df92258da..a4d0719fdc5 100644
--- a
v2: rebase
Signed-off-by: Rhys Perry
---
src/amd/vulkan/radv_nir_to_llvm.c | 14 --
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 2002a744545..01b8b097ea1 100644
--- a/src/amd/vulkan/radv_ni
This patch can be ignored. I forgot to delete it and it ended up getting sent.
"[PATCH v2 37/41] WIP: radv, ac: implement 16-bit interpolation" is
the correct one.
On Sat, 16 Feb 2019 at 00:23, Rhys Perry wrote:
>
> v2: add to patch series
>
> Signed-off-by: Rhys Perry
> ---
> src/amd/common/ac
v2: rebase
v2: fix 64-bit visit_load_var()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 14 ++
src/amd/vulkan/radv_nir_to_llvm.c | 22 +-
2 files changed, 19 insertions(+), 17 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/am
v2: add to patch series
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 33 +---
src/amd/common/ac_llvm_build.h | 3 ++-
src/amd/common/ac_nir_to_llvm.c | 14 +++---
src/amd/vulkan/radv_nir_to_llvm.c| 27 ++-
v2: add to patch series
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 33 +---
src/amd/common/ac_llvm_build.h | 3 ++-
src/amd/common/ac_nir_to_llvm.c | 14 +++---
src/amd/vulkan/radv_nir_to_llvm.c| 27 ++-
v2: rebase
v2: move LLVMAddSLPVectorizePass to after LLVMAddEarlyCSEMemSSAPass
v2: run unconditionally on GFX9 and later
v2: mark as WIP because it can make 32-bit code much worse
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_util.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletio
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 939b8eb13de..8bfc63958ca 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/a
v2: rebase
v2: mark VK_KHR_8bit_storage as DONE in features.txt
Signed-off-by: Rhys Perry
---
docs/features.txt | 2 +-
src/amd/vulkan/radv_device.c | 17 +
src/amd/vulkan/radv_extensions.py | 4
src/amd/vulkan/radv_shader.c | 3 +++
4 files chan
In the hope that one day LLVM will then be able to generate code with
vectorized v_cvt_pkrtz_f16_f32 instructions.
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac
v2: fix C++ style comment
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index ec87a7b9343..c986f800fa4 100644
--- a/src/amd/common/ac_llvm_build.c
+++ b/src/amd/
v2: don't use ac_get_zerof() and ac_get_onef()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index b53d9c7ff8c..667f9700764 100644
---
v2: don't use ac_get_onef()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 23e454385d7..fb871a47400 100644
--- a/src/amd/common/ac_llvm_build.c
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 75bb19031bf..bad1c2a990e 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/co
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 667f9700764..db937eb66fb 100644
--- a/src/amd/common/ac_llvm_build.c
+++ b/src/amd/
v2: don't use ac_get_onef()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index cba0cec3e8f..8b0e07d2930 100644
--- a/src/amd/common/ac_nir_to_l
v2: don't use ac_get_onef()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 741059b5f1a..cba0cec3e8f 100644
--- a/src/amd/common/ac_nir_to_l
Signed-off-by: Rhys Perry
---
src/amd/vulkan/radv_shader.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c
index 1dcb0606246..adba730ad8b 100644
--- a/src/amd/vulkan/radv_shader.c
+++ b/src/amd/vulkan/radv_shader.c
@@ -53,6 +53,7 @@
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 8b0e07d2930..0e5946dfdb3 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/a
v2: don't use ac_get_zero() and ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 33 ++---
1 file changed, 6 insertions(+), 27 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index aa92c55c822..
v2: don't use ac_get_zero(), ac_get_one() and ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 27 ---
1 file changed, 4 insertions(+), 23 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index db937
This seems to generate fine code, even though the IR is a bit ugly.
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 0e5946dfdb
v2: don't use ac_get_zero() and ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 38 +++---
1 file changed, 7 insertions(+), 31 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 61085db9320.
v2: don't use ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 71eaac4b7bd..aa92c55c822 100644
--- a/src/amd/common/
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 26 ++
1 file changed, 6 insertions(+), 20 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 46738faea9d..dff369aae7f 100644
--- a/src/amd/common/ac_llvm_build.c
+
v2: rebase
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 20
1 file changed, 16 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index fb871a47400..71eaac4b7bd 100644
--- a/src/amd/common/ac_llvm_bui
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 33 +++--
1 file changed, 7 insertions(+), 26 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index c986f800fa4..46738faea9d 100644
--- a/src/amd/common/ac_llvm_bu
Signed-off-by: Rhys Perry
---
src/amd/vulkan/radv_nir_to_llvm.c | 55 ---
1 file changed, 35 insertions(+), 20 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 01b8b097ea1..c46eabf3656 100644
--- a/src/amd/vulkan/r
The lowering needs to be disabled for sufficient precision to pass
deqp-vk's 16-bit fma test on radv.
Signed-off-by: Rhys Perry
---
src/broadcom/compiler/nir_to_vir.c| 1 +
src/compiler/nir/nir.h| 1 +
src/compiler/nir/nir_opt_algebraic.py | 4 +++-
src/gallium/drivers/radeon
v2: don't use ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index e459001c1cf..75bb19031bf 100644
--- a/src/amd/common/ac_ni
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 17d952d1ae8..89a78b43c6f 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+
v2: remove ac_get_one(), ac_get_zero(), ac_get_onef() and ac_get_zerof()
v2: remove ac_int_of_size()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 55 ++---
src/amd/common/ac_llvm_build.h | 15 +++--
src/amd/common/ac_nir_to_llvm.c | 30
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 691d444db05..741059b5f1a 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@
So that the signature is correct and consistent, the inputs to a export
intrinsic should always be 32-bit floats.
This and the previous commit fixes a large amount crashes from
dEQP-VK.spirv_assembly.instruction.graphics.16bit_storage.input_output_int_*
tests
Fixes: b722b29f10d ('radv: add suppor
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index b260142c177..f39232b91a1 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
v2: don't use ac_get_zerof() and ac_get_onef()
Signed-off-by: Rhys Perry
---
src/amd/common/ac_llvm_build.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 3b2257e8bf0..23e454385d7 100644
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index f39232b91a1..691d444db05 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/co
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 37 +++--
1 file changed, 31 insertions(+), 6 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index bed52490bad..17d952d1ae8 100644
--- a/src/amd/common/ac_nir_
Signed-off-by: Rhys Perry
---
src/amd/common/ac_nir_to_llvm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 89a78b43c6f..b260142c177 100644
--- a/src/amd/common/ac_nir_to_llvm.c
+++ b/src/amd/common/ac_nir_to_llvm.c
@
16-bit outputs are stored as 16-bit floats in the outputs array, so they
have to be bitcast.
Fixes: b722b29f10d ('radv: add support for 16bit input/output')
Signed-off-by: Rhys Perry
---
src/amd/vulkan/radv_nir_to_llvm.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src
This series add support for:
- VK_KHR_shader_float16_int8
- VK_AMD_gpu_shader_half_float
- VK_AMD_gpu_shader_int16
- VK_KHR_8bit_storage
on VI+. Half floats are disabled on LLVM 7 because of a bug causing large
memory usage and long (or unbounded) compilation times with some CTS
tests.
It is writt
On Fri, Feb 15, 2019 at 2:22 AM Iago Toral Quiroga
wrote:
> Some conversions are not directly supported in hardware and need to be
> split in two conversion instructions going through an intermediary type.
> Doing this at the NIR level simplifies a bit the complexity in the backend.
>
> v2:
> -
On Fri, Feb 15, 2019 at 03:29:41PM +0200, Eleni Maria Stea wrote:
> GPUs Gen < 8 cannot sample ETC2 formats. So far, they converted the
> compressed EAC/ETC2 images to non-compressed RGBA images. When
> GetCompressed* functions were called, the pixels were returned in this
> RGBA format and not the
Reviewed-by: Timothy Arceri
But I think you should add something like the following to the commit
message:
"This was fixed properly by commit ..."
Once you push "glsl/linker: don't fail non static used inputs without
matching outputs"
On 9/2/19 4:06 am, Andres Gomez wrote:
This reverts c
Actually you probably want to add this to the commit message also since
it will also be needed if the revert patch gets picked up by stable:
Fixes: 1aa5738e666 ("glsl: relax input->output validation for SSO programs")
On 7/2/19 2:58 am, Andres Gomez wrote:
If there is no Static Use of an input
If the updated piglit tests pass on the Nvidia blob as per my reply to
those patches and this patch passes on the new and old piglit tests.
Then this patch is:
Reviewed-by: Timothy Arceri
Thanks for fixing this!
On 7/2/19 2:58 am, Andres Gomez wrote:
If there is no Static Use of an input va
https://bugs.freedesktop.org/show_bug.cgi?id=109107
Michael Eagle changed:
What|Removed |Added
See Also||https://bugs.freedesktop.or
v2: Don't check for NULL before free()
Signed-off-by: Alyssa Rosenzweig
---
src/gallium/drivers/panfrost/pan_resource.c | 19 +++
src/gallium/drivers/panfrost/pan_screen.h | 4 +++-
2 files changed, 14 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/panfrost/pa
On Fri, Feb 15, 2019 at 03:29:39PM +0200, Eleni Maria Stea wrote:
> Intel Gen7 GPUs don't support the ETC2 formats natively and in order to
> show the pixels properly we decompress them and create decompressed
> miptrees. The problem with that is that the functions that map the
> miptrees for readi
> Am I reading this correctly, that now we free a slab [for a tiled
> texture] before allocating new one? The commit message seems rather
> cryptic.
Yeah, it's a super minor fix. Rather than allocating a new slab for the
texture and then freeing the old slab, we free then allocate.
Practically, it
https://bugs.freedesktop.org/show_bug.cgi?id=109646
--- Comment #4 from bmil...@gmail.com ---
Ok, some updates:
1. The stutter/framedrop was related to the codec, not your patches. Switching
to dav1d improved it.
2. The last patch you posted fixed the black UI issues.
3. Got a comparison shot of
https://bugs.freedesktop.org/show_bug.cgi?id=109201
Alexander changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=109599
--- Comment #8 from tempel.jul...@gmail.com ---
Regarding the "hedges regression": Sorry, false alarm. It looks the same on
Windows DX11, the game developer apparently has recently downgraded the game's
visuals.
--
You are receiving this mail b
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #30 from Mark Janes ---
(In reply to Mark Janes from comment #28)
> https://android-review.googlesource.com/c/platform/external/deqp/+/901894
Mesa still asserts with this fix. I also tested Andrii's mesa patch with the
dEQP fix a
> Nit: staying consistent with "foo != NULL" vs "foo" checks helps a
> lot.
Which form is preferred?
> free(NULL); is perfectly valid.
Huh, TIL, thank you.
> The function pointer seems to be NULL. Did you forget to git add the
> file which sets it?
See my comment in the other mail about the ov
https://bugs.freedesktop.org/show_bug.cgi?id=109647
Bug ID: 109647
Summary: /usr/include/xf86drm.h:40:10: fatal error: drm.h: No
such file or directory
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS:
> - about 1/5 of the patch seems to be white space changes
...Oops. Any tips for avoiding this type of diff churn in the future? I
suppose it's not inherently harmful, but maybe it could make merging
more difficult than strictly necessary.
> - doesn't seem like BIFROST is defined anywhere
Indeed
> Feel free to reuse:
> meson: panfrost: add missing libdrm dependency
Hmm?
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> Seems like a file is missing - git add pan_screen.c perhaps? Neither
> the function pointer nor the imported/imported_size are set anywhere.
This is defined in the out-of-tree overlay with working with the vendor
kernel. When the DRM driver gains support for these features (Rob and
Tomeu are mak
Fixes: 9364d66cb7f7 (Add video compositor compute shader render)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109646
Problem 1,4: they are caused by imcomplete blend comute shader
implementation. So Reverts rgba back to frament shader.
Signed-off-by: James Zhu
---
src/gallium/auxiliary
Bugzilla bug 109646 - New video compositor compute shader render glitches mpv
https://bugs.freedesktop.org/show_bug.cgi?id=109646
Problem 1,4: they are caused by imcomplete blend comute shader
implementation. So Reverts rgba back to frament shader from
commit 9364d66cb7f7deb83876a44bb4e29e8105141c
On 2/15/19 3:42 PM, Zhu, James wrote:
> Bugzilla Bug 109646 - New video compositor compute shader render glitches mpv
> Problem 1,4: they are caused by imcomplete blend compute shader
> implementation. So Revert rgba back to frament shader.
Please refer to other commit message to put Bugzilla lin
https://bugs.freedesktop.org/show_bug.cgi?id=109646
--- Comment #3 from bmil...@gmail.com ---
(In reply to jam...@amd.com from comment #2)
> bmil...@gmail.com:
> Problem 1,4: the blend implementation seems incomplete with compute shader.
> Problem 3: could you provide the clip and the screen capt
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #31 from andrii simiklit ---
(In reply to Mark Janes from comment #30)
> (In reply to Mark Janes from comment #28)
> > https://android-review.googlesource.com/c/platform/external/deqp/+/901894
>
> Mesa still asserts with this fix.
Bugzilla Bug 109646 - New video compositor compute shader render glitches mpv
Problem 1,4: they are caused by imcomplete blend compute shader
implementation. So Revert rgba back to frament shader.
Signed-off-by: James Zhu
---
src/gallium/auxiliary/vl/vl_compositor.c | 17 +++--
1 fil
https://bugs.freedesktop.org/show_bug.cgi?id=109646
--- Comment #2 from jam...@amd.com ---
bmil...@gmail.com:
Problem 1,4: the blend implementation seems incomplete with compute shader.
Problem 3: could you provide the clip and the screen capture? I want to check
on my bench.
Problem 2: So far I
https://bugs.freedesktop.org/show_bug.cgi?id=109646
--- Comment #1 from leoxs...@gmail.com ---
Thanks for the testing. We'll have a look.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.___
m
https://bugs.freedesktop.org/show_bug.cgi?id=109646
Bug ID: 109646
Summary: New video compositor compute shader render glitches
mpv
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
On Fri, 15 Feb 2019 at 08:55, Alyssa Rosenzweig wrote:
>
> Signed-off-by: Alyssa Rosenzweig
> ---
> src/gallium/drivers/panfrost/meson.build | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/panfrost/meson.build
> b/src/gallium/drivers/panfrost/meson.build
> index 17f
On Fri, 15 Feb 2019 at 08:50, Alyssa Rosenzweig wrote:
>
> Signed-off-by: Alyssa Rosenzweig
> ---
> src/gallium/drivers/panfrost/pan_resource.c | 12 ++--
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/src/gallium/drivers/panfrost/pan_resource.c
> b/src/gallium/drive
https://bugs.freedesktop.org/show_bug.cgi?id=109645
Bug ID: 109645
Summary: build error on arm64: tegra_screen.c:33:
/usr/include/xf86drm.h:41:10: fatal error: drm.h: No
such file or directory
Product: Mesa
Versio
https://bugs.freedesktop.org/show_bug.cgi?id=109535
Mark Janes changed:
What|Removed |Added
Depends on||109594
Referenced Bugs:
https://bugs.fre
On Fri, 15 Feb 2019 at 08:50, Alyssa Rosenzweig wrote:
>
> Signed-off-by: Alyssa Rosenzweig
> ---
> src/gallium/drivers/panfrost/pan_resource.c | 4
> src/gallium/drivers/panfrost/pan_resource.h | 6 ++
> src/gallium/drivers/panfrost/pan_screen.h | 2 ++
> 3 files changed, 12 insertio
Hi Alyssa,
On Fri, 15 Feb 2019 at 08:50, Alyssa Rosenzweig wrote:
>
> Signed-off-by: Alyssa Rosenzweig
> ---
> src/gallium/drivers/panfrost/pan_resource.c | 22 -
> src/gallium/drivers/panfrost/pan_screen.h | 4 +++-
> 2 files changed, 16 insertions(+), 10 deletions(-)
>
Hi Alyssa,
On Thu, 14 Feb 2019 at 01:58, Alyssa Rosenzweig wrote:
>
> There are a few differenes between Mali T860 (Panfrost's primary
> reference target) and the older Midgard generations (T600/T700):
>
> - Miscellaneous different magic numbers. It's not clear what these
> numbers mean on eithe
On Wed, 13 Feb 2019 at 09:32, Luigi Santivetti
wrote:
>
> Hello Emil,
>
> thanks for your feedback, I agree, dri2_make_current() looks complex.
> I'll comment inline.
>
> Emil Velikov writes:
>
> > Hi all,
> >
> > Haven't looked it this has landed or not.
> >
> > On Tue, 5 Feb 2019 at 16:41, Eric
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #29 from Mark Janes ---
*** Bug 109260 has been marked as a duplicate of this bug. ***
--
You are receiving this mail because:
You are the assignee for the bug.___
mesa-dev mailing list
m
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #28 from Mark Janes ---
Chris Forbes at google found a patch that explains at least some of the reason
for this test regression:
https://android-review.googlesource.com/c/platform/external/deqp/+/901894
I just realized that this
On Tue, 5 Feb 2019 at 15:35, Emil Velikov wrote:
>
> From: Emil Velikov
>
> VGEM and kms_swrast were introduced to work with one another.
>
> All we do is CPU rendering to dumb buffers. There is no reason to carve
> out GPU memory, increasing the memory pressure on a device that could
> make a be
Hello list,
The candidate for the Mesa 18.3.4 is now available. Currently we have:
- 34 queued
- 1 nominated (outstanding)
- and 16 rejected patches
The current queue consists of:
A fix in the XvMC state-tracker, which was causing some video attributes to
not take affect. On the video front t
Reviewed-by: Bas Nieuwenhuizen
On Fri, Feb 15, 2019 at 3:57 PM Samuel Pitoiset
wrote:
>
> The elements added into a vector should have the same type as the
> first one, otherwise this hits an assertion in LLVM.
>
> Fixes: 4b3549c0846 ("radv: reduce the number of loaded channels for vertex
> inp
https://bugs.freedesktop.org/show_bug.cgi?id=109333
--- Comment #5 from Emil Velikov ---
Food for though:
AFAICT the concept of reproducible builds relies heavily of having consistent
environment. It even suggests to explicitly set SOURCE_DATE_EPOCH although
there could be others. Be that explic
https://bugs.freedesktop.org/show_bug.cgi?id=109064
--- Comment #1 from Alex Xu (Hello71) ---
Still broken in 19.0.0_rc4.
#0 0x7f0322468821 in get_temp_registers_required_live_ranges(void*,
exec_list*, int, register_live_range*, int, array_live_range*)
() from /usr/lib64/dri/r600_dri.so
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109597
Cc: 18.3 19.0
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 9745a1f2aa7..6b54da2e31b
https://bugs.freedesktop.org/show_bug.cgi?id=109333
--- Comment #4 from Jan Vesely ---
(In reply to Dylan Baker from comment #3)
> What exactly are you wanting? We (meson) have been trying not to use
> environment variables when possible because they are so awful to use on
> Windows. Our goal has
Reviewed-by: Connor Abbott
I agree that if we ever need to bring this back, we should just check for
both branches empty and no phis afterwards.
On Thu, Feb 14, 2019 at 2:38 AM Timothy Arceri
wrote:
> This was probably useful when it was first written, however it
> looks to be no longer necess
https://bugs.freedesktop.org/show_bug.cgi?id=109641
Bug ID: 109641
Summary: GLX swrast driver leaks shared memory segments
Product: Mesa
Version: 18.0
Hardware: All
OS: Linux (All)
Status: NEW
Severity: no
GPUs Gen < 8 cannot sample ETC2 formats. So far, they converted the
compressed EAC/ETC2 images to non-compressed RGBA images. When
GetCompressed* functions were called, the pixels were returned in this
RGBA format and not the compressed format that was expected.
Trying to fix this problem, we use
Signed-off-by: Samuel Iglesias Gonsálvez
---
src/intel/vulkan/anv_image.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c
index 3999c7399d0..f4a65044a3b 100644
--- a/src/intel/vulkan/anv_image.c
+++ b/src/intel/vulkan
From: Nanley Chery
Use more generic field names. We'll reuse these fields for a workaround
with ASTC miptrees.
Reviewed-by: Eleni Maria Stea
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 8
src/mesa/drivers/dri/i965/intel_mipmap_tree.c| 16
src/mesa/driv
There are formats which bpp are not aligned to a power-of-two and
that can cause problems in the checks we do.
The cacheline size was a requirement for using the BLT engine, which
we don't use anymore except for a few things on old HW, so we drop it.
Fixes CTS's CL#3500 test:
dEQP-VK.api.image_c
https://bugs.freedesktop.org/show_bug.cgi?id=109532
--- Comment #27 from asimiklit ---
(In reply to Ilia Mirkin from comment #24)
> """
> But according to next comment it should be 0 because of the BlockB[0] was
> optimized
> and there is the BlockB[1] only:
>
>/* The ARB_shading_language_42
Quoting Chris Wilson (2019-02-14 12:05:00)
> If we hang the GPU and end up banning our context, we will no longer be
> able to submit and abort with an error (exit(1) no less). As we submit
> minimal incremental batches that rely on the logical context state of
> previous batches, we can not rely o
On Thu, Feb 14, 2019 at 05:47:06PM -0500, Rob Clark wrote:
> On Thu, Feb 14, 2019 at 4:00 AM Daniel Vetter wrote:
> >
> > Clear rules avoid arguing.
> >
> > I think it'd be good to have an equally solid list on the kms side.
> > But kms is much more meant to be a standard, and the list of userspac
After the previous changes to emulate the ETC/EAC formats using the
secondary shadow miptree, the etc_format field of the intel_mipmap_tree
struct became redundant and the remaining check that used it has been
replaced. (Nanley Chery)
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 2 +-
s
1 - 100 of 138 matches
Mail list logo