Re: [Intel-gfx] [PATCH 1/4] drm/i915: Fix the missing PPAT cache attributes on CNL

Thanks for merging the patch and test. :) -Original Message- From: Rodrigo Vivi [mailto:rodrigo.v...@gmail.com] Sent: Saturday, September 2, 2017 3:01 AM To: Wang, Zhi A Cc: intel-gfx ; intel-gvt-...@lists.freedesktop.org; Vivi,

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

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

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

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 |

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

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

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

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

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

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

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

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 |

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

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

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

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

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

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

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

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

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

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,

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

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 +

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

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

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

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

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

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] [PATCH i-g-t 3/3] intel_display_poller: Fix truncation of a test name.

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

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

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

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

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