Re: [Mesa-dev] [PATCH 1/2] meson: Fix configuring dri glx with only gallium drivers

2018-01-13 Thread Dylan Baker
meson considers classic swrast to be a dri driver, I know it's not exactly accurate, but, at least for me, it made the build system easier to reason about. Quoting Adam Jackson (2018-01-12 09:06:37) > On Fri, 2018-01-12 at 13:18 +, Jon Turney wrote: > > 'meson -Ddri-drivers= -Dgallium-drivers=

Re: [Mesa-dev] [PATCH 1/2] meson: Fix configuring dri glx with only gallium drivers

2018-01-13 Thread Dylan Baker
Maybe this is correct, but it makes me nervous treating with_gallium as equivalent to with_dri, since gallium drivers can be built dri-less (gallium-xlib, and some other configurations on windows). I think something like: with_glx = get_option('glx') if with_glx == 'auto' if with_dri

[Mesa-dev] [PATCH 0/2] Small fixes for the meson build

2018-01-13 Thread Dylan Baker
Here's a few things I've caught as I've started trying to add the meson build to our CI system. Dylan Baker (2): meson: set proper pkg-config version for libdrm_freedreno meson: set the minimum version correctly freedreno/meson.build | 2 +- meson.build | 2 +- 2 f

[Mesa-dev] [PATCH 1/2] meson: set proper pkg-config version for libdrm_freedreno

2018-01-13 Thread Dylan Baker
Copy and paste error from exynos. Signed-off-by: Dylan Baker --- freedreno/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/freedreno/meson.build b/freedreno/meson.build index b4035e1..de6a413 100644 --- a/freedreno/meson.build +++ b/freedreno/meson.build @@ -64,7

Re: [Mesa-dev] [PATCH 5/7] report.py: Add option to only display measurements that have changes

2018-01-13 Thread Dylan Baker
Quoting Ian Romanick (2018-01-12 12:06:59) > From: Ian Romanick > > This is useful for preparing data to go in a Mesa commit message. > > Signed-off-by: Ian Romanick > --- > report.py | 53 +++-- > 1 file changed, 31 insertions(+), 22 deletions(-

Re: [Mesa-dev] [PATCH 4/7] report.py: Add option to only display the final summary

2018-01-13 Thread Dylan Baker
Quoting Ian Romanick (2018-01-12 12:06:58) > From: Ian Romanick > > This is useful for preparing data to go in a Mesa commit message. > > Signed-off-by: Ian Romanick > --- > report.py | 60 > 1 file changed, 32 insertions(+), 28 dele

[Mesa-dev] [PATCH 2/2] meson: set the minimum version correctly

2018-01-13 Thread Dylan Baker
Currently we ask for 0.42, but we actually require 0.43 because we pass file objects as arguments to tests. If someone needs version 0.42 it wouldn't be hard, just a lot of replacing files() with strings. Signed-off-by: Dylan Baker --- meson.build | 2 +- 1 file changed, 1 insertion(

Re: [Mesa-dev] [PATCH] glx: fix non-dri build

2018-01-14 Thread Dylan Baker
I don't know enough about glx to know if this is correct, but you'll need to add the c files to the meson.build as well On January 13, 2018 3:36:30 AM PST, Samuel Thibault wrote: >glXGetDriverConfig parameters do not provide a context to dynamically >check for the presence of the function, so t

[Mesa-dev] [PATCH] meson: Don't confuse the install and search paths for dri drivers

2018-01-17 Thread Dylan Baker
. Reported-by: Ilia Mirkin Signed-off-by: Dylan Baker --- meson.build | 6 ++ meson_options.txt | 8 +++- src/egl/meson.build | 2 +- src/gbm/meson.build | 2 +- src/glx/meson.build | 3 ++- 5 files changed, 17 insertions(+), 4 deletions(-) diff --git a/meson.build b/meson.build

Re: [Mesa-dev] [PATCH] meson: add llvm dependency for swr build

2018-01-17 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting George Kyriazis (2018-01-16 13:25:45) > cc: Dylan Baker > --- > src/gallium/drivers/swr/meson.build | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/gallium/drivers/swr/meson.build > b/src/gallium/drivers/swr/meson.build >

Re: [Mesa-dev] [PATCH 5/7] report.py: Add option to only display measurements that have changes

2018-01-17 Thread Dylan Baker
Quoting Ian Romanick (2018-01-16 09:39:07) > On 01/12/2018 12:23 PM, Dylan Baker wrote: > > Quoting Ian Romanick (2018-01-12 12:06:59) > >> From: Ian Romanick > >> > >> This is useful for preparing data to go in a Mesa commit message. &

Re: [Mesa-dev] [PATCH 1/2] meson: Fix configuring dri glx with only gallium drivers

2018-01-17 Thread Dylan Baker
Your attached patch looks good to me. You can add my r-b to it. Quoting Jon Turney (2018-01-15 11:50:04) > On 12/01/2018 17:33, Dylan Baker wrote: > > Maybe this is correct, but it makes me nervous treating with_gallium as > > equivalent to with_dri, since gallium drivers can be

Re: [Mesa-dev] [PATCH 00/14] BSD portability (Meson, ANV, RADV, VC4/5, SWR)

2018-01-17 Thread Dylan Baker
Just wanted to see how things were going on this, especially for the meson bits as I'd like to get them merged before the 18.0 merge window closes. Quoting Greg V (2017-12-31 08:55:14) > Hello everyone and happy new year! :) > > This set of patches makes the Meson build work on FreeBSD, including

Re: [Mesa-dev] [Test Patch] Meson: ensure vdpau has proper symbols exposed

2018-01-18 Thread Dylan Baker
I dug into what autotools is doing. It's using -Wl,--whole-archive to link everything, in this target, not just the state tracker. I think I'm okay with this patch given that, so I'm going to apply the same change to the other media state trackers and send it out as a series. Quot

[Mesa-dev] [PATCH 2/6] meson: fix vdpau target linkage

2018-01-18 Thread Dylan Baker
The VDPAU state tracker needs to be linked with whole-archive (autotools does this), and a few dependeies can be cleaned up Signed-off-by: Dylan Baker --- src/gallium/targets/vdpau/meson.build | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 5/6] meson: fix xa target linkage

