Re: [Mesa-dev] [PATCH 1/4] nv50ir: fix memset on non trivial types warning

On Sat, 21 Sep 2019 at 04:27, Karol Herbst wrote: > Signed-off-by: Karol Herbst > Reviewed-by: Rhys Kidd > --- > src/gallium/drivers/nouveau/codegen/nv50_ir.cpp| 4 +--- > src/gallium/drivers/nouveau/codegen/nv50_ir.h | 2 +- > src/gallium/drive

Re: [Mesa-dev] [PATCH 3/4] nv50ir/nir: comparison of integer expressions of different signedness warning

On Sat, 21 Sep 2019 at 04:27, Karol Herbst wrote: > Signed-off-by: Karol Herbst > Reviewed-by: Rhys Kidd > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_from_nir.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/gallium/dr

Re: [Mesa-dev] [PATCH 2/4] nv50ir: fix unnecessary parentheses warning

On Sat, 21 Sep 2019 at 04:27, Karol Herbst wrote: > Signed-off-by: Karol Herbst > Reviewed-by: Rhys Kidd > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_util.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/gallium/drivers/nouveau/c

[Mesa-dev] [PATCH] nv50, nvc0: add missing CAPs for unsupported features

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nv50/nv50_screen.c | 2 ++ src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_screen.c b/src/gallium/drivers/nouveau/nv50/nv50_screen.c index

[Mesa-dev] [MR] nv50, nvc0: add missing CAPs for unsupported features

nouveau warns on unhandled pipe CAPs, including where they may be set by u_pipe_screen_get_param_defaults() This patch avoids spurious text in debug outputs, e.g. when running piglit. https://gitlab.freedesktop.org/mesa/mesa/merge_requests/133 Rhys ___

Re: [Mesa-dev] [PATCH 0/2] Restore optional xf86vidmode

On Tue, 11 Dec 2018 at 10:22, Jon Turney wrote: > Restore optional xf86vidmode, and fix the meson build to actually use it. > > Jon Turney (2): > Revert "glx: make xf86vidmode mandatory for direct rendering" > Fix typo preventing xxf86vm being used in meson build > Jon I don't believe these

Re: [Mesa-dev] [PATCH 1/2] vc4: Wire up core pipe_debug_callback

Awesome, thanks! Apologies that I was unable to get around to looking at your feedback in the intervening period of time. On Thu, Dec 20, 2018 at 2:34 PM Eric Anholt wrote: > Eric Anholt writes: > > > [ Unknown signature status ] > > Rhys Kidd writes: > > >

[Mesa-dev] [PATCH] iris: Fix assertion in iris_resource_from_handle() tiling usage

lx-tfp glx/glx-visuals-depth -pixmap glx/glx-visuals-stencil -pixmap spec/egl 1.4/eglcreatepbuffersurface and then glclear spec/egl 1.4/largest possible eglcreatepbuffersurface and then glclear spec/egl_nok_texture_from_pixmap/basic Cc: Kenneth Graunke Cc: Jason Ekstrand Signed-off-by: Rhys Kidd --

Re: [Mesa-dev] [PATCH 00/18] Meson fixes and Travis integration

acOS as follow-up. > > Let me know what you think. > Emil > > Cc: Rhys Kidd > Cc: Dylan Baker > > For the series: Acked-by: Rhys Kidd > Dylan Baker (3): > travis: meson: use native files to override llvm-config > travis: Don't try to read libdrm out of co

Re: [Mesa-dev] last call for autotools

On Tue, 11 Dec 2018 at 16:46, Emil Velikov wrote: > On Tue, 11 Dec 2018 at 19:49, Dylan Baker wrote: > > > > Quoting Emil Velikov (2018-12-11 10:48:56) > > > On Tue, 11 Dec 2018 at 18:18, Eric Anholt wrote: > > > > > > > > Emil Velikov writes: > > > > > > > > > On Mon, 10 Dec 2018 at 23:11, Dy

[Mesa-dev] [RFC PATCH 3/5] travis: radeonsi and radv require LLVM 7.0 (meson)

Signed-off-by: Rhys Kidd --- .travis.yml | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/.travis.yml b/.travis.yml index 60ef236e7be..070c9ef6773 100644 --- a/.travis.yml +++ b/.travis.yml @@ -81,20 +81,21 @@ matrix: - VULKAN_DRIVERS

[Mesa-dev] [RFC PATCH 5/5] meson: libfreedreno depends upon libdrm (for fence support)

Error message building freedreno Gallium driver with meson: ../src/gallium/drivers/freedreno/freedreno_fence.c:27:21: fatal error: libsync.h: No such file or directory \#include Signed-off-by: Rhys Kidd --- src/gallium/drivers/freedreno/meson.build | 4 +--- 1 file changed, 1 insertion

