Vignesh Raman
fbd6da9bcd
ci: handle missing dri libraries during listing
...
Upreving mesa in drm-ci causes the below error,
ls -1 '/install/lib/dri/*_dri.so'
ls: cannot access '/install/lib/dri/*_dri.so': No such file or directory
trap_err 2
Allow the script to continue execution even if listing dri
libraries command fails.
Reviewed-by: Guilherme Gallo <guilherme.gallo@collabora.com >
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28782 >
2024-04-18 07:47:12 +00:00
Faith Ekstrand
e297fb1bd5
nil: cbindgen is required
...
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28794 >
2024-04-17 23:16:31 -05:00
Samuel Pitoiset
40f39482e1
ci: uprev CTS to vulkan-cts-1.3.8.0
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27459 >
2024-04-17 21:22:36 +00:00
Mike Blumenkrantz
974b3ab964
zink: disable buffer reordering correctly on shader image binds
...
the unordered flags must be set after the barrier to avoid the
scenario where the barrier checks buffer usage and resets the flags
cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28787 >
2024-04-17 20:59:40 +00:00
Yiwei Zhang
737bae4267
venus: fix swapchain image memory bind
...
Fixed dEQP-VK.wsi.*.swapchain.simulate_oom.image_swapchain_create_info
Fixes: 36f639375b ("venus: use STACK_ARRAY to simplify BindImageMemory2")
Signed-off-by: Yiwei Zhang <zzyiwei@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28790 >
2024-04-17 18:32:42 +00:00
Mike Blumenkrantz
dd02ea3769
egl: use os_get_option for MESA_LOADER_DRIVER_OVERRIDE
...
this is better for android
Reviewed-by: Yiwei Zhang <zzyiwei@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28570 >
2024-04-17 17:19:27 +00:00
Mike Blumenkrantz
6f13b201ad
egl/android: fix zink loading
...
should be as simple as checking whether zink is being used
cc: mesa-stable
Reviewed-by: Yiwei Zhang <zzyiwei@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28570 >
2024-04-17 17:19:27 +00:00
Mike Blumenkrantz
ff37271ea7
egl: fix defines for zink's dri3 check
...
if mesa is built without dri3 then dri3 should/can not be checked
cc: mesa-stable
Reviewed-by: Yiwei Zhang <zzyiwei@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28570 >
2024-04-17 17:19:27 +00:00
Nanley Chery
e3a5ade9ee
intel/isl: Disable miptails to align LODs for CCS WA
...
For HSD 22015614752, we enabled Tile64 to ensure image subresources are
64K aligned. However, we neglected to disable miptails so some image
miplevels actually did not achieve the desired image alignment. Do that
now.
Fixes: c6686fda28 ("intel/isl: Use Tile64 to align images for CCS WA")
Reported-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28703 >
2024-04-17 16:52:11 +00:00
Derek Foreman
e2260b8152
vulkan/wsi/wayland: Remove unused get_min_image_count_for_mode_group
...
This function has no callers.
Signed-off-by: Derek Foreman <derek.foreman@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28786 >
2024-04-17 11:04:05 -05:00
Patrick Lerda
4f5e9a21c5
panfrost: remove panfrost_create_shader_state() related dead code
...
The pointer "xfb" is allocated with a clone of "so->nir" and lost
without further processing.
The function panfrost_shader_compile() was the one processing "xfb".
The call of this function was removed with the commit 40372bd720 .
This makes "xfb" not required anymore.
For instance, this issue is triggered on a Mali-T820 with
"piglit/bin/arb_transform_feedback2-change-objects-while-paused -auto":
Indirect leak of 32776 byte(s) in 1 object(s) allocated from:
#0 0xf78f30a6 in malloc (/usr/lib/libasan.so.6+0x840a6)
#1 0xee9cd4ee in ralloc_size ../src/util/ralloc.c:118
#2 0xee9cf7ae in create_slab ../src/util/ralloc.c:801
#3 0xee9cf7ae in gc_alloc_size ../src/util/ralloc.c:840
#4 0xef74ab82 in nir_undef_instr_create ../src/compiler/nir/nir.c:888
#5 0xef76212c in clone_ssa_undef ../src/compiler/nir/nir_clone.c:328
#6 0xef76212c in clone_instr ../src/compiler/nir/nir_clone.c:438
#7 0xef7642d8 in clone_block ../src/compiler/nir/nir_clone.c:501
#8 0xef7642d8 in clone_cf_list ../src/compiler/nir/nir_clone.c:555
#9 0xef7657dc in clone_function_impl ../src/compiler/nir/nir_clone.c:632
#10 0xef766cb8 in nir_shader_clone ../src/compiler/nir/nir_clone.c:743
#11 0xf007673e in panfrost_create_shader_state ../src/gallium/drivers/panfrost/pan_shader.c:434
#12 0xeeb6766c in st_create_common_variant ../src/mesa/state_tracker/st_program.c:781
#13 0xeeb71d1c in st_get_common_variant ../src/mesa/state_tracker/st_program.c:834
#14 0xeeb72ea2 in st_precompile_shader_variant ../src/mesa/state_tracker/st_program.c:1320
#15 0xeeb72ea2 in st_finalize_program ../src/mesa/state_tracker/st_program.c:1421
#16 0xef3806ec in st_link_glsl_to_nir ../src/mesa/state_tracker/st_glsl_to_nir.cpp:748
#17 0xef3806ec in st_link_shader ../src/mesa/state_tracker/st_glsl_to_nir.cpp:984
#18 0xef2992f6 in link_program ../src/mesa/main/shaderapi.c:1336
#19 0xef2992f6 in link_program_error ../src/mesa/main/shaderapi.c:1445
Fixes: 40372bd720 ("panfrost: Implement a disk cache")
Signed-off-by: Patrick Lerda <patrick9876@free.fr >
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28743 >
2024-04-17 15:20:10 +00:00
Derek Foreman
642d32045f
vulkan/wsi/wayland: Fix use after free
...
In 7eaceb0392 I called pthread_mutex_unlock() with a member of a freed
structure.
We can unlock as soon as this element is removed from the list it was in,
so just move the unlock to before the free.
Fixes 7eaceb0392
Signed-off-by: Derek Foreman <derek.foreman@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28432 >
2024-04-17 14:43:18 +00:00
Samuel Pitoiset
74615bb704
radv: clear color attachments without exports before compaction
...
For PS epilogs, this isn't necessary because spi_shader_col_format is
already cleared. This will help for implementing color attachment
remapping because colors_written is always the original mapping.
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28781 >
2024-04-17 13:58:37 +00:00
José Roberto de Souza
68a91242dc
anv: Remove protected memory types from default_buffer_mem_types
...
Without this application could allocate protected memory buffers
and use it in non-protected queues.
Fixes: c6a91f1695 ("anv: add new heap/pool for descriptor buffers")
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Signed-off-by: José Roberto de Souza <jose.souza@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28769 >
2024-04-17 13:33:29 +00:00
Mike Blumenkrantz
fc691d9f37
lavapipe: disable stencil test if no stencil attachment
...
stencil test must not be enabled if there is no stencil attachment
fixes dEQP-VK.pipeline.*.stencil.no_stencil_att.dynamic_rendering.*
fixes #10990
cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28772 >
2024-04-17 12:50:13 +00:00
Connor Abbott
5363f7cce5
freedreno,tu: Disable UBWC for storage images on a750
...
Fixes: a80a23dc49 ("tu: Enable UBWC for storage images on a7xx")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28749 >
2024-04-17 09:55:13 +00:00
Connor Abbott
c234c502aa
freedreno: Make has_ibo_ubwc a7xx specific
...
It's not supported on a6xx, and the next commit will disable it on a750.
Fixes: a80a23dc49 ("tu: Enable UBWC for storage images on a7xx")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28749 >
2024-04-17 09:55:13 +00:00
Eric Engestrom
5c842d95aa
ci: delete mistaken duplicate llvmpipe-{fails,skips}.txt
...
These were added, likely by mistake, in 54050d8844 , and never used, but
they are causing problems because they overwrite the proper files from
src/gallium/drivers/llvmpipe/ci/, causing CI failures because the wrong
file is used.
Fixes: 54050d8844 ("svga/ci: land vmware mesa-ci lava farm")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28773 >
2024-04-17 09:06:36 +00:00
Samuel Pitoiset
26d9e9bdb5
radv/rt: add radv_rt_pipeline_compile()
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Samuel Pitoiset
07f3b9aaed
radv/rt: insert shaders to cache right after they are compiled
...
No need postpone this.
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Samuel Pitoiset
6dc5b6ee60
radv/rt: rework handle_from_stages to pass hashes directly
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Samuel Pitoiset
6979b1ea3d
radv/rt: initialize shader group capture/replay in a separate function
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Samuel Pitoiset
3738331afe
radv/rt: remove unnecessary pipeline parameter to radv_generate_rt_shaders_key()
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Samuel Pitoiset
29338d85a0
radv/rt: remove unnecessary pipeline parameter to radv_rt_fill_group_info()
...
Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28760 >
2024-04-17 06:54:36 +00:00
Joshua Ashton
763b4e992c
radv: Enable KHR_video_queue if encode is enabled
...
KHR_video_encode_queue is dependent on KHR_video_queue.
Not exposing this makes encode-only usecases not functional,
eg. SteamVR Link.
Signed-off-by: Joshua Ashton <joshua@froggi.es >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28779 >
2024-04-17 04:18:49 +01:00
Stéphane Cerveau
363a90d0c4
vulkan/video: hevc: b-frames can be reference or not
...
b-frames can be considered as reference, so the NAL type
should refer to reference type and either RASL or TRAIL
depending on the irap_pic_flag.
Fixes: 72f52329c ("vulkan/video: add a nal_unit lookup for hevc")
Reviewed-by: Hyunjun Ko <zzoon@igalia.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28657 >
2024-04-17 01:45:54 +00:00
Jose Maria Casanova Crespo
eb4dbe59b3
ci: re-enable Igalia farm
...
Re-enable the farm after detecting that job failures were caused
by a new CI host to be deployed announcing rpi4 devices from other
host.
Signed-off-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28776 >
2024-04-17 01:23:25 +00:00
Mike Blumenkrantz
6ae2147dd6
nir: print i/o variables in location order
...
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25036 >
2024-04-17 00:46:40 +00:00
Constantine Shablia
cdcb2ab538
mesa: fix typo
...
Reviewed-by: Alyssa Rosenzweig <alyssa@rosenzweig.io >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28695 >
2024-04-16 23:56:19 +00:00
Constantine Shablia
6b04843537
vulkan/runtime: fix typo
...
Reviewed-by: Alyssa Rosenzweig <alyssa@rosenzweig.io >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28695 >
2024-04-16 23:56:19 +00:00
Guilherme Gallo
69eac6dd15
ci/a618: Add zink-tu-a618-full
...
We needed to halve the number of tests in `zink-tu-a618` due to the
reduced number of a618-limozeen in Collabora's farm, let's create a job
in the nightly pipeline to keep track of zink tests.
Signed-off-by: Guilherme Gallo <guilherme.gallo@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28770 >
2024-04-16 23:33:38 +00:00
Guilherme Gallo
659f5d3271
ci/a618: Rebalance a618-limozeen jobs
...
Recently, `sc7180-trogdor-lazor-limozeen-cbg-2` was retired from the
farm, so we need to rebalance the jobs again. The current flow is
causing the job queue to be huge.
Signed-off-by: Guilherme Gallo <guilherme.gallo@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28770 >
2024-04-16 23:33:38 +00:00
Rob Clark
453e9a24fd
ci: Add deqp fix for pipeline_statistics_3 tests
...
Signed-off-by: Rob Clark <robdclark@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28701 >
2024-04-16 22:44:24 +00:00
Rob Clark
7688d5a062
freedreno: Update a618 xfails
...
I'm not sure why deqp-runner isn't tagging things as UnexpectedPass.
Signed-off-by: Rob Clark <robdclark@chromium.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28701 >
2024-04-16 22:44:24 +00:00
David Heidelberg
f56fc16824
ci: disable Igalia farm
...
Multiple jobs failing.
Signed-off-by: David Heidelberg <david.heidelberg@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28775 >
2024-04-16 22:41:31 +00:00
Surafel Assefa
eaf8c56a11
radeonsi: Adds return on failure to get plane info
...
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27456 >
2024-04-16 22:14:49 +00:00
Iván Briano
53e130e333
nir/lower_doubles: preserve NaN when asked to do so
...
v2: avoid generating unnecessary bcsel (Caio)
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28702 >
2024-04-16 13:34:48 -07:00
Iván Briano
c6c52113a0
nir/lower_doubles: preserve sign of zero if we are asked to
...
v2: avoid generating unused instructions (Caio)
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28702 >
2024-04-16 13:34:46 -07:00
Iván Briano
5218cff34b
nir/algebraic: avoid double lowering of some fp64 operations
...
The ffloor@64 case, which lowers to use ffract, is already ignored if
nir_lower_dfract is set. Do the same thing for ftrunc@64 and ffract@64
and let nir_lower_doubles take care of them directly instead.
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28702 >
2024-04-16 13:34:36 -07:00
Mike Blumenkrantz
73bf648f04
ci: kill piano trace globally
...
fixes #11016
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28767 >
2024-04-16 15:28:42 -04:00
Rhys Perry
8eb2743b1f
aco/tests: remove LLVM 11 code
...
RADV requires LLVM 15+.
Signed-off-by: Rhys Perry <pendingchaos02@gmail.com >
Reviewed-by: Timur Kristóf <timur.kristof@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28748 >
2024-04-16 15:46:19 +00:00
Rhys Perry
13faa231c2
aco/tests: don't assume constructor order
...
"tests" might not be initialized when this constructor is called. Just use
a pointer instead.
Fixes aco_tests with LTO enabled.
Signed-off-by: Rhys Perry <pendingchaos02@gmail.com >
Reviewed-by: Timur Kristóf <timur.kristof@gmail.com >
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/11009
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28748 >
2024-04-16 15:46:19 +00:00
Hans-Kristian Arntzen
71fdc67682
radv: Store range rather than bo_size in VkBuffer/VkImage.
...
For purposes of address reports, it makes far more sense to report the
actually bound range rather than the full bo_size. RMV code used
effective size, so reproduce that here.
No other code looks at bo_size, so this should be quite safe.
Also fixes a theoretical correctness issue where plane aspect for
DISJOINT image was not passed to GetImageMemoryRequirements2 in internal
code.
Signed-off-by: Hans-Kristian Arntzen <post@arntzen-software.no >
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/10996
Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28718 >
2024-04-16 16:29:57 +02:00
Eric R. Smith
c939111f3f
panfrost: mark separate_stencil as valid when surface is valid
...
panfrost_initialize_surface is called when a surface is written to,
and marks that surface as valid. If the surface is a depth buffer
with a separate stencil, that separate stencil should also be marked
as valid; otherwise, readpixel will skip reading it (and hence the
stencil data will be read as uninitialized). This only affects
DEPTH32F_STENCIL8 formats.
Cc: mesa-stable
Signed-off-by: Eric R. Smith <eric.smith@collabora.com >
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28738 >
2024-04-16 12:58:09 +00:00
Timur Kristóf
9f36fba9be
ac/nir/ngg: Enable packing 16-bit mesh shader outputs.
...
This prepares ac_nir_lower_ngg_ms for the possibility of packing
two 16-bit outputs into the same 32-bit output slot, by handling
the high_16bits flag in NIR IO semantics.
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00
Timur Kristóf
07cedd6486
ac/nir/ngg: Split 16-bit MS output stores by components.
...
This is to prevent overwriting the high 16 bits.
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00
Timur Kristóf
c0c04d244c
ac/nir/ngg: Refactor MS output store into two functions.
...
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00
Timur Kristóf
10a71984d6
ac/nir/ngg: Refactor update_ms_output_info.
...
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00
Timur Kristóf
238c7f0ea2
ac/nir/ngg: Use just one IO semantics variable in MS output store.
...
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00
Timur Kristóf
d0018901f9
ac/nir/ngg: Slightly refactor mesh shader cull flag stores.
...
No functional changes, just improve code readability a little.
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Georg Lehmann <dadschoorse@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/28704 >
2024-04-16 14:08:31 +02:00