2018-01-18 Thread Dylan Baker
This needs to link the state-tracker with whole archive, and needs a few extra window-system deps. Signed-off-by: Dylan Baker --- src/gallium/targets/xa/meson.build | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/gallium/targets/xa/meson.build b/src

Re: [Mesa-dev] [PATCH 2/6] st/omx/tizonia: Add --enable-omx-tizonia flag and build files

2018-01-18 Thread Dylan Baker
The meson build for omx-bellagio has landed along with the other media state trackers that are in tree. Currently there is a known bug with at least vdpau and va-api (I suspect that the omx, va, and xvmc are affected as well) which causes required symbols to not be exported, I'm still trying to res

[Mesa-dev] [PATCH 4/6] meson: Fix omx-bellagio target linkage

2018-01-18 Thread Dylan Baker
ff-by: Dylan Baker --- meson.build | 5 - src/gallium/targets/omx-bellagio/meson.build | 18 -- 2 files changed, 12 insertions(+), 11 deletions(-) diff --git a/meson.build b/meson.build index ae31cdd..bd4afdf 100644 --- a/meson.build +++ b/meson.

[Mesa-dev] [PATCH 0/6] meson: fix gallium media target linkage

2018-01-18 Thread Dylan Baker
The linkage of all of the gallium media targets is broken in various ways in the meson build. This series should correct that by doing more what the autotools build does. Dylan Baker (6): meson: link dri3 xcb libs into vlwinsys instead of into each target meson: fix vdpau target linkage

[Mesa-dev] [PATCH 6/6] meson: fix xvmc target linkage

2018-01-18 Thread Dylan Baker
This needs to link the state tracker with whole-archive, it needs to link with libxvmc, and it needs a few extra window system dependencies. Signed-off-by: Dylan Baker --- meson.build | 5 - src/gallium/targets/xvmc/meson.build | 18 -- 2 files

[Mesa-dev] [PATCH 3/6] meson: fix va-api target linkage

2018-01-18 Thread Dylan Baker
The state tracker needs to be linked with whole-archive (like autotools), and we need a few window system deps. Signed-off-by: Dylan Baker --- src/gallium/targets/va/meson.build | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/src/gallium/targets/va

Re: [Mesa-dev] [PATCH] meson: Don't confuse the install and search paths for dri drivers

2018-01-18 Thread Dylan Baker
Quoting Eric Anholt (2018-01-17 12:32:32) > Dylan Baker writes: > > > Currently there is not a separate option for setting the search path of > > DRI drivers in meson, like there is in scons and autotools. This is an > > oversight and needs to be fixed. This adds an extr

[Mesa-dev] [PATCH 1/6] meson: link dri3 xcb libs into vlwinsys instead of into each target

2018-01-18 Thread Dylan Baker
This makes the dependencies easier to manage, since each media target doesn't need to worry about linking to half a dozen libraries. Signed-off-by: Dylan Baker --- src/gallium/auxiliary/meson.build | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxi

Re: [Mesa-dev] [PATCH] meson: generate translations for driconf

2018-01-18 Thread Dylan Baker
ping Quoting Dylan Baker (2018-01-09 11:36:36) > Currently meson implements the same logic as SCons for translations, > namely it doesn't do them. This patch changes meson to use logic more > like autotools, and generate translations. To do this we have to go > behind meson'

Re: [Mesa-dev] Mesa 18.0.0 release plan

2018-01-18 Thread Dylan Baker
I'd like to get meson into the tarball for this release. There's a couple of patches on the list that will need to land, but I can CC those to stable if they don't land before the branch on Friday. Dylan Quoting Emil Velikov (2018-01-16 11:18:13) > Hi all, > > As you've know the Mesa 18.0.0 rele

Re: [Mesa-dev] [PATCH 00/14] BSD portability (Meson, ANV, RADV, VC4/5, SWR)

