Commits on Source (49)
-
The SPIR-V parser catches its failures and failed assertions, clean up and return a failure to the caller. This is a good default behavior, but when debugging sometimes is useful to hook the debugger right in the failed assertion. Reviewed-by:
Jason Ekstrand <jason.ekstrand@collabora.com> Part-of: <!19663>
-
Gert Wollny authored
Fixes: 79ca456b r600/sfn: rewrite NIR backend Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19707>
-
Gert Wollny authored
The two conversion ops are now handled in nir_lower_int64, but the fixup for the input to f2u32 has to be handled there and not in f2u64. Fixes: 29da9856 nir/lower_int64: Enable lowering of 64-bit float to 64-bit integer conversions. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19707>
-
Fixes: 1d726940 ("vulkan: Add a common CmdBegin/EndRederPass implementation") Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Part-of: <!19599>
-
This is a very scorched-earth approach which doesn't take into account whether or not there are any explicitly provided dependencies. We could take a finer-grained approach in theory but it's unlikely to matter in practice since you usually stall in Begin/EndRenderPass anyway. Fixes: 1d726940 ("vulkan: Add a common CmdBegin/EndRederPass implementation") Closes: #6203 Closes: #7650 Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Part-of: <!19599>
-
Timothy Arceri authored
shader-db results (BDW): total instructions in shared programs: 17527053 -> 17526931 (<.01%) instructions in affected programs: 5116 -> 4994 (-2.38%) helped: 25 HURT: 0 helped stats (abs) min: 2 max: 15 x̄: 4.88 x̃: 3 helped stats (rel) min: 0.25% max: 5.34% x̄: 3.39% x̃: 3.90% 95% mean confidence interval for instructions value: -6.19 -3.57 95% mean confidence interval for instructions %-change: -3.98% -2.81% Instructions are helped. total cycles in shared programs: 856680230 -> 856682009 (<.01%) cycles in affected programs: 6583780 -> 6585559 (0.03%) helped: 117 HURT: 77 helped stats (abs) min: 1 max: 854 x̄: 68.56 x̃: 16 helped stats (rel) min: <.01% max: 35.34% x̄: 2.12% x̃: 0.76% HURT stats (abs) min: 1 max: 2188 x̄: 127.27 x̃: 18 HURT stats (rel) min: 0.01% max: 22.66% x̄: 1.86% x̃: 0.67% 95% mean confidence interval for cycles value: -30.07 48.41 95% mean confidence interval for cycles %-change: -1.28% 0.19% Inconclusive result (value mean confidence interval includes 0). LOST: 3 GAINED: 1 Reviewed-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19666>
-
Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
To prepare for a future sharing in vertex shader register allocator. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
We can have up to 6 in vertex shaders. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
We will add a new one for vp in a next commit Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
It can now be used to init rc_regalloc_state for both vertex and fragment shaders. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
This function is used by rc_variable_change_dst when rewriting variables and even though we don't need to rewrite the index for RC_FILE_NONE, we still need to properly reswizzle. All other users of the function are properly checking for RC_FILE_NONE later. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
Vertex shaders can have a per-channel negate so we need to properly update the negate mask when rewriting swizzles. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
This helps in some cases where we fail to detect that the write mask is still alive after the loop and thus fail to detect some readers. Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
This is a port of the fp pair regalloc. It is however much simpler as contrary to the fp, we don't have to care about texturing, we can use any swizzle we want and we don't have to track the inputs. Using the register class machinery might actually be a slight overkill right now, however the infrastructure will hopefully come in handy if someone decides to implement the vp pair scheduling eventually. Shader-db stats: RV530: total temps in shared programs: 18594 -> 17000 (-8.57%) temps in affected programs: 5753 -> 4159 (-27.71%) helped: 665 HURT: 0 RV370: total temps in shared programs: 13555 -> 12181 (-10.14%) temps in affected programs: 5116 -> 3742 (-26.86%) helped: 633 HURT: 0 Closes: #5972 Signed-off-by:
Pavel Ondračka <pavel.ondracka@gmail.com> Reviewed-by:
Filip Gawin <filip@gawin.net> Tested-by:
Filip Gawin <filip@gawin.net> Part-of: <!19618>
-
(VS doesn't have limitations like FS) total instructions in shared programs: 27439 -> 26928 (-1.86%) instructions in affected programs: 6219 -> 5708 (-8.22%) helped: 157 HURT: 0 helped stats (abs) min: 1 max: 13 x̄: 3.25 x̃: 3 helped stats (rel) min: 1.03% max: 66.67% x̄: 17.66% x̃: 11.11% 95% mean confidence interval for instructions value: -3.58 -2.93 95% mean confidence interval for instructions %-change: -20.12% -15.20% Instructions are helped. Part-of: <!19711>
-
Samuel Pitoiset authored
VGPR allocation changed on GFX11 and this might have changed how shared VGPRs work, so it's probably more secure to lower in NIR. Cc: 22.3 mesa-stable Signed-off-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Timur Kristóf <timur.kristof@gmail.com> Part-of: <!19679>
-
We only support lazy descriptors these days, so having the infrastructure around to support automatic selection of that one mode is kinda silly. And it's not like setting an environment variable that is never read is going to cause any issues, so we don't even need this to avoid breaking existing setups. Let's just rip it out. We can reintroduce it again on the off-chance that someone has a new clever descriptor mode they want to experiment with. Reviewed-by:
Hoe Hao Cheng <haochengho12907@gmail.com> Part-of: <!19670>
-
This reverts commit cb02cf46. There are 3 reported flakes over a period of a month, and we have been unable to reproduce it even once. It clearly doesn't happen often enough to warrant disabling our vulkan CI, so let's restore it while we continue to try to reproduce the issue on our side. Signed-off-by:
Eric Engestrom <eric@igalia.com> Acked-by:
Iago Toral Quiroga <itoral@igalia.com> Part-of: <!19720>
-
Alejandro Piñeiro authored
After 'v3dv: fix debug dump on BO free' we changed the order, and this lead to the following test dEQP-VK.api.object_management.multithreaded_per_thread_resources.device_memory_small v2: Expanded comment just before the reset, explaining that we need to do the reset before we free the BO from the kernel (Iago) Raising this assertion: deqp-vk: ../src/broadcom/vulkan/v3dv_bo.c:281: v3dv_bo_alloc: Assertion `bo && bo->handle == 0' failed. Fixes: 2c445971 ('v3dv: fix debug dump on BO free') Reviewed-by:
Iago Toral Quiroga <itoral@igalia.com> Part-of: <!19693>
-
Move shader key independent lowering and some optimizations to finalize_nir. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19708>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19729>
-
Gert Wollny authored
Replace some asserts with unreachable too. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19729>
-
Gert Wollny authored
Also simplify some use of these flags. Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19729>
-
Gert Wollny authored
Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19729>
-
Gert Wollny authored
gallium-nine seems to ignore the shader options and emits the instruction despite "options::lower_flrp32 = true" Signed-off-by:
Gert Wollny <gert.wollny@collabora.com> Part-of: <!19729>
-
If the intersection shader is VK_SHADER_UNUSED_KHR (which is only allowed for a zero shader group) then no further processing of the intersection candidate occurs. Part-of: <!19705>
-
The code builds up the dynamic array of objects (spirv_objs) and collect pointers to each of them into another dynamic array (spirv_ptr_objs). If the growth of the first array cause a reallocation, it is possible that the previous pointers end up invalid. Fixes: 77e929a5 ("intel/clc: allow multiple CL files to be compiled together") Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Jordan Justen <jordan.l.justen@intel.com> Part-of: <!19730>
-
Per Ken suggestion, use ascending order for the start offset. Fixes: 6d28c6e5 ("i965: Select ranges of UBO data to be uploaded as push constants.") Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> Part-of: <!19731>
-
Signed-off-by:
Eric Engestrom <eric@igalia.com> Part-of: <!19536>
-
Not sure if it used to work, but it doesn't anymore. Signed-off-by:
Eric Engestrom <eric@igalia.com> Part-of: <!19536>
-
Signed-off-by:
Eric Engestrom <eric@igalia.com> Part-of: <!19536>
-
Signed-off-by:
Eric Engestrom <eric@igalia.com> Part-of: <!19536>
-
Signed-off-by:
Eric Engestrom <eric@igalia.com> Part-of: <mesa/mesa!19536>
-
Acked-by:
Yonggang Luo <luoyonggang@gmail.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <!19587>
-
See https://www.openwall.com/lists/musl/2015/09/11/5 ``` ../src/gallium/drivers/crocus/crocus_bufmgr.c:102: error: "PAGE_SIZE" redefined [-Werror] ``` Acked-by:
Yonggang Luo <luoyonggang@gmail.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <!19587>
-
Fixes multiple warnings as this one: ``` /usr/include/sys/poll.h:1:2: warning: #warning redirecting incorrect #include <sys/poll.h> to <poll.h> [-Wcpp] 1 | #warning redirecting incorrect #include <sys/poll.h> to <poll.h> | ^~~~~~~ ``` Acked-by:
Yonggang Luo <luoyonggang@gmail.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <!19587>
-
The helper used ralloc which is unusual for vulkan objects, did not handle allocation failures properly and was only useful for RADV. Reviewed-by:
Tatsuyuki Ishi <ishitatsuyuki@gmail.com> Part-of: <!19700>
-
This fixes leaking shader modules on failure and simplifies freeing. Reviewed-by:
Tatsuyuki Ishi <ishitatsuyuki@gmail.com> Part-of: <!19700>
-
Fixes a vtn crash with dEQP-VK.ray_tracing_pipeline.pipeline_library.configurations.singlethreaded_compilation.s0_l1 and validation enabled. Closes: #7642 Reviewed-by:
Tatsuyuki Ishi <ishitatsuyuki@gmail.com> Part-of: <!19700>
-
Reviewed-by:
Tatsuyuki Ishi <ishitatsuyuki@gmail.com> Part-of: <!19700>
-
Signed-off-by:
Guilherme Gallo <guilherme.gallo@collabora.com> Acked-by:
Karol Herbst <kherbst@redhat.com> Part-of: <!19608>
-
The usual reasons: Flake handling, familiar skips/xfails handling, faster parallelization. This also sets us up for running a subset of the CL CTS once we decide to build it in our containers. Acked-by:
Karol Herbst <kherbst@redhat.com> Part-of: <!19608>
-
av1 decoding supports NV12 and P010 output. Reviewed-by:
Leo Liu <leo.liu@amd.com> Signed-off-by:
Ruijing Dong <ruijing.dong@amd.com> Part-of: <!19660>
-
add the logic for calculating film grain related coefficients for VCN to generate film grain output. Reviewed-by:
Leo Liu <leo.liu@amd.com> Signed-off-by:
Ruijing Dong <ruijing.dong@amd.com> Part-of: <!19660>
Showing
- .gitlab-ci/container/build-deqp-runner.sh 1 addition, 1 deletion.gitlab-ci/container/build-deqp-runner.sh
- .gitlab-ci/deqp-runner.sh 1 addition, 0 deletions.gitlab-ci/deqp-runner.sh
- .gitlab-ci/image-tags.yml 4 additions, 4 deletions.gitlab-ci/image-tags.yml
- .gitlab-ci/piglit/run_cl.sh 0 additions, 75 deletions.gitlab-ci/piglit/run_cl.sh
- docs/drivers/zink.rst 0 additions, 13 deletionsdocs/drivers/zink.rst
- docs/releasing.rst 39 additions, 12 deletionsdocs/releasing.rst
- src/amd/vulkan/radv_pipeline.c 1 addition, 11 deletionssrc/amd/vulkan/radv_pipeline.c
- src/amd/vulkan/radv_pipeline_rt.c 54 additions, 47 deletionssrc/amd/vulkan/radv_pipeline_rt.c
- src/amd/vulkan/radv_private.h 3 additions, 0 deletionssrc/amd/vulkan/radv_private.h
- src/amd/vulkan/radv_shader.c 2 additions, 0 deletionssrc/amd/vulkan/radv_shader.c
- src/broadcom/ci/gitlab-ci.yml 1 addition, 1 deletionsrc/broadcom/ci/gitlab-ci.yml
- src/broadcom/vulkan/v3dv_bo.c 12 additions, 7 deletionssrc/broadcom/vulkan/v3dv_bo.c
- src/compiler/nir/nir_opt_algebraic.py 3 additions, 0 deletionssrc/compiler/nir/nir_opt_algebraic.py
- src/compiler/spirv/spirv_to_nir.c 4 additions, 0 deletionssrc/compiler/spirv/spirv_to_nir.c
- src/gallium/drivers/crocus/crocus_bufmgr.c 2 additions, 0 deletionssrc/gallium/drivers/crocus/crocus_bufmgr.c
- src/gallium/drivers/llvmpipe/ci/deqp-llvmpipe-clover.toml 4 additions, 0 deletionssrc/gallium/drivers/llvmpipe/ci/deqp-llvmpipe-clover.toml
- src/gallium/drivers/llvmpipe/ci/deqp-llvmpipe-rusticl.toml 4 additions, 0 deletionssrc/gallium/drivers/llvmpipe/ci/deqp-llvmpipe-rusticl.toml
- src/gallium/drivers/llvmpipe/ci/gitlab-ci.yml 5 additions, 7 deletionssrc/gallium/drivers/llvmpipe/ci/gitlab-ci.yml
- src/gallium/drivers/llvmpipe/ci/llvmpipe-clover-fails.txt 53 additions, 0 deletionssrc/gallium/drivers/llvmpipe/ci/llvmpipe-clover-fails.txt
- src/gallium/drivers/llvmpipe/ci/llvmpipe-clover-flakes.txt 4 additions, 0 deletionssrc/gallium/drivers/llvmpipe/ci/llvmpipe-clover-flakes.txt
.gitlab-ci/piglit/run_cl.sh
deleted
100755 → 0