- Mar 10, 2023
-
-
Some of the symbols listed in PLATFORM_SYMBOLS are not only specific to Linux, but rather specific to the GNU toolchain. Hence, use them when inspecting ELF binaries produced by a GNU toolchain: this means on Hurd ('GNU'), and on e.g. kFreeBSD ('GNU/kFreeBSD'). Signed-off-by:
Pino Toscano <toscano.pino@tiscali.it> Part-of: <mesa/mesa!21825>
-
This reverts commit 7ae0d9d2. Part-of: <mesa/mesa!21836>
-
This is an inline function with a compile-constant switch, so I expect the compiler wouldn't produce any better code like this, but for humans it's easier to read when function calls are not embedded into other function calls. Signed-off-by:
Eric Engestrom <eric@igalia.com> Reviewed-by:
Alejandro Piñeiro <apinheiro@igalia.com> Part-of: <mesa/mesa!21835>
-
Part-of: <mesa/mesa!21485>
-
This reduces the pain of Meson having picked inconsistent value names. See also https://github.com/mesonbuild/meson/pull/11279 where Meson devs argue that allowing `foo=false` could be interpreted by users to mean either `disabled` or `auto`, which I personally don't see. Part-of: <!21485>
-
Part-of: <!21485>
-
When enabled, on gfx12 plus, we will add the sync nop instruction after each instruction to make sure that current instruction depends on the previous instruction explicitly. This option will help us to get a hint if something is missing or broken in software scoreboard pass. Signed-off-by:
Sagar Ghuge <sagar.ghuge@intel.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Part-of: <mesa/mesa!21797>
-
Not sure how this one slipped in. Signed-off-by:
Alyssa Rosenzweig <alyssa@rosenzweig.io> Part-of: <mesa/mesa!21828>
-
This reverts commit aef0f3ef. We've got a new set of runners now (mesa-swrast-4, 5, and 7 because counting is hard) Part-of: <mesa/mesa!21826>
-
Signed-off-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
José Roberto de Souza <jose.souza@intel.com> Part-of: <mesa/mesa!21648>
-
Signed-off-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
José Roberto de Souza <jose.souza@intel.com> Part-of: <!21648>
-
- Mar 09, 2023
-
-
Now that we aren't using them on Gfx8+ we can drop a lot of cruft. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
This mainly lets the software scoreboarding pass correctly mark the instructions, without needing to resort to fragile manual handling in the generator. We can also make small improvements. On Gfx 8LP-12.0, we no longer have the restrictions about DWord alignment, so we can simply write each half into its intended location, rather than writing it to the low DWord and then shifting it in place. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
I originally thought that we were intentionally emitting the legacy opcodes here to make them opaque to the optimizer, so that it wouldn't eliminate the explicit type conversions, as they're actually required to do the quantization. But...we don't actually optimize those away currently anyway. So...go ahead and use the helpers for consistency. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
With the previous patch, we no longer need to special case this, as we emit a MOV with an HF source, rather than F16TO32 with an UW source, on all platforms that need scoreboarding. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
This gets us a MOV at the IR level on Gfx8+ which should be more optimizable than F16TO32. It also removes confusion about which pipe which the instruction will run on. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
We can just use the new builder helpers to get the optimization advantages of a MOV on Gfx8+ while also getting the necessary F32TO16 on Gfx7.x and yet not worry too hard about it. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
These take care of emitting the F32TO16/F16TO32 instructions on Gfx7.x but otherwise just emit a type converting MOV on Gfx8+. Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> Part-of: <mesa/mesa!21783>
-
For converting half-float to float, we currently emit BRW_OPCODE_F16TO32 with a UW source, to match legacy Gfx7 behavior. In the generator, this becomes a MOV with a HF source on Gfx8+. Unfortunately, this UW source confuses the scoreboarding pass into thinking it's an integer source, leading to incorrect SWSB annotations on Alchemist. We should ultimately fix the IR to stop being so...legacy...here, but this is the simplest fix for stable branches. Fixes misrendering in Elden Ring and likely Sekiro: Shadows Die Twice. Cc: mesa-stable Tested-by:
Chuansheng Liu <chuansheng.liu@intel.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Sagar Ghuge <sagar.ghuge@intel.com> References: mesa/mesa#8018 References: mesa/mesa#8375 Part-of: <mesa/mesa!21783>
-
Wa_1409600907 was enabled for gen12+. It should not be applied for platforms after gen12.0. Use generated helpers to ensure application to all relevant platforms. Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> Part-of: <mesa/mesa!21743>
-
We used to set RADEON_FLAG_GTT_WC when wsi_info is set. This changes it to set the flag for any external mem on vram, extending the logic for apps using external memory directly. Part-of: <mesa/mesa!21803>
-
Needed for process tracking for LAVA. Reviewed-by:
Erico Nunes <nunes.erico@gmail.com> Reviewed-by:
Daniel Stone <daniels@collabora.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <mesa/mesa!21711>
-
When Xorg already running, Weston XWayland should pick DISPLAY=:1 Reviewed-by:
Erico Nunes <nunes.erico@gmail.com> Reviewed-by:
Daniel Stone <daniels@collabora.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <mesa/mesa!21711>
-
Before: kernelci 38; Mesa3D 75 Priority now: - 38 ‒ kernelci - 40 ‒ after merge and performance - 50 ‒ user runs - 75 ‒ marge-bot (MUST be prioritized) Reviewed-by:
Guilherme Gallo <guilherme.gallo@collabora.com> Reviewed-by:
Daniel Stone <daniels@collabora.com> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <mesa/mesa!21812>
-
CTS runs on stoney are currently taking ~20min to complete, which seems to have begun with the upgrade to CTS 1.3.5.0. This is a bit too long in and of itself, but it means that - assuming zero contention - a job that has to be retried because the machine hung can take 40 minutes. Aim to drop this to 15min turnaround by lowering the overall fraction from 1/8th of the CTS to 1/11th. As the jobs we run have been reshuffled, this adds a lot more expected fails. As most of them categorise easily into patterns, group the failures together in the file. Non-strict wide lines has passed since we last ran it; the other failures all group into existing classes seen for a long time. Part-of: <mesa/mesa!21791>
-
Part-of: <mesa/mesa!21820>
-
Temporarily. Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <mesa/mesa!21815>
-
It looks like descriptors are generically a bit broken, which takes out a massive number of tests periodically. The pipeline-library tests also have some unknowable issues. cf. #8219 Part-of: <mesa/mesa!21740>
-
Reviewed-by:
Rhys Perry <pendingchaos02@gmail.com> Part-of: <mesa/mesa!21766>
-
Reviewed-by:
Rhys Perry <pendingchaos02@gmail.com> Part-of: <mesa/mesa!21766>
-
Reviewed-by:
Rhys Perry <pendingchaos02@gmail.com> Part-of: <mesa/mesa!21766>
-
Reviewed-by:
Rhys Perry <pendingchaos02@gmail.com> Part-of: <mesa/mesa!21766>
-
Reviewed-by:
Rhys Perry <pendingchaos02@gmail.com> Part-of: <mesa/mesa!21766>
-
Signed-off-by:
Tapani Pälli <tapani.palli@intel.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Part-of: <mesa/mesa!21723>
-
It currently takes ~21 minutes to complete. That's not quick. Part-of: <mesa/mesa!21793>
-
New test since the 1.3.5 update, and running it on older mesa it would have always failed, so it's not a regression -> let's just mark it as a known failure Part-of: <mesa/mesa!21792>
-
Otherwise the end wanders off to some unrelated cmdbuf. Fixes: 271ebdd7 ("zink: pass cmdbuf to debug marker begin") Part-of: <mesa/mesa!21799>
-
Closes: mesa/mesa#8501 Fixes: 4cc0cec4 ("ci: implement unified sections") Reviewed-by:
Emma Anholt <emma@anholt.net> Signed-off-by:
David Heidelberg <david.heidelberg@collabora.com> Part-of: <mesa/mesa!21796>
-
Quake II uses GL_POINT_SMOOTH to render particles. Zink currently requires `zink_emulate_point_smooth` to support that feature. Acked-by:
Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Reviewed-by:
Erik Faye-Lund <erik.faye-lund@collabora.com> Part-of: <mesa/mesa!21731>
-
Reviewed-By:
Mike Blumenkrantz <michael.blumenkrantz@gmail.com> Reviewed-by:
Erik Faye-Lund <erik.faye-lund@collabora.com> Part-of: <mesa/mesa!21731>
-