2018-01-18 Thread Dylan Baker
No worries, December is a busy time of year. Quoting Greg V (2018-01-17 12:57:39) > On 01/17/2018 00:59, Dylan Baker wrote: > > Just wanted to see how things were going on this, especially for the meson > > bits > > as I'd like to get them merged before the 18.0 merge

Re: [Mesa-dev] [PATCH] meson: generate translations for driconf

2018-01-18 Thread Dylan Baker
Quoting Eric Anholt (2018-01-17 12:58:27) > Dylan Baker writes: > > > Currently meson implements the same logic as SCons for translations, > > namely it doesn't do them. This patch changes meson to use logic more > > like autotools, and generate translations. To do

[Mesa-dev] [PATCH] autotools: include meson build files in tarball

2018-01-18 Thread Dylan Baker
This adds the meson.build, meson_options.txt, and a few scripts that are used exclusively by the meson build. Signed-off-by: Dylan Baker --- Makefile.am | 7 ++- src/Makefile.am | 2 +- src/amd/Makefile.am

[Mesa-dev] [PATCH v2] meson: Don't confuse the install and search paths for dri drivers

2018-01-18 Thread Dylan Baker
. v2: - Split input list before joining. Reported-by: Ilia Mirkin Signed-off-by: Dylan Baker --- meson.build | 6 ++ meson_options.txt | 8 +++- src/egl/meson.build | 2 +- src/gbm/meson.build | 2 +- src/glx/meson.build | 3 ++- 5 files changed, 17 insertions(+), 4 deletions

[Mesa-dev] [PATCH] meson: ensure that xmlpool_options.h is generated for targets that need it

2018-01-18 Thread Dylan Baker
Currently a couple of gallium targets race with xmlpool_options.h being generated, don't do that. Signed-off-by: Dylan Baker --- src/gallium/auxiliary/pipe-loader/meson.build | 4 ++-- src/gallium/targets/d3dadapter9/meson.build | 2 +- src/gallium/targets/pipe-loader/meson.build

Re: [Mesa-dev] [PATCH 02/14] vc4, vc5: add ETIME fallback define

2018-01-18 Thread Dylan Baker
I'm not sure how Emil feels about this (and the related patches) but I think it might be better to handle this at the build system level, for meson it would look something like (in the top level meson.build): if cc.get_define('ETIME') == '' pre_args += '-DETIME=ETIMEDOUT' endif Which should be

Re: [Mesa-dev] [PATCH v2 3/4] meson: fix getting cflags from pkg-config

2018-01-18 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting Greg V (2018-01-17 12:54:17) > get_pkgconfig_variable('cflags') always returns an empty list, it's a > function for getting *custom* variables. > > Meson does not yet support asking for cflags, so explicitly invoke > pkg-config for

Re: [Mesa-dev] [PATCH v2 4/4] meson: fix missing dependencies

2018-01-18 Thread Dylan Baker
Quoting Greg V (2018-01-17 12:54:18) > --- > meson.build | 4 > src/gallium/drivers/radeonsi/meson.build | 2 +- > src/mesa/meson.build | 2 +- > 3 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/meson.build b/meson.build > ind

Re: [Mesa-dev] [PATCH 02/14] vc4, vc5: add ETIME fallback define

2018-01-18 Thread Dylan Baker
Forgot to CC Emil... Quoting Dylan Baker (2018-01-18 09:13:13) > I'm not sure how Emil feels about this (and the related patches) but I think > it > might be better to handle this at the build system level, for meson it would > look something like (in the top level meso

Re: [Mesa-dev] [PATCH 3/6] meson: fix va-api target linkage

2018-01-18 Thread Dylan Baker
Quoting Marc Dietrich (2018-01-18 04:03:21) > Am Mittwoch, 17. Januar 2018, 19:34:48 CET schrieb Dylan Baker: > > The state tracker needs to be linked with whole-archive (like > > autotools), and we need a few window system deps. > > > > Signed-off-by: Dylan Bake

[Mesa-dev] [PATCH 7/6] meson: use va-api version reported by pkg-config

2018-01-18 Thread Dylan Baker
Signed-off-by: Dylan Baker --- meson.build | 2 +- src/gallium/state_trackers/va/meson.build | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/meson.build b/meson.build index 1ec9e4c2221..c95c44641f9 100644 --- a/meson.build +++ b

Re: [Mesa-dev] [PATCH] meson: fix the install path of amdgpu.ids

2018-01-18 Thread Dylan Baker
Reviewed-by: Dylan Baker I'll go ahead and push this. Quoting Christoph Haag (2018-01-18 10:01:55) > --- > amdgpu/meson.build | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/amdgpu/meson.build b/amdgpu/meson.build > index 55ab9d1d..8b04520

Re: [Mesa-dev] [PATCH 3/6] meson: fix va-api target linkage

2018-01-18 Thread Dylan Baker
Quoting Marc Dietrich (2018-01-18 04:03:21) > Am Mittwoch, 17. Januar 2018, 19:34:48 CET schrieb Dylan Baker: > > The state tracker needs to be linked with whole-archive (like > > autotools), and we need a few window system deps. > > > > Signed-off-by: Dylan Bake

