Commits on Source (59)
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Found by inspection. Cc: 17.3 <mesa-stable@lists.freedesktop.org> Signed-off-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> (cherry picked from commit 9f54675d)
-
This add support for a 3D image reading path to the blit 2d paths, like I did for the clear paths. Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Tested-by:
Alex Smith <asmith@feralinteractive.com> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 9f675bf9)
-
If we are doing a general->general transfer with HIZ enabled, we want to hit the tile surface disable bits in radv_emit_fb_ds_state, however we never get the current layout to know we are in general and meta hardcoded the transfer layout which is always tile enabled. This fixes: dEQP-VK.api.copy_and_blit.core.image_to_image.all_formats.depth_stencil.d32_sfloat_s8_uint_d32_sfloat_s8_uint.optimal_general dEQP-VK.api.copy_and_blit.core.image_to_image.all_formats.depth_stencil.d32_sfloat_s8_uint_d32_sfloat_s8_uint.general_general v2: refactor some shared helpers for blit patches v3: we only need multiple render passes as they should be compatible. v3.1: use enum (Bas) Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Cc: "17.2 17.3" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 821b5379)
-
This fixes the layout issue for the blit path as well. This fixes: dEQP-VK.api.copy_and_blit.core.blit_image.all_formats.depth_stencil.d32_sfloat_s8_uint_d32_sfloat_s8_uint* v2: use compatible render passes. v2.1: use enum Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Cc: "17.2 17.3" <mesa-stable@lists.freedesktop.org> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit fbac9f86)
-
This fixes vmfaults seen on vega with: dEQP-VK.pipeline.multisample_interpolation.sample_interpolate_at_single_sample_.128_128_1.samples_1 These were caused by the don't allocate cmask but it was just accidental. The actual problem was the shader was trying to get the sample positions from a buffer, but the buffer was never getting configured to contain them, as the previous shader never needed them. Reviewed-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Fixes: 1171b304 (radv: overhaul fragment shader sample positions.) Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit b81f1a59)
-
On GFX9 we must access 3D textures with 3D samplers AFAICS. This fixes: dEQP-VK.api.image_clearing.core.clear_color_image.3d.single_layer on GFX9 for me. v1.1: fix tex->sampler_dim to dim v2: send layer in from outside v3: don't regress on pre-gfx9 Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Tested-by:
Alex Smith <asmith@feralinteractive.com> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit a99fa7e8)
-
DCC was disabled when the image format is !!supported, which is one ! too many. Ironically the commit that introduced it was supposed to lead to more DCC use ... Fixes: 969537d9 "radv: Add support for more DCC compression with VK_KHR_image_format_list." Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 70b5e85f)
-
This patch is ported from RadeonSI and it has two effects. It fixes a rendering issue which affects F1 2017 and Dawn of War 3 (Vega only) because LLVM was ending up by generating the new v_mad_mix_{hi,lo} instructions which appear to be buggy in some way. Not sure if Mesa is generating something wrong or if the issue is in LLVM only. Anyway, that explains why the DOW3 issue can't be reproduced with GL on Vega. It also improves performance because v_cvt_pkrtz_f16 is faster, and because I guess the rounding mode behaviour is similar between GL and VK, we can use it. About performance, it improves Talos by +3/4% but I don't see any other impacts. No CTS regressions on Polaris. Signed-off-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 5f81a435)
-
This fixes issues seen with certain versions of Unreal Engine 4 editor and games built with that using GLSL 4.30. v2: add driinfo_gallium change (Emil Velikov) Signed-off-by:
Tapani Pälli <tapani.palli@intel.com> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97852 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103801 Acked-by:
Andres Gomez <agomez@igalia.com> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit faccbaf3) [Emil Velikov: resolve trivial conflicts] Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Conflicts: src/mesa/drivers/dri/i965/brw_context.c
-
Signed-off-by:
Tapani Pälli <tapani.palli@intel.com> Suggested-by:
Darius Spitznagel <d.spitznagel@goodbytez.de> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104288 Acked-by:
Kenneth Graunke <kenneth@whitecape.org> (cherry picked from commit fcfb4236)
-
For copies the texture unit needs to know the depth format so it can read the htile data properly. This fixes: dEQP-VK.renderpass.suballocation.formats.d32_sfloat_s8_uint.load.clear Fixes: ad3d98da (radv: enable tc compatible htile for d32s8 also.) Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit d2acf97e)
-
Note: the following happens only when using slibtool. Since this is a very serious breakage, we will keep the workaround until a better solution is available. DRI modules store the address of the dispatch table in a TLS variable, _glapi_tls_Dispatch. Changes to the way libEGL is built in d884d8d0 resulted in it being statically linked against libglapi, and thus containing its own copy of _glapi_tls_Dispatch. The result was that some applications would fail to work (e.g. deqp-egl, which dynamically loads libEGL), due to the DRI module storing the dispatch table address in one copy of _glapi_tls_Dispatch, and libEGL obtaining the address from another copy of the variable. Fixes: d884d8d0 "egl/dri: link directly to libglapi.so" Signed-off-by:
Brendan King <Brendan.King@imgtec.com> Signed-off-by:
Eric Engestrom <eric.engestrom@imgtec.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> (cherry picked from commit e491bffc)
-
My refactor in 47273d73 missed this early return; because of it, setting UseFallback one layer above actually prevented the software path from being used. Remove this early return and let each platform's dri2_initialize_*() decide what it can do with the LIBGL_ALWAYS_SOFTWARE restriction. platform_{surfaceless,x11,wayland} were already handling it themselves. Fixes: 47273d73 "egl: set UseFallback if LIBGL_ALWAYS_SOFTWARE is set" Signed-off-by:
Eric Engestrom <eric.engestrom@imgtec.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Reported-by:
Brendan King <Brendan.King@imgtec.com> (cherry picked from commit 2f421651)
-
Commit 2f421651 ("egl: let each platform decided how to handle LIBGL_ALWAYS_SOFTWARE") broke the build due to copy-n-paste of misnamed function parameter.: src/egl/drivers/dri2/platform_android.c:1183:8: error: use of undeclared identifier 'disp' Rather than just fixing 'disp', rename the function parameter 'dpy' to 'disp' to align with the other EGL platforms' implementations. Fixes: 2f421651 ("egl: let each platform decided how to handle LIBGL_ALWAYS_SOFTWARE") Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> Acked-by:
Eric Engestrom <eric.engestrom@imgtec.com> Signed-off-by:
Rob Herring <robh@kernel.org> (cherry picked from commit aa187fe7) Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Conflicts: src/egl/drivers/dri2/platform_android.c
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Juan A. Suarez Romero authored
regression: The commit is causing a regression (https://bugs.freedesktop.org/show_bug.cgi?id=103626 ) Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Lucas Stach authored
The in-place resolve probably has some additional restrictions when not operating on a super tiled surface. Disable it on non-supertiled surfaces for now to work around a GPU hang. Fixes: 78ade659 ("etnaviv: Do GC3000 resolve-in-place when possible") Cc: mesa-stable@lists.freedesktop.org Signed-off-by:
Lucas Stach <l.stach@pengutronix.de> Reviewed-by:
Christian Gmeiner <christian.gmeiner@gmail.com> (cherry picked from commit 01585659)
-
Alex Smith authored
Fixes hangs seen due to the lock not being released here. Signed-off-by:
Alex Smith <asmith@feralinteractive.com> Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit 00a81e99)
-
Tim Rowley authored
Should be 0x80000000 instead of 0x8000000. Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Bruce Cherniak <bruce.cherniak@intel.com> (cherry picked from commit 396c006d)
-
Alex Smith authored
If we have a color attachment, but its writes are masked, this would have still returned true. This is inconsistent with how HasWriteableRT in 3DSTATE_PS_BLEND is set, which does take the mask into account. This could lead to PixelShaderHasUAV not being set in 3DSTATE_PS_EXTRA if the fragment shader does use UAVs, meaning the fragment shader may not be invoked because HasWriteableRT is false. Specifically, this was seen to occur when the shader also enables early fragment tests: the fragment shader was not invoked despite passing depth/stencil. Fix by taking the color write mask into account in this function. This is consistent with how things are done on i965. Signed-off-by:
Alex Smith <asmith@feralinteractive.com> Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Iago Toral Quiroga <itoral@igalia.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> (cherry picked from commit 12f4e00b)
-
Kenneth Graunke authored
intel_batchbuffer_emit_dword doesn't reserve space for the DWord it emits. In the past, we had some reserved batch space to ensure this worked. With the switch to growing batches, we need to actually request space so that we grow if necessary. Fixes: 2c46a67b (i965: Delete BATCH_RESERVED handling.) Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit 1c9f1a28)
-
Kenneth Graunke authored
intel_batchbuffer_emit_float is dead code, it should go. intel_batchbuffer_emit_dword only had one user, which had bungled using them by forgetting to call intel_batchbuffer_require_space first. So it seems wise to delete these unsafe helpers. Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit be144e25)
-
Alex Smith authored
After executing a secondary command buffer, we need to update certain state on the primary command buffer to reflect changes by the secondary. Otherwise subsequent commands may not have the correct state set. This fixes various issues (rendering errors, GPU hangs) seen after executing secondary command buffers in some cases. v2 (Jason Ekstrand): - Reset to invalid values instead of pulling from the secondary - Change the comment to be more descriptive Signed-off-by:
Alex Smith <asmith@feralinteractive.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> Cc: mesa-stable@lists.freedesktop.org (cherry picked from commit 4fd85617)
-
Juan A. Suarez Romero authored
fixes: The commit addresses earlier commit 6132992c which did not land in branch. Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Samuel Iglesias Gonsálvez authored
From Vulkan spec: "descriptorCount is the number of descriptors contained in the binding, accessed in a shader as an array. If descriptorCount is zero this binding entry is reserved and the resource must not be accessed from any stage via this binding within any pipeline using the set layout." Fixes: dEQP-VK.binding_model.descriptor_update.empty_descriptor.uniform_buffer Signed-off-by:
Samuel Iglesias Gonsálvez <siglesias@igalia.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Cc: mesa-stable@lists.freedesktop.org (cherry picked from commit e63adf8b)
-
Juan A. Suarez Romero authored
extra: The commit just references a fix for an additional change in its v2. Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Juan A. Suarez Romero authored
fixes: The commit addresses earlier commits 40a01c9a and 8d745abc which did not land in branch. Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Bas Nieuwenhuizen authored
the samples_identical instruction returns 0 if they are differet, so we have to do the extra work if the result is 0, not if it is != 0. Fixes: f4e499ec "radv: add initial non-conformant radv vulkan driver" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit c39947ce)
-
Bas Nieuwenhuizen authored
Fixes: f4e499ec "radv: add initial non-conformant radv vulkan driver" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit cebc9a11)
-
Bas Nieuwenhuizen authored
The position start at (dst.x, dst.y), so if we want the source to start at (src.x, src.y), we have to offset by (src.x-dst.x,src.y-dst.y). Haven't tested that this fixed anything yet, but found by inspection. Fixes: 69136f4e "radv/meta: add resolve pass using fragment/vertex shaders" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 73279da4)
-
Bas Nieuwenhuizen authored
Framebuffer is from 0,0, not (dst.x, dst.y). Fixes: 69136f4e "radv/meta: add resolve pass using fragment/vertex shaders" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit da192b50)
-
Bas Nieuwenhuizen authored
HW resolve does not support it either. Fixes: 2a04f548 "radv/meta: select resolve paths" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit a636208a)
-
Juan A. Suarez Romero authored
fixes: The commit addresses earlier commit d50937f1 which did not land in branch. Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Dave Airlie authored
The event emission wasn't sending the correct packet for gfx8 compute queues, which explains why it works on vega fine. This fixes the mpv vulkan hang. Fixes: ad61eac2 (radv: factor out eop event writing code. (v2)) Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 38e4467e)
-
Dave Airlie authored
It's legal to a pipeline stat query on a compute queue, but we'd emit the wrong packet here. This should fix it to emit the correct packet. Noticed while inspecting the mpv hang. Fixes: ad61eac2 (radv: factor out eop event writing code. (v2)) Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit ec1edd0f)
-
Dave Airlie authored
This fixes some of the broken: dEQP-VK.synchronization.op.multi_queue.*64x64x8* tests. Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit d08f2678)
-
Dave Airlie authored
This fixes some of the broken: dEQP-VK.synchronization.op.multi_queue.*64x64x8* tests. Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 09612a62)
-
Dave Airlie authored
This fixes some of the broken: dEQP-VK.synchronization.op.multi_queue.*64x64x8* tests. Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 420627e6)
-
Dave Airlie authored
This ports a fix from amdvlk, to fix the sizing for mip levels when block compressed images are viewed using uncompressed views. Fixes: dEQP-VK.image.texel_view_compatible.graphic.extended*bc* Fixes: e38685cc 'Revert "radv: disable support for VEGA for now."' Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 59515780)
-
Dave Airlie authored
amdvlk is probably more subtle than this but it never uses the inv cb/db variants, we fail some CTS tests without this. Fixes: dEQP-VK.renderpass.dedicated_allocation.formats.d32_sfloat_s8_uint.input*. Fixes: c2fbeb7c (radv: add GFX9 cache flushing support.) Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> (for now :-) Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 868377ab)
-
Dave Airlie authored
This should fix: dEQP-VK.pipeline.sampler.view_type.*.format.b4g4r4a4_unorm_pack16.address_modes.all_mode_clamp_to_border_opaque_black and a few others in that area. Fixes: b11c4a55 (radv: add texture descriptor/fmask/cmask support for GFX9) Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl> Signed-off-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit a4c23ce1)
-
Bas Nieuwenhuizen authored
This reverts commit 59515780. The mentioned commit causes a hang in DoW3 on Vega. Fixes: 59515780 "radv/gfx9: fix block compression texture views." Acked-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 516a80b5)
-
Bas Nieuwenhuizen authored
Seems like users are actually hitting 0xFFFFFFFF actually making things broken for them, and the mad max regression is fixed, so lets put this in once more. v2: Use 0xf for depth-only htile. (Dave) Fixes: af284411 "radv: Revert HTILE reset word to 0xFFFFFFFF." Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 51586031)
-
Bas Nieuwenhuizen authored
When rasterization is disabled we can have that few. Fixes: 76603aa9 "radv: Drop the default viewport when 0 viewports are given." Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit 1c78e4f0)
-
Bas Nieuwenhuizen authored
Fixes: f4e499ec "radv: add initial non-conformant radv vulkan driver" Reviewed-by:
Dave Airlie <airlied@redhat.com> (cherry picked from commit c99426ea)
-
Bas Nieuwenhuizen authored
These are just shaders reads, so we need to invalidate L1. Fixes: 6dbb0eac "radv: handle subpass cache flushes" Reviewed-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> (cherry picked from commit f2c9f13e)
-
Bas Nieuwenhuizen authored
If they were promoted from inputs/outputs, they could have a non-zero value left over, which messed with our store handling. Fixes: 06f05040 "radv: Link shaders." Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> (cherry picked from commit 67e09c8b)
-
Andres Gomez authored
anv_extensions usage from anv_icd was bringing the unwanted dependency of mako templates for the latter. We don't want that since it will force the dependency even for distributable tarballs which was not needed until now. Jason suggested this approach. v2: Patch simplification (Jason). Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104551 Fixes: 0ab04ba9 ("anv: Use python to generate ICD json files") Cc: Jason Ekstrand <jason.ekstrand@intel.com> Cc: Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
Andres Gomez <agomez@igalia.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit a1901d09)
-
Thomas Hellstrom authored
Upon reception of an event that lowered the number of active back buffers, the code would immediately try to free all back buffers with an id equal to or higher than the new number of active back buffers. However, that could lead to an active or to-be-active back buffer being freed, since the old number of back buffers was used when obtaining an idle back buffer for use. This lead to crashes when lowering the number of active back buffers by transitioning from page-flipping to non-page-flipping presents. Fix this by computing the number of active back buffers only when trying to obtain a new back buffer. Fixes: 15e208c4 ("loader/dri3: Don't accidently free buffer holding new back content") Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104214 Cc: "17.3" <mesa-stable@lists.freedesktop.org> Tested-by:
Andriy.Khulap <andriy.khulap@globallogic.com> Tested-by:
Vadym Shovkoplias <vadym.shovkoplias@globallogic.com> Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> Signed-off-by:
Thomas Hellstrom <thellstrom@vmware.com> (cherry picked from commit 897c54d5)
-
Józef Kucia authored
If alpha-to-coverage is enabled, we have to compute alpha even if color writes are disabled. Signed-off-by:
Józef Kucia <joseph.kucia@gmail.com> Signed-off-by:
Marek Olšák <marek.olsak@amd.com> (cherry picked from commit f222cf3c)
-
Florian Will authored
Respect the std430 rules for determining offset and size of struct members when using a std430 buffer. std140 rules lead to wrong buffer offsets in that case. Fixes my test case attached in Bugzilla. No piglit changes. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104492 Reviewed-by:
Timothy Arceri <tarceri@itsqueeze.com> (cherry picked from commit 7e025def)
-
Juan A. Suarez Romero authored
Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Juan A. Suarez Romero authored
Signed-off-by:
Juan A. Suarez Romero <jasuarez@igalia.com>
-
Timo Aaltonen authored
mesa-17.3.3
-
Timo Aaltonen authored
-
Timo Aaltonen authored
docs/relnotes/17.3.2.html
0 → 100644
docs/relnotes/17.3.3.html
0 → 100644
This diff is collapsed.