Re: [Mesa-dev] [Mesa-stable] [PATCH 3/7] configure.ac: rework llvm libs handling for 3.9+

2017-10-25 Thread Andres Gomez
Emil, this patch series introduces a lot of changes on how we treat LLVM's dependency. After applying this patch in the stable queue, I get these errors: For the "make Gallium Drivers SWR" and "make Gallium Drivers Other" targets: " ... CXXLDgallium_dri.la /usr/bin/ld: /usr/lib/llvm-3.9/li

Re: [Mesa-dev] [Mesa-stable] [PATCH 4/7] Travis: add binutils 2.26 for a few more LLVM 3.9 builds

2017-10-26 Thread Andres Gomez
Scratch my previous mail about binutils 2.26 not helping. The problem is that this patch needs previous commit b9a358a3e6c Emil, I will keep this series on hold. Can you do a backport of the 2 patches I've mentioned? (Alternative, I can do the backport myself, if you prefer that ...) On Thu, 20

Re: [Mesa-dev] [Mesa-stable] [PATCH v2] glsl: fix derived cs variables

2017-10-26 Thread Andres Gomez
Ilia, for stable, this is quite a big patch, adding a new class and source file. As per documentation, our criteria to accept a patch in the stable queue includes not being larger than 100 lines, which this ones exceeds by far: https://www.mesa3d.org/submittingpatches.html#criteria Hence, by now

Re: [Mesa-dev] [Mesa-stable] [PATCH] amd/common/gfx9: workaround DCC corruption more conservatively

2017-10-26 Thread Andres Gomez
Nicolai, this patch depends on 759526813be which didn't make it for 17.2 Hence, I could recover that if you want to nominate it or we would need a backport. Otherwise, I'm dropping this patch. Let me know what you think. On Thu, 2017-10-12 at 11:24 +0200, Nicolai Hähnle wrote: > From: Nicolai Hä

Re: [Mesa-dev] [PATCH 2/2] radv: Disallow indirect outputs for GS on GFX9 as well.

2017-10-27 Thread Andres Gomez
Bas, this patch looks like it should have been marked as fixing 6ce550453f1 instead of e38685cc62e (?). In any case, I was wondering whether it would be interesting to bring them both to the 17.2 stable queue and whether we would also want Timothy's preceding patch: 087e010b2b3dd83a539f97203909d6

[Mesa-dev] Mesa 17.2.4 release candidate

2017-10-27 Thread Andres Gomez
(1): i965: unref push_const_bo in intelDestroyContext Queued (21) === Andres Gomez (7): cherry-ignore: configure.ac: rework llvm detection and handling cherry-ignore: glsl: fix derived cs variables cherry-ignore: added 17.3 nominations. cherry-ignore: radv: Don

Re: [Mesa-dev] Mesa 17.2.4 release candidate

2017-10-28 Thread Andres Gomez
On Fri, 2017-10-27 at 14:14 -0400, Ilia Mirkin wrote: > On Fri, Oct 27, 2017 at 1:43 PM, Andres Gomez wrote: > > Rejected (6) > > > > > > Ilia Mirkin (1): > > glsl: fix derived cs variables > > > > Reason: Commit is too big for sta

[Mesa-dev] [ANNOUNCE] mesa 17.2.4

2017-10-30 Thread Andres Gomez
ing after a specific clang revision. Vulkan's WSI has gotten plugged a memory leak for X11. Andres Gomez (9): cherry-ignore: configure.ac: rework llvm detection and handling cherry-ignore: glsl: fix derived cs variables cherry-ignore: added 17.3 nominations. cherr

Re: [Mesa-dev] Mesa 17.2.4 release candidate

2017-10-30 Thread Andres Gomez
On Mon, 2017-10-30 at 11:39 -0400, Ilia Mirkin wrote: > On Mon, Oct 30, 2017 at 11:29 AM, Emil Velikov > wrote: > > On 28 October 2017 at 21:35, Andres Gomez wrote: > > > On Fri, 2017-10-27 at 14:14 -0400, Ilia Mirkin wrote: > > > > On Fri, Oct 27, 201

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] radv: Disallow indirect outputs for GS on GFX9 as well.

2017-10-30 Thread Andres Gomez
On Fri, 2017-10-27 at 19:50 +0200, Bas Nieuwenhuizen wrote: > On Fri, Oct 27, 2017 at 5:03 PM, Andres Gomez wrote: [...] > > In any case, I was wondering whether it would be interesting to bring > > them both to the 17.2 stable queue and whether we would also want > > Timo

Re: [Mesa-dev] [PATCH] i965: fix blorp stage_prog_data->param leak

2017-11-01 Thread Andres Gomez
Tapani, this patch landed with the "mesa-sta...@lists.freedesktop.org" tag without specifying branch. However, it addresses a fix for 8d90e28839, which didn't land in 17.2 so I'm skipping this patch for that queue. I hope this is OK with you. Let me know otherwise. On Fri, 2017-10-27 at 15:55 +0

Re: [Mesa-dev] [PATCH] gallium: fix typos in README.portability