Re: [Mesa-dev] [PATCH] anv_icd.py: improve reproducible builds

2018-01-18 Thread Dylan Baker
Reviewed-by: Dylan Baker Quoting maxin.j...@gmail.com (2018-01-18 03:33:37) > From: "Maxin B. John" > > Sort the output to ensure build reproducibility > > Signed-off-by: Maxin B. John > --- > src/intel/vulkan/anv_icd.py | 2 +- > 1 file changed, 1 ins

Re: [Mesa-dev] [PATCH v2 2/4] meson: handle LLVM 'x.x.xgit-revision' versions

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 10:22:50) > On 17 January 2018 at 20:54, Greg V wrote: > > When LLVM is built inside of a git repo (even way below, e.g. > > /usr/ports/.git > > exists, and LLVM is built in /usr/ports/devel/llvm50/work), its version > > becomes something like 5.0.0git-f8ab206b217

Re: [Mesa-dev] [PATCH 0/6] meson: fix gallium media target linkage

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 11:12:30) > On 17 January 2018 at 18:34, Dylan Baker wrote: > > The linkage of all of the gallium media targets is broken in various > > ways in the meson build. This series should correct that by doing more > > what the autotools build does

Re: [Mesa-dev] [PATCH v2 3/4] meson: fix getting cflags from pkg-config

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 10:26:20) > On 17 January 2018 at 20:54, Greg V wrote: > > get_pkgconfig_variable('cflags') always returns an empty list, it's a > > function for getting *custom* variables. > > > > Meson does not yet support asking for cflags, so explicitly invoke > > pkg-config f

Re: [Mesa-dev] [PATCH 02/14] vc4, vc5: add ETIME fallback define

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 10:40:40) > On 18 January 2018 at 17:13, Dylan Baker wrote: > > I'm not sure how Emil feels about this (and the related patches) but I > > think it > > might be better to handle this at the build system level, for meson it would >

Re: [Mesa-dev] [PATCH 12/14] swr: build on FreeBSD/DragonFlyBSD

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 10:50:05) > On 18 January 2018 at 18:09, Greg V wrote: > > On 01/18/2018 21:01, Emil Velikov wrote: > >> > >> On 31 December 2017 at 16:55, Greg V wrote: > >>> > >>> (Needs CPU topology detection to actually work) > >> > >> IMHO it might be better to drop the patc

Re: [Mesa-dev] [PATCH 00/14] BSD portability (Meson, ANV, RADV, VC4/5, SWR)

2018-01-18 Thread Dylan Baker
Quoting Emil Velikov (2018-01-18 10:57:02) > On 18 January 2018 at 18:29, Greg V wrote: > > > > On 01/18/2018 21:18, Emil Velikov wrote: > >> > >> Hi Greg, > >> > >> On 31 December 2017 at 16:55, Greg V wrote: > >>> > >>> Hello everyone and happy new year! :) > >>> > >>> This set of patches makes

[Mesa-dev] [PATCH v2] autotools: include meson build files in tarball

2018-01-18 Thread Dylan Baker
This adds the meson.build, meson_options.txt, and a few scripts that are used exclusively by the meson build. v2: - Remove accidentally included changes needed to test make dist with LLVM > 3.9 Signed-off-by: Dylan Baker Acked-by: Eric Engestrom --- Makefile

Re: [Mesa-dev] [PATCH] autotools: include meson build files in tarball

2018-01-18 Thread Dylan Baker
Quoting Eric Engestrom (2018-01-18 13:04:00) > On Thursday, 2018-01-18 17:09:04 +0000, Dylan Baker wrote: > > This adds the meson.build, meson_options.txt, and a few scripts that are > > used exclusively by the meson build. > > > > Signed-off-by: Dylan Baker > >

Re: [Mesa-dev] [PATCH 00/14] BSD portability (Meson, ANV, RADV, VC4/5, SWR)

2018-01-19 Thread Dylan Baker
Quoting Emil Velikov (2018-01-19 08:26:41) > On 18 January 2018 at 21:30, Dylan Baker wrote: > > Quoting Emil Velikov (2018-01-18 10:57:02) > >> On 18 January 2018 at 18:29, Greg V wrote: > >> > > >> > On 01/18/2018 21:18, Emil Velikov wrote: >

Re: [Mesa-dev] [PATCH 00/14] BSD portability (Meson, ANV, RADV, VC4/5, SWR)

2018-01-19 Thread Dylan Baker
Quoting Daniel Stone (2018-01-19 08:50:17) > Hi, > > On 19 January 2018 at 16:26, Emil Velikov wrote: > > On 18 January 2018 at 21:30, Dylan Baker wrote: > >> I'm willing to defer to you on this, if you think that checking headers for > >> ETIME is

