[Mesa-dev] [PATCH] anv: FORMAT_FEATURE_TRANSFER_SRC/DST_BIT_KHR not used with VkFormatProperties.bufferFeatures

ntel.com> Cc: Iago Toral Quiroga <ito...@igalia.com> Cc: Lionel Landwerlin <lionel.g.landwer...@intel.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- src/intel/vulkan/anv_formats.c | 5 - 1 file changed, 5 deletions(-) diff --git a/src/intel/vulkan/anv_formats.c

[Mesa-dev] [PATCH] docs: small release calendar fixes

Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/release-calendar.html | 2 +- docs/releasing.html| 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/docs/release-calendar.html b/docs/release-calendar.html index 19f518db1a..730cff8921 100644 --- a/docs/r

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

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 <ago...@igalia.com> --- bin/get-fixes-pick-list.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

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

Signed-off-by: Andres Gomez <ago...@igalia.com> --- 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/b

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

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] [ANNOUNCE] mesa 17.0.6

. 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-dxtn library

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

On Fri, 2017-05-12 at 13:54 +0100, Emil Velikov wrote: > On 12 May 2017 at 09:33, Andres Gomez <ago...@igalia.com> wrote: > > We warn again if there is more than one line with the "fixes:" tag. > > > > The warning is only silenced when the commit has la

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

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 <ago...@igalia.com> --- bin/get-fixes-pick-list.sh |

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

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 tha

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

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] [PATCH] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

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 &

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

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

[Mesa-dev] Mesa 17.0.6 release candidate

3040b4d366fbc) 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   t

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

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 <ago...@igalia.com> ---

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

On Mon, 2017-05-08 at 13:41 +0100, Emil Velikov wrote: > On 8 May 2017 at 11:52, Andres Gomez <ago...@igalia.com> wrote: > > On Mon, 2017-05-08 at 10:56 +0100, Emil Velikov wrote: > > > On 6 May 2017 at 15:09, Andres Gomez <ago...@igalia.com> wrote: > > > &g

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

On Mon, 2017-05-08 at 10:56 +0100, Emil Velikov wrote: > On 6 May 2017 at 15:09, Andres Gomez <ago...@igalia.com> wrote: > > If an identified commit was having more than one fix, we would warn > > about that and only treat the first. > > > > Now, we

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

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: Mauro my prov

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

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 <emil.veli...@collabora.com> > > Cc: Juan A. S

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

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

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

Completed the 17.1 cycle and added the beginning of the 17.2 one. Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Juan A. Suarez Romero <jasua...@igalia.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/release-calendar.html | 62 +++

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

Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Juan A. Suarez Romero <jasua...@igalia.com> Signed-off-by: Andres Gomez <ago...@igalia.com> --- docs/release-calendar.html | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/release-calendar.h

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

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] bin/get-fixes-pick-list.sh: don't warn if more than one, go over them

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 <ago...@igalia.com> --- bin/get-fixes-pick-list.sh | 51 +++--- 1 file c

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

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

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

Signed-off-by: Andres Gomez <ago...@igalia.com> --- 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-an

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

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

Signed-off-by: Andres Gomez <ago...@igalia.com> --- 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 +-- b

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

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 <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Tapani Pälli <tapani.pa...@intel

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

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

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> 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 buffe

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

v2: left code style/formatting corrections out. Signed-off-by: Andres Gomez <ago...@igalia.com> --- 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 1159

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

v2: Tapani as main contact and Mauro just for help with debugging/building (Mauro). Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Tapani Pälli <tapani.pa...@intel.com> Cc: Mauro Rossi <issor.or...@gmail.com> Acked-by:

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

v2: Brian Paul as main contact point and Jose Fonseca as fallback (Vinson, Jose) Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Vinson Lee <v...@freedesktop.org> Cc: Brian Paul <bri...@vmware.com> Cc: Jose Fons

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

ON && ./configure --prefix=$HOME/prefix && make > install); > fi > > + # Generate the header since one is missing on the Travis instance > + - mkdir -p linux > + - echo "#ifndef _LINUX_MEMFD_H" > linux/memfd.h > + - echo

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Fri, 2017-04-28 at 19:25 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Should make things a bit more consistent across the board. > > Cc: Eric Engestrom <e...@engestr

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Fri, 2017-04-28 at 19:25 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Provides a small, but consistent improvement. > Example numbers of the jobs added later in the series. &g

[Mesa-dev] [ANNOUNCE] mesa 17.0.5

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" cherry-igno

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