2017-11-01 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez I would drop the mesa-stable nomination, though. These are just typos and not even in the docs/ directory ... On Mon, 2017-10-30 at 10:55 -0400, Ben Crocker wrote: > Cc: "17.2" > Signed-off-by: Ben Crocker > --- > src/gallium/

Re: [Mesa-dev] [PATCH] etnaviv: bugfix: Don't do resolve-in-place without valid TS

2017-11-03 Thread Andres Gomez
On Wed, 2017-11-01 at 12:47 +0100, Lucas Stach wrote: > Am Mittwoch, den 01.11.2017, 12:34 +0100 schrieb Wladimir J. van der Laan: > > On Wed, Nov 01, 2017 at 11:52:55AM +0100, Lucas Stach wrote: > > > Am Mittwoch, den 01.11.2017, 11:17 +0100 schrieb Wladimir J. van der Laan: > > > > GC3000 resolve

Re: [Mesa-dev] [PATCH 1/2] intel/fs: Alloc pull constants off mem_ctx

2017-11-03 Thread Andres Gomez
On Wed, 2017-11-01 at 20:58 +0200, Tapani Pälli wrote: > > On 11/01/2017 06:33 PM, Jason Ekstrand wrote: > > Do either of you mind if I cc the first one to stable? It does fix a > > potential memory leak in the case where compilation fails. > > IMO CC stable would be fine for that one. Jason,

Re: [Mesa-dev] [PATCH] radv: Don't expose heaps with 0 memory.

2017-11-03 Thread Andres Gomez
Bas, this commit landed without mentioning any specific stable queue. For 17.2, this is quite an extensive change and has several trivial conflicts. I think it is OK to merge if you really meant it for 17.2 too, but just double checking. Thanks! On Wed, 2017-11-01 at 20:58 +0100, Bas Nieuwenhuiz

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] radv: Disallow indirect outputs for GS on GFX9 as well.

2017-11-06 Thread Andres Gomez
Bas, how is the backport for this patch going? The pre-release announce is planned for this Wednesday so it'd great to have it latest by tomorrow. Thanks! On Mon, 2017-10-30 at 18:55 +0200, Andres Gomez wrote: > On Fri, 2017-10-27 at 19:50 +0200, Bas Nieuwenhuizen wrote: > > On Fri

Re: [Mesa-dev] [Mesa-stable] [PATCH 2/2] radv: Disallow indirect outputs for GS on GFX9 as well.

2017-11-07 Thread Andres Gomez
Queued. Thanks a lot, Bas. Very much appreciated ☺ On Tue, 2017-11-07 at 10:02 +0100, Bas Nieuwenhuizen wrote: > The backports and cherry-picks are available at > https://patchwork.freedesktop.org/series/33323/ . > > On Mon, Nov 6, 2017 at 11:25 AM, Andres Gomez wrote: > &g

[Mesa-dev] Mesa 17.2.5 release candidate

2017-11-08 Thread Andres Gomez
intel/nir: Add a helper for getting the NoIndirect mask inte l/nir: Break the linking code into a helper in brw_nir.c Queued (30) === Andres Gomez (7): docs: add sha256 checksums for 17.2.4 cherr y-ignore: radv: copy indirect lowering settings from radeonsi cher ry-ignore

[Mesa-dev] [PATCH 0/3] docs/releasing: enhance the releasing documentation

2017-11-09 Thread Andres Gomez
Just some small updates on the releasing documentation. Andres Gomez (3): docs/releasing: drop custom LLVM_CONFIG if previously manually set docs/releasing: drop manually exported variables during smoke test docs/releasing: improve the pre-announce template and examples docs

[Mesa-dev] [PATCH 1/3] docs/releasing: drop custom LLVM_CONFIG if previously manually set

2017-11-09 Thread Andres Gomez
Cc: Emil Velikov Cc: Eric Engestrom Signed-off-by: Andres Gomez --- docs/releasing.html | 4 1 file changed, 4 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html index 66742b0bcb4..e588435e813 100644 --- a/docs/releasing.html +++ b/docs/releasing.html @@ -471,6 +471,10

[Mesa-dev] [PATCH 2/3] docs/releasing: drop manually exported variables during smoke test

2017-11-09 Thread Andres Gomez
Cc: Emil Velikov Cc: Eric Engestrom Signed-off-by: Andres Gomez --- docs/releasing.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html index e588435e813..944b6e88bc1 100644 --- a/docs/releasing.html +++ b/docs/releasing.html @@ -504,8 +504,10

[Mesa-dev] [PATCH 3/3] docs/releasing: improve the pre-announce template and examples

2017-11-09 Thread Andres Gomez
Cc: Emil Velikov Cc: Eric Engestrom Signed-off-by: Andres Gomez --- docs/releasing.html | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/docs/releasing.html b/docs/releasing.html index 944b6e88bc1..40cbe232014 100644 --- a/docs/releasing.html +++ b/docs

[Mesa-dev] [ANNOUNCE] mesa 17.1.5

2017-07-14 Thread Andres Gomez
cmpswap intrinsics Andres Gomez (4):   docs: add sha256 checksums for 17.1.4   cherry-ignore: i965: Fix anisotropic filtering for mag filter   Update version to 17.1.5   docs: add release notes for 17.1.5 Anuj Phogat (2):   intel/isl: Use uint64_t to store total surface size   

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

