- Apr 30, 2017
-
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Emil Velikov authored
The instance should have 2 cores, yet bumping the jobs to 4 should give us a minor speed improvement. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit b1d45c33)
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 27a0b383)
-
Emil Velikov authored
Split into OpenCL and others, since the former is quite time consuming. v2: - explicitly enable/disable components - build libvdpau 1.1 requirement - enable st/vdpau - build libva 1.6.2 (API 0.38) requirement v3: Drop ubuntu-toolchain-r-test from sources (Andres) Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 0e6a36cd)
-
Emil Velikov authored
Should make things a bit more consistent across the board. Cc: Eric Engestrom <eric@engestrom.ch> Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit b3f20765)
-
Emil Velikov authored
Split the target to allow faster builds for each run. The overall build time will be more, yet Travis runs multiple builds in parallel so we're limited by the slowest one. Things are split roughly as: - DRI loaders, classic DRI drivers, classic OSMesa, make check - All Gallium drivers (minus the SWR) alongside st/dri (mesa) - The Vulkan drivers - ANV and RADV, make check (anv) v2: - rework RUN_CHECK to MAKE_CHECK_COMMAND - explicitly disable DRI loaders - generate linux/memfd.h locally and enable ANV - add libedit-dev v3: Use printf to create the header (Andres). v4: Really add the libedit + printf hunks. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 7e2af374)
-
Emil Velikov authored
v2: Quote OVERRIDE variables. v3: Add missplaced libedit-dev hunk (Andres). Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 8479fd8a)
-
Emil Velikov authored
Requires GCC 5.0 (due to the C++14 requirement) and LLVM 3.9. v2: Enable the target, add libedit-dev, rework check target. v3: Comment the current check target, add -j4 SCONSFLAGS, quote OVERRIDE variables. v4: Keep check target as-is (Andres) Cc: Tim Rowley <timothy.o.rowley@intel.com> Cc: George Kyriazis <george.kyriazis@intel.com> Reviewed-by:
George Kyriazis <george.kyriazis@intel.com> Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit f55d98ac)
-
Emil Velikov authored
The former does not require any LLVM, while the latter uses LLVM 3.3. This way we'll quickly catch any LLVM 3.3+ functionality that gets introduced where it shouldn't. Add the full list of addons for each build permutation. v2: Keep libedit-dev, rework check target. v3: Comment the current check target, add -j4 SCONSFLAGS v4: - Remove llvm-toolchain-trusty-3.3 source (Andres) - Keep check target as-is (Andres) Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 85ee2c6c)
-
Emil Velikov authored
With next commits we'll add a couple of more options. v2: Rework check target. v3: Comment the current check target, add -j4 SCONSFLAGS v4: Keep check target as-is, will rework with later patch. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 56ba252e)
-
Emil Velikov authored
Split the "if test" blocks so that we get more sensible output in case of a failure. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit abcfea23)
-
Emil Velikov authored
We effectively override libdrm-dev and libxcb-dri2-0-dev since we build and install the package locally. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit ae713a7b)
-
Emil Velikov authored
According to the manual "If you are using ccache, use: language: c # or other C/C++ variants cache: ccache to cache $HOME/.ccache and automatically add /usr/lib/ccache to your $PATH." Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 6431b98c)
-
Emil Velikov authored
Provides a small, but consistent improvement. Example numbers of the jobs added later in the series. "make loaders/classic DRI" - 1s "scons SWR" - 6s Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit 486f28ba)
-
The txc-dxtn library implements the patented S3 Texture Compression algorithm. By default it won't be used but we add the possibility of setting the USE_TXC_DXTN variable to yes in the travis web UI so it will be installed and used for the scons tests. Cc: Eric Anholt <eric@anholt.net> Cc: Rhys Kidd <rhyskidd@gmail.com> Signed-off-by:
Andres Gomez <agomez@igalia.com> [Emil Velikov: keep the LIB prefix, drop the LD_LIBRARY_PATH, fold URL] Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> (cherry picked from commit 29322dae)
-
Trusty's LLVM toochain repository was whitelisted some time ago. See: https://github.com/travis-ci/apt-source-whitelist/commit/479067c5e74cb0c1e2419209179b1afe2edce274 Signed-off-by:
Andres Gomez <agomez@igalia.com> [Emil Velikov] - set sudo to false - reference the Trusty change (Rhys) - keep libedit-dev Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> (cherry picked from commit 7819d265)
-
Emil Velikov authored
Some of the libraries may be dlopened, which may not always work due to the non-standard prefix that we're using. Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Andres Gomez <agomez@igalia.com> (cherry picked from commit cb820daa)
-
The command is really operating on a Queue not a command buffer and the nearest object to that with an allocator is VkDevice. Reviewed-by:
Chad Versace <chadversary@chromium.org> Cc: "17.0 17.1" <mesa-dev@lists.freedesktop.org> (cherry picked from commit bd3a9813)
-
val_bool and val_int are in a union. val_bool gets the first byte, which happens to work on LE when setting via the int, but breaks on BE. By setting the value properly, we are able to use DRI3 on BE architectures. Tested by running glxgears with a NV34 in a G5 PPC. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Cc: mesa-stable@lists.freedesktop.org [Emil Velikov: squash the vmwgfx hunk] Reviewed-by:
Emil Velikov <emil.l.velikov@gmail.com> Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> (cherry picked from commit 6af14778)
-
The incomplete framebuffer is set for a surfaceless context. This leads to the following error in piglit spec@egl_khr_surfaceless_context@viewport: ==26703==ERROR: AddressSanitizer: global-buffer-overflow on address 0x7f6886e43240 at pc 0x7f68854db0fd bp 0x7ffca404b3b0 sp 0x7ffca404b3a0 READ of size 8 at 0x7f6886e43240 thread T0 #0 0x7f68854db0fc in st_viewport ../../../mesa-src/src/mesa/state_tracker/st_cb_viewport.c:57 #1 0x556840176cdb in main tests/egl/spec/egl_khr_surfaceless_context/viewport.c:101 #2 0x7f688edcf3f0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x203f0) #3 0x556840176e19 in _start (/home/nha/amd/piglit/bin/egl-surfaceless-context-viewport+0xe19) 0x7f6886e43240 is located 32 bytes to the left of global variable 'DummyRenderbuffer' defined in '../../../mesa-src/src/mesa/main/fbobject.c:69:31' (0x7f6886e43260) of size 112 0x7f6886e43240 is located 8 bytes to the right of global variable 'IncompleteFramebuffer' defined in '../../../mesa-src/src/mesa/main/fbobject.c:73:30' (0x7f6886e42de0) of size 1112 SUMMARY: AddressSanitizer: global-buffer-overflow ../../../mesa-src/src/mesa/state_tracker/st_cb_viewport.c:57 in st_viewport Cc: mesa-stable@lists.freedesktop.org Reviewed-by: Marek Olšák <marek@olsak@amd.com> (cherry picked from commit 19b61799)
-
The more I think about it the more this seems like a bad idea. When we were deleting old cache dirs this wasn't so bad as it was unlikely we would ever hit the actual limit before things were cleaned up. Now that we only start cleaning up old cache items once the limit is reached the a percentage based max cache limit is more risky. For the inital release of shader cache I think its better to stick to a more conservative cache limit, at least until we have some way of cleaning up the cache more aggressively. Cc: "17.1" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com> (cherry picked from commit 22fa3d90)
-
The majority of cache files are less than 1kb this resulted in us greatly miscalculating the amount of disk space used by the cache. Using the number of blocks allocated to the file is more conservative and less likely to cause issues. This change will result in cache sizes being miscalculated further until old items added with the previous calculation have all been removed. However I don't see anyway around that, the previous patch should help limit that problem. Cc: "17.1" <mesa-stable@lists.freedesktop.org> Reviewed-and-Tested-by:
Michel Dänzer <michel.daenzer@amd.com> (cherry picked from commit 4e1f3afe)
-
Modern disks are extremely large and are only going to get bigger. Usage has shown frequent Mesa upgrades can result in the cache growing very fast i.e. wasting a lot of disk space unnecessarily. 5% seems like a more reasonable default. Cc: "17.1" <mesa-stable@lists.freedesktop.org> Acked-by:
Michel Dänzer <michel.daenzer@amd.com> (cherry picked from commit ce412371)
-
This fixes rendering corruptions in DOOM. Hopefully, it will also make Jenkins a bit more stable as we've been seeing some random failures and GPU hangs ever since turning on 48bit. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100620 Fixes: 651ec926 "anv: Add support for 48-bit addresses" Tested-by:
Grazvydas Ignotas <notasas@gmail.com> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org> Cc: "17.1" <mesa-stable@lists.freedesktop.org> (cherry picked from commit c43b4bc8)
-
Cc: 17.0 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 3f2a0649)
-
Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 80814819)
-
Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 60a20e68)
-
Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 8e8570a9)
-
Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 482e6b07)
-
There are 2 major hw changes: - The address must always point to the address of level 0. GFX9 tiling modes don't allow binding to a non-0 level. - 3D must always be bound as 3D, because 2D and 3D use entirely different tiling modes, and the texture target determines which set of modes is used. Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 5c947795)
-
Cc: 17.1 <mesa-stable@lists.freedesktop.org> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 65e0c3fb)
-
Until now the spilling cost calculation was neglecting the amount of data read from the register during the spilling cost calculation. This caused it to make suboptimal decisions in some cases leading to higher memory bandwidth usage than necessary. Improves Unigine Heaven performance by ~4% on BDW, reversing an unintended FPS regression from my previous commit 147e7124 with n=12 and statistical significance 5%. In addition SynMark2 OglCSDof performance is improved by an additional ~5% on SKL, and a Kerbal Space Program apitrace around the Moho planet I can provide on request improves by ~20%. Cc: <mesa-stable@lists.freedesktop.org> Reviewed-by:
Plamena Manolova <plamena.manolova@intel.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit 58324389)
-
This is what we use later on to compute the number of registers that will actually get spilled to memory, so it's more likely to match reality than the current open-coded approximation. Cc: <mesa-stable@lists.freedesktop.org> Reviewed-by:
Plamena Manolova <plamena.manolova@intel.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit ecc19e12)
-
opt_register_coalesce() was optimizing sequences such as: mul(8) acc0:D, attr18.xyyy:D, attr19.xyyy:D mach(8) vgrf5.xy:D, attr18.xyyy:D, attr19.xyyy:D mov(8) m4.zw:F, vgrf5.xxxy:F into: mul(8) acc0:D, attr18.xyyy:D, attr19.xyyy:D mach(8) m4.zw:D, attr18.xxxy:D, attr19.xxxy:D This doesn't work - if we're going to reswizzle MACH, we'd need to reswizzle the MUL as well. Here, the MUL fills the accumulator's .zw components with attr18.yy * attr19.yy. But the MACH instruction expects .z to contain attr18.x * attr19.x. Bogus results ensue. No change in shader-db on Haswell. Prevents regressions in Timothy's patches to use enhanced layouts for varying packing (which rearrange code just enough to trigger this pre-existing bug, but were fine themselves). Acked-by:
Timothy Arceri <tarceri@itsqueeze.com> Reviewed-by:
Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit 2faf227e) Squashed with commit: i965/vec4: Use reads_accumulator_implicitly(), not MACH checks. Curro pointed out that I should not just check for MACH, but use the reads_accumulator_implicitly() helper, which would also prevent the same bug with MAC and SADA2 (if we ever decide to use them). Cc: mesa-stable@lists.freedesktop.org Reviewed-by:
Francisco Jerez <currojerez@riseup.net> (cherry picked from commit 6b10c37b)
-
- Apr 24, 2017
-
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Emil Velikov authored
As pointed out by compiler ./llvm/codegen.hpp:52:22: error: ‘<::’ cannot begin a template-argument list [-fpermissive] ./llvm/codegen.hpp:52:22: note: ‘<:’ is an alternate spelling for ‘[’. Insert whitespace between ‘<’ and ‘::’ Cc: Francisco Jerez <currojerez@riseup.net> Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> Reviewed-by:
Vedran Miletić <vedran@miletic.net> (cherry picked from commit dd6ec78b)
-
Signed-off-by:
Vinson Lee <vlee@freedesktop.org> Reviewed-by:
Alejandro Piñeiro <apinheiro@igalia.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Cc: <mesa-stable@lists.freedesktop.org> (cherry picked from commit b81d85f1)
-
Currently we were only making sure types were the same within a single stage. This looks to have regressed with 953a0af8. Fixes: 953a0af8 ("mesa: validate sampler uniforms during gluniform calls") Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> Reviewed-by:
Tapani Pälli <tapani.palli@intel.com> https://bugs.freedesktop.org/show_bug.cgi?id=97524 (cherry picked from commit d682f8aa)
-
Emil Velikov authored
As mentioned in the manual - comparing pthread_t handles via the C comparison operator is incorrect and pthread_equal() should be used instead. Cc: Timothy Arceri <tarceri@itsqueeze.com> Fixes: d8d81fbc ("mesa: Add infrastructure for a worker thread to process GL commands.") Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Plamena Manolova <plamena.manolova@intel.com> Reviewed-by:
Brian Paul <brianp@vmware.com> Reviewed-by:
Eric Anholt <eric@anholt.net> (cherry picked from commit 52df318d)
-
Emil Velikov authored
If VDPAU is installed in the non-default location, we'll fail to find the headers and error at build time. ../../src/gallium/include/state_tracker/vdpau_dmabuf.h:37:25: fatal error: vdpau/vdpau.h: No such file or directory #include <vdpau/vdpau.h> ^ Fixes: faba96bc ("st/vdpau: add new interop interface") Cc: Christian König <christian.koenig@amd.com> Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Nicolai Hähnle <nicolai.haehnle@amd.com> (cherry picked from commit 51c0c213)
-