Juan A. Suarez Romero
8ab6d2b4c4
ci/broadcom: use new piglit runner
...
Switch from the old piglit to the new piglit-runner executor.
Acked-by: Andres Gomez <agomez@igalia.com >
Reviewed-by: Eric Anholt <eric@anholt.net >
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9680 >
2021-03-18 16:31:45 +00:00
Vasily Khoruzhick
e3f703e464
lima: update dEQP fails and skips lists
...
Reviewed-by: Erico Nunes <nunes.erico@gmail.com >
Reviewed-by: Andreas Baierl <ichgeh@imkreisrum.de >
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9586 >
2021-03-18 16:11:39 +00:00
Eric Anholt
fc6a815401
freedreno/a5xx: Fix stream-output binning handling.
...
This makes it match (to the best I was able) a6xx's behavior, with the
exception of the XXX note in fd5_gmem.c highlighting what I think is an
issue on both a5xx and a6xx.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9295 >
2021-03-18 08:47:48 -07:00
Jesse Natalie
69c2a472c2
u_format: Add restrict to fn pointer and manual format pack/unpack/fetch
...
MSVC warns (loudly) about a mismatch between the generated functions in
u_format_table.c and the definition of util_format_[un]pack_description,
specifically having 'restrict' in the function but not in the pointer
type in the struct.
So, add 'restrict' everywhere - to the function types, and to the rest
of the implementations that are assigned to those structs.
v2: util_format_unpack_description is used in gallium/auxiliary/translate
Fixes: 5785fdac ("u_format: Mark the generated pack/unpack src/dst args as restrict.")
Reviewed-by: Eric Anholt <eric@anholt.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9670 >
2021-03-18 14:53:52 +00:00
Mike Blumenkrantz
3419b16b6d
zink: handle nir_intrinsic_image_deref_samples
...
same as the texop but for images
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9682 >
2021-03-18 14:16:35 +00:00
Mike Blumenkrantz
50881d59e6
compiler/spirv: fix image sample queries
...
this was only implemented for textures (I assume because drivers which implement
the corresponding intrinsic don't support multisampled images), but it's also
used for shader images
Fixes: 22fdb2f855 ("nir/spirv: Update to the latest revision")
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9682 >
2021-03-18 14:16:35 +00:00
Mike Blumenkrantz
861c6a86dc
zink: enable spirv extension for post depth coverage
...
Fixes: 3c72c86742 ("zink: Wire up ARB_post_depth_coverage")
Reviewed-by: Adam Jackson <ajax@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9683 >
2021-03-18 10:01:13 -04:00
Timur Kristóf
89c8e22cc6
aco: Fix constant address offset calculation for ds_read2 instructions.
...
Cc: mesa-stable
Signed-off-by: Timur Kristóf <timur.kristof@gmail.com >
Reviewed-by: Rhys Perry <pendingchaos02@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9678 >
2021-03-18 10:43:41 +00:00
Erik Faye-Lund
0b090d8e67
gallium/st: reserve space in default uniform block for lowered constants
...
If we don't reserve these, we risk these lowering passes eating up more
uniforms than we have available.
This fixes a crash due to an assert in Zink, because we end up trying to
use a too large UBO after lowering.
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9631 >
2021-03-18 11:17:05 +01:00
Erik Faye-Lund
78de2de8f3
gallium/st: fix shader_has_one_variant
...
I think we need to care about this here as well, otherwise the variant
logic might be short-circuited.
Fixes: 7eb5fd98fd ("mesa/st: handle running nir lower passes for ucp and psiz in tess stage")
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9631 >
2021-03-18 11:16:06 +01:00
Erik Faye-Lund
bdfe0e3ef4
gallium/st: fix shader_has_one_variant
...
I think we need to care about this here as well, otherwise the variant
logic might be short-circuited.
Fixes: ad0037fcfe ("mesa/st: set lower_point_size for tes/gs during program update")
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9631 >
2021-03-18 11:16:06 +01:00
Neha Bhende
cc314361e3
nir_to_tgsi: Fix indices for CMP in nir_to_tgsi for nir_op_fcsel
...
nir_to_tgsi utiliy was using wrong src for nir_op_fcsel.
Fixes regression with solidworks2012_viewport.trace and mudbox-2010sp1-basic-head.trace
v2: as per suggested by Brian, instead of relying assumption that
value will be 0.0 or 1.0 all the time, use -abs(src) to make any non-zero
value as negative.
Reviewed-by: Brian Paul <brianp@vmware.com >
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Reviewed-by: Eric Anholt <eric@anholt.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9658 >
2021-03-18 04:20:16 +00:00
Eric Anholt
4258ab45b7
nir_to_tgsi: Respect PIPE_SHADER_CAP_TGSI_SQRT_SUPPORTED.
...
If the driver can't do it, we shouldn't be putting it in the TGSI,
regardless of what the NIR compiler above us decided.
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Reviewed-by: Neha Bhende <bhenden@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9662 >
2021-03-18 03:35:14 +00:00
Mike Blumenkrantz
5d031d8f31
zink: clear framebuffer state on context destroy
...
ensure we don't leak surface references
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:29 -04:00
Mike Blumenkrantz
321a569656
zink: move fence reset to zink_fence_init()
...
fence stuff goes in fence file
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:29 -04:00
Mike Blumenkrantz
bb90c8b926
zink: remove query batch-tracking init from begin_query()
...
this is always created on startup now
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:29 -04:00
Mike Blumenkrantz
cb70132778
zink: use macro to streamline batch struct member init
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:29 -04:00
Mike Blumenkrantz
89dce254d8
zink: move other batch-tracking implementations to unified codepath
...
prep for monotonic batch ids
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:29 -04:00
Mike Blumenkrantz
57021b1d4a
zink: also move batch destructor into zink_batch.c
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
56452230ab
zink: move batch init into zink_batch.c
...
this consolidates more batch code into the batch file
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
fc27594943
zink: move active query pruning to batch reset
...
this didn't ever really need to be on the fence in the first place
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
6aa791f626
zink: split out batch resource-set clearing into separate function
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
ecb50b37a9
zink: return enum zink_queue from zink_batch_reference_resource_rw()
...
this is a little cleaner
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
870e0e73d7
zink: abstract zink_get_resource_usage() and move it to be internal
...
I'll be rewriting how resource tracking works, so abstracting it and removing
direct uses is going to reduce the chances of breaking things as well as code churn
plus it's a bit easier to use
downside is that until that rewrite happens, this will be a (very small) perf hit and
there's some kinda gross macros involved to consolidate code
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
62b10ad1b8
zink: convert ZINK_RESOURCE_ACCESS defines to enum
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
c5390b2ef7
zink: refactor resource_sync_writes_from_batch_usage() to manage batch id internally
...
this is always used the same way, so we can simplify the code a bit for future use
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
304ee56cab
zink: add enum for different queues
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
49efef9d55
zink: add a pipe_context::fence_server_sync hook
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9626 >
2021-03-17 23:01:28 -04:00
Mike Blumenkrantz
6d005ba3fc
zink: more consolidation for null sampler/image view hashing
...
by using the new helper functions, this is even more consistent
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9625 >
2021-03-18 02:51:21 +00:00
Mike Blumenkrantz
c22e56f454
zink: ci updates
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9673 >
2021-03-18 02:40:56 +00:00
Mike Blumenkrantz
e02d1eb950
zink: always use requested format for sampler view creation
...
this should be fine and good
Fixes: c768c5297a ("zink: force stencil format for stencil-only samplers and swizzle the right component")
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9673 >
2021-03-18 02:40:56 +00:00
Neha Bhende
c6c532faa8
gallium/u_vbuf: use updated pipe_draw_start_count while using draw_vbo
...
new_draw has updated draw count. Stale info in draw caused regression
with piglit gl-3.0-multidrawarrays-vertexid -indirect
fixes piglit test ./gl-3.0-multidrawarrays-vertexid -indirect
Fixes: 1cd455b17b ("gallium: extend draw_vbo to support multi draws")
Reviewed-by: Brian Paul <brianp@vmware.com >
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9656 >
2021-03-18 02:09:45 +00:00
Mauro Rossi
1b9d8d7fdf
android: util: create some standalone compression helpers
...
Fixes the following building errors:
FAILED: out/target/product/x86_64/obj/SHARED_LIBRARIES/vulkan.android-x86_intermediates/LINKED/vulkan.radv.so
...
ld.lld: error: undefined symbol: util_compress_inflate
>>> referenced by disk_cache_os.c:459 (external/mesa/src/util/disk_cache_os.c:459)
...
ld.lld: error: undefined symbol: util_compress_max_compressed_len
>>> referenced by disk_cache_os.c:614 (external/mesa/src/util/disk_cache_os.c:614)
...
ld.lld: error: undefined symbol: util_compress_deflate
>>> referenced by disk_cache_os.c:622 (external/mesa/src/util/disk_cache_os.c:622)
Fixes: d7ecbd5bf8 ("util: create some standalone compression helpers")
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9613 >
2021-03-18 01:31:31 +00:00
Neha Bhende
edbbf4537b
mesa: set states in fast path for restoring light attributes
...
Since states were not updated in fast path for restoring light attributes,
seen darker images in solidworks2012_viewprt.trace
Fixes regression seen with solidworks2012_viewport.trace
Fixes: 7fa9d9d06c ("mesa: add a fast path for restoring light attributes")
Reviewed-by: Brian Paul <brianp@vmware.com >
Reviewed-by: Marek Olšák <marek.olsak@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9657 >
2021-03-18 01:06:39 +00:00
Jordan Justen
df5607d2ef
anv: Use fallback paths if DRM_I915_QUERY_ENGINE_INFO fails
...
Anvil can handle if this call fails, but not if we assert. :)
Reported-by: Brian Paul <brianp@vmware.com >
Fixes: 5d84c764fd ("anv: Gather engine info from i915 if available")
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9664 >
2021-03-18 00:54:29 +00:00
Jason Ekstrand
91192696e6
intel/fs: Add support for 16-bit A64 float and integer atomics
...
The messages for those 16-bit operations still use 32-bit sources and
destinations, so expand them accordingly when building the payload.
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8750 >
2021-03-18 00:13:40 +00:00
Jason Ekstrand
a572471edc
spirv: Add support for SPV_EXT_shader_atomic_float_min_max
...
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8750 >
2021-03-18 00:13:40 +00:00
Jason Ekstrand
1ba9c262fd
nir: Add image atomic_fmin/fmax intrinsics
...
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8750 >
2021-03-18 00:13:40 +00:00
Caio Marcelo de Oliveira Filho
302183d635
nir: Handle deref_atomic_fadd in a couple of passes
...
Acked-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8750 >
2021-03-18 00:13:40 +00:00
Caio Marcelo de Oliveira Filho
ba8ef1df9b
spirv: Update headers and metadata from latest Khronos commit
...
This corresponds to bcf55210f13a4fa3c3d0963b509ff1070e434c79
("Merge pull request #178 from orbea/datadir") in
https://github.com/KhronosGroup/SPIRV-Headers .
Acked-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8750 >
2021-03-18 00:13:40 +00:00
Dave Airlie
23100f3b65
lavapipe: bump maxMemoryAllocationCount
...
not sure why this was so low
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Reviewed-by: Adam Jackson <ajax@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9644 >
2021-03-18 00:01:08 +00:00
Marek Olšák
0bbae3139e
st/mesa: add a driconf option to transcode ETC2 to DXTC
...
for performance analysis
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9357 >
2021-03-17 23:33:09 +00:00
Lionel Landwerlin
8b6d22109f
intel/fs/vec4: add missing dependency in write-on-write fixed GRFs
...
If we load constant data using pull constant SENDS, and we later load
that register with some other data, we can end up in a situation where
we don't track the initial fixed register write and therefore end up
using uninitialized registers.
This tracks write-on-write of fixed GRFs like we do for normal virtual
GRFs.
v2: Fix post_alloc_reg case (Jason)
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Cc: <mesa-stable@lists.freedesktop.org >
Reviewed-by: Francisco Jerez <currojerez@riseup.net >
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9667 >
2021-03-17 23:25:02 +00:00
Adam Jackson
3c72c86742
zink: Wire up ARB_post_depth_coverage
...
Just a matter of passing the bits through in the right place.
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9637 >
2021-03-17 23:15:12 +00:00
Marek Olšák
c49d88f6b2
mesa: fix parameter reservation size
...
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00
Marek Olšák
94f41b8a09
mesa: clear reserved parameter storage because it's stored in the shader cache
...
The elements might not be initialized and we don't want random bytes
in the shader cache.
Discovered by valgrind.
Reviewed-by: Zoltán Böszörményi <zboszor@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00
Marek Olšák
bcc61a01d4
mesa: don't overallocate ParameterValues 4 times (v2)
...
The additional memory was never used.
v2: rework
Reviewed-by: Zoltán Böszörményi <zboszor@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00
Marek Olšák
e91dec36bc
mesa: fix a oldNum typo in reallocation in _mesa_reserve_parameter_storage
...
oldNum was incorrect. oldValNum is the correct number of elements
to copy inside realloc. (oldNum is for Parameters, not ParameterValues)
Reviewed-by: Zoltán Böszörményi <zboszor@gmail.com >
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00
Marek Olšák
aa748ef7e5
mesa: add assertions for buffer reference counts
...
Reviewed-by: Zoltán Böszörményi <zboszor@gmail.com >
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00
Marek Olšák
ae0ce3e3ba
mesa: fix Blender crash due to optimizations in buffer reference counting
...
The problem was that I assumed that deleted zombie buffers can't have any
references in the context, so buffers were released sooner than they should
have been.
The fix is to count the non-atomic references in the new field
gl_buffer_object::CtxRefCount. When we detach the context from the buffer,
we can just add CtxRefCount to RefCount to re-enable atomic reference
counting. This also allows removing code that was doing a similar thing.
Fixes: e014e3b6be "mesa: don't count buffer references for the context that created them"
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/4259
Reviewed-by: Zoltán Böszörményi <zboszor@gmail.com >
Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9360 >
2021-03-17 22:43:01 +00:00