On Fri, 2017-04-28 at 19:27 +0100, Emil Velikov wrote: > On 28 April 2017 at 19:15, Emil Velikov <emil.l.veli...@gmail.com> wrote: > > On 28 April 2017 at 11:50, Andres Gomez <ago...@igalia.com> wrote: > > > Do we want to do this? > > > > > &

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

On Fri, 2017-04-28 at 17:43 +0100, Emil Velikov wrote: > On 28 April 2017 at 16:00, Andres Gomez <ago...@igalia.com> wrote: > > On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > > > From: Emil Velikov <emil.veli...@collabora.com> > > > > > >

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

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. > >

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

On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > 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 13/15] travis: add Gallium state-tracker targets

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 > -

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

libexpat1-dev > - libx11-xcb-dev > @@ -337,6 +345,10 @@ install: >- tar -jxvf $LIBVA_VERSION.tar.bz2 >- (cd $LIBVA_VERSION && ./configure --prefix=$HOME/prefix > --disable-wayland --disable-dummy-driver && make install) > > +

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

--disable-xa --disable-nine --disable-xvmc --disable-vdpau > --disable-va --disable-omx --disable-gallium-osmesa" > +# i915 most likely doesn't work with OpenCL. > +# Regardless - we're doing a quick build test here. > +- GALLIUM_DRIVERS="i915" > +- VULKAN_DRIVERS="" > + addons: > +apt: > + sources: > +- ubuntu-toolchain-r-test g++-4.7 comes with Trusty: http://packages.ubuntu.com/trusty/g++-4.7 Therefore, the line above is not needed. With that change, this is: Reviewed-by: Andres Gomez <ago...@igalia.com> -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

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

; +- ubuntu-toolchain-r-test > +- llvm-toolchain-trusty-3.9 > + packages: > +# From sources above > +- g++-5 > +- llvm-3.9-dev > +# Common > +- x11proto-xf86vidmode-dev > +

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

eview that lead to v2 but I wonder 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 <ago...@igalia.com> -- Br, Andres ___ mes

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

# Keep it symmetrical to the make build. > - SCONS_CHECK_COMMAND="scons llvm=1 check" > +- LLVM_VERSION=3.3 > +- LLVM_CONFIG="llvm-config-${LLVM_VERSION}" > + addons: > +apt: > + sources: > +

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

> +- SCONS_TARGET="llvm=1" > +# Keep 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

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Split the "if test" blocks so that we get more sensible output in case > of a failure. > >

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > We effectively override libdrm-dev and libxcb-dri2-0-dev since we build > and install the package locally.

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

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

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > According to the manual > > "If you are using ccache, use: > > language: c # or oth

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-04-27 at 19:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Some of the libraries may be dlopened, which may not always work due to > the non-standard prefix that we'r

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

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-04-27 at 16:38 +0100, Emil Velikov wrote: > From: Emil Velikov <emil.veli...@collabora.com> > > Add a page that has information which release is expected when and > associated information. &

[Mesa-dev] Mesa 17.0.5 release candidate

n 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 ctx torn down with no

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

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

Re: [Mesa-dev] [PATCH v2 1/2] bin/get-{extra, fixes}-pick-list.sh: add support for ignore list

On Mon, 2017-04-10 at 12:15 +0200, Juan A. Suarez Romero wrote: > On Fri, 2017-04-07 at 19:38 +0100, Emil Velikov wrote: > > On 7 April 2017 at 12:30, Juan A. Suarez Romero wrote: > > > Both scripts does not use a file with the commits to ignore. So if we > > > have handled

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

Jose, is it OK that I mention you as a reference for Windows in the releasing doc? On Mon, 2017-03-27 at 12:03 -0700, Vinson Lee wrote: > On Mon, Mar 27, 2017 at 11:48 AM, Andres Gomez <ago...@igalia.com> wrote: > > Signed-off-by: Andres Gomez <ago...@igalia.com> > >

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

Brian, is it OK that I mention you as a reference for Windows in the releasing doc? On Mon, 2017-03-27 at 12:03 -0700, Vinson Lee wrote: > On Mon, Mar 27, 2017 at 11:48 AM, Andres Gomez <ago...@igalia.com> wrote: > > Signed-off-by: Andres Gomez <ago...@igalia.com> > >

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

Jeremy, is it OK that I mention you as a reference for MacOSX in the releasing doc? On Mon, 2017-03-27 at 21:48 +0300, Andres Gomez wrote: > Signed-off-by: Andres Gomez <ago...@igalia.com> > Cc: Emil Velikov <emil.veli...@collabora.com> > Cc: Jeremy Huddleston Sequoi

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

