summaryrefslogtreecommitdiff
path: root/src/gallium/drivers/v3d/v3dx_state.c
AgeCommit message (Collapse)AuthorFilesLines
2024-02-07gallium: remove take_ownership from set_vertex_buffers, assume it's trueMarek Olšák1-2/+1
This removes the take_ownership parameter and defines the behavior as if take_ownership was always true, which is the fast path. This way, we don't have to have 2 codepaths in set_vertex_buffers of every driver. The old behavior is optionally available through util_set_vertex_buffers. It also documents a new constraint that count in set_vertex_buffers must be equal to the number of vertex buffers referenced by vertex elements or 0. Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27492>
2024-01-14gallium: remove unbind_trailing_count from set_vertex_buffersMarek Olšák1-3/+1
It should implicitly unbind all bound buffers after "count". This also slightly simplies u_vbuf. This is a cleanup suggested by: https://gitlab.freedesktop.org/mesa/mesa/-/issues/8142 Reviewed-by: Alyssa Rosenzweig <alyssa@rosenzweig.io> (asahi & panfrost) Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/26309>
2023-11-02broadcom: only support v42 and v71Alejandro Piñeiro1-111/+7
Acked-by: Emma Anholt <emma@anholt.net> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25851>
2023-10-13v3d,v3dv: fix depth bias for v3d 7.xIago Toral Quiroga1-1/+3
In v3d 7.x we don't need to scale up depth bias for D16 buffers. Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3dv: use new texture shader state rb_swap and reverse fields in v3d 7.xIago Toral Quiroga1-6/+0
In v3d 4.x we handle formats that are reversed or R/B swapped by applying a format swizzle. This doesn't work on border colors though, and for that there is a specific bit to reverse the border color in the texture shader state. In v3d 7.x we have new reverse and swap R/B bits and we no longer have a bit to reverse the border color because the new reverse bit applies to border texels too. Because of this, we absolutely need to use these new bits in order to get correct border colors in all cases with these formats. When we enable the reverse and/or swap R/B bits, we are effectively applying the format swizzle through them, so in these cases we need to make sure the swizzle we program in the texture shader state is the view swizzle provided by the API and not the composition of the format swizzle with the view swizzle like we do in 4.x for all formats. The same applies to custom border colors: we must not apply the format swizzle to them for formats that are reversed or R/B swapped, because again, this format swizzle is already applied through these new bits. While we are doing this, we also fully adopt the texture shader state spec from v3d 7.1.5 for v3d 7.x instead of using a description from 7.1.2 which is incompatible and required the driver to manually pack some of the bits. Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d,v3dv: support up to 8 render targets in v7.1+Iago Toral Quiroga1-2/+3
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d/v3dv: fix texture state array stride packing for V3D 7.1.5Iago Toral Quiroga1-5/+15
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d: don't convert floating point border colors in v71Alejandro Piñeiro1-14/+15
The TMU does this for us now. Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d: handle new TEXTURE_SHADER_STATE v71 YCbCr fieldsAlejandro Piñeiro1-4/+13
There are some new fields for YCbCr with pointers for the various planes in multi-planar formats. These need to match the base address pointer in the texture state, or the hardware will assume this is a multi-planar texture. Notice we don't use an address type for these fields in the XML description. This is because the addresses are 64-bit aligned (even though the PRM doesn't say it) which means the 6 LSB bits are implicitly 0, but the fields are encoded before the 6th bit of their starting byte, so we can't use the usual trick we do with address types where the first 6 bits in the byte are implicitly overwritten by other fields and we have to encode this manually as a uint field. This would mean that if we had an actual BO we would also need to add it manually to the job's list, but since we don't have one, we don't have to do anything about it. Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d: handle new texture state transfer functions in v71Alejandro Piñeiro1-6/+3
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d: default vertex attributes values are not needed for v71Alejandro Piñeiro1-17/+36
Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-10-13v3d: add v71 hw generationAlejandro Piñeiro1-0/+12
Starting point for v71 version inclusion: * Adds as one of the versions to be compiled on meson * Updated the v3d_X and v3dX macros to include version 71 * Update the code enough to get it building when using v71. Any real v71 support will be implemented on following commits. Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25450>
2023-08-11gallium: remove start_slot parameter from pipe_context::set_vertex_buffersYogesh Mohan Marimuthu1-3/+2
This patch removes start_slot from set_vertex_buffers() as suggested in https://gitlab.freedesktop.org/mesa/mesa/-/issues/8142 compilation testing: all gallium drivers, nine frontend compilation has been tested. d3d10umd compilation has not been tested driver, frontend testing: only llvmpipe and radeonsi driver was tested running game only the nine frontend changes are complex. All other changes are easy. nine front end was using start slot and also using multi context. nine frontend code changes: In update_vertex_elements() and update_vertex_buffers(), the vertex buffers or streams are ordered removing the holes. In update_vertex_elements() the vertex_buffer_index is updated for pipe driver to match the ordered list. v2: remove start_slot usage code from Marek (Marek Olšák) v3: nine stream number holes mask code from Axel (Axel Davy) Reviewed-by: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> (except nine, which is Ab) Reviewed-by: Marek Olšák <marek.olsak@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22436>
2023-05-23v3d: remove v3d_get_internal_type_bpp_for_output_formatAlejandro Piñeiro1-4/+3
This is a one-line wrapper, so let's just use the v3d_X or v3dX macros instead. Acked-by: Iago Toral Quiroga <itoral@igalia.com> Reviewed-by: Juan A. Suarez <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23172>
2023-05-23v3d: remove v3d_create_texture_shader_state_boAlejandro Piñeiro1-2/+2
This is a one-line wrapper, so let's just use the v3d_X or v3dX macros instead. Acked-by: Iago Toral Quiroga <itoral@igalia.com> Reviewed-by: Juan A. Suarez <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/23172>
2023-04-24v3d: add support for ARB_texture_cube_map_arrayJuan A. Suarez Romero1-3/+22
This implements support for texture cubemap arrays. Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22482>
2023-04-21broadcom: fix typosHarri Nieminen1-1/+1
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22591>
2023-04-14v3d: set depth compare function correctlyJuan A. Suarez Romero1-3/+6
Depth compare function must be set to the configured one only when compare mode is enabled; otherwise it must be configured to never. v2 (Eric): - Handle V3D < 4.0 case CC: mesa-stable Reviewed-by: Eric Engestrom <eric@igalia.com> Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22470>
2023-04-05v3d,v3dv: stop trying to force 16-bit TMU output for shadow comparisonsIago Toral Quiroga1-2/+1
In V3D we were doing this incorrectly by peeking into the sampler state unconditionally, which is not correct if the TMU operations don't use sampler state at all (like PBOs). This was causing us to fail the second test in this sequence when both tests run back back to back in the same process: dEQP-GLES3.functional.texture.shadow.2d.linear.greater_or_equal_depth_component32f dEQP-GLES3.functional.texture.specification.teximage2d_pbo.rg32f_cube Here, the first test would setup sampler state for shadow comparisons and the second test would setup a PBO upload, which would incorrectly pick up the sampler state to decide about the TMU output size for the PBO operation. In V3DV we were doing this right looking through each texture/sampler instruction and checking if they all involved shadow comparisons or had relaxed precission, defaulting to 32-bit otherwise. This special-casing for shadow comparisons also leaks from drivers into the compiler where we are forced to emit some pieces of sampler state for 32-bit outputs, so we had to special-case shadow instructions there as well and we also had a fix for CS textures not having correct sampler state representing shadow operations too. Finally, we also had at least a couple of bugs where forcing 32-bit TMU output through V3D_DEBUG wasn't correctly forcing shadow comparisons to actually be 32-bit in all the right places, leading to visual bugs with the option enabled (Sponza being one example of this). This change eliminates all of these issues. Finally, the performance improvement observed from special casing shadow comparison is negligible, and in specific scenarios it can even be detrimental to performance due to increased register pressure (Sponza with PCF filtering set to 4 is an example of this again). Fixes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/8684 Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22284>
2023-03-23v3d: fix condition for EZ disabling when stencil onJuan A. Suarez Romero1-1/+1
When stencil is enabled and it isn't non-op, Early-Z must be disabled. The condition that checks this for stencil[0] is correct, but the one for stencil[1] is wrong: it uses an "and" instead of "or" condition. This affects dEQP-GLES3.functional.fragment_ops.interaction.basic_shader.14 Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22081>
2023-03-21v3d: include offset as part of streamout targetJuan A. Suarez Romero1-2/+5
When dealing with multiple Transform Feedback buffers, each of them needs to have their own offset, so when resuming from one to another we know exactly were to continue adding primitives. Fixes "spec@arb_transform_feedback2@change objects while paused (gles3)" piglit test. Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Eric Engestrom <eric@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17373>
2022-08-04gallium: fixup some inconsistent uses of enum pipe_shader_type.Dave Airlie1-1/+1
These should be the enum not unsigned. Reviewed-by: Marek Olšák <marek.olsak@amd.com> Reviewed-by: Alyssa Rosenzweig <alyssa@collabora.com> (panfrost) Reviewed-by: Juan A. Suarez Romero <jasuarez@igalia.com> (v3d) Reviewed-by: Rob Clark <robdclark@chromium.org> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17747>
2022-07-13v3d,v3dv: stop copying and pasting the translate_swizzle helperIago Toral Quiroga1-28/+9
Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17509>
2022-06-09v3d: add ARB_polygon_offset_clamp extension supportJuan A. Suarez Romero1-0/+6
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16925>
2022-06-06v3d: use function to initialize refcountJuan A. Suarez Romero1-1/+1
Call proper pipe reference function to initialize the reference counting. Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16837>
2022-05-13v3d: add builtin support for white/black clamp-to-borderJuan A. Suarez Romero1-8/+34
Besides transparent border color, the hardware has special support for white and black borders when clamping to border is enabled, which should be more efficient. v2: - Rename enumerations (Iago) - Add comment to border_color_variant field (Iago) - Refactor the sampler variant code selection (Iago) v3: - Add comment for case of not clamping to border (Iago). - Generate only required sampler states (Iago). v4: - Use one array element for standard border color set (Iago). v5: - Fix variant setting for transparent black borders (Iago). Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16422>
2022-03-04v3d: rebind sampler view if resource changed the BOJuan A. Suarez Romero1-0/+17
When discarding the whole resource to create a new one, if this resource is used by a sampler view, a rebind must be done to use the new resource. But this must be done when setting the sampler views, because we don't have access to those samplers before. v2: - Pack shader state on setting sampler views (Iago) - Use a serial ID to know when to rebind sampler views (Juan) v3: - Move check to caller (Iago) - Keep rebind sampler view on BO change (Iago) v4: - Rename "serial_bo" to "serial_id" (Iago) - Add comments (Iago) Fixes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/6027 Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/15171>
2021-12-03v3d: support for texture buffer objectsAlejandro Piñeiro1-11/+54
This commit handles the support for texture buffer objects. In general it is mostly about using the buffer info from the pipe_image_view instead of the texture info. v2: - Rework some assertions (Iago) - Remove needless comment (Alejandro) - Fix comment typos (Iago) v3: - Fix typos (Iago) Signed-off-by: Alejandro Piñeiro <apinheiro@igalia.com> Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13409>
2021-11-11v3d: Update prim_counts when prims generated query in flight without TFNeil Roberts1-8/+2
In order to implement GL_PRIMITIVES_GENERATED, v3d allocates a small resource and adds a command to the job to store the prim counts to it. However it was only doing this when TF was enabled which meant that if the query was used with a geometry shader but no TF then the query would always be zero. This patch makes the driver keep track of how many PRIMITIVES_GENERATED queries are in flight and then enable writing the prim count if its more than zero. Fix dEQP-GLES31.functional.geometry_shading.query.primitives_generated_* v2: Update CI expectations and references to fixed tests in commit log. v3: - Add comment that GL_PRIMITIVES_GENERATED query is included because OES_geometry_shader, but it is not part of OpenGL ES 3.1. (Iago) - Update Fixes to commit introducing geometry shaders. (Iago) Fixes: a1b7c084 ("v3d: fix primitive queries for geometry shaders") Signed-off-by: Neil Roberts <nroberts@igalia.com> Signed-off-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com> Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13712>
2021-09-07v3d: Enable PIPE_CAP_TEXTURE_MIRROR_CLAMP_TO_EDGEJose Maria Casanova Crespo1-0/+2
Reviewed-by: Juan A. Suarez <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12744>
2021-08-20gallium: add take_ownership into set_sampler_views to skip reference countingMarek Olšák1-1/+7
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/12334>
2021-06-27st/mesa: remove the sampler min_lod/max_lod value swapMarek Olšák1-1/+2
It might not be needed at all. v2: fix v3d assertion failure Acked-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11428>
2021-06-04broadcom/common: move v3d_tiling to commonAlejandro Piñeiro1-1/+1
We initially just copied on v3dv, just in case we needed to modify it. One year later the code is exactly the same, so let's move it to common. This fix an additional issue, as we were not using NEON when building v3d_tiling.c for v3dv. v2: * Add "#include util/u_box.h" at v3d_tiling.h, so we can't avoid the need to include it on other places. (Juan and Iago) Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11121>
2021-04-29v3d: rename VC5 enums and definitionsJuan A. Suarez Romero1-25/+25
As the driver was renamed in the past from VC5 to V3D, let's rename also the definitions and enumerations to keep it consistent across the code. Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com> Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/10402>
2021-02-24v3d: Replace driver lowering of GL_CLAMP with mesa/st's.Eric Anholt1-21/+9
Mesa core can do this logic for us now. Reviewed-by: Iago Toral Quiroga <itoral@igalia.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9228>
2021-01-27gallium: add take_ownership param into set_vertex_buffers to eliminate atomicsMarek Olšák1-1/+3
There are a few places (mainly u_threaded_context) that do: set_vertex_buffers(...); for (i = 0; i < count; i++) pipe_resource_reference(&buffers[i].resource.buffer, NULL); set_vertex_buffers increments the reference counts while the loop decrements them. This commit eliminates those reference count changes by adding a parameter into set_vertex_buffers that tells the callee to accept all buffers without incrementing the reference counts. AMD Zen benefits from this because it has slow atomics if they come from different CCXs. Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8298>
2021-01-27gallium: add unbind_num_trailing_slots to set_sampler_viewsMarek Olšák1-0/+1
Instead of calling this functions again to unbind trailing slots, extend it to do it when binding. This reduces CPU overhead. A lot of drivers ignore "start" and always unbind all slots after "count". Such drivers don't need any changes here. Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8298>
2021-01-27gallium: add unbind_num_trailing_slots to set_shader_imagesMarek Olšák1-0/+6
Instead of calling this function again to unbind trailing slots, extend it to do it when images are being set. This reduces CPU overhead. Only st/mesa benefits. Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8298>
2021-01-27gallium: add unbind_num_trailing_slots to set_vertex_buffersMarek Olšák1-1/+3
Instead of calling this functions again to unbind trailing slots, extend it to do it as part of the call that sets vertex buffers. This reduces CPU overhead. Only st/mesa benefits from this. Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8298>
2021-01-27gallium: add take_ownership param into set_constant_buffer to eliminate atomicsMarek Olšák1-1/+2
We often do this: pipe->set_constant_buffer(pipe, shader, slot, &cb); pipe_resource_reference(&cb->buffer, NULL); That results in atomic increment in set_constant_buffer followed by atomic decrement after set_constant_buffer. This new interface eliminates those atomics. For the case above, this should be used instead: pipe->set_constant_buffer(pipe, shader, slot, true, &cb); cb->buffer = NULL; // if cb is not a local variable, else do nothing AMD Zen benefits from this. The perf improvement is ~3% for Viewperf13/Catia. Reviewed-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8298>
2020-12-22gallium: inline pipe_depth_state to decrease DSA state size by 4 bytesMarek Olšák1-2/+2
Depth and alpha states are now packed together, interleaved somewhat. Reviewed-by: Eric Anholt <eric@anholt.net> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7940>
2020-12-22gallium: pass pipe_stencil_ref by value (it has only 2 bytes)Marek Olšák1-2/+2
This changes pipe_context::set_stencil_ref to pass the parameter by value. Reviewed-by: Eric Anholt <eric@anholt.net> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7940>
2020-10-06gallium/util: remove empty file u_half.hMarek Olšák1-1/+1
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Reviewed-by: Matt Turner <mattst88@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6987>
2020-10-06util: remove util_float_to_half and util_half_to_float wrappersMarek Olšák1-5/+5
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Reviewed-by: Matt Turner <mattst88@gmail.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6987>
2020-10-04v3d: Split the creating of TEXTURE_SHADER_STATE into a helper functionNeil Roberts1-75/+88
This extracts the part of v3d_create_sampler_view that creates and fills in the buffer for the TEXTURE_SHADER_STATE record into a helper function. This will be used in a later patch to update the record when the information changes. v2: Also put the part that creates the buffer into the helper function so that it won’t override the contents of an inflight buffer. Reviewed-by: Eric Anholt <eric@anholt.net> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6914>
2020-10-04v3d: Make the function to set tex dirty state for a stage globalNeil Roberts1-22/+0
This moves the static function v3d_dirty_tex_state_for_shader_type from v3dx_state.c to v3d_context.c and adds a declaration for it in the header so that it can be used as a general utility function in a later patch. Reviewed-by: Eric Anholt <eric@anholt.net> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6914>
2020-10-01Revert F16C series (MR 6774)Matt Turner1-6/+6
This reverts commit 4fb2eddfdf9adafde2e6f94de23202ee44123d59. This reverts commit 7a1deb16f8af4e0ae4ed64511cbfcc606087f0ee. This reverts commit 2b6a17234376817e75d1f81edf5bd1b28eefb374. This reverts commit 5af81393e419eaf086e4de2a1d149af78cd1f54d. This reverts commit 87900afe5bbe90c5f3ad0921b28ae1c889029ada. A couple of problems were discovered after this series was merged that cause breakage in different configurations: (1) It seems that using -mf16c also enables AVX, leading to SIGILL on platforms that do not support AVX. (2) Since clang only warns about unknown flags, and as I understand it Meson's handling in cc.has_argument() is broken, the F16C code is wrongly enabled when clang is used, even for example on ARM, leading to a compilation error. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/3583 Reviewed-by: Eric Anholt <eric@anholt.net> Reviewed-by: Rob Clark <robdclark@chromium.org> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6969>
2020-09-30gallium/util: remove empty file u_half.hMarek Olšák1-1/+1
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6774>
2020-09-30util: remove util_float_to_half and util_half_to_float wrappersMarek Olšák1-5/+5
Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6774>
2020-05-13gallium: change comments to remove 'state tracker'Marek Olšák1-1/+1
Acked-by: Eric Anholt <eric@anholt.net> Acked-by: Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com> Acked-by: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/4902>