2017-07-15 Thread Andres Gomez
On Mon, 2017-07-10 at 12:15 +0100, Eric Engestrom wrote: > On Saturday, 2017-07-08 22:59:40 +0300, Andres Gomez wrote: > > The LD_LIBRARY_PATH environment variable could be already defined so > > we extend it and restore it rather than just overwriting it. > > > >

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

2017-07-15 Thread Andres Gomez
On Mon, 2017-07-10 at 12:19 +0100, Emil Velikov wrote: > On 8 July 2017 at 20:59, Andres Gomez wrote: > > The LD_LIBRARY_PATH environment variable could be already defined so > > we extend it and restore it rather than just overwriting it. > > > > Hmm, what are yo

Re: [Mesa-dev] [PATCH 2/4] docs: remove redundant parameter for scons build basic testing

2017-07-15 Thread Andres Gomez
On Mon, 2017-07-10 at 12:22 +0100, Emil Velikov wrote: > On 8 July 2017 at 20:59, Andres Gomez wrote: > > toolchain=crossmingw is the default option for platform=windows when > > invoking scons. > > > > It may be a bit redundant, but I'd rather keep it. I doub

[Mesa-dev] [PATCH 0/3] docs: update basic testing on the releasing instructions (v2)

2017-07-15 Thread Andres Gomez
e actually doing. Andres Gomez (3): docs: add instructions to specify LLVM version for basic testing docs: avoid overwrite of LD_LIBRARY_PATH during basic testing docs: update master's release notes, news and calendar commit docs/releasing.html | 27 +++ 1 f

[Mesa-dev] [PATCH 1/3] docs: add instructions to specify LLVM version for basic testing

2017-07-15 Thread Andres Gomez
d the need to define LLVM_CONFIG also before running the release.sh script. Signed-off-by: Andres Gomez Reviewed-by: Eric Engestrom Reviewed-by: Emil Velikov --- docs/releasing.html | 8 1 file changed, 8 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html index 99707bee3

[Mesa-dev] [PATCH 2/3] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

2017-07-15 Thread Andres Gomez
The LD_LIBRARY_PATH environment variable could be already defined so we extend it and restore it rather than just overwriting it. v2: - Unset the __old_ld helper variable when we are done with it. - Corrected test for and escaping of variables (Eric). Signed-off-by: Andres Gomez Reviewed-by

[Mesa-dev] [PATCH 3/3] docs: update master's release notes, news and calendar commit

2017-07-15 Thread Andres Gomez
This reflects closer what we are actually doing. Signed-off-by: Andres Gomez Reviewed-by: Eric Engestrom Reviewed-by: Emil Velikov --- docs/releasing.html | 15 --- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/docs/releasing.html b/docs/releasing.html index

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

2017-07-18 Thread Andres Gomez
On Tue, 2017-07-18 at 10:48 +0100, Emil Velikov wrote: > On 15 July 2017 at 17:01, Andres Gomez wrote: > > On Mon, 2017-07-10 at 12:19 +0100, Emil Velikov wrote: > > > On 8 July 2017 at 20:59, Andres Gomez wrote: > > > > The LD_LIBRARY_PATH environment variab

Re: [Mesa-dev] [PATCH 3/4] docs: avoid overwrite of LD_LIBRARY_PATH during basic testing

2017-07-19 Thread Andres Gomez
On Wed, 2017-07-19 at 13:19 +0100, Emil Velikov wrote: > On 18 July 2017 at 14:05, Andres Gomez wrote: > > On Tue, 2017-07-18 at 10:48 +0100, Emil Velikov wrote: > > > On 15 July 2017 at 17:01, Andres Gomez wrote: > > > > On Mon, 2017-07-10 at 12:19 +0100, Emil Veli

[Mesa-dev] [PATCH] travis: add missing wayland-protocols

2017-07-19 Thread Andres Gomez
the pkg-config man page for more details. Also, added extra path to PKG_CONFIG_PATH env variable. Signed-off-by: Andres Gomez --- .travis.yml | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/.travis.yml b/.travis.yml index fa52bf96f1..246ad30eff 100644

[Mesa-dev] [PATCH 0/2] Fix distcheck

2017-07-19 Thread Andres Gomez
A couple of patches to fix recent problems while running make distcheck. Andres Gomez (2): broadcom: correct header file in BROADCOM_FILES egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist" src/broadcom/Makefile.sources | 2 +- src/egl/Makefile.am | 10 -

[Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

2017-07-19 Thread Andres Gomez
cal/mesa/src/egl' > make[2]: *** [distdir] Error 1 > make[1]: *** [distdir] Error 1 > make: *** [dist] Error 2 Fixes: 02cc359372 ("egl/wayland: Use linux-dmabuf interface for buffers") Cc: Emil Velikov Signed-off-by: Andres Gomez --- src/egl/Makefile.am | 10 +++

[Mesa-dev] [PATCH 1/2] broadcom: correct header file in BROADCOM_FILES

2017-07-19 Thread Andres Gomez
e[2]: Leaving directory '/home/local/mesa/src' > make[2]: *** [distdir] Error 1 > make[1]: *** [distdir] Error 1 Fixes: 42799c ("broadcom: Introduce a header for talking about chip revisions.") Cc: Emil Velikov Signed-off-by: Andres Gomez --- src/broadcom/Makefil

Re: [Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

2017-07-20 Thread Andres Gomez
On Thu, 2017-07-20 at 14:37 +0100, Emil Velikov wrote: > On 20 July 2017 at 13:54, Daniel Stone wrote: > > Hi Emil, > > > > On 20 July 2017 at 13:51, Emil Velikov wrote: > > > On 19 July 2017 at 23:44, Andres Gomez wrote: > > > > -dri2_backen

Re: [Mesa-dev] [PATCH] configure.ac: rework wayland-protocols handling

2017-07-23 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Fri, 2017-07-21 at 13:02 +0100, Emil Velikov wrote: > From: Emil Velikov > > At dist/distcheck time we need to ensure that all the files and their > respective dependencies are handled. > > At the moment we'll bail out as the

Re: [Mesa-dev] [PATCH] configure.ac: rework wayland-protocols handling

2017-07-23 Thread Andres Gomez
On Mon, 2017-07-24 at 01:27 +0300, Andres Gomez wrote: > This is: > > Reviewed-by: Andres Gomez Mmmm ... I hit the send button too quickly. Just wanted to mention that I share the same concerns than Daniel. Would it be up to me, I would not be adding the generated files in the rele

Re: [Mesa-dev] [PATCH 2/2] egl: add linux-dmabuf-unstable-v1-protocol.c to "nodist"

2017-07-23 Thread Andres Gomez
I'll be pushing already 1/2 and keep this on hold to see whether we favor instead: https://lists.freedesktop.org/archives/mesa-dev/2017-July/163730.html On Thu, 2017-07-20 at 01:44 +0300, Andres Gomez wrote: > This fixes `make distcheck` > > > make[3]: *** No rule to make tar

Re: [Mesa-dev] [Mesa-stable] [PATCH] i965/vec4: Use reads_accumulator_implicitly(), not MACH checks.

2017-04-25 Thread Andres Gomez
On Tue, 2017-04-25 at 11:57 +0200, Juan A. Suarez Romero wrote: > On Sat, 2017-04-22 at 16:28 -0700, Kenneth Graunke wrote: > > 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 SADA

[Mesa-dev] Mesa 17.0.5 release candidate

2017-04-26 Thread Andres Gomez
E in the correct register   radeonsi/gfx9: don't set deprecated field PARTIAL_ES_WAVE_ON   radeonsi: adjust ESGS ring buffer size computation on VI Queued (38) === Andres Gomez (15):   cherry-ignore: Add the pci_id into the shader cache UUID   cherry-ignore: fix crash if c

Re: [Mesa-dev] [PATCH v2] docs: add release calendar page and references to it

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Thu, 2017-04-27 at 16:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > Add a page that has information which release is expected when and > associated information. > > Reference to it from the "Releasing process" and &q

Re: [Mesa-dev] [PATCH 01/15] travis: explicitly LD_LIBRARY_PATH the local libraries

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > 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 &

Re: [Mesa-dev] [PATCH 05/15] travis: automatically manage ccache caching

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > According to the manual > > "If you are using ccache, use: > > language: c # or other C/C++ variants > > cache: ccache > > to cac

Re: [Mesa-dev] [PATCH 04/15] travis: enable apt cache

2017-04-28 Thread Andres Gomez
Do we want to do this? According to Travis own doc, there is little to no gain: https://docs.travis-ci.com/user/caching/#Things-not-to-cache On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > Signed-off-by: Emil Velikov > --- > .travis.yml | 1 + > 1 file changed,

Re: [Mesa-dev] [PATCH 06/15] travis: remove unused -dev packages

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > We effectively override libdrm-dev and libxcb-dri2-0-dev since we build > and install the package locally. > > Signed-off-by: Emil Velikov > --- > .tra

Re: [Mesa-dev] [PATCH 07/15] travis: rework "if test" blocks in the script section

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > Split the "if test" blocks so that we get more sensible output in case > of a failure. > > Signed-off-by: Emil Velikov > --- > .travis

Re: [Mesa-dev] [PATCH 08/15] travis: split out matrix from env

2017-04-28 Thread Andres Gomez
eep it symmetrical to the make build. > +- SCONS_CHECK_COMMAND="scons llvm=1 check" This is kind of confusing since MAKE_CHECK_COMMAND is not introduced until patch 12 in this series. I would rather have a specific patch just for that change and use until then: scons $SCONS_TA

Re: [Mesa-dev] [PATCH 09/15] travis: add separate "scons" and "scons llvm" targets

2017-04-28 Thread Andres Gomez
- SCONS_CHECK_COMMAND="scons llvm=1 check" > +- LLVM_VERSION=3.3 > +- LLVM_CONFIG="llvm-config-${LLVM_VERSION}" > + addons: > +apt: > + sources: > +- llvm-toolchain-trusty-3.3 llvm-toolchain-trusty-3.3 is disallowed (doesn't exist). Additionally, the LLVM 3.3 packages exist in Trusty, from which they are pulled. http://packages.ubuntu.com/trusty/llvm-3.3-dev With the above 2 lines removed, this is: Reviewed-by: Andres Gomez -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 10/15] travis: add "scons swr" to the build matrix

2017-04-28 Thread Andres Gomez
why we need libedit-dev. As far as I can see, the builds work OK without it. At least, until this point. With that change or not, this is: Reviewed-by: Andres Gomez -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://l

Re: [Mesa-dev] [PATCH 11/15] travis: add "make swr" to the build matrix

2017-04-28 Thread Andres Gomez
gt; +- llvm-toolchain-trusty-3.9 > + packages: > +# From sources above > +- g++-5 > +- llvm-3.9-dev > +# Common > +- x11proto-xf86vidmode-dev > + - libexpat1-dev > +

Re: [Mesa-dev] [PATCH 13/15] travis: add Gallium state-tracker targets

2017-04-28 Thread Andres Gomez
g++-4.7 > +- DRI_LOADERS="--disable-glx --disable-gbm --disable-egl" > +- DRI_DRIVERS="" > +- GALLIUM_ST="--disable-dri --enable-opencl --enable-opencl-icd > --enable-llvm --disable-xa --disable-nine --disable-xvmc --disable-vdpau > --

Re: [Mesa-dev] [PATCH 14/15] travis: enable wayland support

2017-04-28 Thread Andres Gomez
all: >- tar -jxvf $LIBVA_VERSION.tar.bz2 >- (cd $LIBVA_VERSION && ./configure --prefix=$HOME/prefix > --disable-wayland --disable-dummy-driver && make install) > > + - wget http://wayland.freedesktop.org/releases/$LIBWAYLAND_VERSION.tar.

Re: [Mesa-dev] [PATCH 13/15] travis: add Gallium state-tracker targets

2017-04-28 Thread Andres Gomez
On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > 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) req

Re: [Mesa-dev] [PATCH 15/15] travis: bump MAKEFLAGS to -j4

2017-04-28 Thread Andres Gomez
On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > The instance has 2 cores, yet bumping the jobs to 4 gives a minor speed > improvement. It probably won't make any harm in any case. This is: Reviewed-by: Andres Gomez

Re: [Mesa-dev] [PATCH 12/15] travis: split the make target to three separate ones

2017-04-28 Thread Andres Gomez
On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov > > 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: > -

Re: [Mesa-dev] [PATCH 12/15] travis: split the make target to three separate ones

2017-04-28 Thread Andres Gomez
On Fri, 2017-04-28 at 17:43 +0100, Emil Velikov wrote: > On 28 April 2017 at 16:00, Andres Gomez wrote: > > On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > > > From: Emil Velikov > > > > > > Split the target to allow faster builds for each run. >

Re: [Mesa-dev] [PATCH 04/15] travis: enable apt cache

2017-04-28 Thread Andres Gomez
On Fri, 2017-04-28 at 19:27 +0100, Emil Velikov wrote: > On 28 April 2017 at 19:15, Emil Velikov wrote: > > On 28 April 2017 at 11:50, Andres Gomez wrote: > > > Do we want to do this? > > > > > > According to Travis own doc, there is little to no gain: &

[Mesa-dev] [ANNOUNCE] mesa 17.0.5

2017-04-28 Thread Andres Gomez
lidate the readpix cache. Andres Gomez (17): cherry-ignore: Add the pci_id into the shader cache UUID cherry-ignore: fix crash if ctx torn down with no rendering cherry-ignore: Fix typos. cherry-ignore: Revert "etnaviv: Cannot render to rb-swapped formats" c

Re: [Mesa-dev] [PATCH 04/16] travis: enable apt cache

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Fri, 2017-04-28 at 19:25 +0100, Emil Velikov wrote: > From: Emil Velikov > > Provides a small, but consistent improvement. > Example numbers of the jobs added later in the series. > > "make loaders/classic DRI" - 1s > &quo

Re: [Mesa-dev] [PATCH 13/16] travis: model scons check target like the make one

2017-04-28 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Fri, 2017-04-28 at 19:25 +0100, Emil Velikov wrote: > From: Emil Velikov > > Should make things a bit more consistent across the board. > > Cc: Eric Engestrom > CC: Andres Gomez > Signed-off-by: Emil Velikov > --- > .travis

Re: [Mesa-dev] [PATCH 12/16] travis: split the make target to three separate ones

2017-04-28 Thread Andres Gomez
> > + # Generate the header since one is missing on the Travis instance > + - mkdir -p linux > + - echo "#ifndef _LINUX_MEMFD_H" > linux/memfd.h > + - echo "#define _LINUX_MEMFD_H" >> linux/memfd.h > + - ech

[Mesa-dev] [PATCH v2 5/7] docs/releasing: added relevant people for build/check with Windows

2017-05-03 Thread Andres Gomez
v2: Brian Paul as main contact point and Jose Fonseca as fallback (Vinson, Jose) Signed-off-by: Andres Gomez Cc: Emil Velikov Cc: Vinson Lee Cc: Brian Paul Cc: Jose Fonseca --- docs/releasing.html | 5 + 1 file changed, 5 insertions(+) diff --git a/docs/releasing.html b/docs

[Mesa-dev] [PATCH v2 6/7] docs/releasing: added relevant people for build/check with Android

2017-05-03 Thread Andres Gomez
v2: Tapani as main contact and Mauro just for help with debugging/building (Mauro). Signed-off-by: Andres Gomez Cc: Emil Velikov Cc: Tapani Pälli Cc: Mauro Rossi Acked-by: Tapani Pälli --- docs/releasing.html | 6 ++ 1 file changed, 6 insertions(+) diff --git a/docs/releasing.html

[Mesa-dev] [PATCH v2] glsl: Corrected some typos and error messages

2017-05-03 Thread Andres Gomez
v2: left code style/formatting corrections out. Signed-off-by: Andres Gomez --- src/compiler/glsl/ast_to_hir.cpp | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp index 1159b2cdfb..0703ed664c 100644

Re: [Mesa-dev] [PATCH] glsl: rename image_* qualifiers to memory_*

2017-05-03 Thread Andres Gomez
This is: Reviewed-by: Andres Gomez On Wed, 2017-05-03 at 11:16 +0200, Samuel Pitoiset wrote: > It doesn't make sense to prefix them with 'image' because > they are called "Memory Qualifiers" and they can be applied > to members of storage buffer blocks. &

Re: [Mesa-dev] [PATCH 2/2] nir: add pass to lower atomic counters to SSBO

2017-05-05 Thread Andres Gomez
Rob, I believe this is causing a linking failure in the Windows scons build: https://ci.appveyor.com/project/AndresGomez/mesa/build/321    Creating library build\windows-x86-debug\gallium\targets\osmesa\osmesa.lib and object build\windows-x86-debug\gallium\targets\osmesa\osmesa.exp nir.lib(nir_lo

[Mesa-dev] [PATCH v3 6/7] docs/releasing: added relevant people for build/check with Android

2017-05-05 Thread Andres Gomez
v2: Tapani as main contact and Mauro just for help with debugging/building (Mauro). v3: Mauro my provide feedback for android-x86 only (Mauro). Signed-off-by: Andres Gomez Cc: Emil Velikov Cc: Tapani Pälli Cc: Mauro Rossi Acked-by: Tapani Pälli --- docs/releasing.html | 6 ++ 1

[Mesa-dev] [PATCH 0/2] bin: honor editorconfig formatting

2017-05-05 Thread Andres Gomez
Just a series to correct the formating under bin/* Andres Gomez (2): bin/*sh: honor editorconfig formatting bin/*py: honor editorconfig formatting bin/bugzilla_mesa.sh | 14 - bin/get-extra-pick-list.sh | 48 ++-- bin/get-fixes-pick-list.sh | 78

[Mesa-dev] [PATCH 1/2] bin/*sh: honor editorconfig formatting

2017-05-05 Thread Andres Gomez
Signed-off-by: Andres Gomez --- bin/bugzilla_mesa.sh | 14 - bin/get-extra-pick-list.sh | 48 ++-- bin/get-fixes-pick-list.sh | 78 +++--- bin/get-pick-list.sh | 32 +-- bin/get-typod-pick-list.sh

[Mesa-dev] [PATCH 2/2] bin/*py: honor editorconfig formatting

2017-05-05 Thread Andres Gomez
Signed-off-by: Andres Gomez --- bin/perf-annotate-jit.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bin/perf-annotate-jit.py b/bin/perf-annotate-jit.py index 746434008f..4f05585bd4 100755 --- a/bin/perf-annotate-jit.py +++ b/bin/perf-annotate-jit.py @@ -133,7 +133,7

[Mesa-dev] [PATCH v2 1/2] bin: use tabs for coding style on *.sh files

2017-05-05 Thread Andres Gomez
v2: Instead of changing *.sh, adapt the editorconfig file (Emil). Signed-off-by: Andres Gomez --- bin/.editorconfig | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/bin/.editorconfig b/bin/.editorconfig index b9a948f405..ef92655823 100644 --- a/bin/.editorconfig +++ b/bin

[Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-06 Thread Andres Gomez
If an identified commit was having more than one fix, we would warn about that and only treat the first. Now, we don't warn but treat all of them. Signed-off-by: Andres Gomez --- bin/get-fixes-pick-list.sh | 51 +++--- 1 file changed, 26 inser

[Mesa-dev] [PATCH 0/2] Extended the release calendar

2017-05-06 Thread Andres Gomez
Calendar updated and extended until end of August. Andres Gomez (2): docs: update "Release manager" column docs: extend until the end of August docs/release-calendar.html | 68 +++--- 1 file changed, 64 insertions(+), 4 deletions(-)

[Mesa-dev] [PATCH 1/2] docs: update "Release manager" column

2017-05-06 Thread Andres Gomez
Cc: Emil Velikov Cc: Juan A. Suarez Romero Signed-off-by: Andres Gomez --- docs/release-calendar.html | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/release-calendar.html b/docs/release-calendar.html index fa67987da1..897fb41ead 100644 --- a/docs/release

[Mesa-dev] [PATCH 2/2] docs: extend until the end of August

2017-05-06 Thread Andres Gomez
Completed the 17.1 cycle and added the beginning of the 17.2 one. Cc: Emil Velikov Cc: Juan A. Suarez Romero Signed-off-by: Andres Gomez --- docs/release-calendar.html | 62 +- 1 file changed, 61 insertions(+), 1 deletion(-) diff --git a/docs

Re: [Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-08 Thread Andres Gomez
On Mon, 2017-05-08 at 00:48 +0100, Eric Engestrom wrote: > On Saturday, 2017-05-06 17:09:35 +0300, Andres Gomez wrote: > > If an identified commit was having more than one fix, we would warn > > about that and only treat the first. > > > > Now, we don't warn but

Re: [Mesa-dev] [PATCH 2/2] docs: extend until the end of August

2017-05-08 Thread Andres Gomez
On Mon, 2017-05-08 at 00:59 +0100, Eric Engestrom wrote: > On Saturday, 2017-05-06 18:46:13 +0300, Andres Gomez wrote: > > Completed the 17.1 cycle and added the beginning of the 17.2 one. > > > > Cc: Emil Velikov > > Cc: Juan A. Suarez Romero > > Signed-off-b

Re: [Mesa-dev] [PATCH v3 6/7] docs/releasing: added relevant people for build/check with Android

2017-05-08 Thread Andres Gomez
I'm not going to wait any longer and push this series now. I'm using Mauro's words so I hope this is OK. On Fri, 2017-05-05 at 12:44 +0300, Andres Gomez wrote: > v2: Tapani as main contact and Mauro just for help with > debugging/building (Mauro). > > v3: Ma

Re: [Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-08 Thread Andres Gomez
On Mon, 2017-05-08 at 10:56 +0100, Emil Velikov wrote: > On 6 May 2017 at 15:09, Andres Gomez wrote: > > If an identified commit was having more than one fix, we would warn > > about that and only treat the first. > > > > Now, we don't warn but treat all of the

Re: [Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-08 Thread Andres Gomez
On Mon, 2017-05-08 at 13:41 +0100, Emil Velikov wrote: > On 8 May 2017 at 11:52, Andres Gomez wrote: > > On Mon, 2017-05-08 at 10:56 +0100, Emil Velikov wrote: > > > On 6 May 2017 at 15:09, Andres Gomez wrote: > > > > If an identified commit was having mo

[Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: bring back the warning

2017-05-08 Thread Andres Gomez
We warn again if there is more than one line with the "fixes:" tag. The warning is only silenced when the commit has landed already or we output a message with the specific commit the "fixes:" tag is referring to. Signed-off-by: Andres Gomez --- bin/get-fix

[Mesa-dev] Mesa 17.0.6 release candidate

2017-05-10 Thread Andres Gomez
7a3040b4d366fbc) Cheers, Andres Mesa stable queue - Nominated (0) = Queued (38) === Adam Jackson (1):   egl/platform/drm: Don't take display ownership until gbm is initialized Andres Gomez (6):   docs: add sha256 checksums for 17.0.5   travi

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] st/mesa: don't cast the incomplete framebufer to st_framebuffer

2017-05-11 Thread Andres Gomez
Michel, Nicolai, this patch is queued for 17.0. Does this finding mean that we rather shouldn't add it in the release? So far it seems to me that it is still better having it in than not. Also, if we decide not to include it, should I keep the patch 2/2 of this series out too? Let me know what d

Re: [Mesa-dev] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

2017-05-11 Thread Andres Gomez
Emil, to bring the warning back in the same conditions there is: https://lists.freedesktop.org/archives/mesa-dev/2017-May/154907.html Could you give it a look and may be grant a R-b ? Thanks! On Tue, 2017-05-09 at 10:42 +0100, Emil Velikov wrote: > On 8 May 2017 at 14:49, Andres Gomez wr

Re: [Mesa-dev] [PATCH 0/2] Extended the release calendar

2017-05-11 Thread Andres Gomez
Emil, Juan, could you give a look to this series? Thanks! On Sat, 2017-05-06 at 18:46 +0300, Andres Gomez wrote: > Calendar updated and extended until end of August. > > Andres Gomez (2): > docs: update "Release manager" column > docs: extend until the end of

Re: [Mesa-dev] [Mesa-stable] [PATCH 1/2] st/mesa: don't cast the incomplete framebufer to st_framebuffer

2017-05-11 Thread Andres Gomez
On Thu, 2017-05-11 at 21:05 +0200, Nicolai Hähnle wrote: > On 11.05.2017 21:01, Andres Gomez wrote: > > Michel, Nicolai, this patch is queued for 17.0. > > > > Does this finding mean that we rather shouldn't add it in the release? > > So far it seems to me that

[Mesa-dev] [PATCH v2] bin/get-fixes-pick-list.sh: bring back the warning

2017-05-12 Thread Andres Gomez
We warn again if there is more than one line with the "fixes:" tag. The warning is only silenced when the commit has landed already or we output another message for every "fixes:" tag. Signed-off-by: Andres Gomez --- bin/get-fixes-pick-list.sh | 19 +--

Re: [Mesa-dev] [PATCH v2] bin/get-fixes-pick-list.sh: bring back the warning

2017-05-12 Thread Andres Gomez
On Fri, 2017-05-12 at 13:54 +0100, Emil Velikov wrote: > On 12 May 2017 at 09:33, Andres Gomez wrote: > > We warn again if there is more than one line with the "fixes:" tag. > > > > The warning is only silenced when the commit has landed already or we > > ou

[Mesa-dev] [ANNOUNCE] mesa 17.0.6

2017-05-12 Thread Andres Gomez
M 4.0. Adam Jackson (1):   egl/platform/drm: Don't take display ownership until gbm is initialized Andres Gomez (8):   docs: add sha256 checksums for 17.0.5   travis: replace Trusty-based LLVM toolchain apt-get with apt addon   travis: add the possibility of using the txc-dx

[Mesa-dev] [PATCH 0/2] Improve get-fixes-pick-list.sh

2017-05-12 Thread Andres Gomez
Some more improvements: - Only parse the commit messages. - Split "fixes:" tags on its own lines. - Join in a single line split lines like "fixes:\n$sha" Andres Gomez (2): bin/get-fixes-pick-list.sh: parse just the commit message bin/get-fixes-pick-list.sh: better iden

[Mesa-dev] [PATCH 2/2] bin/get-fixes-pick-list.sh: better identify multiple "fixes:" tags

2017-05-12 Thread Andres Gomez
Signed-off-by: Andres Gomez --- bin/get-fixes-pick-list.sh | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/bin/get-fixes-pick-list.sh b/bin/get-fixes-pick-list.sh index f9afcc49ce..32d830cda0 100755 --- a/bin/get-fixes-pick-list.sh +++ b/bin/get-fixes-pick-list.sh

[Mesa-dev] [PATCH 1/2] bin/get-fixes-pick-list.sh: parse just the commit message

2017-05-12 Thread Andres Gomez
We were parsing the whole diff, although the candidates were identified only by the commit message. Now, we only use the commit message for parsing. Signed-off-by: Andres Gomez --- bin/get-fixes-pick-list.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bin/get-fixes

Re: [Mesa-dev] [Mesa-announce] [ANNOUNCE] mesa 17.3.0-rc3

2017-11-10 Thread Andres Gomez
Hi, It seems we are a bit delayed with regard to the release-calendar [1]. Let's see when we finally get the final 17.3.0 but it may be that we will need to add a new cycle for 17.2.x and, also, move the 17.3.x release dates. Mentioning this specifically because I won't be able to be to the task

[Mesa-dev] [ANNOUNCE] mesa 17.2.5

2017-11-10 Thread Andres Gomez
ease build. Andres Gomez (9): docs: add sha256 checksums for 17.2.4 cherry-ignore: radv: copy indirect lowering settings from radeonsi cherry-ignore: i965: fix blorp stage_prog_data->param leak cherry-ignore: etnaviv: don't do resolve-in-place without valid TS cherr

Re: [Mesa-dev] [Mesa-stable] [PATCH] intel/fs: Use a pure vertical stride for large register strides

2017-11-10 Thread Andres Gomez
Jason, having this into account, I'll leave this patch out of 17.2 so far we don't have another one that fixes this regression (?) I noticed that the patch bisected by Mark is a different one so I'm not sure I'm understanding the status, though. Let me know what you think. On Thu, 2017-11-09 at

Re: [Mesa-dev] [Mesa-stable] [PATCH] intel/fs: Use a pure vertical stride for large register strides

2017-11-10 Thread Andres Gomez
On Fri, 2017-11-10 at 16:26 -0800, Jason Ekstrand wrote: > On Fri, Nov 10, 2017 at 4:12 PM, Andres Gomez > wrote: > > Jason, having this into account, I'll leave this patch out of 17.2 > > so > > far we don't have another one that fixes this regression (?) >

Re: [Mesa-dev] [Mesa-stable] [PATCH 17/21] intel/nir: Add a helper for getting the NoIndirect mask

2017-11-10 Thread Andres Gomez
Jason, this series of nominated commits landed without mentioning any specific stable queue. From what I'm seeing, they depend on 379b24a40d3 which didn't make it for 17.2 so I'm dropping the 3 of them for that queue: 7364f080f9a272323ed3491f278a1eed3eb9b1a7 (intel/nir: Add a helper for getting

Re: [Mesa-dev] [Mesa-stable] [PATCH v3 01/48] intel/fs: Pass builders instead of blocks into emit_[un]zip

2017-11-14 Thread Andres Gomez
Jason, after checking more in detail these 3 nominated commits from this series: fcd4adb9d08094520fb8d118d3448b04c6ec1fd1 intel/fs: Pass builders instead of blocks into emit_[un]zip. 0d905597fe2997c89022c76cdf84dc4fba5eb055 intel/fs: Be more explicit about our placement of [un]zip. 6c00240bc6508

<    1   2   3   4   5   6   7   >