[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/3] Use PATH_MAX to fix some sprintf-into-short-buffers warnings.

2017-09-02 Thread Patchwork
== Series Details == Series: series starting with [1/3] Use PATH_MAX to fix some sprintf-into-short-buffers warnings. URL : https://patchwork.freedesktop.org/series/29752/ State : success == Summary == Test kms_atomic_transition: Subgroup plane-use-after-nonblocking-unbind-fencing:

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] Use PATH_MAX to fix some sprintf-into-short-buffers warnings.

2017-09-02 Thread Patchwork
== Series Details == Series: series starting with [1/3] Use PATH_MAX to fix some sprintf-into-short-buffers warnings. URL : https://patchwork.freedesktop.org/series/29752/ State : success == Summary == IGT patchset tested on top of latest successful build 5ce65a9a51f17e0183e3e4f8943981ee7b96c

Re: [Intel-gfx] [PATCH i-g-t 05/12] tests/gem_spin_batch: Fix warning

2017-09-02 Thread Eric Anholt
Daniel Vetter writes: > Signed-off-by: Daniel Vetter 1-5 are: Reviewed-by: Eric Anholt My meson branch has some stuff you probably want to squash into your meson build system commit. signature.asc Description: PGP signature ___ Intel-gfx mailing

[Intel-gfx] [PATCH i-g-t 1/3] Use PATH_MAX to fix some sprintf-into-short-buffers warnings.

2017-09-02 Thread Eric Anholt
Signed-off-by: Eric Anholt --- This little series cleans up many compiler warnings I saw when testing danvet's meson branch. lib/igt_debugfs.c | 4 ++-- lib/igt_sysfs.c | 4 ++-- tests/kms_hdmi_inject.c | 2 +- tests/pm_rpm.c | 4 ++-- 4 files changed, 7 insertions(+), 7

[Intel-gfx] [PATCH i-g-t 3/3] intel_display_poller: Fix truncation of a test name.

2017-09-02 Thread Eric Anholt
Signed-off-by: Eric Anholt --- tools/intel_display_poller.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/intel_display_poller.c b/tools/intel_display_poller.c index c501c79d6367..828ca52b35ee 100644 --- a/tools/intel_display_poller.c +++ b/tools/intel_display_poller.c

[Intel-gfx] [PATCH i-g-t 2/3] intel_watermark: Fix a warning about "const char" return being silly.

2017-09-02 Thread Eric Anholt
Signed-off-by: Eric Anholt --- tools/intel_watermark.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/intel_watermark.c b/tools/intel_watermark.c index d98ef19b0abd..d8c784802c5b 100644 --- a/tools/intel_watermark.c +++ b/tools/intel_watermark.c @@ -110,7 +110,7 @@ stru

Re: [Intel-gfx] [RFC PATCH 5/5] drm/atomic: Make async plane update checks work as intended.

2017-09-02 Thread Gustavo Padovan
2017-08-30 Daniel Vetter : > On Wed, Aug 30, 2017 at 02:17:52PM +0200, Maarten Lankhorst wrote: > > By always keeping track of the last commit in plane_state, we know > > whether there is an active update on the plane or not. With that > > information we can reject the fast update, and force the s

[Intel-gfx] [PATCH i-g-t 11/12] assembler/test: Prep work for meson

2017-09-02 Thread Daniel Vetter
Again we want to be able to run each testcase individually. Also, we need to make sure the target directory for the temp files exists - meson always builds with a build-dir outside of the source tree. Signed-off-by: Daniel Vetter --- assembler/test/run-test.sh | 16 1 file chan

[Intel-gfx] [PATCH i-g-t 06/12] lib: prefix frame_dump_path

2017-09-02 Thread Daniel Vetter
Just a bit of ocd for anything non-static. Signed-off-by: Daniel Vetter --- lib/igt_core.c | 12 ++-- lib/igt_core.h | 2 +- lib/igt_frame.c | 10 +- 3 files changed, 12 insertions(+), 12 deletions(-) diff --git a/lib/igt_core.c b/lib/igt_core.c index cf15db69c661..017c63048f

[Intel-gfx] [PATCH i-g-t 10/12] demos: remove

2017-09-02 Thread Daniel Vetter
The igt testcase themselves contain plenty of demos, libdrm also contains a bunch of demos, this here just bitrots. So let's remove it. Signed-off-by: Daniel Vetter --- Makefile.am | 2 +- demos/.gitignore| 1 - demos/Android.mk| 32 -- demos/Makefile.am |