[Mesa-dev] [RFC 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-20 Thread Dylan Baker
It is possible to have DEBUG disabled but asserts on, which cannot build because these asserts work on members that are only present when DEBUG is on. Signed-off-by: Dylan Baker --- src/util/ralloc.c | 8 1 file changed, 8 insertions(+) diff --git a/src/util/ralloc.c b/src/util

[Mesa-dev] [RFC 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-20 Thread Dylan Baker
This builds and installs, but I haven't had a chance to test it yet. Signed-off-by: Dylan Baker --- meson.build | 17 ++ src/{ => amd/addrlib}/meson.build | 69 - src/{ => amd/common}/meson.build | 65 s

[Mesa-dev] [RFC 1/4] intel: use a flag instead of setting PYTHONPATH

2017-09-20 Thread Dylan Baker
Meson doesn't allow setting environment variables for custom targets, so we either need to not pass this as an environment variable or use a shell script to wrap the invocation. The chosen solution has the advantage of working for both autotools and meson. Signed-off-by: Dylan Baker ---

[Mesa-dev] [RFC 3/4] meson: Add build Intel "anv" vulkan driver

2017-09-20 Thread Dylan Baker
pendency of a target that will actually be installed) this allows us to avoid massive if chains, while ensuring that only the bits that need to be built are. Signed-off-by: Dylan Baker --- include/meson.build | 22 ++ meson.build

[Mesa-dev] initial meson port

2017-09-20 Thread Dylan Baker
Hi everyone, A long time ago I made some rumbling about porting mesa to meson (isn't that confusing). In the mean time I've been bogged down with other projects, including adding features to meson itself, and trying to write and rebase meson patches for all of mesa. Unfortunately mesa is a large f

Re: [Mesa-dev] [RFC 1/4] intel: use a flag instead of setting PYTHONPATH

2017-09-20 Thread Dylan Baker
Quoting Kenneth Graunke (2017-09-20 13:44:58) > On Wednesday, September 20, 2017 1:27:38 PM PDT Dylan Baker wrote: > > Meson doesn't allow setting environment variables for custom targets, so > > we either need to not pass this as an environment variable or use a > >

Re: [Mesa-dev] [RFC 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-20 Thread Dylan Baker
Quoting Kenneth Graunke (2017-09-20 13:42:45) > On Wednesday, September 20, 2017 1:27:39 PM PDT Dylan Baker wrote: > > It is possible to have DEBUG disabled but asserts on, which cannot build > > because these asserts work on members that are only present when DEBUG > > is on

Re: [Mesa-dev] [RFC 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-20 Thread Dylan Baker
of this for android eventually. Dylan Quoting Nicolai Hähnle (2017-09-20 13:57:17) > On 20.09.2017 22:27, Dylan Baker wrote: > > +amd_common_files = files( > > + 'ac_binary.c', > > + 'ac_binary.h', > > + 'ac_exp_param.h', > > + &#x

Re: [Mesa-dev] [PATCH 3/6] vulkan: enum generator: make registry more flexible

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:20) > From: Lionel Landwerlin > > It will be used to store extension numbers as well. > > Signed-off-by: Lionel Landwerlin > Reviewed-by: Jason Ekstrand > --- > src/vulkan/util/gen_enum_to_str.py | 22 +++--- > 1 file changed, 11 insert

Re: [Mesa-dev] [PATCH 4/6] vulkan: enum generator: generate extension number defines

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:21) > From: Lionel Landwerlin > > New extensions can introduce additional enums. Most of the new enums > will have disjoint numbers from the initial enums. For example new > formats introduced by VK_IMG_format_pvrtc : > > VK_FORMAT_ASTC_10x8_UNORM_BLOCK =

Re: [Mesa-dev] [PATCH 5/6] vulkan: enum generator: Stop using iterparse

2017-09-21 Thread Dylan Baker
Quoting Jason Ekstrand (2017-09-21 08:32:22) > While using iterparse is potentially a little more efficient, the Vulkan > registry XML is not large and using regular element tree simplifies the > parsing logic substantially. > --- > src/vulkan/util/gen_enum_to_str.py | 41 > +++---

Re: [Mesa-dev] [PATCH v3 1/6] vulkan: enum generator: align function declarations/prototypes

2017-09-21 Thread Dylan Baker
Thanks for updating. For the series: Reviewed-by: Dylan Baker Quoting Jason Ekstrand (2017-09-21 14:19:43) > From: Lionel Landwerlin > > Signed-off-by: Lionel Landwerlin > Acked-by: Jason Ekstrand > --- > src/vulkan/util/gen_enum_to_str.py | 42 > +++---

Re: [Mesa-dev] Meson mesademos (Was: [RFC libdrm 0/2] Replace the build system with meson)

2017-09-22 Thread Dylan Baker
Quoting Nirbheek Chauhan (2017-09-22 10:47:51) > So it seems to me that wrapping binaries is the only major feature > that is being an obstacle for you to port mesademos? I will publish an > example for doing that via subprojects, that should help! > > Cheers, > Nirbheek That would be very useful

Re: [Mesa-dev] Meson mesademos (Was: [RFC libdrm 0/2] Replace the build system with meson)

2017-09-22 Thread Dylan Baker
Quoting Jose Fonseca (2017-09-22 11:10:34) > On 22/09/17 18:47, Nirbheek Chauhan wrote: > > On Fri, Sep 22, 2017 at 10:19 PM, Jose Fonseca wrote: > >> cmake provides a generic mechanism to set any variable, either from a > >> command line -DFOO=boo, or via cache files. But meson didn't provide su

Re: [Mesa-dev] Meson mesademos (Was: [RFC libdrm 0/2] Replace the build system with meson)

2017-09-22 Thread Dylan Baker
Quoting Jose Fonseca (2017-09-22 11:58:17) > On 22/09/17 19:38, Dylan Baker wrote: > > Quoting Jose Fonseca (2017-09-22 11:10:34) > >> On 22/09/17 18:47, Nirbheek Chauhan wrote: > >>> On Fri, Sep 22, 2017 at 10:19 PM, Jose Fonseca > >>> wrote: > >&

[Mesa-dev] [PATCH v2 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-23 Thread Dylan Baker
It is possible to have DEBUG disabled but asserts on (NDEBUG(, which cannot build because these asserts work on members that are only present when DEBUG is on. Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/util/ralloc.c | 8 1 file changed, 8 insertions(+) diff

[Mesa-dev] [PATCH v2 1/4] intel: use a flag instead of setting PYTHONPATH

2017-09-23 Thread Dylan Baker
(Ken) Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/intel/Makefile.compiler.am | 2 +- src/intel/compiler/brw_nir_trig_workarounds.py | 33 +++--- 2 files changed, 26 insertions(+), 9 deletions(-) diff --git a/src/intel/Makefile.compiler.am

[Mesa-dev] [PATCH v2 3/4] meson: Add build Intel "anv" vulkan driver

2017-09-23 Thread Dylan Baker
pendency of a target that will actually be installed) this allows us to avoid massive if chains, while ensuring that only the bits that need to be built are. v2: - enable anv, x11, and wayland by default - add configure option to disable valgrind Signed-off-by: Dylan Baker ---

[Mesa-dev] [PATCH v2 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-23 Thread Dylan Baker
This builds and installs, but I haven't had a chance to test it yet. v2: - enable radv by default - add shader cache support and enforce that it's built for radv Signed-off-by: Dylan Baker --- meson.build | 27 - meson_options.txt

Re: [Mesa-dev] [PATCH v2 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-23 Thread Dylan Baker
I have tested this, and as of v2 it actually works. I'll update the commitment message. Dylan On September 23, 2017 8:39:06 AM PDT, Dylan Baker wrote: >This builds and installs, but I haven't had a chance to test it yet. > >v2: - enable radv by default >- add sh

Re: [Mesa-dev] [PATCH v2 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-25 Thread Dylan Baker
Quoting Nicholas Miell (2017-09-23 21:17:48) > On 09/23/2017 08:39 AM, Dylan Baker wrote: > > diff --git a/meson_options.txt b/meson_options.txt > > index e52cec31f11..854cba851d7 100644 > > --- a/meson_options.txt > > +++ b/meson_options.txt > > @@ -20,8 +20,10

Re: [Mesa-dev] [PATCH v2 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-25 Thread Dylan Baker
21:16:36) > Hi Dylan, > > Awesome work. I noticed though that when llvm-config gives 6.0.0svn we > don't strip the svn away when passing to HAVE_LLVM and > MESA_LLVM_VERSION_PATCH, which results in compile errors. > > - Bas > > On Sun, Sep 24, 2017 at 12:08 AM, Dyla

[Mesa-dev] [PATCH v3 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-26 Thread Dylan Baker
setting the version preprocessor flag (Bas) - Check for LLVM module requirements Signed-off-by: Dylan Baker --- meson.build | 40 +++- meson_options.txt | 4 +- src/{ => amd/addrlib}/meson.build | 69 - src/{ => amd/

[Mesa-dev] [PATCH v3 1/4] intel: use a flag instead of setting PYTHONPATH

2017-09-26 Thread Dylan Baker
(Ken) Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/intel/Makefile.compiler.am | 2 +- src/intel/compiler/brw_nir_trig_workarounds.py | 33 +++--- 2 files changed, 26 insertions(+), 9 deletions(-) diff --git a/src/intel/Makefile.compiler.am

[Mesa-dev] [PATCH v3 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-26 Thread Dylan Baker
It is possible to have DEBUG disabled but asserts on (NDEBUG(, which cannot build because these asserts work on members that are only present when DEBUG is on. Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/util/ralloc.c | 8 1 file changed, 8 insertions(+) diff

[Mesa-dev] [PATCH v3 3/4] meson: Add build Intel "anv" vulkan driver

2017-09-26 Thread Dylan Baker
pendency of a target that will actually be installed) this allows us to avoid massive if chains, while ensuring that only the bits that need to be built are. v2: - enable anv, x11, and wayland by default - add configure option to disable valgrind v3: - fix typo in meson_options (Nicholas)

Re: [Mesa-dev] [PATCH v3 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-26 Thread Dylan Baker
Quoting Nirbheek Chauhan (2017-09-26 12:29:30) > On Tue, Sep 26, 2017 at 9:46 PM, Dylan Baker wrote: > > This builds and installs, but I haven't had a chance to test it yet. > > > > v2: - enable radv by default > > - add shader cache support and enforce that i

Re: [Mesa-dev] [PATCH v3 3/4] meson: Add build Intel "anv" vulkan driver

2017-09-26 Thread Dylan Baker
Quoting Eric Anholt (2017-09-26 11:46:50) > Dylan Baker writes: > > > This allows building and installing the Intel "anv" Vulkan driver using > > meson and ninja, the driver has been tested against the CTS and has > > seems to pass the same series of test

Re: [Mesa-dev] [PATCH v3 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-26 Thread Dylan Baker
Quoting Nirbheek Chauhan (2017-09-26 12:58:12) > On Wed, Sep 27, 2017 at 1:11 AM, Dylan Baker wrote: > > Quoting Nirbheek Chauhan (2017-09-26 12:29:30) > >> On Tue, Sep 26, 2017 at 9:46 PM, Dylan Baker wrote: > >> > This builds and installs, but I haven

[Mesa-dev] [PATCH v4 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-26 Thread Dylan Baker
It is possible to have DEBUG disabled but asserts on (NDEBUG(, which cannot build because these asserts work on members that are only present when DEBUG is on. Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/util/ralloc.c | 8 1 file changed, 8 insertions(+) diff

[Mesa-dev] [PATCH v4 1/4] intel: use a flag instead of setting PYTHONPATH

2017-09-26 Thread Dylan Baker
(Ken) Reviewed-by: Kenneth Graunke Signed-off-by: Dylan Baker --- src/intel/Makefile.compiler.am | 2 +- src/intel/compiler/brw_nir_trig_workarounds.py | 33 +++--- 2 files changed, 26 insertions(+), 9 deletions(-) diff --git a/src/intel/Makefile.compiler.am

[Mesa-dev] [PATCH v4 4/4] meson: build "radv" vulkan driver for radeon hardware

2017-09-26 Thread Dylan Baker
po in meson_options (Nicholas) - strip trailing 'svn' from llvm version before setting the version preprocessor flag (Bas) - Check for LLVM module requirements Signed-off-by: Dylan Baker --- meson.build | 39 +++- meson_options.txt

[Mesa-dev] [PATCH v4 3/4] meson: Add build Intel "anv" vulkan driver

2017-09-26 Thread Dylan Baker
: - Remove dead code (Eric) - Remove change to generator that was from v0 (Eric) - replace if chain with loop (Eric) - Fix typos (Eric) - define HAVE_DLOPEN for both libdl and builtin dl cases (Eric) Signed-off-by: Dylan Baker Reviewed-by: Eric Anholt --- include/meson.build

Re: [Mesa-dev] [PATCH v4 2/4] util/ralloc: Don't define assert with magic member without DEBUG

2017-09-27 Thread Dylan Baker
Quoting Eric Engestrom (2017-09-27 04:10:37) > On Tuesday, 2017-09-26 23:38:11 +0000, Dylan Baker wrote: > > It is possible to have DEBUG disabled but asserts on (NDEBUG(, which > > parentheses typo > > other than that: > Reviewed-by: Eric Engestrom > > and I th

Re: [Mesa-dev] initial meson port

2017-09-27 Thread Dylan Baker
le to support windows in the future, I don't know what will happen when android moves to blueprint/soong). If there is anything else please let me know. Dylan Quoting Dylan Baker (2017-09-20 13:27:37) > Hi everyone, > > A long time ago I made some rumbling about porting mesa to meso

[Mesa-dev] [PATCH] meson: remove duplicate libisl dependency in anv

2017-09-27 Thread Dylan Baker
Signed-off-by: Dylan Baker CC: Kristian Høgsberg --- src/intel/vulkan/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/meson.build b/src/intel/vulkan/meson.build index 9f0ee558e8a..a0be95c747c 100644 --- a/src/intel/vulkan/meson.build +++ b/src

Re: [Mesa-dev] [PATCH mesa] REVIEWERS: add Meson

2017-09-28 Thread Dylan Baker
I use my personal email for working with mailing lists (for a host of reasons), so with the email changed to , Reviewed-by: Dylan Baker I set my signoff's using my Intel address because management feels that that is sufficient to convey that I'm working on the behalf of Intel and

Re: [Mesa-dev] initial meson port

2017-09-28 Thread Dylan Baker
Quoting Tapani Pälli (2017-09-27 23:34:26) > > > On 09/27/2017 08:57 PM, Rob Herring wrote: > > On Wed, Sep 27, 2017 at 12:38 PM, Eric Anholt wrote: > >> Dylan Baker writes: > >> > >>> [ Unknown signature status ] > >>> I

Re: [Mesa-dev] [PATCH mesa] REVIEWERS: add Meson

2017-09-28 Thread Dylan Baker
I don't have access to my Intel email except when physically at work, I'd prefer to use dy...@pnwbakers.com Otherwise I probably am the guy to look at meson, Reviewed-by: Dylan Baker Quoting Eric Engestrom (2017-09-28 05:37:02) > Cc: Dylan Baker > Signed-off-by

Re: [Mesa-dev] [PATCH mesa] meson: fix version typo + grammar

2017-09-29 Thread Dylan Baker
I had this exact same change in my wip tree, Reviewed-by: Dylan Baker Quoting Eric Engestrom (2017-09-29 08:06:39) > Signed-off-by: Eric Engestrom > --- > meson.build | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/meson.build b/m

[Mesa-dev] [PATCH 1/2] meson: check for python2 mako

2017-09-29 Thread Dylan Baker
Signed-off-by: Dylan Baker --- meson.build | 4 1 file changed, 4 insertions(+) diff --git a/meson.build b/meson.build index 5353a417484..add27b5dbac 100644 --- a/meson.build +++ b/meson.build @@ -53,6 +53,10 @@ if _vulkan_drivers != '' endif prog_python2 = find_progra

[Mesa-dev] [PATCH 2/2] meson: change vulkan icd config to - instead of _

2017-09-29 Thread Dylan Baker
Just to be consistent. Signed-off-by: Dylan Baker --- meson_options.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meson_options.txt b/meson_options.txt index 082ade7f480..d0f7cc9c7ed 100644 --- a/meson_options.txt +++ b/meson_options.txt @@ -22,9 +22,9 @@ option

[Mesa-dev] [PATCH v2] meson: change vulkan icd config to - instead of _

2017-09-29 Thread Dylan Baker
Just to be consistent. v2: - update meson.build too Signed-off-by: Dylan Baker --- meson.build | 2 +- meson_options.txt | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/meson.build b/meson.build index add27b5dbac..2e1de2b2e20 100644 --- a/meson.build +++ b

[Mesa-dev] [PATCH 2/5] meson: add window system deps to intel vulkan common

2017-09-30 Thread Dylan Baker
Signed-off-by: Dylan Baker --- src/intel/vulkan/meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/meson.build b/src/intel/vulkan/meson.build index a0be95c747c..6ce092e05ce 100644 --- a/src/intel/vulkan/meson.build +++ b/src/intel/vulkan/meson.build

[Mesa-dev] [PATCH 5/5] travis: add meson build for vulkan drivers.

2017-09-30 Thread Dylan Baker
Signed-off-by: Dylan Baker --- .travis.yml | 36 1 file changed, 36 insertions(+) diff --git a/.travis.yml b/.travis.yml index d9a8bf5a9d4..e8fff1b66ab 100644 --- a/.travis.yml +++ b/.travis.yml @@ -27,6 +27,7 @@ env: - WAYLAND_PROTOCOLS_VERSION=wayland

[Mesa-dev] [PATCH 3/5] meson: set C++ standard to C++11

2017-09-30 Thread Dylan Baker
RadeonSI requires C++11, clover requires C++11, LLVM requires it, so llvmpipe may require it, and that cover most of the C++ code in mesa. Signed-off-by: Dylan Baker --- meson.build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 15f7f0ea64f

[Mesa-dev] [PATCH 4/5] meson: convert gtest to an internal dependency

2017-09-30 Thread Dylan Baker
In truth gtest is an external dependency that upstream expects you to "vendor" into your own tree. As such, it makes sense to treat it more like a dependency than an internal library, and collect it's requirements together in a dependency object. Signed-off-by: Dylan Baker --- s

[Mesa-dev] Add a travis job for meson vulkan

2017-09-30 Thread Dylan Baker
This adds a job to travis to build the vulkan driver and run some driver checks, and fix building on travis (which uses a particularly old Ubuntu). ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/me

[Mesa-dev] [PATCH 1/5] meson: look for libelf as a library if there is no pkgconfig

2017-09-30 Thread Dylan Baker
Required for older versions of libelf that don't have a pkgconfig file. Signed-off-by: Dylan Baker --- meson.build | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 5353a417484..15f7f0ea64f 100644 --- a/meson.build +++ b/meson.

Re: [Mesa-dev] [PATCH v2] meson: change vulkan icd config to - instead of _

2017-10-02 Thread Dylan Baker
Quoting Eric Engestrom (2017-10-02 02:49:51) > On Saturday, 2017-09-30 01:00:56 +0000, Dylan Baker wrote: > > Just to be consistent. > > > > v2: - update meson.build too > > > > Signed-off-by: Dylan Baker > > --- > > meson.build | 2 +- > &g

Re: [Mesa-dev] [PATCH 4/5] meson: convert gtest to an internal dependency

2017-10-02 Thread Dylan Baker
Quoting Eric Engestrom (2017-10-02 08:33:14) > On Sunday, 2017-10-01 03:58:27 +0000, Dylan Baker wrote: > > In truth gtest is an external dependency that upstream expects you to > > "vendor" into your own tree. As such, it makes sense to treat it more > > like a depe

<    4   5   6   7   8   9   10   11   12   13   >