xed cull/clip distances
Samuel Pitoiset (1):
Revert "radv: re-apply "Do not access set layout during
vkCmdBindDescriptorSets.""
Hi Dylan, can we add
commit 66f7289d568db8711adb885acc56622e2aff252a
Author: Samuel Pitoiset
Date: Wed Jan 19 16:15:33 2022 +0100
radv:
For RADV: https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10174
On 4/9/21 6:39 PM, Dylan Baker wrote:
Hi all,
I've been a little behind on release work recently, and I'm tryinng to
cleanup the backlog of patches against the 21.0 branch that haven't been
applied but have been
+1
On 11/20/20 10:44 AM, Daniel Schürmann wrote:
Hi,
I would like to request developer access for our new team member, Tony
Wasserka.
He has proven himself capable with a number of MRs and works actively
on the ACO backend.
His gitlab nick is neobrain.
Thanks in advance,
Daniel
On RADV, we already support fp16 with the LLVM backend (not saying it's
always optimized though) and with ACO it should be mostly working but
not yet enabled because I think we would like fast packed math support
first and I'm not sure if fp16 I/O are implemented.
There is also some missing
no/ir3/ra: fix array liveranges
util: fix u_fifo_pop()
Samuel Pitoiset (7):
radv/gfx10: fix required subgroup size with VK_EXT_subgroup_size_control
radv/gfx10: fix required ballot size with VK_EXT_subgroup_size_control
radv: fix optional pSizes parameter when bindi
On 1/28/20 8:46 PM, Dylan Baker wrote:
Quoting Dylan Baker (2020-01-22 10:27:05)
Hi list, due to some last minute changes in plan I'll be managing the 20.0
release. The release calendar has been updated, but the gitlab milestone wasn't
opened. That has been corrected, and is here
, Samuel Pitoiset wrote:
If we can wait end of january that would be highly appreciated :-)
On 1/15/20 5:13 PM, Jason Ekstrand wrote:
When were we planning to cut the 20.0 release? We just landed Vulkan
1.2 support for ANV and RADV this morning so it seems like a good
time to me. The release
If we can wait end of january that would be highly appreciated :-)
On 1/15/20 5:13 PM, Jason Ekstrand wrote:
When were we planning to cut the 20.0 release? We just landed Vulkan
1.2 support for ANV and RADV this morning so it seems like a good time
to me. The release calendar has nothing for
LGTM
It's easier to figure how old is a release with year based versioning.
On 10/10/19 10:14 PM, Marek Olšák wrote:
Hi,
I expect to make a new libdrm release soon. Any objections to changing
the versioning scheme?
Current: 2.4.n
n = starts from 0, incremented per release
New proposals:
+1
On 8/29/19 8:52 PM, Kenneth Graunke wrote:
Hi all,
As a lot of you have probably noticed, Bugzilla seems to be getting a
lot of spam these days - several of us have been disabling a bunch of
accounts per day, sweeping new reports under the rug, hiding comments,
etc. This bug spam causes
ACK.
On 8/26/19 9:05 PM, Marek Olšák wrote:
Hi,
I'd like to push the Navi14 merge request to 19.2 no later than
Tuesday August 27.
https://gitlab.freedesktop.org/mesa/mesa/merge_requests/1726
Please ack if it's OK with you,
Thanks,
Marek
___
gfx10: implement scan & reduce operations"
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 05871f5ea98..5abae00d8f6 100644
On 8/21/19 3:59 PM, Bas Nieuwenhuizen wrote:
On Wed, Aug 21, 2019 at 3:45 PM Samuel Pitoiset
wrote:
This fixes a regression introduced with scan operations
on GFX10. Note that some subgroups CTS still fail on GFX10 but
I assume it's a different issue.
This fixes dEQP-VK.subgroups.arithmetic
s"
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c | 7 +++
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/src/amd/common/ac_llvm_build.c b/src/amd/common/ac_llvm_build.c
index 05871f5ea98..d72eaa2db46 100644
--- a/src/amd/common/ac_llvm_build.c
+++ b/
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 64bd0d64401..c049a2844b8 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan/radv_pipeline.c
Only gfx9 and older use it to get InstanceID in VGPR1.
Ported from RadeonSI.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/si_cmd_buffer.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index e37b9498a71
Trivial extension that matches PAL.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 9 +
src/amd/vulkan/radv_extensions.py | 1 +
2 files changed, 10 insertions(+)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index cc45ac95c08
Scans aren't implemented on SI/CIK.
Cc: 19.2
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 3 ++-
src/amd/vulkan/radv_shader.c | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index
Shader ballot will be enabled by default for Wolfenstein
Youngblood. This follows what we did for sisched.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_debug.h | 1 +
src/amd/vulkan/radv_device.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/amd/vulkan/radv_debug.h b/src
of some minor artifacts that only
happens when ballot is disabled for some reasons.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 49518d43218..c04f6a27e82
The script doesn't handle them correctly and D16_UNORM_S8_UINT
isn't supported by the hardware, mark it as invalid.
This fixes warning when generating gfx10_format_table.h.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=111393
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/gfx10_format_table.py | 20 +---
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/src/amd/vulkan/gfx10_format_table.py
b/src/amd/vulkan/gfx10_format_table.py
index 34ad5f6cdf2..81b0bed92aa 100644
--- a/src/amd
CLEAR_STATE emits it for us.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/si_cmd_buffer.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index a5057fe25a2..68ec925f2b5 100644
--- a/src/amd/vulkan/si_cmd_buffer.c
+++ b
It's emitted by the kernel.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 9 ++---
src/amd/vulkan/si_cmd_buffer.c | 9 ++---
2 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index
10: add missing dcc_tile_swizzle tweak")
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_private.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h
index 49d3c78db98..ee0761e69fe 100644
--- a/src/amd/vulkan/
It's unnecessary to duplicate fields in another struct.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 12 -
src/amd/vulkan/radv_image.c | 44 +++-
src/amd/vulkan/radv_meta_clear.c | 12 ++---
src/amd/vulkan/radv_private.h
It's unnecessary to duplicate fields in another struct.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 4 ++--
src/amd/vulkan/radv_image.c | 38 +---
src/amd/vulkan/radv_meta_clear.c | 11 +
src/amd/vulkan/radv_private.h| 13
It's 0 for depth surfaces with TC compat HTILE enabled.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_image.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c
index f3237dd5985..221b554e73e 100644
Fixes: c90f46700dd ("radv/gfx10: mask DCC tile swizzle by alignment")
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_image.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_image.c b/src/amd/vulkan/radv_image.c
index 221b554e73e..8
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_shader.c | 30 +-
1 file changed, 17 insertions(+), 13 deletions(-)
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c
index 97fa80b348c..f0ab2d5e467 100644
--- a/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_llvm_helper.cpp | 30 +
src/amd/vulkan/radv_shader.c| 3 ++-
src/amd/vulkan/radv_shader_helper.h | 3 ++-
3 files changed, 26 insertions(+), 10 deletions(-)
diff --git a/src/amd/vulkan
It can be enabled with RADV_PERFTEST=pswave32.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_debug.h | 1 +
src/amd/vulkan/radv_device.c | 6 ++
src/amd/vulkan/radv_nir_to_llvm.c | 2 ++
src/amd/vulkan/radv_pipeline.c| 3 ++-
src/amd/vulkan/radv_private.h | 1
It can be enabled with RADV_PERFTEST=gewave32.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_debug.h | 1 +
src/amd/vulkan/radv_device.c | 5 +
src/amd/vulkan/radv_nir_to_llvm.c | 13 +++--
src/amd/vulkan/radv_pipeline.c| 10 +-
src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 14 ++
src/amd/vulkan/si_cmd_buffer.c | 9 +++--
2 files changed, 21 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index 37026246aa9
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 4d4f86a7e24..b3952846f43 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/si_cmd_buffer.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index 3d6c672dd0f..d48ed804e63 100644
--- a/src/amd/vulkan/si_cmd_buffer.c
+++ b/src/amd
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 17 -
src/amd/vulkan/si_cmd_buffer.c | 13 +
2 files changed, 13 insertions(+), 17 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 5c913f29a5a
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index e0ea47b5745..37026246aa9 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
+++ b/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 27 +++
1 file changed, 23 insertions(+), 4 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index b3952846f43..d62066cbee4 100644
--- a/src/amd/vulkan
It can be enabled with RADV_PERFTEST=cswave32.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_debug.h | 1 +
src/amd/vulkan/radv_device.c | 12 +++-
src/amd/vulkan/radv_nir_to_llvm.c | 14 +-
src/amd/vulkan/radv_pipeline.c| 3 ++-
src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 583b600dfdd..e11196bd82e 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan
On 7/29/19 2:30 PM, Bas Nieuwenhuizen wrote:
On Mon, Jul 29, 2019 at 2:20 PM Samuel Pitoiset
wrote:
On 7/29/19 2:15 PM, Bas Nieuwenhuizen wrote:
On Mon, Jul 29, 2019 at 2:11 PM Samuel Pitoiset
wrote:
The HTILE masks seem to be different and so we need to rework that
path. Just disabled
On 7/29/19 2:15 PM, Bas Nieuwenhuizen wrote:
On Mon, Jul 29, 2019 at 2:11 PM Samuel Pitoiset
wrote:
The HTILE masks seem to be different and so we need to rework that
path. Just disabled for now and implement later.
The HTILE masks are not different per amdvlk?
Can you at least rework
The HTILE masks seem to be different and so we need to rework that
path. Just disabled for now and implement later.
This fixes rendering issues with vkmark and Wreckfest at least.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_meta_clear.c | 5 +++--
1 file changed, 3 insertions(+), 2
This might happen when a pipeline doesn't define the vertex input
state, so the buffer data format is 0 (aka INVALID).
This fixes crashes when compiling some shaders on GFX10.
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_llvm_build.c | 1 +
1 file changed, 1 insertion(+)
diff --git
Natively supported on VI+.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 60 +++
src/amd/vulkan/radv_device.c | 6
src/amd/vulkan/radv_extensions.py | 1 +
3 files changed, 61 insertions(+), 6 deletions(-)
diff --git a/src/amd
Reviewed-by: Samuel Pitoiset
On 7/25/19 4:55 PM, Bas Nieuwenhuizen wrote:
Without correct size, radeonsi assumes the metadata is incorrect,
which can and will cause issues.
Since the metadata is really incorrect without the size, let us
fix that.
Fixes: e43cc3e3afc "radv/gfx9: handle
It's already disabled later in this function?
On 7/25/19 4:34 PM, Bas Nieuwenhuizen wrote:
(a) radv does not set the DCC fields required yet.
(b) radeonsi just broke their DCC metadata.
Fixes: f8b6c5a1a63 "radeonsi: rewrite si_get_opaque_metadata, also for gfx10
support"
---
On 7/25/19 3:39 PM, Bas Nieuwenhuizen wrote:
r-b
though it sounds like some of our cache flushes might be not ideal.
Yes.
On Thu, Jul 25, 2019 at 3:35 PM Samuel Pitoiset
wrote:
It's coherent and faster. GFX7-GFX9 should also support this but
for now only uses L2 for GFX10 because it's
-off-by: Samuel Pitoiset
---
src/amd/vulkan/si_cmd_buffer.c | 16
1 file changed, 16 insertions(+)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index 21a90cb2514..94f759139ee 100644
--- a/src/amd/vulkan/si_cmd_buffer.c
+++ b/src/amd/vulkan
This fixes
dEQP-VK.rasterization.primitive_size.points.point_size_*
This also fixes some black squares with the Sascha SSAO demo.
v2: - do not set for multiple channels
- call vi_alpha_is_on_msb() for pre-GFX10
- remove unused 'swap'
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan
This fixes
dEQP-VK.rasterization.primitive_size.points.point_size_*
This also fixes some black squares with the Sascha SSAO demo.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_image.c | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan
On 7/23/19 9:31 PM, Bas Nieuwenhuizen wrote:
On Tue, Jul 23, 2019 at 3:21 PM Samuel Pitoiset
wrote:
NGG GS for streamout requires a bunch of work, so enable it with
the legacy path only for now.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 28
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index 84d627340e9..c2e3f3b5fd0 100644
--- a/src/amd/vulkan/radv_cmd_buffer.c
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_shader.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c
index 5fd1022b05a..56f421026b7 100644
--- a/src/amd/vulkan/radv_shader.c
+++ b/src/amd
NGG GS for streamout requires a bunch of work, so enable it with
the legacy path only for now.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 28
1 file changed, 28 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan
When a pipeline uses transform feedback, the driver fallbacks to
the legacy path because NGG support for streamout is a non-trivial
amount of work.
AMDVLK also uses the legacy path for streamout, while RadeonSI
uses the new NGG path.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan
Required for legacy streamout.
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
index cf73cdc692b..020c6d17771 100644
--- a/src/amd/vulkan
For some reasons, InstanceID is VGPR3 although StepRate0 is set to 1.
v3: fix instanceID input VGPR for geometry
v2: fix instanceID
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 12 +---
src/amd/vulkan/radv_shader.c | 8 ++--
2 files changed, 15
For some reasons, InstanceID is VGPR3 although StepRate0 is set to 1.
v2: fix instanceID
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 12 +---
src/amd/vulkan/radv_shader.c | 2 +-
2 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/amd
On 7/23/19 1:37 PM, Bas Nieuwenhuizen wrote:
So does this work with tests that use multiple instances?
Apparently no.
If so, r-b.
On Tue, Jul 23, 2019 at 1:29 PM Samuel Pitoiset
wrote:
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 3 ++-
1 file changed, 2
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 336bae28614..9cea92e8a69 100644
--- a/src/amd/vulkan/radv_nir_to_llvm.c
+++ b
Fixes: a20a9d0c5e7 ("radv: dont store disasm string unless keep_shader_info
flag set")
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_shader.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shad
It actually works.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 992e12840f7..93b03afda22 100644
--- a/src/amd/vulkan/radv_device.c
+++ b/src
On 7/22/19 6:01 PM, Ilia Mirkin wrote:
On Mon, Jul 22, 2019 at 11:49 AM Samuel Pitoiset
wrote:
For TES as NGG.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan
For TES as NGG.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 336bae28614..6e5a283f923 100644
--- a/src/amd
Same as RadeonSI.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index a7ff0e2d139..fce60a62ee9 100644
--- a/src/amd/vulkan/radv_pipeline.c
depth_stencil_attachment and/or ds_resolve attachment can be NULL.
This fixes crashes with
dEQP-VK.renderpass.suballocation.unused_clear_attachments.*
Cc: 19.1
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_meta_clear.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Reviewed-by: Samuel Pitoiset
On 7/19/19 9:17 PM, Rhys Perry wrote:
Seems to fix some hair artifacts in Max Payne 3:
https://github.com/daniel-schuermann/mesa/issues/76
Signed-off-by: Rhys Perry
Fixes: f4e499ec791 ('radv: add initial non-conformant radv vulkan driver')
---
src/amd/common
This field doesn't exist.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 6e313aa9aa1..3c553cb93e7 100644
--- a/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 7e623414adc..6feb55e3916 100644
--- a/src/amd/vulkan/radv_nir_to_llvm.c
GFX10 isn't affected.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c
index b6ac14f63a9..84d627340e9 100644
--- a/src/amd
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 6 +++---
src/amd/vulkan/radv_device.c | 2 +-
src/amd/vulkan/radv_image.c | 10 +-
src/amd/vulkan/si_cmd_buffer.c | 12 ++--
4 files changed, 15 insertions(+), 15 deletions(-)
diff --git a/src
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_device.c | 34 +-
1 file changed, 9 insertions(+), 25 deletions(-)
diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 15bda6822e8..6e313aa9aa1 100644
--- a/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd/vulkan/radv_nir_to_llvm.c
index 6feb55e3916..19dcae3a476 100644
--- a/src/amd/vulkan
RadeonSI only uses Z32_FLOAT_CLAMP for upgraded depth textures
on GFX10 and RADV doesn't promotes Z16 or Z24.
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_nir_to_llvm.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 14 --
1 file changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 9338fcd550a..bcb7ccc803d 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b
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 b11d79f4811..a7ff0e2d139 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan
Unnecessary.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index bcb7ccc803d..b11d79f4811 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan
On 7/18/19 2:29 AM, Bas Nieuwenhuizen wrote:
On Wed, Jul 17, 2019 at 3:44 PM Samuel Pitoiset
wrote:
With NGG, empty waves may still be required to export data.
This fixes dEQP-VK.ycbcr.format.*_unorm.geometry_*.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 31
On 7/18/19 2:10 AM, Bas Nieuwenhuizen wrote:
On Thu, Jul 18, 2019 at 2:05 AM Bas Nieuwenhuizen
wrote:
On Wed, Jul 17, 2019 at 3:44 PM Samuel Pitoiset
wrote:
It's useless for NGG anyways.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 43
Reviewed-by: Samuel Pitoiset
On 7/18/19 2:51 AM, Dave Airlie wrote:
From: Dave Airlie
Enabling tracing, and then having a vmfault, can leads to a segfault
before we print out the traces, as if a meta shader is executing
and we don't have the NIR for it.
Just pass the stage and give back
Reviewed-by: Samuel Pitoiset
On 7/18/19 3:20 AM, Dave Airlie wrote:
From: Dave Airlie
IF we don't have clear state (which gfx10 doesn't currently)
we will fix to reset the scissor. AMDVLK will leave it set
to something else.
Marek also has this fix for radeonsi pending.
---
src/amd/vulkan
Reviewed-by: Samuel Pitoiset
On 7/18/19 8:14 AM, Dave Airlie wrote:
From: Dave Airlie
I can find no evidence that removing this is a good idea.
Fixes: 9b116173b6a ("radv: do not emit VGT_FLUSH on GFX10")
---
src/amd/vulkan/radv_device.c | 6 ++
1 file changed, 2 insert
With NGG, empty waves may still be required to export data.
This fixes dEQP-VK.ycbcr.format.*_unorm.geometry_*.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 31 ++-
1 file changed, 30 insertions(+), 1 deletion(-)
diff --git a/src/amd
It's useless for NGG anyways.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 43 ++
1 file changed, 33 insertions(+), 10 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index fdeb31c453e..686fd371f0f
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 de933937f03..8b6e62a75f5 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd/vulkan
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 42 --
1 file changed, 15 insertions(+), 27 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index 686fd371f0f..de933937f03 100644
--- a/src/amd/vulkan
Found by inspection.
Cc:
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index a3323ae8135..f6cb3611c9d 100644
--- a/src/amd/vulkan
On 7/17/19 10:25 AM, Juan A. Suarez Romero wrote:
On Tue, 2019-07-16 at 08:37 +0200, Samuel Pitoiset wrote:
For NGG, the driver relies on the VS outinfo struct.
This fixes
dEQP-VK.clipping.user_defined.clip_*_vert_tess_geom_*
Should this be included in 19.1 stable branch?
No, it's GFX10
Reviewed-by: Samuel Pitoiset
On 7/17/19 6:59 AM, Dave Airlie wrote:
From: Dave Airlie
this shouldn't matter, but it's good to be correct.
---
src/amd/vulkan/radv_pipeline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan
Unnecessary.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 7 ++-
src/amd/vulkan/radv_device.c | 2 ++
src/amd/vulkan/radv_image.c | 7 ---
src/amd/vulkan/radv_private.h| 1 +
4 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/src/amd
I did implement this extension a while ago but it didn't work
on pre GFX10 for some reasons. Now all CTS pass.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_extensions.py | 1 +
src/amd/vulkan/radv_nir_to_llvm.c | 1 +
src/amd/vulkan/radv_pipeline.c| 1 +
src/amd/vulkan
This is unsupported and hangs.
This fixes GPU hangs with
dEQP-VK.tessellation.geometry_interaction.limits.output_required_*.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 12
src/amd/vulkan/radv_shader.c | 2 +-
2 files changed, 13 insertions(+), 1
It should be possible to build it on-demand too but it requires
more work. On GFX10, the GS copy shader is required when tess
is enabled with extreme geometry.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 8
src/amd/vulkan/radv_pipeline.c | 21
Acked-by: Samuel Pitoiset
On 7/10/19 9:13 AM, Mauro Rossi wrote:
This patch adds gfx10_format_table.h in Makefile.sources
and the rules for Android to fix following building errors:
In file included from external/mesa/src/amd/vulkan/radv_debug.c:35:
In file included from external/mesa/src/amd
Mirror RadeonSI.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/si_cmd_buffer.c | 60 --
1 file changed, 42 insertions(+), 18 deletions(-)
diff --git a/src/amd/vulkan/si_cmd_buffer.c b/src/amd/vulkan/si_cmd_buffer.c
index a832dbd89eb..e996fa250a9 100644
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_pipeline.c | 8 +++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_pipeline.c b/src/amd/vulkan/radv_pipeline.c
index fdb0ed29ea4..31495ec078d 100644
--- a/src/amd/vulkan/radv_pipeline.c
+++ b/src/amd
For NGG, the driver relies on the VS outinfo struct.
This fixes
dEQP-VK.clipping.user_defined.clip_*_vert_tess_geom_*
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_nir_to_llvm.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/amd/vulkan/radv_nir_to_llvm.c
b/src/amd
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_shader.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/amd/vulkan/radv_shader.c b/src/amd/vulkan/radv_shader.c
index f6b0297d4a3..1e9399de193 100644
--- a/src/amd/vulkan/radv_shader.c
+++ b/src/amd/vulkan
1 - 100 of 5019 matches
Mail list logo