Mauro, is it OK that I mention you as a reference for Android in the releasing doc? On Mon, 2017-03-27 at 21:48 +0300, Andres Gomez wrote: > Signed-off-by: Andres Gomez <ago...@igalia.com> > Cc: Emil Velikov <emil.veli...@collabora.com> > Cc: Tapani Pälli <tapani.pa..

Re: [Mesa-dev] [PATCH 2/2] travis: add the txc-dxtn library so scons check can pass

On Thu, 2017-04-06 at 09:59 -0700, Eric Anholt wrote: > Andres Gomez <ago...@igalia.com> writes: > > > The packaged libtxc-dxtn-s2tc0 will make scons check fail. Manually > > remove this package and install libtxc-dxtn instead. > > I don't think we should be auto

Re: [Mesa-dev] [PATCH 2/2] travis: add the txc-dxtn library so scons check can pass

On Thu, 2017-04-06 at 01:35 -0400, Rhys Kidd wrote: > On 5 April 2017 at 13:27, Andres Gomez <ago...@igalia.com> wrote: > > The packaged libtxc-dxtn-s2tc0 will make scons check fail. Manually > > remove this package and install libtxc-dxtn instead. > > > >

[Mesa-dev] [PATCH 3/3] travis: add the possibility of using the txc-dxtn library

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. Signed-off-by: Andres Gomez <

[Mesa-dev] [PATCH 2/3] travis: remove the libtxc-dxtn-s2tc0 library so scons check can pass

-by: Andres Gomez <ago...@igalia.com> Cc: Eric Anholt <e...@anholt.net> Cc: Emil Velikov <emil.l.veli...@gmail.com> Cc: Rhys Kidd <rhysk...@gmail.com> --- .travis.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.travis.yml b/.travis.yml index 0667c7488d..3c51c06a73

[Mesa-dev] [PATCH 1/3] travis: replace Trusty-based LLVM toolchain apt-get with apt addon

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 <ago...@igalia.com> Cc: Rhys Kidd <rhysk...@gmail.com> Cc: Emil Velikov <emil.l.ve

[Mesa-dev] [PATCH v2 0/3] Make travis green

A couple of fixes to use a bit more standard syntax to pull the LLVM dependencies and to make scons check pass. Andres Gomez (3): travis: replace Trusty-based LLVM toolchain apt-get with apt addon travis: remove the libtxc-dxtn-s2tc0 library so scons check can pass travis: add

Re: [Mesa-dev] [PATCH 1/2] travis: replace Trusty-based LLVM toolchain apt-get with apt addon

gt; Dylan > > Quoting Andres Gomez (2017-04-05 10:27:30) > > Trusty's LLVM toochain repository was whitelisted some time ago. See: > > https://github.com/travis-ci/apt-source-whitelist/commit/ebc278924a0b5e282dac685683faf6a318dd424c > > > > Signed-off-by: Andres Gomez <a

Re: [Mesa-dev] [PATCH 2/2] travis: add the txc-dxtn library so scons check can pass

On Wed, 2017-04-05 at 18:05 +, Rhys Kidd wrote: > On Wed, Apr 5, 2017 at 1:49 PM Emil Velikov <emil.l.veli...@gmail.com> wrote: > > On 5 April 2017 at 18:27, Andres Gomez <ago...@igalia.com> wrote: > > > > > The packaged libtxc-dxtn-s2tc0 will make scons ch

[Mesa-dev] [PATCH 2/2] travis: add the txc-dxtn library so scons check can pass

The packaged libtxc-dxtn-s2tc0 will make scons check fail. Manually remove this package and install libtxc-dxtn instead. Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Eric Anholt <e...@anholt.net> Cc: Emil Velikov <emil.l.veli...@gmail.com> --- .travis.yml | 8

[Mesa-dev] [PATCH 1/2] travis: replace Trusty-based LLVM toolchain apt-get with apt addon

Trusty's LLVM toochain repository was whitelisted some time ago. See: https://github.com/travis-ci/apt-source-whitelist/commit/ebc278924a0b5e282dac685683faf6a318dd424c Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Rhys Kidd <rhysk...@gmail.com> Cc: Emil Velikov <emil.l.ve

[Mesa-dev] [PATCH 0/2] Make travis green

A couple of fixes to use a bit more standard syntax to pull the LLVM dependencies and to make scons check pass. Andres Gomez (2): travis: replace Trusty-based LLVM toolchain apt-get with apt addon travis: add the txc-dxtn library so scons check can pass .travis.yml | 19

Re: [Mesa-dev] [PATCH] travis: Add radeonsi to continuous integration