[Mesa-dev] [RFC PATCH 2/5] travis: Add libclc-dev, clang and libclang-dev dependencies of clover

Fixes: 98a3f027aaf ("travis: enable nine and clover") Signed-off-by: Rhys Kidd --- .travis.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.travis.yml b/.travis.yml index aec1b001083..60ef236e7be 100644 --- a/.travis.yml +++ b/.travis.yml @@ -90,8 +90,11 @@ matrix:

[Mesa-dev] [RFC PATCH 1/5] travis: fix python3.5 dependency for "meson Gallium Drivers"

oring debug log for failure in /home/travis/.pip/pip.log Fixes: 05edf75f42e ("wip") Signed-off-by: Rhys Kidd --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.travis.yml b/.travis.yml index fb86418b2b0..aec1b001083 100644 --- a/.travis.yml +++ b/.travis.yml

[Mesa-dev] [RFC PATCH 0/5] last call for autotools (meson travis fixes)

travis-ci.org/Echelon9/mesa/builds/466840665 * gallium pipeloaders: There's an ongoing scoping problem with the LLVM libraries as a dependency. I was unable to fix yet. See further here: https://travis-ci.org/Echelon9/mesa/jobs/466861661#L9112 Rhys Kidd (5): travis: fix python3

[Mesa-dev] [RFC PATCH 4/5] travis: delete remaining scons residuals

Fixes: 53d5129ded5 ("delete me: remove scons") Signed-off-by: Rhys Kidd --- .travis.yml | 12 1 file changed, 12 deletions(-) diff --git a/.travis.yml b/.travis.yml index 070c9ef6773..442afea3a74 100644 --- a/.travis.yml +++ b/.travis.yml @@ -138,12 +138,6 @@ install:

[Mesa-dev] [PATCH] travis: radeonsi and radv require LLVM 7.0

Fixes: 3fbdcd942fe ("amd: remove support for LLVM 6.0") Cc: Samuel Pitoiset Cc: Marek Olšák Cc: Emil Velikov Cc: Jan Vesely Cc: Andres Gomez Cc: Dylan Baker Signed-off-by: Rhys Kidd --- .travis.yml | 29 - 1 file changed, 16 insertions(+), 13 deletion

[Mesa-dev] [PATCH] nv50/ir: silence partitionLoadStore() unused function warning

nt8_t size[2], uint8_t mask) Fixes: 86e4440361 ("nouveau: codegen: Disable more old resource handling code") Signed-off-by: Rhys Kidd Cc: Hans de Goede Cc: Samuel Pitoiset --- src/gallium/drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deleti

[Mesa-dev] [PATCH 2/2] v3d: Wire up core pipe_debug_callback

Signed-off-by: Rhys Kidd --- src/gallium/drivers/v3d/v3d_context.c | 13 + src/gallium/drivers/v3d/v3d_context.h | 1 + 2 files changed, 14 insertions(+) diff --git a/src/gallium/drivers/v3d/v3d_context.c b/src/gallium/drivers/v3d/v3d_context.c index 2fd2fa0bbf..0737713e8b 100644

[Mesa-dev] [PATCH 1/2] vc4: Wire up core pipe_debug_callback

Signed-off-by: Rhys Kidd --- src/gallium/drivers/vc4/vc4_context.c | 13 + src/gallium/drivers/vc4/vc4_context.h | 1 + 2 files changed, 14 insertions(+) diff --git a/src/gallium/drivers/vc4/vc4_context.c b/src/gallium/drivers/vc4/vc4_context.c index 9ff39c2655..6f91d0567c 100644

Re: [Mesa-dev] [PATCH 10/17] glsl/tests: Fix strict aliasing warning about int64/double.

On 10 February 2018 at 11:33, Eric Anholt wrote: > Cc: Rhys Kidd > Fixes: 4bf986274728 ("glsl/tests: Add UINT64 and INT64 types") > Reviewed-by: Rhys Kidd > --- > .../glsl/tests/uniform_initializer_utils.cpp | 22 > +++--- > 1 fi

[Mesa-dev] [PATCH] i915: No longer rely on compatability define in intel_bufmgr.h

Symbol rename from dri_* to drm_intel_* introduced a number of compatability defines within intel_bufmgr.h. Replace the old function with the new function, consistent with the balance of this file. Signed-off-by: Rhys Kidd --- src/mesa/drivers/dri/i915/intel_regions.c | 2 +- 1 file changed, 1

Re: [Mesa-dev] [PATCH v2 0/3] nouveau compiler warning cleanups

..60c2d09483 master -> master > > On 2017-12-16 — 23:00, Rhys Kidd wrote: > > Couple of little compiler warning cleanups so that nouveau builds > without any > > warnings for meson's debug and release builds with gcc 7.2.0 > > > > v2: Add preventative co