[Intel-gfx] [PATCH i-g-t 12/12] RFC: meson build system support

2017-09-02 Thread Daniel Vetter
Why? Because it's fast. Like really, really fast. Some data (from a snb laptop, so rather lower-powered): - Incremental build after $ touch lib/igt_core.c with meson: 0.6s It notices that the symbol list of the libigt.so hasn't changed and doesn't bother re-linking the almost 300 binaries w

[Intel-gfx] [PATCH i-g-t 08/12] tests/igt_command_line.sh: Allow testing individual tests

2017-09-02 Thread Daniel Vetter
meso will use this to run the tests for all testcases in parallel, for great speedup! Signed-off-by: Daniel Vetter --- tests/igt_command_line.sh | 39 ++- 1 file changed, 26 insertions(+), 13 deletions(-) diff --git a/tests/igt_command_line.sh b/tests/igt_com

[Intel-gfx] [PATCH i-g-t 09/12] lib/uwildmat: Use include paths

2017-09-02 Thread Daniel Vetter
meson runs gcc from where meson.build that includes those files is, which means we need to add the directory ourselves. For automake it doesn't matter, so let's just do it for simplicity. Signed-off-by: Daniel Vetter --- lib/uwildmat/uwildmat.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[Intel-gfx] [PATCH i-g-t 01/12] build: Define _GNU_SOURCE in Makefile.am

2017-09-02 Thread Daniel Vetter
In meson I want to just set this everywhere (no reason not to), and doing so will allow us to clean up a few things. But that means autofoo needs to follow suit. Signed-off-by: Daniel Vetter --- benchmarks/Makefile.am | 2 +- lib/Makefile.am| 1 + lib/tests/Makefile.am | 1 + tests/Mak

[Intel-gfx] [PATCH i-g-t 03/12] build: use HAVE_LIBGEN_H consistently

2017-09-02 Thread Daniel Vetter
Also, we are _GNU_SOURCE, so simplify the conditions accordingly. The next patch will remove _GNU_SOURCE everywhere else. Signed-off-by: Daniel Vetter --- lib/drmtest.c| 4 +--- lib/igt_aux.c| 4 +--- lib/igt_core.c | 4 +--- lib/ioctl_wrappers.c | 4 +--- lib/sw_sync.c

[Intel-gfx] [PATCH i-g-t 02/12] build: Nuke #ifdef HAVE_CONFIG_H cargo-cult

2017-09-02 Thread Daniel Vetter
We have it. Daniel Stone said it comes from the X11 transition to the modular build. Signed-off-by: Daniel Vetter --- lib/igt_alsa.h | 2 -- lib/igt_audio.h | 2 -- lib/igt_chamelium.h | 2 -- lib/igt_frame.h | 2 -- lib/igt_x86.c

[Intel-gfx] [PATCH i-g-t 04/12] build: remove _GNU_SOURCE from source files

2017-09-02 Thread Daniel Vetter
We are, the build system takes care of that. Signed-off-by: Daniel Vetter --- benchmarks/gem_exec_tracer.c | 2 -- benchmarks/gem_latency.c | 1 - benchmarks/gem_syslatency.c| 2 -- lib/igt_fb.c | 1 - lib/igt_vgem.c | 2 -- lib/tests/igt_exit_handle

[Intel-gfx] [PATCH i-g-t 05/12] tests/gem_spin_batch: Fix warning

2017-09-02 Thread Daniel Vetter
Signed-off-by: Daniel Vetter --- tests/gem_spin_batch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/gem_spin_batch.c b/tests/gem_spin_batch.c index 941aa1399ae1..09bb3cb37108 100644 --- a/tests/gem_spin_batch.c +++ b/tests/gem_spin_batch.c @@ -48,7 +48,7 @@ static vo

[Intel-gfx] [PATCH i-g-t 07/12] lib: clean up header includes

2017-09-02 Thread Daniel Vetter
Just a bit of OCD, I like it when connections within library modules are a bit more obvious. igt.h is ok for tests, but let's use individual include lines for libraries consistently. Also order standard includes before igt ones. Signed-off-by: Daniel Vetter --- lib/igt_alsa.c | 3 ++- lib