Hi, This is Reviewed-by: Andres Gomez <ago...@igalia.com> On Tue, 2017-04-04 at 23:56 -0400, Rhys Kidd wrote: > Signed-off-by: Rhys Kidd <rhysk...@gmail.com> > --- > .travis.yml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/.travis.yml

Re: [Mesa-dev] [PATCH] travis: Add radv vulkan driver to continuous integration

Hi, This is Reviewed-by: Andres Gomez <ago...@igalia.com> On Tue, 2017-04-04 at 23:58 -0400, Rhys Kidd wrote: > Signed-off-by: Rhys Kidd <rhysk...@gmail.com> > --- > .travis.yml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/.travis.yml b/.travis.yml &

[Mesa-dev] [ANNOUNCE] mesa 17.0.3

and some other fixes. radv, freedreno and clover include some few fixes too. On integration side - we had a swr build fix with llvm >= 5.0 and also a fix for building with MSVC. Alex Deucher (1):   radeonsi: add new polaris12 pci id Andres Gomez (6):   glsl: on UBO/SSBOs link error re

Re: [Mesa-dev] [Mesa-announce] Mesa 17.0.3 release candidate

On Thu, 2017-03-30 at 18:09 +0300, Andres Gomez wrote: ... > Rejected (2) > ... > Thomas Hellstrom (1): >   gbm/dri: Flush after unmap > > The commit caused a regression in i965 (and possibly others) since it This should have said i915 -^^^

[Mesa-dev] Mesa 17.0.3 release candidate

1a65dc43be3a4bf6b8a901586f718f4b6b3) Cheers, Andres Mesa stable queue - Nominated (1) = Craig Stout (1):   anv/cmd_buffer: fix host memory leak Queued (32) === Alex Deucher (1):   radeonsi: add new polaris12 pci id Andres Gomez (1):   glsl: o

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

Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Jeremy Huddleston Sequoia <jerem...@apple.com> --- docs/releasing.html | 5 + 1 file changed, 5 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html

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

Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Tapani Pälli <tapani.pa...@intel.com> Cc: Mauro Rossi <issor.or...@gmail.com> --- docs/releasing.html | 5 + 1 file changed, 5 insertions(+) diff --git a/do

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

Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> Cc: Vinson Lee <v...@freedesktop.org> --- docs/releasing.html | 5 + 1 file changed, 5 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html index ea0088342f8..

[Mesa-dev] [PATCH 4/7] docs/releasing: if possible, do some every day use on the RC

Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> --- docs/releasing.html | 6 ++ 1 file changed, 6 insertions(+) diff --git a/docs/releasing.html b/docs/releasing.html index 10a18790300..ea0088342f8 100644 --- a/docs/releasing.htm

[Mesa-dev] [PATCH 3/7] docs/releasing: further explain the build/check testing process

The build/check test should be done with an appropriate combination of flags, depending on the changes introduced by the patch set. Also, mention to cross compile with mingw-w64 for Windows. Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com&g

[Mesa-dev] [PATCH 2/7] docs/releasing: check in master for forgotten nomination candidates

The maintanier should not just rely on the mesa-stable@ mailing list but actually check the master branch in search for suitable nomination candidates. Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> --- docs/releasing.html | 9 +

[Mesa-dev] [PATCH 1/7] docs/releasing: format/style homogenization

Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Emil Velikov <emil.veli...@collabora.com> --- docs/releasing.html | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/docs/releasing.html b/docs/releasing.html index fc7f2a5aff9.

[Mesa-dev] [PATCH 0/7] Adding more documentation to the release process

A series of patches adding more details to the release process. Andres Gomez (7): docs/releasing: format/style homogenization docs/releasing: check in master for forgotten nomination candidates docs/releasing: further explain the build/check testing process docs/releasing: if possible, do

Re: [Mesa-dev] [PATCH 3/3] glsl: Interface Block instances don't need linking validation