[Mesa-dev] [PATCH v2 0/3] nouveau compiler warning cleanups

Couple of little compiler warning cleanups so that nouveau builds without any warnings for meson's debug and release builds with gcc 7.2.0 v2: Add preventative comment (Ilia Mirkin) Rhys Kidd (3): nv50: Fix unused var warning in release build nvc0: Fix unused var warnings in release

[Mesa-dev] [PATCH v2 2/3] nvc0: Fix unused var warnings in release build

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nvc0/nvc0_video.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_video.c b/src/gallium/drivers/nouveau/nvc0/nvc0_video.c index b5e7bba5f9..5c1cf899ca 100644 --- a/src

[Mesa-dev] [PATCH v2 3/3] nv50/ir: Fix unused var warnings in release build

v2: Add preventative comment (Ilia Mirkin) Reviewed-by: Eric Engestrom Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nv50.cpp | 2 +- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH v2 1/3] nv50: Fix unused var warning in release build

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nv50/nv98_video.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv98_video.c b/src/gallium/drivers/nouveau/nv50/nv98_video.c index 92526d9f64..da0267e646 100644 --- a/src/gallium

[Mesa-dev] [PATCH 0/3] nouveau compiler warning cleanups

Couple of little compiler warning cleanups so that nouveau builds without any warnings for meson's debug and release builds with gcc 7.2.0 Rhys Kidd (3): nv50: Fix unused var warning in release build nvc0: Fix unused var warnings in release build nv50/ir: Fix unused var warnings in re

[Mesa-dev] [PATCH 1/3] nv50: Fix unused var warning in release build

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nv50/nv98_video.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv98_video.c b/src/gallium/drivers/nouveau/nv50/nv98_video.c index 92526d9f64..da0267e646 100644 --- a/src/gallium

[Mesa-dev] [PATCH 3/3] nv50/ir: Fix unused var warnings in release build

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nv50.cpp | 2 +- src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nv50.cpp b/src/gallium

[Mesa-dev] [PATCH 2/3] nvc0: Fix unused var warnings in release build

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nvc0/nvc0_video.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_video.c b/src/gallium/drivers/nouveau/nvc0/nvc0_video.c index b5e7bba5f9..5c1cf899ca 100644 --- a/src

Re: [Mesa-dev] [PATCH] meson: fix egl build for meson version < 0.43

libglapi, > libmesa_util], >link_args : [ld_args_bsymbolic, ld_args_gc_sections], >dependencies : [deps_for_egl, dep_dl, dep_libdrm, dep_clock, > dep_thread], > -- > 2.14.2 > > ___ > mesa-dev mailing list > mesa-dev@li

Re: [Mesa-dev] [PATCH] build: Fix up spirv_info.Plo

On 10 August 2017 at 12:46, Emil Velikov wrote: > Thanks Matt. Patch is: > > Reviewed-by: Emil Velikov > It's a little late, but gets my: Tested-by: Rhys Kidd Thanks Matt for fixing the out-of-tree builds. > > -Emil > _

Re: [Mesa-dev] [shader-db 0/3] Small README clean ups

On 24 July 2017 at 23:42, Rhys Kidd wrote: > Update or fix a few small items that I noticed when running through piglit > setup on a new development environment. > Ah, whoops. The subject prefix for this series should be [piglit] not [shader-db]. > > The first two s

[Mesa-dev] [shader-db 0/3] Small README clean ups

