URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=825e950aeae92b71ce4cee1df8c70cf81664cb6e
Author: Marc Dietrich <[email protected]>
Date: Wed Jan 24 22:03:51 2018 +0100
meson: fix HAVE_LLVM version define in meson build
LLVM patch level is not included in HAVE_LLVM.
Fixes: e6418ab1566d ("meson: build "radv" vulkan driver for radeon
hardware")
Reviewed-by: Eric Engestrom <[email protected]>
Reviewed-by: Dylan Baker <[email protected]>
Signed-off-by: Marc Dietrich <[email protected]>
(cherry picked from commit a2a1b0e75ef9adeb17468509c5f6331b42f6dc2e)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a989e999b41f2e6c48261c0a2b8aba8258dc97dc
Author: Samuel Pitoiset <[email protected]>
Date: Wed Apr 11 21:34:43 2018 +0200
radv: fix radv_layout_dcc_compressed() when image doesn't have DCC
num_dcc_levels means that DCC is supported, but this doesn't
mean that it's enabled by the driver. Instead, we should rely
on radv_image_has_dcc().
This fixes some multisample regressions since 0babc8e5d66
("radv: fix picking the method for resolve subpass") on Vega.
This is because the resolve method changed from HW to FS, but
those fails are totally unexpected, so there might some
differences between Polaris and Vega here.
Fixes: 44fcf587445 ("radv: Disable DCC for GENERAL layout and compute
transfer dest.")
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
(cherry picked from commit 9eac49246cdc501530418e8bd2a3e6d47173332b)
[Juan A. Suarez: do not call radv_image_has_dcc(), as it is not defined]
Signed-off-by: Juan A. Suarez Romero <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1d44ea348ef5fd8d4fcd032fc3beafb16bda7294
Author: Samuel Pitoiset <[email protected]>
Date: Tue Apr 10 16:00:56 2018 +0200
radv: fix picking the method for resolve subpass
The source and destination image parameters were swapped.
No CTS changes on Polaris10, but I suspect this might
fix something.
Fixes: 2a04f5481df ("radv/meta: select resolve paths")
Signed-off-by: Samuel Pitoiset <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
(cherry picked from commit 0babc8e5d665e54783c926b89183ab9a596aa04c)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=362c4f4c72966e50b47999ea16225b095f52a38d
Author: Bas Nieuwenhuizen <[email protected]>
Date: Mon Apr 9 16:54:55 2018 +0200
radv: Always reset draw user SGPRs after secondary command buffer.
As we sometimes reset them to -1, -1 does not mean that they are
not written by the secondary command buffer.
Fixes: ad11fc3571 "radv: don't emit unneeded vertex state."
Reviewed-by: Samuel Pitoiset <[email protected]>
(cherry picked from commit 41fbcc7901bc7a95fb7d5ccffdb3c18ba0361c40)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=d2991fc2c6076a5efd2fd9e5a3ecd0f0e186807a
Author: Bas Nieuwenhuizen <[email protected]>
Date: Mon Apr 9 16:53:57 2018 +0200
radv: Don't set instance count using predication.
The packet can sometimes be skipped, but we still think the change takes
effect.
This just makes the packet always take effect.
Fixes: ad11fc3571 "radv: don't emit unneeded vertex state."
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105942
Reviewed-by: Samuel Pitoiset <[email protected]>
(cherry picked from commit 74b0b869ddd4dbd36482aa9bec3403d45396af2d)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=66964df17a6e494bd1711712f56e9df3abf9b425
Author: Andres Gomez <[email protected]>
Date: Fri Mar 2 15:02:33 2018 +0200
mesa: adds some comments regarding MESA_GLES_VERSION_OVERRIDE usage
Fixes: 03fd6704db9 ("mesa: Add support for a new override string
MESA_GLES_VERSION_OVERRIDE")
Cc: Jordan Justen <[email protected]>
Cc: Ian Romanick <[email protected]>
Signed-off-by: Andres Gomez <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit 7cf3932098aba5fefaf241e35ee276b82e6e8ec7)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=5eef557ddd1f89e3004d16ba0dd0b4bf19bf6900
Author: Marek Olšák <[email protected]>
Date: Tue Mar 6 18:27:30 2018 -0500
mesa: simplify MESA_GL_VERSION_OVERRIDE behavior of API override
v2:
- Provide a correct explanation on the envvars documentation (Ian).
- Provide a more correct explanation on the function comments (Andres).
v3:
- Homogenize documentation and inline comments (Emil).
- Correct a typo (Emil).
Fixes: 2599b92eb97 ("mesa: allow forcing >=3.1 compatibility contexts
with MESA_GL_VERSION_OVERRIDE")
Cc: Jordan Justen <[email protected]>
Cc: Ian Romanick <[email protected]>
Cc: Eric Engestrom <[email protected]>
Cc: Emil Velikov <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit 806ab42c0f53064a774f002e311cfbb7ff10a667)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7711ae298777b3f1f95dd96fa7f9dffc10f95c1b
Author: Andres Gomez <[email protected]>
Date: Fri Mar 2 15:25:32 2018 +0200
dri_util: when overriding, always reset the core version
This way we won't fail when validating just because we may have a non
overriden core version that is lower than the requested one, even when
the compat version is high enough.
For example, running glcts from VK-GL-CTS with i965, this will
succeed:
$ MESA_GL_VERSION_OVERRIDE=4.6 ./glcts --deqp-case=KHR-GL46.info.vendor
While, this will fail:
$ MESA_GL_VERSION_OVERRIDE=4.6COMPAT ./glcts
--deqp-case=KHR-GL46.info.vendor
Fixes: 464c56d3d5c ("dri_util: Use
_mesa_override_gl_version_contextless")
Cc: Ian Romanick <[email protected]>
Cc: Tapani Pälli <[email protected]>
Cc: Marek Olšák <[email protected]>
Signed-off-by: Andres Gomez <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
(cherry picked from commit 044acd3569cbe689712be3c35544ceb7da4e5347)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=5f4009079cd31cfeac5fbb2b30b2a3197797437f
Author: Dylan Baker <[email protected]>
Date: Wed Apr 4 10:23:02 2018 -0700
meson: fix megadriver symlinking
Which should be relative instead of absolute.
Fixes: f7f1b30f81e842db6057591470ce3cb6d4fb2795
("meson: extend install_megadrivers script to handle symmlinking")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105567
Signed-off-by: Dylan Baker <[email protected]>
Reviewed-and-Tested-by: Eric Engestrom <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit 6ac87c17695ebf300a81ecc61e5703c6492b6e5c)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c9b6960f34bbfff21d9c66cc7278a99af8b4cb44
Author: Dylan Baker <[email protected]>
Date: Wed Apr 4 10:53:16 2018 -0700
meson: Set .so version for xa like autotools does
Fixes: 0ba909f0f111824223bc38563d1a6bc73e69c2cc
("meson: build gallium xa state tracker")
Signed-off-by: Dylan Baker <[email protected]>
Reviewed-by: Eric Engestrom <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit 19dbed6477525bff2918182b06bf01314f3cd5e9)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=e49d7abf87e33520298d99e0fb117b715a1b721a
Author: Jason Ekstrand <[email protected]>
Date: Fri Mar 23 11:05:04 2018 -0700
nir/lower_vec_to_movs: Only coalesce if the vec had a SSA destination
Otherwise we may end up trying to coalesce in a case such as
ssa_1 = fadd r1, r2
r3.x = fneg(r2);
r3 = vec4(ssa_1, ssa_1.y, ...)
and that would cause us to move the writes to r3 from the vec to the
fadd which would re-order them with respect to the write from the fneg.
In order to solve this, we just don't coalesce if the destination of the
vec is not SSA. We could try to get clever and still coalesce if there
are no writes to the destination of the vec between the vec and the ALU
source. However, since registers only come from phi webs and indirects,
the chances of having a vec with a register destination that is actually
coalescable into its source is very slim.
Shader-db results on Haswell:
total instructions in shared programs: 13657906 -> 13659101 (<.01%)
instructions in affected programs: 149291 -> 150486 (0.80%)
helped: 0
HURT: 592
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105440
Fixes: 2458ea95c56 "nir/lower_vec_to_movs: Coalesce movs on-the-fly when
possible"
Reported-by: Vadym Shovkoplias <[email protected]>
Tested-by: Vadym Shovkoplias <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
(cherry picked from commit 800df942eadc5356840f5cbc2ceaa8a65c01ee91)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1ec916659831ac1da1b31475bfe18422275ba76d
Author: Timothy Arceri <[email protected]>
Date: Tue Apr 3 11:38:13 2018 +1000
glsl: always call do_lower_jumps() after loop unrolling
This fixes a bug in radeonsi where LLVM cannot handle the case where
a break exists but its not the last instruction in the block.
LLVM would fail with:
Terminator found in the middle of a basic block!
LLVM ERROR: Broken function found, compilation aborted!
Fixes: 96fe8834f539 "glsl_to_tgsi: do fewer optimizations with
GLSLOptimizeConservatively"
Reviewed-by: Matt Turner <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105317
(cherry picked from commit b42633db8e3711e54a5bd10495b1436b8e362801)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=f1604f69c20d85d7e3e112ecf7a373d6143094aa
Author: Timothy Arceri <[email protected]>
Date: Sun Apr 1 09:32:28 2018 +1000
gallium/pipebuffer: fix parenthesis location
Without this the return value will never get set to -1. This
was first added in 49866c8f3457 and copied in 2b396eeed983.
Fixes: 2b396eeed983 "gallium/pb_cache: add a copy of cache bufmgr
independent of pb_manager"
Reviewed-by: Marek Olšák <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=102342
(cherry picked from commit 7e9b7ec094500f1245eed518592f99244e54a753)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=44c7d1aa2ea46b019b132ac095b50341aa13d26c
Author: Daniel Stone <[email protected]>
Date: Mon Apr 2 13:20:34 2018 +0100
st/dri: Initialise modifier to INVALID for DRI2
When allocating a buffer for DRI2, set the modifier to INVALID to inform
the backend that we have no supplied modifiers and it should do its own
thing. The missed initialisation forced linear, even if the
implementation had made other decisions.
This resulted in VC4 DRI2 clients failing with:
Modifier 0x0 vs. tiling (0x700000000000001) mismatch
Signed-off-by: Daniel Stone <[email protected]>
Reported-by: Andreas Müller <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Fixes: 3f8513172ff6 ("gallium/winsys/drm: introduce modifier field to
winsys_handle")
(cherry picked from commit 4cbecb61682a0ee426faaa03d824fc8fd7aef826)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=f2a13363cf1a1b982ab7ce6bfe7867b97e86d947
Author: Jason Ekstrand <[email protected]>
Date: Fri Mar 23 09:27:55 2018 -0700
intel/vec4: Set channel_sizes for MOV_INDIRECT sources
Otherwise, any indirect push constant access results in an assertion
failure when we start digging through the channel_sizes array. This
fixes dEQP-VK.pipeline.push_constant.graphics_pipeline.dynamic_index_vert
on Haswell. It should be a harmless no-op for GL since indirect push
constants aren't used there.
Reviewed-by: Kenneth Graunke <[email protected]>
Fixes: e69e5c7006d "i965/vec4: load dvec3/4 uniforms first in the..."
(cherry picked from commit 2b977989f3f01c186677988494bbf9b7342b31f2)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=df6c2bef90f0b6e84c72cfdb68e8bdba9d558f58
Author: Bas Nieuwenhuizen <[email protected]>
Date: Wed Mar 28 23:54:40 2018 +0200
ac/nir: Add workaround for GFX9 buffer views.
On GFX9 whether the buffer size is interpreted as elements or bytes
depends on whether IDXEN is enabled in the instruction. If the index
is a constant zero, LLVM optimizes IDXEN to 0.
Now the size in elements is interpreted in bytes which of course
results in out of bounds accesses.
The correct fix is most likely to disable the LLVM optimization,
but we need something to work with LLVM <= 6.0.
radeonsi does the max between stride and element count on the CPU
but that results in the size intrinsics returning the wrong size
for the buffer. This would cause CTS errors for radv.
v2: Also include the store changes.
Fixes: e38685cc62e 'Revert "radv: disable support for VEGA for now."'
Reviewed-by: Samuel Pitoiset <[email protected]>
(cherry picked from commit 4503ff760c794c3bb15b978a47c530037d56498e)
[Juan A. Suarez: partially backported from 908a0cd1dbe5, a backport for
17.3 stable branch; resolved trivial conflicts]
Conflicts:
src/amd/common/ac_nir_to_llvm.c
src/amd/vulkan/radv_nir_to_llvm.c
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=1550c67a3aadc783b4fc209974195c6a2e9aea74
Author: Dylan Baker <[email protected]>
Date: Mon Mar 26 11:10:16 2018 -0700
autotools: include meson_get_version
Otherwise meson won't read the VERSION file and won't set a version.
That means that pkg-config files will have version unset as well.
Fixes: 3e9533d9b88d75d99632fa40e38cfed842d10842
("meson: Add script to use VERSION file for getting version")
Signed-off-by: Dylan Baker <[email protected]>
Reviewed-by: Eric Engestrom <[email protected]>
(cherry picked from commit bc2fdb9759dc702ec351a044b3fd408c0701fedb)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=92cb895316b34374e65b6903c52811ffb2ee4747
Author: Eric Engestrom <[email protected]>
Date: Mon Mar 26 15:11:45 2018 +0100
gbm: remove never-implemented function
I assume this was implemented in a previous version of that commit, but
was removed in the version that actually landed.
Fixes: 8430af5ebe1ee8119e14 "Add support for swrast to the DRM EGL platform"
Cc: Giovanni Campagna <[email protected]>
Signed-off-by: Eric Engestrom <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit 431a1d12cc48060366caf49da76108cd0406b0f6)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9710a7042c4781c41d97646f3aac6c0eb6da0236
Author: Timothy Arceri <[email protected]>
Date: Mon Mar 26 11:41:51 2018 +1100
nir: fix crash in loop unroll corner case
When an if nesting inside anouther if is optimised away we can
end up with a loop terminator and following block that looks like
this:
if ssa_596 {
block block_5:
/* preds: block_4 */
vec1 32 ssa_601 = load_const (0xffffffff /* -nan */)
break
/* succs: block_8 */
} else {
block block_6:
/* preds: block_4 */
/* succs: block_7 */
}
block block_7:
/* preds: block_6 */
vec1 32 ssa_602 = phi block_6: ssa_552
vec1 32 ssa_603 = phi block_6: ssa_553
vec1 32 ssa_604 = iadd ssa_551, ssa_66
The problem is the phis. Loop unrolling expects the last block in
the loop to be empty once we splice the instructions in the last
block into the continue branch. The problem is we cant move phis
so here we lower the phis to regs when preparing the loop for
unrolling. As it could be possible to have multiple additional
blocks/ifs following the terminator we just convert all phis at
the top level of the loop body for simplicity.
We also add some comments to loop_prepare_for_unroll() while we
are here.
Fixes: 51daccb289eb "nir: add a loop unrolling pass"
Reviewed-by: Jason Ekstrand <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105670
(cherry picked from commit 629ee690addad9b3dc8f68cfff5ae09858f31caf)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7fe3731e9fa884d1b449af3dd8706aec3970ef77
Author: Timothy Arceri <[email protected]>
Date: Mon Mar 26 10:31:26 2018 +1100
glsl: fix infinite loop caused by bug in loop unrolling pass
Just checking for 2 jumps is not enough to be sure we can do a
complex loop unroll. We need to make sure we also have also found
2 loop terminators.
Without this we were attempting to unroll a loop where the second
jump was nested inside multiple ifs which loop analysis is unable
to detect as a terminator. We ended up splicing out the first
terminator but failed to actually unroll the loop, this resulted
in the creation of a possible infinite loop.
Fixes: 646621c66da9 "glsl: make loop unrolling more like the nir unrolling
path"
Tested-by: Gert Wollny <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105670
(cherry picked from commit 56b867395dee1a48594b27987d3bf68a4e745dda)
Squashed with:
glsl: remove unreachable assert()
Earlier commit enforced that we'll bail out if the number of terminators
is different than 2. With that in mind, the assert() will never trigger.
Fixes: 56b867395de ("glsl: fix infinite loop caused by bug in loop
unrolling pass")
Reviewed-by: Timothy Arceri <[email protected]>
Signed-off-by: Emil Velikov <[email protected]>
(cherry picked from commit 8eceac9de7d3cd4fddabbe61d512acfed9812169)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=4cfb3553eb40b26c1c2931e13136c2ceb216e114
Author: Lionel Landwerlin <[email protected]>
Date: Thu Mar 22 16:02:11 2018 +0000
i965/perf: fix config registration when uploading to kernel
When registring configurations to the kernel for the first time, we
run into an issue where the id number is not properly set (we're using
the wrong variable). As a result when trying to use that id later on,
we get an error.
This issue manifest itself the first time you use frameretrace after
reboot, subsequent runs are fine.
Fixes: 27ee83eaf7e9 ("i965: perf: add support for userspace configurations")
Signed-off-by: Lionel Landwerlin <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
(cherry picked from commit 1603ce1921a511f128025a49d055283440376231)
[Juan A. Suarez: resolve trivial conflicts]
Signed-off-by: Juan A. Suarez Romero <[email protected]>
Conflicts:
src/mesa/drivers/dri/i965/brw_performance_query.c
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=31f323165c380a1d79d6aaa2fcb9c2a9cec61dc8
Author: Juan A. Suarez Romero <[email protected]>
Date: Wed Apr 11 16:55:14 2018 +0200
cherry-ignore: omx: always define ENABLE_ST_OMX_{BELLAGIO,TIZONIA}
fixes: The commit fixes earlier commits 83d4a5d5aea5a8a05be2,
b2f2236dc565dd1460f0 and c62cf1f165919bc74296 which did not land in
branch.
Signed-off-by: Juan A. Suarez Romero <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7279b0c5ce3d707939a00055d757ebec414ffb3e
Author: Caio Marcelo de Oliveira Filho <[email protected]>
Date: Thu Mar 15 13:09:29 2018 -0700
anv/pipeline: fail if TCS/TES compile fail
v2: Add Fixes tag. (Lionel)
Fixes: e50d4807a35e679 ("anv: Compile TCS/TES shaders.")
Reviewed-by: Lionel Landwerlin <[email protected]>
(cherry picked from commit 318073ce660ca72b47ba83e37d1d0bc756f779b7)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=08b7ec9b2083e8f3482f651c20419b549fa1654b
Author: Juan A. Suarez Romero <[email protected]>
Date: Wed Apr 11 16:43:45 2018 +0200
cherry-ignore: radv: handle exporting view index to fragment shader. (v1.1)
fixes: The commit requieres earlier commits 639c4f2b54a6 and
2cfba40eea4c which did not land in branch.
Signed-off-by: Juan A. Suarez Romero <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=e26892d90271645efeb5dcda7116881aee9a3d0a
Author: Juan A. Suarez Romero <[email protected]>
Date: Wed Apr 11 16:30:54 2018 +0200
cherry-ignore: ac/shader: fix vertex input with components.
fixes: The commit fixes earlier commit 1c57a6da5e3 which did not land in
branch.
Signed-off-by: Juan A. Suarez Romero <[email protected]>
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c9e2de3398edccea06b62a4d1c3d11456b97f6c3
Author: Timothy Arceri <[email protected]>
Date: Wed Mar 7 11:10:54 2018 +1100
ac: make use of if/loop build helpers
These helpers insert the basic block in the same order as they
appear in NIR making it easier to follow LLVM IR dumps. The helpers
also insert more useful labels onto the blocks.
TGSI use the line number of the corresponding opcode in the TGSI
dump as the label id, here we use the corresponding block index
from NIR.
Reviewed-by: Marek Olšák <[email protected]>
(cherry picked from commit 99cdc019bf6fe11c135b7544ef6daf4ac964fa24)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=7a02062da57f41fac4f056e7b93ed019997d4087
Author: Timothy Arceri <[email protected]>
Date: Wed Mar 7 10:55:47 2018 +1100
radeonsi: make use of if/loop build helpers in ac
Reviewed-by: Marek Olšák <[email protected]>
(cherry picked from commit 6e1a142863b368a032e333f09feb107241446053)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=48cbac76a6c444b4f4119b3ddca7a69a4b4404b9
Author: Timothy Arceri <[email protected]>
Date: Wed Mar 7 10:53:34 2018 +1100
ac: add if/loop build helpers
These have been ported over from radeonsi.
Reviewed-by: Marek Olšák <[email protected]>
(cherry picked from commit 42627dabb4db3011825a022325be7ae9b51103d6)
[Juan A. Suarez: resolve trivial conflicts]
Signed-off-by: Juan A. Suarez Romero <[email protected]>
Conflicts:
src/amd/common/ac_llvm_build.c
src/amd/common/ac_llvm_build.h
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=42cf180fb5eba6efdfa66eb99f3c819607ecb908
Author: Dylan Baker <[email protected]>
Date: Mon Mar 12 11:19:52 2018 -0700
meson: don't use compiler.has_header
Meson's compiler.has_header is completely useless, it only checks that a
header exists, not whether it's usable. This creates problems if a
header contains a conditional #error declaration, like so:
> #if __x86_64__
> # error "Doesn't work with x86_64!"
> #endif
Compiler.has_header will return true in this case, even when compiling
for x86_64. This is useless.
Instead, we'll do a compile check so that any #error declarations will
be treated as errors, and compilation will work.
Fixes compilation on x32 architecture.
Gentoo Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=649746
meson bug: https://github.com/mesonbuild/meson/issues/2246
Signed-off-by: Dylan Baker <[email protected]>
Acked-by: Matt Turner <[email protected]>
Reviewed-by: Eric Engestrom <[email protected]>
(cherry picked from commit 8247a30838a74dcdd27cc2468bff8a3d8def640e)
[Juan A. Suarez: resolve trivial conflicts]
Signed-off-by: Juan A. Suarez Romero <[email protected]>
Conflicts:
meson.build
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=50e3fb590c615155ad9fb694b7c49992e0319a24
Author: Derek Foreman <[email protected]>
Date: Thu Mar 22 10:20:43 2018 -0500
egl/wayland: Make swrast display_sync the correct queue
commit 03dd9a88b0be17ff0ce91e92f6902a9a85ba584a introduced per surface
queues, but the display_sync for swrast_commit_backbuffer remained on
the old queue. This is likely to break when dispatching the correct
queue at the top of function (which can't dispatch the sync callback
we're waiting for).
The easiest known reproduction case is running weston-subsurfaces under
weston --use-pixman
Signed-off-by: Derek Foreman <[email protected]>
Reviewed-by: Daniel Stone <[email protected]>
(cherry picked from commit aa18a63512ccfa4eb8bc5d043e8967738a465af4)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9776580d9713be55098e776954796377dc0ca201
Author: Xiong, James <[email protected]>
Date: Thu Apr 5 11:58:14 2018 -0700
i965: return the fourcc saved in __DRIimage when possible
When creating a image from a texture, the image's dri_format is
set to the first plane's format, and used to look up for the
fourcc. e.g. for FOURCC_NV12 texture, the dri_format is set to
__DRI_IMAGE_FORMAT_R8, we end up with a wrong entry in function
intel_lookup_fourcc():
{ __DRI_IMAGE_FOURCC_R8, __DRI_IMAGE_COMPONENTS_R, 1,
{ { 0, 0, 0, __DRI_IMAGE_FORMAT_R8, 1 }, } },
instead of the correct one:
{ __DRI_IMAGE_FOURCC_NV12, __DRI_IMAGE_COMPONENTS_Y_UV, 2,
{ { 0, 0, 0, __DRI_IMAGE_FORMAT_R8, 1 },
{ 1, 1, 1, __DRI_IMAGE_FORMAT_GR88, 2 } } },
as a result, a wrong fourcc __DRI_IMAGE_FOURCC_R8 was returned.
To fix this bug, the image inherits the texture's planar_format that
has the original fourcc; Upon querying, if planar_format is set,
return the saved fourcc; Otherwise fall back to the old way.
v3: add a bug description and "cc mesa-stable" tag (Jason)
remove redundant null pointer check (Tapani)
squash 2 patches into one (James)
v2: fall back to intel_lookup_fourcc() when planar_format is NULL
(Dongwon & Matt Roper)
Cc: [email protected]
Signed-off-by: Xiong, James <[email protected]>
Reviewed-by: Tapani Pälli <[email protected]>
(cherry picked from commit f23b45dce3888112b7d26a623ab1280ce86533a1)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=2165cc0a1718346991367d61b9ed7e4c369ef4d6
Author: Axel Davy <[email protected]>
Date: Thu Apr 5 23:46:48 2018 +0200
st/nine: Do not use scratch for face register
Scratch registers are reused every instructions.
Since vFace is reused, a new temporary register
should be used.
Fixes: https://github.com/iXit/Mesa-3D/issues/311
Signed-off-by: Axel Davy <[email protected]>
CC: "17.3 18.0" <[email protected]>
(cherry picked from commit d899826733b1f6614c913c1c216f8157bf9e297d)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=8521e00b5c3ef627f1c302582dc6757052b3ddcb
Author: Axel Davy <[email protected]>
Date: Mon Apr 2 18:25:35 2018 +0200
st/nine: Declare lighting consts for ff shaders
The lighting constants were not declared previously,
but were accessed with indirect addressing, which is
illegal.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=105442
Signed-off-by: Axel Davy <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
CC: "17.3 18.0" <[email protected]>
(cherry picked from commit 39240926cd45519f35a6fa576c387f727b057aa1)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=0007574c41e1a3af269f0ac0ddb204791637d17c
Author: Iago Toral Quiroga <[email protected]>
Date: Mon Apr 2 11:39:41 2018 +0200
compiler/spirv: set is_shadow for depth comparitor sampling opcodes
From the SPIR-V spec, OpTypeImage:
"Depth is whether or not this image is a depth image. (Note that
whether or not depth comparisons are actually done is a property of
the sampling opcode, not of this type declaration.)"
The sampling opcodes that specify depth comparisons are
OpImageSample{Proj}Dref{Explicit,Implicit}Lod, so we should set
is_shadow only for these (we were using the deph property of the
image until now).
v2:
- Do the same for OpImageDrefGather.
- Set is_shadow to false if the sampling opcode is not one of these (Jason)
- Reuse an existing switch statement instead of adding a new one (Jason)
Fixes crashes in:
dEQP-VK.spirv_assembly.instruction.graphics.image_sampler.depth_property.*
Reviewed-by: Jason Ekstrand <[email protected]>
Cc: [email protected]
(cherry picked from commit 41ac0b1443ca7c8c3481eab978a41b7caba5503a)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=755d07c2692ea7f84bf15a545271311332e5b1da
Author: Sergii Romantsov <[email protected]>
Date: Mon Apr 2 09:59:06 2018 +0300
i965: Extend the negative 32-bit deltas to 64-bits
Gen8+ use 48-bit address relocations so need to extend the sign
to 64-bit return value. Without it we have higher bits zeroed
and missing the negavive values.
Haswell and older use 32-bit deltas so are unaffected by this issue.
v2:
used int32_t fucntion parameter instead of explicit type conversion.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101408
Signed-off-by: Sergii Romantsov <[email protected]>
Tested-by: Andriy Khulap <[email protected]>
Tested-by: Stuart Young <[email protected]>
Reviewed-by: Chris Wilson <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Cc: "18.0 17.3" <[email protected]>
(cherry picked from commit 98b860e3115ff937152dbf4c843e1ecb9244734c)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b44df1d118cc0880891ccd2c10f6175c835bf09d
Author: Rob Clark <[email protected]>
Date: Sun Apr 1 11:26:01 2018 -0400
freedreno/a5xx: don't align height for PIPE_BUFFER
Buffers can be large, so we probably don't want to make them all 32x
bigger. But they can't be rendered to (at least in GL) so we don't
need this workaround to prevent page faults on mem<->gmem.
Cc: "18.0" <[email protected]>
Signed-off-by: Rob Clark <[email protected]>
(cherry picked from commit 2f175bfe5d8ca59a8a68b6d6d072cd7bf2f8baa9)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b582a4e910cec834944e71809d08e100e113aaf1
Author: Rob Clark <[email protected]>
Date: Sun Apr 1 10:32:36 2018 -0400
freedreno/a5xx: fix page faults on last level
We could alternatively fall back to using "old style" draw's for
mem<->gmem (ie. what <= a4xx do) when height is not aligned to 32,
but that is somewhat more work (and not really something that could
be applied to stable)
Cc: "18.0" <[email protected]>
Signed-off-by: Rob Clark <[email protected]>
(cherry picked from commit 1866f76f7bc3ec54b4e91eb7d329b2e6f7b6277c)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=ab520c95bd4fca8e4e79f301ceb4884058b636d6
Author: Jason Ekstrand <[email protected]>
Date: Tue Mar 20 12:12:12 2018 -0700
nir/lower_indirect_derefs: Support interp_var_at intrinsics
This fixes the fs-interpolateAtCentroid-block-array piglit test on i965.
Reviewed-by: Kenneth Graunke <[email protected]>
Cc: [email protected]
(cherry picked from commit 6018f5b07966a0f85dea1ee6775d50a8c85fdee1)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=490c34281c609efe7af46dcf4f77ddade6b595cc
Author: Jason Ekstrand <[email protected]>
Date: Thu Mar 15 16:42:13 2018 -0700
nir/vars_to_ssa: Remove copies from the correct set
Reviewed-by: Kenneth Graunke <[email protected]>
Cc: [email protected]
(cherry picked from commit 0517d65f9639349d626aeb2af48ba9e4e605900d)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=510f4f33623080ecce2718a0839ad0502e45860a
Author: Henri Verbeet <[email protected]>
Date: Mon Mar 26 19:06:08 2018 +0430
mesa: Inherit texture view multi-sample information from the original
texture images.
Found running "The Witness" in Wine. Without this patch, texture views
created
on multi-sample textures would have a GL_TEXTURE_SAMPLES of 0. All things
considered such views actually work surprisingly well, but when combined
with
(plain) multi-sample textures in a framebuffer object, the resulting FBO is
incomplete because the sample counts don't match.
CC: <[email protected]>
Signed-off-by: Henri Verbeet <[email protected]>
Reviewed-by: Brian Paul <[email protected]>
(cherry picked from commit 0b73c86b8030a7f7cb35fc85c83eff7f2b8c24a4)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=51265b844b2365f43749566130aea2f4b77fb464
Author: Eric Engestrom <[email protected]>
Date: Wed Mar 28 11:14:40 2018 +0100
docs: fix 18.0 release note version
Fixes: 839fb3a696679bfe975c2 "docs: Update 18.0.0 release notes"
Cc: "18.0" <[email protected]>
Signed-off-by: Eric Engestrom <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
(cherry picked from commit d77844a5290948a490ce6921c1623d1dd7af6c31)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=b3343407a8e34e2393d3f87eab5feb5ce6580239
Author: Rob Clark <[email protected]>
Date: Fri Mar 16 13:10:18 2018 -0400
nir: fix per_vertex_output intrinsic
This is supposed to have both BASE and COMPONENT but num_indices was
inadvertantly set to 1.
Cc: <[email protected]>
Signed-off-by: Rob Clark <[email protected]>
Reviewed-by: Jason Ekstrand <[email protected]>
(cherry picked from commit cc3a88e81dbceb12b79eb4ebe7a4ce5ba97fc291)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=af4f8d426a70f31f57d590524e14a09b67a482a4
Author: Ian Romanick <[email protected]>
Date: Fri Mar 23 11:46:12 2018 -0700
i965/vec4: Fix null destination register in 3-source instructions
A recent commit (see below) triggered some cases where conditional
modifier propagation and dead code elimination would cause a MAD
instruction like the following to be generated:
mad.l.f0 null, ...
Matt pointed out that fs_visitor::fixup_3src_null_dest() fixes cases
like this in the scalar backend. This commit basically ports that code
to the vec4 backend.
NOTE: I have sent a couple tests to the piglit list that reproduce this
bug *without* the commit mentioned below. This commit fixes those
tests.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
Tested-by: Tapani Pälli <[email protected]>
Cc: [email protected]
Fixes: ee63933a7 ("nir: Distribute binary operations with constants into
bcsel")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105704
(cherry picked from commit 91225cb33f0baede872114bd416084b3b52937a1)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=a602ef93524d92695fc3df4b463ecf6fb50765be
Author: Eric Engestrom <[email protected]>
Date: Wed Mar 21 17:04:06 2018 +0000
meson/configure: detect endian.h instead of trying to guess when it's
available
Cc: Maxin B. John <[email protected]>
Cc: Khem Raj <[email protected]>
Cc: Rob Herring <[email protected]>
Suggested-by: Jon Turney <[email protected]>
Signed-off-by: Eric Engestrom <[email protected]>
Reviewed-by: Emil Velikov <[email protected]>
Reviewed-by: Dylan Baker <[email protected]>
Cc: <[email protected]>
(cherry picked from commit cbee1bfb34274668a05995b9d4c78ddec9e5ea4c)
[Juan A. Suarez: resolve trivial conflicts]
Signed-off-by: Juan A. Suarez Romero <[email protected]>
Conflicts:
meson.build
Squashed with:
configure: use AC_CHECK_HEADERS to check for endian.h
The currently we use the singular CHECK_HEADER combined with explicit
append to the DEFINES variable. That is a legacy misnomer, since it
requires us to add $DEFINES to every piece that we build.
Using the plural version of the helper sets the HAVE_ macro for us, plus
ensures it's passed to the compiler - if config.h is available in there
(not in the case of mesa) otherwise on the command line.
In hindsight, we should replace all the AC_CHECK_{FUNC,HEADER} instances
with the plural version (or even the _ONCE suffixed version) and drop
the DEFINES hacks.
Fixes: cbee1bfb342 ("meson/configure: detect endian.h instead of trying
to guess when it's available")
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105717
Signed-off-by: Emil Velikov <[email protected]>
Acked-by: Eric Engestrom <[email protected]>
Tested-by: Clayton Craft <[email protected]>
(cherry picked from commit 5a75019ad0270a974788a9b8648ba98ff4203768)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=fd6ded8f9cc3c95f25ee2623a397aff08cb158e2
Author: Leo Liu <[email protected]>
Date: Mon Mar 19 11:16:46 2018 -0400
radeon/vce: move feedback command inside of destroy function
On the CI family, firmware requires the destory command have to be the
last command in the IB, moving feedback command after destroy is causing
issues on CI cards, so we have to keep the previous logic that moves
destroy back to the last command.
But as the original issue fixed previously, with the newer family like
Vega10,
feedback command have to be included inside of the task info command along
with destroy command.
Fixes: 6d74cb25("radeon/vce: move destroy command before feedback command")
Signed-off-by: Leo Liu <[email protected]>
Acked-by: Christian König <[email protected]>
Cc: [email protected]
(cherry picked from commit c4de2f0880cfa49bd6fd3138564ee64ef4e637a1)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=bcdbcfca6e0d7f62da8989f31d78feae48eb3189
Author: Axel Davy <[email protected]>
Date: Sat Mar 10 18:49:59 2018 +0100
st/nine: Fix non inversible matrix check
There was a missing absolute value when
checking if the determinant was big enough.
Fixes: https://github.com/iXit/Mesa-3D/issues/292
Signed-off-by: Axel Davy <[email protected]>
Reviewed-by: Patrick Rudolph <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
CC: "17.3 18.0" <[email protected]>
(cherry picked from commit dbc24835d75466951a44b391b42e39461a6ac5a2)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=9ea807c8eee0a90b8774fc5b09a24779a492e35c
Author: Axel Davy <[email protected]>
Date: Sat Mar 10 14:28:10 2018 +0100
st/nine: Fixes warning about implicit conversion
Makes the conversion explicit.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=102542
Signed-off-by: Axel Davy <[email protected]>
Reviewed-by: Patrick Rudolph <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
CC: "17.3 18.0" <[email protected]>
(cherry picked from commit f61e9a958bd8d61cb7ca575ca987caefc6edbffd)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=25bf9b2bb71cc4182a127770a4dcec564dea22e8
Author: Axel Davy <[email protected]>
Date: Sat Mar 10 14:23:43 2018 +0100
st/nine: Fix bad tracking of vs textures for NINESBT_ALL
Stateblocks with NINESBT_ALL should track all textures.
For better performance they have a faster path which
copies all the required.
This path was only tracking ps textures.
Fixes: https://github.com/iXit/Mesa-3D/issues/303
Signed-off-by: Axel Davy <[email protected]>
Reviewed-by: Patrick Rudolph <[email protected]>
Tested-by: Dieter Nützel <[email protected]>
CC: "17.3 18.0" <[email protected]>
(cherry picked from commit 71eae7940ef7fa92e01cdc9afa1172f92d4b489e)
URL:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=85a862949d4c19fd4d623e2cdbdde081ba464f38
Author: Juan A. Suarez Romero <[email protected]>
Date: Thu Apr 12 21:48:14 2018 +0200
cherry-ignore anv: Be more careful about fast-clear colors
stable: There is a specific version for this patch for stable branches,
but it is causing regressions.
Signed-off-by: Juan A. Suarez Romero <[email protected]>
_______________________________________________
mesa-commit mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-commit