On Sun, 2017-03-26 at 21:28 +1100, Timothy Arceri wrote: > Patches 2 & 3 are: > > Reviewed-by: Timothy Arceri Thanks! I've already pushed this. Additionally, could you take a look to this piglit patch related to patch 3 in this series?

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Sun, 2017-03-26 at 21:08 +1100, Timothy Arceri wrote: > Ok. So lets start again. :) ... > With those this is: > > Reviewed-by: Timothy Arceri I'll apply the changes. Thanks a lot for your patience during the reviewing process ☺ -- Br, Andres

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Sun, 2017-03-12 at 16:34 +1100, Timothy Arceri wrote: > > On 12/03/17 12:17, Andres Gomez wrote: > > On Sun, 2017-03-12 at 10:07 +1100, Timothy Arceri wrote: > > > > > > On 12/03/17 00:29, Andres Gomez wrote: > > > > On Sat, 2017-

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Sun, 2017-03-12 at 10:07 +1100, Timothy Arceri wrote: > > On 12/03/17 00:29, Andres Gomez wrote: > > On Sat, 2017-03-11 at 23:24 +1100, Timothy Arceri wrote: > > > > > > On 10/03/17 18:53, Andres Gomez wrote: > > > > > > > > Accordi

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Sat, 2017-03-11 at 23:24 +1100, Timothy Arceri wrote: > > On 10/03/17 18:53, Andres Gomez wrote: > > > > According with that text it would be OK to report a different number > > than 0 when asking for the active UBOs to the failed link program but > > it still wi

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Fri, 2017-03-10 at 14:28 +1100, Timothy Arceri wrote: > > On 10/03/17 08:46, Andres Gomez wrote: > > On Fri, 2017-03-10 at 08:32 +1100, Timothy Arceri wrote: > > > On 23/02/17 19:55, Andres Gomez wrote: > > > > Commit f1293b2f9bc3 split apart buffer block a

Re: [Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

On Fri, 2017-03-10 at 08:32 +1100, Timothy Arceri wrote: > On 23/02/17 19:55, Andres Gomez wrote: > > Commit f1293b2f9bc3 split apart buffer block arrays but introduced > > also the possibility of a recount of active > > blocks (NumUniformBlocks/NumShaderStora

Re: [Mesa-dev] [PATCH] glsl: add missing blend_support qualifier in validate_flags()

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-02-23 at 19:17 +0100, Samuel Pitoiset wrote: > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > src/compiler/glsl/ast_type.cpp | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(

Re: [Mesa-dev] [PATCH] glsl: use an enum for AMD_conservative_depth layout qualifiers

>data.depth_layout = ir_depth_layout_greater; > - else if (qual->flags.q.depth_less) > + else if (qual->depth_type == ast_depth_less) >var->data.depth_layout = ir_depth_layout_less; > - else if (qual->flags.q.depth_unchanged) > + else if (qual->depth_type == ast_depth_unchanged) > var->data.depth_layout = ir_depth_layout_unchanged; > else > var->data.depth_layout = ir_depth_layout_none; Maybe replace this with a ? switch(qual->depth_type) { case ast_depth_any: ... } Other than that nitpick, this is: Reviewed-by: Andres Gomez <ago...@igalia.com> -- Br, Andres ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] glsl: Fix missing-braces warning.

This is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Thu, 2017-02-23 at 22:31 +, Vinson Lee wrote: > CXXglsl/ast_to_hir.lo > glsl/ast_to_hir.cpp: In member function 'virtual ir_rvalue* > ast_declarator_list::hir(exec_list*, _mesa_glsl_parse_state*)': > glsl/ast_

Re: [Mesa-dev] [PATCH] glsl: Raise a link error for non-SSO ES programs with a TES but no TCS.

I would welcome a reference to the text in Secption 7.3 of the OpenGL ES 3.2 specs as a code comment and commit text but, other than that, this is: Reviewed-by: Andres Gomez <ago...@igalia.com> On Wed, 2017-02-22 at 17:54 -0800, Kenneth Graunke wrote: > OpenGL allows the TCS to b

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

Signed-off-by: Andres Gomez <ago...@igalia.com> --- src/compiler/glsl/ast_to_hir.cpp | 37 + 1 file changed, 21 insertions(+), 16 deletions(-) diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp index b90ad97b1de..d7a2b

[Mesa-dev] [PATCH 2/3] glsl: UBOs and SSBOs must match the binding qualifier too

} uni_block2; ..." Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Ian Romanick <ian.d.roman...@intel.com> --- src/compiler/glsl/link_uniform_blocks.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/compiler/glsl/link_uniform_blocks.cpp b/src/compiler/glsl/link_unif

[Mesa-dev] [PATCH 0/3] Validate IB bindings through its name, no instance

ld succeed on linking: "# VS layout(binding = 1) Block1 { vec4 color; } uni_block; ... # FS layout(binding = 2) Block2 { vec4 color; } uni_block; ..." Currently, it fails. Fixes: - GL45-CTS.enhanced_layouts.ssb_layout_qualifier_conflict Andr

[Mesa-dev] [PATCH 1/3] glsl: on UBO/SSBOs link error, the number of active blocks remains 0

a segmentation fault if trying to use the index of a block in a link failed program. Signed-off-by: Andres Gomez <ago...@igalia.com> Cc: Timothy Arceri <tarc...@itsqueeze.com> --- src/compiler/glsl/linker.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/glsl/link

<    1   2   3   4   5   6   7   >