there was a difference in the output format for no particular reason midway through. The change is not carried forward either past those two commands. This might be a difference without a distinction, but it draws unnececssary attention away when a user should be learning the principles. Rhys Kidd (3

[Mesa-dev] [shader-db 2/3] README: Correct typo deQP -> dEQP

Signed-off-by: Rhys Kidd --- README | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/README b/README index cb49516b7..56ea52b62 100644 --- a/README +++ b/README @@ -414,13 +414,13 @@ deqp_egl.py Support for running dEQP's EGL profile with piglit. deqp_gle

[Mesa-dev] [shader-db 1/3] README: Update the list of test profiles

Signed-off-by: Rhys Kidd --- README | 30 ++ 1 file changed, 30 insertions(+) diff --git a/README b/README index 43bdfcae7..cb49516b7 100644 --- a/README +++ b/README @@ -386,6 +386,9 @@ glslparser.py shader.py A subset of all.py which runs only shader tests

[Mesa-dev] [shader-db 3/3] README: Make initial smoke tests steps internally consistent

No other stage of the suggested initial sanity.py test or other tests uses a folder structure with *.results. Use a consistent naming approach throughout the initial README to make initial setup clearer. Signed-off-by: Rhys Kidd --- README | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[Mesa-dev] [PATCH] nvc0: Clean up unnecessary includes from gallium/auxiliary/vl/

Signed-off-by: Rhys Kidd --- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c b/src/gallium/drivers/nouveau/nvc0/nvc0_screen.c index f2ae4f3ae3..c636926994 100644 --- a/src/gallium/drivers/nouveau

Re: [Mesa-dev] [PATCH] u_format_test: Ignore S3TC errors.

On Mon, May 22, 2017 at 1:07 PM Jose Fonseca wrote: > This prevents spurious failures when libtxc-dxtn-s2tc is installed. > > Note: lp_test_format does need any change s/does/doesn't/ With that commit message fix: Reviewed-by: Rhys Kidd since we were already ignoring > S3

Re: [Mesa-dev] [PATCH] travis: Disable scons tests, since the gallium unit tests fail.

On 21 May 2017 at 19:33, Roland Scheidegger wrote: > I suppose the s3tc problems should go away rather sooner than later, but > isn't it possible to just disable the formats tests? > Yes, see USE_TXC_DXTN that was introduced in 29322daef2b77c4d869d2945fa1226e6b433c68 to address this issue with a

[Mesa-dev] [PATCH] vc4: Remove dead code in vc4_dump_surface_msaa()

Coverity caught the use of dead code copy-paste for found_colors[] and num_found_colors. CID: 1341850 Signed-off-by: Rhys Kidd --- src/gallium/drivers/vc4/vc4_resource.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/src/gallium/drivers/vc4/vc4_resource.c b/src/gallium/drivers/vc4

[Mesa-dev] [PATCH shader-db] Add ".so" shared objects to .gitignore

For intel_stubs.so Signed-off-by: Rhys Kidd --- I don't have commit access, so I would appreciate a reviewer pushing this to master. .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index f69750a..95a04f6 100644 --- a/.gitignore +++ b/.gitignore @@

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

On 5 April 2017 at 13:27, Andres Gomez wrote: > The packaged libtxc-dxtn-s2tc0 will make scons check fail. Manually > remove this package and install libtxc-dxtn instead. > > Signed-off-by: Andres Gomez > Cc: Eric Anholt > Cc: Emil Velikov > Tested-by: Rhys Kidd Al

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

On 5 April 2017 at 13:27, Andres Gomez wrote: > 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 > Cc: Rhys K

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

On Wed, Apr 5, 2017 at 1:49 PM Emil Velikov wrote: > On 5 April 2017 at 18:27, Andres Gomez wrote: > > The packaged libtxc-dxtn-s2tc0 will make scons check fail. Manually > > remove this package and install libtxc-dxtn instead. > > > Don't recall seeing it fail. > > Is that a recent thing ? Any

Re: [Mesa-dev] [PATCH] travis: remove r600 as it requires llvm 3.8

On 2 April 2017 at 17:26, Dave Airlie wrote: > On 3 April 2017 at 06:44, Rhys Kidd wrote: > > On 2 April 2017 at 08:42, Christian Gmeiner > > > wrote: > >> > >> At the moment travis ci failes with the following error: > >> > >> chec

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

Signed-off-by: Rhys Kidd --- .travis.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.travis.yml b/.travis.yml index aa212b1..0b03ae7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -115,6 +115,7 @@ script: --with-egl-platforms=x11,drm --with-dri-drivers=i915,i965,radeon

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

Signed-off-by: Rhys Kidd --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index aa212b1..c0b2c1f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -114,7 +114,7 @@ script: ./autogen.sh --enable-debug --with-egl-platforms

[Mesa-dev] [PATCH] travis: Support LLVM 3.8+ on Trusty-based Travis-CI via apt-get not apt addon

ty-ci-environment/#Fully-virtualized-via-sudo%3A-required https://github.com/travis-ci/apt-source-whitelist/pull/205#issuecomment-216054237 Signed-off-by: Rhys Kidd --- .travis.yml | 20 +++- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/.travis.yml b/.travis.yml

Re: [Mesa-dev] [PATCH] travis: remove r600 as it requires llvm 3.8

On 2 April 2017 at 08:42, Christian Gmeiner wrote: > At the moment travis ci failes with the following error: > > checking for RADEON... yes > checking for RADEON... yes > configure: error: LLVM 3.8.0 or newer is required for r600 > make: *** No targets specified and no makefile found. Stop. > >

Re: [Mesa-dev] [PATCH] travis: bring the scons build on par with AppVeyor

On 16 February 2017 at 09:09, Emil Velikov wrote: > From: Emil Velikov > > Namely, always build with LLVM and run the check target. > > Cc: Rhys Kidd > Cc: Eric Anholt > Signed-off-by: Emil Velikov > Gets my Reviewed-by: Rhys Kidd I had a similar testing branch [

Re: [Mesa-dev] [PATCH] glsl/tests: Add UINT64 and INT64 types

On Mon, Feb 13, 2017 at 9:43 AM Elie Tournier wrote: > Seems good to me. > > Reviewed-by: Elie Tournier > Thanks Elie. Given we are both "new-er" Mesa contributors I might wait for one more Reviewed-by before asking for it to be merged. > On Sat, Feb 11, 2017 at 05

[Mesa-dev] [PATCH] glsl/tests: Add UINT64 and INT64 types

tch [-Wswitch] Fixes: 8ce53d4a2f3 ("glsl: Add basic ARB_gpu_shader_int64 types") Signed-off-by: Rhys Kidd --- src/compiler/glsl/tests/uniform_initializer_utils.cpp | 18 ++ 1 file changed, 18 insertions(+) diff --git a/src/compiler/glsl/tests/uniform_initializer_utils.cpp b

Re: [Mesa-dev] [PATCH 3/5] travis: correct libdrm required regex to also track libdrm itself

On 2 February 2017 at 02:53, Andres Gomez wrote: > LGTM > > Reviewed-by: Andres Gomez > Reviewed-by: Rhys Kidd > > On Wed, 2017-02-01 at 22:30 +, Emil Velikov wrote: > > From: Emil Velikov > > > > The current regex was tracking only the libdrm

Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

On Sun, Jan 22, 2017 at 1:49 PM Rhys Kidd wrote: > On 20 January 2017 at 17:06, Christian Gmeiner < > christian.gmei...@gmail.com> wrote: > > Hi Rhys, > > 2017-01-19 7:02 GMT+01:00 Rhys Kidd : > > Use of unsigned loop control variable with '>= 0' woul

Re: [Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

On 20 January 2017 at 17:06, Christian Gmeiner wrote: > Hi Rhys, > > 2017-01-19 7:02 GMT+01:00 Rhys Kidd : > > Use of unsigned loop control variable with '>= 0' would lead to infinite > loop. > > > > Reported by clang: > > > > etna

[Mesa-dev] [PATCH] etnaviv: Avoid infinite loop in find_frame()

ame_sp; sp >= 0; sp--) ~~ ^ ~ Signed-off-by: Rhys Kidd --- src/gallium/drivers/etnaviv/etnaviv_compiler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) I compile tested etnaviv but have not functionally tested on real hardware. diff --git a/src/gallium/drivers/etnaviv/etn

Re: [Mesa-dev] [PATCH] travis: Add the new drivers etnaviv and imx

On 13 January 2017 at 02:05, Christian Gmeiner wrote: > 2017-01-13 5:06 GMT+01:00 Rhys Kidd : > > Signed-off-by: Rhys Kidd > > --- > > .travis.yml | 6 +++--- > > 1 file changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/.travis.yml b/.travi

[Mesa-dev] [PATCH] travis: Add the new drivers etnaviv and imx

Signed-off-by: Rhys Kidd --- .travis.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.travis.yml b/.travis.yml index 3d107aa..dfbc052 100644 --- a/.travis.yml +++ b/.travis.yml @@ -32,7 +32,7 @@ env: - DRI3PROTO_VERSION=dri3proto-1.0 - PRESENTPROTO_VERSION

Re: [Mesa-dev] [PATCH v2] glsl: Add pthread libs to cache_test

On 26 October 2016 at 00:47, Timothy Arceri wrote: > Reviewed-by: Timothy Arceri > > I guess you don't have commit access? > I do not have commit access, so would appreciate if you could please push to master. > > On Wed, 2016-10-26 at 00:13 -0400, Rhys Kidd wrote:

[Mesa-dev] [PATCH v2] glsl: Add pthread libs to cache_test

ned-off-by: Rhys Kidd --- v2: Add additional comment to the commit message (Timothy Arceri) src/compiler/Makefile.glsl.am | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/compiler/Makefile.glsl.am b/src/compiler/Makefile.glsl.am index 80dfb73..15bea6b 100644 --- a/sr

[Mesa-dev] [PATCH] glsl: Add pthread libs to cache_test

Fixes the following compile error: CCLD glsl/tests/cache-test glsl/.libs/libglsl.a(libmesautil_la-mesa-sha1.o): In function `call_once': /mesa/src/util/../../include/c11/threads_posix.h:96: undefined reference to `pthread_once' Signed-off-by: Rhys Kidd --- src/compiler/Makefi

Re: [Mesa-dev] [PATCH] vulkan: Initial partial documentation for Vulkan internals.

On 14 October 2016 at 10:36, wrote: > From: Kevin Rogovin > > A main page ala Doxygen together with a Doxygen file added. > In addition, documentation for portions of anv_private.h: > - placing portions into doxygen groups > - some cross linking doxygen style to other files > Hello Kevin, Th

Re: [Mesa-dev] Proposal: move the Mesa documentation to readthedocs.org

On 27 August 2016 at 01:09, Nicholas Bishop wrote: > Hi, > > I'd like to propose a conversion of Mesa's documentation to > reStructuredText (RST) and hosting the result on readthedocs.org. The > intent is to make Mesa's documentation more accessible, searchable, and > easier to edit. > > I put to

Re: [Mesa-dev] [PATCH 2/2] r600g: Clean up defined magic numbers for TGSI opcodes

On Monday, August 29, 2016, Marek Olšák wrote: > For the series: > > Reviewed-by: Marek Olšák > > > Marek Thanks Marek. As I don't have commit access, when you have a moment I would appreciate if you could please push to master. Regards, Rhys > > On Sat, Aug 2

Re: [Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

On Monday, August 29, 2016, James Harvey wrote: > Hi Rhys, > > I ran piglit on my Evergreen HD5850 with your patches. > > No regressions here. > > Tested-by: James Harvey Thanks James. > > Thanks, > James > > > On 08/27/2016 09:05 AM, Rhys Kidd wrote

Re: [Mesa-dev] [PATCH 1/2] r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA

On Saturday, August 27, 2016, Jan Vesely wrote: > On Sat, 2016-08-27 at 12:05 -0400, Rhys Kidd wrote: > > As reported by Clang, TGSI_OPCODE_DFMA (defined magic number 118) is > > currently initialized twice for Cayman and Evergreen. > > When Jan Vesely added double precis

[Mesa-dev] [PATCH 0/2] r600g: Pair of small code clean ups with TGSI

re to test this so would appreciated Tested-by's. I do not have commit rights to fd.o so after R-B would appreciate if the reviewer(s) could push to master. Rhys Kidd (2): r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA r600g: Clean up defined magic numbers for TGSI opcode

[Mesa-dev] [PATCH 1/2] r600g: Avoid duplicated initialization of TGSI_OPCODE_DFMA

: https://bugs.freedesktop.org/show_bug.cgi?id=96782 Fixes: 54c4d525da7c7fc1e103d7a3e6db015abb132d5d ("r600g: Enable FMA on chips that support it") Signed-off-by: Jan Vesely Tested-by: James Harvey Signed-off-by: Marek Olšák Signed-off-by: Rhys Kidd --- src/galli

[Mesa-dev] [PATCH 2/2] r600g: Clean up defined magic numbers for TGSI opcodes

Small code clean up that removes magic numbers where a TGSI opcode has been defined. No functional change expected as each opcode is unsupported on the respective hardware. Signed-off-by: Rhys Kidd --- src/gallium/drivers/r600/r600_shader.c | 14 +++--- src/gallium/include/pipe

[Mesa-dev] [PATCH] gallium/ttn: Remove duplicated TGSI_OPCODE_DP2A initialization

Duplicate line is currently on 1535. Identified by Clang, when run through Eric Anholt's Travis harness. Signed-off-by: Rhys Kidd --- src/gallium/auxiliary/nir/tgsi_to_nir.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c b/src/gallium/auxiliar

Re: [Mesa-dev] [PATCH 0/4] Travis CI build fixes

> ___ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev > The series gets my Reviewed-by: Rhys Kidd ___ mesa-dev mailing list me

Re: [Mesa-dev] [PATCH] egl/dri2: dri2_make_current: Make sure display is initialized before using it

Whilst I don't know that code as well as I'd like. It does get my Tested-by: Rhys Kidd for fixing that piglit on i965 ILK. Thanks for the two patches! On 15 July 2016 at 04:28, Nicolas Boichat wrote: > android.opengl.cts.WrapperTest#testGetIntegerv1 CTS test calls > eglTermin

Re: [Mesa-dev] [PATCH 1/2] i965/gen4: Pull texture formats from the texture object not the miptree

mpler->sRGBDecode); > > if (tObj->Target == GL_TEXTURE_EXTERNAL_OES) { > > For what it is worth, this series is: Reviewed-by: Rhys Kidd Tested-by: Rhys Kidd ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [GSoC lib soft float] Blog update

On Monday, June 13, 2016, tournier.elie wrote: > Hi, > > After a few weeks of coding, I wrote a new post on my blog ( > https://hopetech.github.io/). I speak about the progress of my project, > the difficulties encountered and the tools I use. > > Happy reading, > Elie > Hello Elie, Thanks for

Re: [Mesa-dev] [PATCH] vc4: Fix failed instruction path of QIR validate pass

On 19 May 2016 at 23:17, Rhys Kidd wrote: > Correct use of qir_dump_inst() within QIR validate pass. > > Reported by the following GCC warning: > > mesa/src/gallium/drivers/vc4/vc4_qir_validate.c: In function 'fail_instr': > mesa/src/gallium/drivers/vc4/vc4_qir_valida

[Mesa-dev] [PATCH 2/2] vc4: Fix doxygen warnings

Now that vc4 automated code documentation can be generated with doxygen, fix the warnings issued by Doxygen 1.8.11. Signed-off-by: Rhys Kidd --- src/gallium/drivers/vc4/vc4_qir_schedule.c | 4 ++-- src/gallium/drivers/vc4/vc4_tiling.c | 8 2 files changed, 6 insertions(+), 6

[Mesa-dev] [PATCH 0/2] vc4: Add automated code documentation

Simple patchset to provide doxygen-based automated code documentation of the vc4 driver for developers. By plumbing this through we get the same for all Gallium-based drivers for free. Post review I will need someone with Mesa commit access to push these for me. Rhys Kidd (2): doxygen: Plumb

[Mesa-dev] [PATCH 1/2] doxygen: Plumb through gallium/ to automated documentation

Signed-off-by: Rhys Kidd --- doxygen/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/doxygen/Makefile b/doxygen/Makefile index 6685b99..e5fac32 100644 --- a/doxygen/Makefile +++ b/doxygen/Makefile @@ -25,7 +25,8 @@ full: $(FULL:.doxy=.tag) SUBSET = \

Re: [Mesa-dev] [PATCH v2] doxygen: Add missing modules to Windows runner

On 11 May 2016 at 07:51, Rhys Kidd wrote: > On 10 May 2016 at 16:04, Elie TOURNIER wrote: > >> --- >> doxygen/doxy.bat | 7 +++ >> 1 file changed, 7 insertions(+) >> >> diff --git a/doxygen/doxy.bat b/doxygen/doxy.bat >> index e566ca3..408964e

[Mesa-dev] [PATCH] vc4: Fix failed instruction path of QIR validate pass

uct _IO_FILE *' void qir_dump_inst(struct vc4_compile *c, struct qinst *inst); ^ Introduced in 8e2d0843c02daf5280184f179ae8ed440ac90d7f. Signed-off-by: Rhys Kidd --- src/gallium/drivers/vc4/vc4_qir_validate.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(

Re: [Mesa-dev] [PATCH v2] doxygen: Add missing modules to Windows runner

; or '--signoff' parameter to git format-patch). Shouldn't be a need to send a v3 for this patch as you did do so on your v1. Whomever pushes the patch can add that for you I'd expect. Whilst I don't have a Windows development environment to test this on it looks sensible so

Re: [Mesa-dev] [PATCH 1/2] i965: Reduce the SIMD8 GS push constant threshold from 32 to 24.

Hello Kenneth, On 10 May 2016 at 03:46, Kenneth Graunke wrote: > Three Shadow of Mordor geometry shaders increase by a single > instruction, but the number of spills/fills in Orbital Explorer > is reduced from 194:1279 -> 82:454. No other programs are affected. > > Signed-off-by: Kenneth Graunk

Re: [Mesa-dev] [PATCH] Update Doxygen for Windows users

On 20 April 2016 at 01:46, Elie TOURNIER wrote: > Now Windows users have the same doxygen files than *nix users > Not tested (I don't have a windows) > > Signed-off-by: Elie TOURNIER > --- > doxygen/doxy.bat | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/doxygen/doxy.bat b/doxy

[Mesa-dev] [PATCH] glsl: Update hash table comments in constant propagation

Signed-off-by: Rhys Kidd --- src/compiler/glsl/opt_constant_propagation.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/compiler/glsl/opt_constant_propagation.cpp b/src/compiler/glsl/opt_constant_propagation.cpp index 416ba16..4764d16 100644 --- a/src/compiler

[Mesa-dev] [PATCH 8/9] doxygen: Fix typo in doxygen/tnl.doxy

TAGFILE relative folder should match .tag file Signed-off-by: Rhys Kidd --- doxygen/tnl.doxy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doxygen/tnl.doxy b/doxygen/tnl.doxy index b985b8d..e6d6572 100644 --- a/doxygen/tnl.doxy +++ b/doxygen/tnl.doxy @@ -40,7 +40,7

[Mesa-dev] [PATCH 5/9] doxygen: Remove references to miniglx

miniglx was removed in February 2010. Clean up remaining unnecessary doxygen references. commit a9e36696837b2c31ecee4017a0adffbd987c1eff Author: Kristian Høgsberg Date: Thu Feb 25 16:17:04 2010 -0500 Remove remaining miniglx references Signed-off-by: Rhys Kidd --- doxygen

[Mesa-dev] [PATCH 9/9] doxygen: Remove deprecated settings in common.doxy

27; has become obsolete. To avoid this warning please remove this line from your configuration file or upgrade it using "doxygen -u" Signed-off-by: Rhys Kidd --- doxygen/common.doxy | 51 --- 1 file changed, 51 deletions(-) diff -

[Mesa-dev] [PATCH 2/9] doxygen: Fix doxygen/glapi.doxy

-by: Rhys Kidd --- doxygen/glapi.doxy | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/doxygen/glapi.doxy b/doxygen/glapi.doxy index 3290f84..0e169e4 100644 --- a/doxygen/glapi.doxy +++ b/doxygen/glapi.doxy @@ -9,7 +9,7 @@ PROJECT_NAME = "Mesa GL API dispa

[Mesa-dev] [PATCH 3/9] doxygen: Correct TAGFILE relative paths

relative path it should be relative with respect to the directory where the HTML output of your project is generated; so a relative path from the HTML output directory of a project to the HTML output of the other project that is linked to. Signed-off-by: Rhys Kidd --- doxygen/gbm.doxy| 2

[Mesa-dev] [PATCH 7/9] doxygen: Correct TAGFILE linkage of main

core' with 'main'. Other minor updates. Signed-off-by: Rhys Kidd --- doxygen/gbm.doxy | 2 +- doxygen/glapi.doxy| 2 +- doxygen/math.doxy | 2 +- doxygen/swrast.doxy | 2 +- doxygen/swrast_setup.doxy | 2 +- doxygen/tnl.doxy | 2 +- doxy

[Mesa-dev] [PATCH 4/9] doxygen: Fix doxygen/gbm.doxy TAGFILES

There has never been a doxygen/gbm_setup output folder. Appears to have been a copy-paste error from original commit in 245341f406bbdf23fec3d8ea8ad95c147c11ad1c. Signed-off-by: Rhys Kidd --- doxygen/gbm.doxy | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doxygen/gbm.doxy b

[Mesa-dev] [PATCH 6/9] doxygen: Update .gitignore

minor updates. commit 69db632a9df035cdc852cc64fb14038919929a70 Author: Jose Fonseca Date: Thu May 1 23:32:54 2003 + Move the Doxygen configuration files into the usual places and integrate with the build system. Signed-off-by: Rhys Kidd --- doxygen/.gitignore | 5 - 1 file changed, 5 deletions

[Mesa-dev] [PATCH 1/9] doxygen: Remove src/mesa/shader/ references

in the consolidated Doxygen output caused by doxygen/shader.doxy inadvertently overwriting doxygen/swrast.tag via its GENERATE_TAGFILE setting. This bug depended upon the specific order each *.tag was built. Signed-off-by: Rhys Kidd --- doxygen/.gitignore | 1 - doxygen/Makefile| 1

[Mesa-dev] [PATCH 0/9] doxygen: Clean up configuration file bit rot

Elie Tournier given recent work in this area. Hopefully also assists in getting the right eyes reviewing the patchset. Rhys Kidd (9): doxygen: Remove src/mesa/shader/ references doxygen: Fix doxygen/glapi.doxy doxygen: Correct TAGFILE relative paths doxygen: Fix doxygen/gbm.doxy TAGFIL

Re: [Mesa-dev] Updating mesa3d.org docs?

On 9 April 2016 at 16:19, Brian Paul wrote: > On 04/09/2016 01:09 PM, Rhys Kidd wrote: > >> On Thursday, October 15, 2015, Brian Paul > <mailto:bri...@vmware.com>> wrote: >> >> On 10/15/2015 01:18 PM, Sarah Sharp wrote: >> >> Hi Bria

Re: [Mesa-dev] Updating mesa3d.org docs?

On Thursday, October 15, 2015, Brian Paul wrote: > On 10/15/2015 01:18 PM, Sarah Sharp wrote: > >> Hi Brian! >> > > Hi Sarah, > > > I'm a new Mesa developer in Intel's OTC graphics team (although not new >> to open source, I've been a Linux kernel developer for the last seven >> years). >> >> I h

Re: [Mesa-dev] [PATH v2] doxygen: Generate Doxygen for NIR

LGTM, Reviewed-by: Rhys Kidd On Saturday, April 9, 2016, Elie TOURNIER wrote: > Now, one can do the following to generate and read the nir Doxygen: > cd $MESA_TOP/doxygen > make > firefox nir/index.html > > Update v2: > Correct TAGFILES in nir.doxy > > S

Re: [Mesa-dev] [PATCH] doxygen: Generate Doxygen for NIR

rnal references > > +#--- > +TAGFILES = glsl.tag=../glsl \ > + main.tag=../main \ > + math.tag=../math \ > + swrast.tag=.

  1   2   3   >