Re: [Intel-gfx] [PATCH v3 0/2] drm/edid: transparent low-level override/firmware EDIDs

tically > it should be part of patch 1, but this division should be easier to > grasp in review. > > BR, > Jani. > > > [1] > http://patchwork.freedesktop.org/patch/msgid/1487344854-18777-5-git-send-email-jani.nik...@intel.com > > Cc: Abdiel Janulgue <a

[Intel-gfx] [PATCH i-g-t] tests/tools_test: Make sure l3_parity is supported

Check support before executing test. v2: Skip test only if intel_l3_parity tool tells us to skip. (Petri) bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101650 Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com>

[Intel-gfx] [PATCH i-g-t] tests/tools_test: check if l3_parity is supported

This is a wrapper for tools/intel_l3_parity which fails if l3_parity is not supported in the kernel. Check support before executing test. bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=101650 Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/tools_test

Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/debugfs_test: Fix testcases to pass

alid argument > grep: /sys/kernel/debug/dri/0/crtc-1/crc/data: Input/output error > > Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > Cc: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > Cc: Arkadiusz Hiler <arka

[Intel-gfx] [PATCH i-g-t] tests/debugfs_test: Add a shorter timeout when reading sysfs entries

This allows us to proceed gracefully without invoking the global CI-level timeout especially when opening the buggy crtc-data file descriptor. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/debugfs_test.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

Re: [Intel-gfx] [PATCH i-g-t] igt/debugfs_test: Skip dummy reads for crtc-n/crc/data

On 06.07.2017 09:05, Maarten Lankhorst wrote: > Hey, > > Op 03-07-17 om 12:19 schreef Abdiel Janulgue: >> When descending thru debugfs directory. Doing this tends to >> takes forever. CRC pipe read tests for this are already >> covered under kms_pipe_crc_basic

[Intel-gfx] [PATCH i-g-t] igt/debugfs_test: Skip dummy reads for crtc-n/crc/data

When descending thru debugfs directory. Doing this tends to takes forever. CRC pipe read tests for this are already covered under kms_pipe_crc_basic. v2: Be more verbose that the check is meant for the crtc control data Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com&

Re: [Intel-gfx] [PATCH i-g-t] igt/debugfs_test: Skip dummy reads for sysfs pipe crc data

On 28.06.2017 14:06, Arkadiusz Hiler wrote: > On Wed, Jun 28, 2017 at 01:33:16PM +0300, Abdiel Janulgue wrote: >> Doing this tends to takes forever. >> >> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> >> --- >> tests/debugfs_test.c |

[Intel-gfx] [PATCH i-g-t] igt/debugfs_test: Skip dummy reads for sysfs pipe crc data

Doing this tends to takes forever. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/debugfs_test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/debugfs_test.c b/tests/debugfs_test.c index 3f0eaae..3d973d9 100644 --- a/tests/debugfs_

[Intel-gfx] [i-g-t PATCH v4 2/2] Convert tools shell tests to C version

v4: Go thru /dev/kmsg instead of doing "dmesg | grep ..." (Arek). Split conversion to couple of patches. Converted: - sysfs_l3_parity - test_rte_check (same as check_drm_clients) - tools_test - ZZ_check_dmesg Signed-off-by: Abdiel Janulgue <

[Intel-gfx] [i-g-t v4 PATCH 1/2] Convert debugfs shell tests to C version

ent debugfs tests. Convert sysfs_l3_parity properly. Rename redundant names in tests. Converted: - debugfs_emon_crash - debugfs_wedged - drv_debugfs_reader Cc: Arkadiusz Hiler <arkadiusz.hi...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com

Re: [Intel-gfx] [i-g-t PATCH v3 3/3] Convert shell script tests to C version

On 12.06.2017 14:14, Arkadiusz Hiler wrote: > On Tue, Jun 06, 2017 at 11:54:14AM +0300, Abdiel Janulgue wrote: >> v3: Drop redundant test covered by drv_hangman/basic. Descend thru >> debugfs path when reading sysfs entries (Chris). >> >> v2: Use internal igt_debug

[Intel-gfx] [i-g-t PATCH v3 3/3] Convert shell script tests to C version

atv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 9 +--- tests/ZZ_check_dmesg | 11 - tests/check_drm_clients | 6 --- tests/debugfs.c | 96 + tests/debugfs_emon

[Intel-gfx] [i-g-t PATCH 2/3] igt/igt_core: Provide an option to check for the log buffer contents

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_core.c | 24 lib/igt_core.h | 3 +++ 2 files changed, 27 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.c index 26f7fc5..94f5529 100644 --- a/lib/igt_core.c +++ b/lib/igt_

[Intel-gfx] [i-g-t PATCH v3 1/3] lib/igt_core: Add igt_system helpers

, like full dmesg. (Petri). Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_core.c | 157 + lib/igt_core.h | 10 2 files changed, 167 insertions(+)

Re: [Intel-gfx] [i-g-t PATCH v2 1/4] lib/igt_core: Add igt_exec helpers

On 29.05.2017 16:01, Chris Wilson wrote: > On Mon, May 29, 2017 at 03:08:07PM +0300, Abdiel Janulgue wrote: >> Support executing external processes with the goal of capturing its >> standard streams to the igt logging infrastructure in addition to its >> exit status. >

[Intel-gfx] [i-g-t PATCH v2 1/4] lib/igt_core: Add igt_exec helpers

ala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_core.c | 154 + lib/igt_core.h | 10 2 files changed, 164 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.

[Intel-gfx] [i-g-t PATCH v2 4/4] Convert ddx_intel_after_fbdev to C.

v2: Use igt_exec_cmd macro. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources| 1 + tests/ddx_intel_after_fbdev | 73 --- tests/ddx_intel_after_fbdev.c | 113 ++ 3 files c

[Intel-gfx] [i-g-t PATCH v2 3/4] Convert shell script tests to C version

). - debugfs_emon_crash - debugfs_wedged - drv_debugfs_reader - sysfs_l3_parity - test_rte_check (same as check_drm_clients) - tools_test - ZZ_check_dmesg Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sou

[Intel-gfx] [i-g-t PATCH v2 2/4] igt/igt_core: Provide an option to check for the log buffer contents

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_core.c | 24 lib/igt_core.h | 3 +++ 2 files changed, 27 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.c index c2c8872..81d2039 100644 --- a/lib/igt_core.c +++ b/lib/igt_

Re: [Intel-gfx] [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers

On 10.05.2017 15:14, Petri Latvala wrote: > On Wed, May 10, 2017 at 03:10:16PM +0300, Abdiel Janulgue wrote: >> In case the stream output is more than the pipe buf, the read loop above >> would just unblock the rest of the entries. From pipe(2) page: > > > But you do

Re: [Intel-gfx] [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers

On 09.05.2017 13:18, Petri Latvala wrote: snip 8< - >> +memset(buf, 0, sizeof(buf)); >> +while (read(current->read_fd, buf, sizeof(buf)) > 0) { >> +if (current->redirected) { >> +if (!unredirect_output(current)) >> +

[Intel-gfx] [i-g-t PATCH] Convert ddx_intel_after_fbdev to C.

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources| 1 + tests/ddx_intel_after_fbdev | 73 - tests/ddx_intel_after_fbdev.c | 121 ++ 3 files changed, 122 insertions(

Re: [Intel-gfx] [PATCH i-g-t v2] tests/meta_test: Add a meta test for sanity checks of CI systems

On 19.04.2017 13:13, Marta Lofstedt wrote: > + > +__attribute__((format(printf, 1, 2))) > +static void kmsg(const char *format, ...) > +#define KERN_EMER"<0>" > +#define KERN_ALERT "<1>" > +#define KERN_CRIT"<2>" > +#define KERN_ERR "<3>" > +#define KERN_WARNING "<4>" > +#define

Re: [Intel-gfx] [i-g-t PATCH v4 1/2] lib/igt_kms: Add support for 4K and audio HDMI EDID injection.

On 19.04.2017 11:13, Petri Latvala wrote: > > The only Intel-specific part I can spot is that one chunk where you > check for at-least-hsw. Is that so you can properly report skip > instead of fail? It would be good to make this all suitable for > non-Intel hw. > That is meant to report skip

[Intel-gfx] [i-g-t PATCH 2/4] igt/igt_core: Provide an option to check for the log buffer contents

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_core.c | 24 lib/igt_core.h | 3 +++ 2 files changed, 27 insertions(+) diff --git a/lib/igt_core.c b/lib/igt_core.c index 8a7ba0d..e80a32a 100644 --- a/lib/igt_core.c +++ b/lib/igt_

[Intel-gfx] [i-g-t PATCH 3/4] lib/igt_debugfs: Add helper to return path to device.

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_debugfs.c | 26 ++ lib/igt_debugfs.h | 1 + 2 files changed, 27 insertions(+) diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c index 7584be5..b019c3b 100644 --- a/lib/igt_debugfs.c +++

[Intel-gfx] [i-g-t PATCH 1/4] lib/igt_core: Add igt_exec helpers

Support executing external processes with the goal of capturing its standard streams to the igt logging infrastructure in addition to its exit status. Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue &

[Intel-gfx] [i-g-t PATCH 4/4] Convert shell script tests to C version

Vetter <daniel.vet...@ffwll.ch> Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 9 +--- tests/ZZ_check_dmesg | 11 - tests/check_drm_clients | 6 --- tests/debugf

Re: [Intel-gfx] [i-g-t PATCH 2/2] lib/tests: Add kmstest_edid_add_* selftests

On 30.03.2017 11:09, Abdiel Janulgue wrote: > Sanity check the edid block generation capabilities. > > Cc: Petri Latvala <petri.latv...@intel.com> Ping! > Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > --- > lib/tests/Makefil

[Intel-gfx] [i-g-t PATCH v5 1/2] tests/kms_hdmi_inject: Add test for HDMI injection capabilities.

ed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 1 + tests/kms_hdmi_inject.c | 272 2 files changed, 273 insertions(+) create mode 100644 tests/kms_hdmi_inject.c diff --git a/tests/Makefile.sou

[Intel-gfx] [i-g-t PATCH 2/2] lib/tests: Add kmstest_edid_add_* selftests

Sanity check the edid block generation capabilities. Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/tests/Makefile.sources | 1 + lib/tests/igt_hdmi_inject.c | 96 ++

Re: [Intel-gfx] [i-g-t PATCH v4] tests/kms_hdmi_inject: Add test for HDMI injection capabilities.

On 30.03.2017 09:43, Petri Latvala wrote: > On Wed, Mar 29, 2017 at 12:52:29PM +0300, Abdiel Janulgue wrote: >> Original author: Marius Vlad. Includes fixes below. >> >> v4: Add a unit test to make sure synthetic EDID blocks generated by >> IGT i

[Intel-gfx] [i-g-t PATCH v4] tests/kms_hdmi_inject: Add test for HDMI injection capabilities.

Original author: Marius Vlad. Includes fixes below. v4: Add a unit test to make sure synthetic EDID blocks generated by IGT is valid (suggested by Petri). v3: Make audio injection work. Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abd

[Intel-gfx] [i-g-t PATCH v3 2/2] tests/kms_hdmi_inject: Add test for HDMI injection capabilities.

From Marius Vlad. Fixed to include proper detection of asound ELD entry. v3: Make audio injection work Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 1 + tests/kms_hdmi_in

[Intel-gfx] [i-g-t PATCH v4 1/2] lib/igt_kms: Add support for 4K and audio HDMI EDID injection.

Based on the initial work by Marius Vlad. v3: Bring back audio injection v4: Make a helper for 3d/4k/audio HDMI injection (Petri) Cc: Petri Latvala <petri.latv...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt

[Intel-gfx] [i-g-t PATCH v3 1/2] lib/igt_kms: Add support for 4K and audio HDMI EDID injection.

From: Marius Vlad <marius.c.v...@intel.com> v3: Bring back audio injection Signed-off-by: Marius Vlad <marius.c.v...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_kms.c | 181 +++

[Intel-gfx] [i-g-t PATCH v3 2/2] tests/kms_hdmi_inject: Test to make use of HDMI injection capabilities.

From: Marius Vlad <marius.c.v...@intel.com> v3: Make audio injection work Signed-off-by: Marius Vlad <marius.c.v...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 1 + tests/kms_h

[Intel-gfx] [i-g-t PATCH 2/2] tests/kms_hdmi_inject: Test to make use of HDMI injection capabilities.

From: Marius Vlad <marius.c.v...@intel.com> v2: - Make test work >= GEN8. - Skip audio injection for now. Signed-off-by: Marius Vlad <marius.c.v...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/Makefile.sources | 1 + tests/km

[Intel-gfx] [i-g-t PATCH 1/2] lib/igt_kms: Add support for 4K HDMI EDID injection.

From: Marius Vlad <marius.c.v...@intel.com> v2: Skip audio injection for now Signed-off-by: Marius Vlad <marius.c.v...@intel.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_kms.c | 85

[Intel-gfx] [i-g-t PATCH] intel-ci: Add initial generic testlist

Similar to fast-feedback test list. The point of the generic test list is to gather pass-rate when running generic tests - i.e., tests that don't rely on specific GPUs. Again, this test list is dynamic and will either shrink or grow over time. Signed-off-by: Abdiel Janulgue <abdiel.ja

[Intel-gfx] [PATCH] drm/i915: Support HDMI EDID injection

From: marius vlad <marius.c.v...@intel.com> Make a copy of drm_property_blob data for user-supplied EDID blobs. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> Signed-off-by: Marius Vlad <marius.c.v...@intel.com> --- drivers/gpu/drm/i915/intel_hdmi.c | 10

Re: [Intel-gfx] [PATCH] igt/gem_exec_schedule: Use igt_spin_batch

On 24.01.2017 11:26, Chris Wilson wrote: > On Tue, Jan 24, 2017 at 08:33:13AM +0200, Abdiel Janulgue wrote: > > It is not equivalent. gem_exec_schedule needs to emit multiple recursive > batches to ensure that the ELSP ports are filled (a bit of inside driver > knowledge leakin

[Intel-gfx] [PATCH] igt/gem_exec_schedule: Use igt_spin_batch

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_exec_schedule.c | 120 ++ 1 file changed, 15 insertions(+), 105 deletions(-) diff --git a/tests/gem_exec_schedule.c b/tests/gem_exec_schedule.c index d8454d5..87d5a85

Re: [Intel-gfx] [PATCH i-g-t] igt_kms: Dynamically allocate igt_display->pipes

dynamically allocating > igt_display->pipes using the number of pipes we've detected on the GPU. > > Signed-off-by: Lyude <ly...@redhat.com> Reviewed-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > --- > lib/igt_kms.c | 2 ++ > lib/igt_kms.h | 2 +- &g

Re: [Intel-gfx] [PATCH i-g-t] lib: Use igt_assert_eq in CHECK_RETURN

On 12/07/2016 08:36 AM, Tomeu Vizoso wrote: > So that debug logs contain the unexpected value. > > Signed-off-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Reviewed-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > --- > lib/igt_kms.c | 2 +- > 1 f

[Intel-gfx] [i-g-t PATCH 2/2] igt/kms_flip: Fix set_dpms called with an idle bo

Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/kms_flip.c b/tests/kms_flip.c index 289335a..f744b3d 100644 --- a/tests/kms_flip.c +++ b/tests/kms_flip.c @@ -757,9 +757,9 @@

[Intel-gfx] [i-g-t PATCH 1/2] igt_dummyload: clear signal handler on the desructor

Fixes an issue when calling igt_spin_batch_set_timeout and then tearing down the spinner right away before it has the chance to timeout, causes the associated signal handler to linger. Make sure to remove the handler on the destructor as well. Signed-off-by: Abdiel Janulgue <abdiel.ja

Re: [Intel-gfx] [PATCH i-g-t] lib/igt_aux.h: Cast is confusing old gcc

properly. Thanks, this is how kernel's LIST_HEAD_INIT looks like anyway Reviewed-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> > > Compilation Error log: > CC igt_dummyload.lo > In file included from igt.h:30:0, > from igt_dummyload.c:25: > ig

Re: [Intel-gfx] [i-g-t PATCH] lib: add option for running generic tests cases only

On 11/28/2016 02:20 PM, Chris Wilson wrote: > On Mon, Nov 28, 2016 at 01:07:36PM +0100, Maarten Lankhorst wrote: >> Op 28-11-16 om 10:37 schreef Abdiel Janulgue: >>> Pretend to run on a non-intel machine even when running on i915.ko, >>> so that we could run and gath

[Intel-gfx] [i-g-t PATCH] lib: add option for running generic tests cases only

Pretend to run on a non-intel machine even when running on i915.ko, so that we could run and gather passrate data. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/drmtest.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/drmtest.c

[Intel-gfx] [i-g-t PATCH v15 3/6] lib: add igt_dummyload

ille.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- .../intel-gpu-tools/intel-gpu-tools-docs.xml

[Intel-gfx] [i-g-t PATCH 2/6] igt_aux: Add some list helpers from wayland

From: Lyude Since we're going to be using lists for keeping track of EDIDs we've allocated on the chamelium, add some generic list helpers from the wayland project. Signed-off-by: Lyude Changes since v1: - Rename list helpers to be more like those from the

[Intel-gfx] [i-g-t PATCH v14 3/6] lib: add igt_dummyload

n from Chamelium patches. Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue &l

[Intel-gfx] [i-g-t PATCH 2/6] igt_aux: Add some list helpers from the kernel

Since we're going to be using lists for keeping track of spinners add some generic list helpers from the kernel. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_aux.c | 39 +++ lib/igt_aux.

[Intel-gfx] [i-g-t PATCH v13 3/6] lib: add igt_dummyload

: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- .../intel-gpu-tools/intel-gpu-tools-docs.xml | 1 + lib/Makefile.sources

[Intel-gfx] [i-g-t PATCH v12 2/5] lib: add igt_dummyload

he GPU if the user terminates tests with ^C Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com>

[Intel-gfx] [i-g-t PATCH v11 2/5] lib: add igt_dummyload

ille Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- .../intel-gpu-tools/intel-g

[Intel-gfx] [i-g-t PATCH v10 1/5] lib: Make signal helper definitions reusable

Lots of test cases are re-declaring this. v2: Remove definition in benchmarks/gem_syslatency.c Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- benchmarks/gem_syslatency.c | 4 lib/igt_aux.c | 11 --- lib/igt_aux.h

[Intel-gfx] [i-g-t PATCH v10 2/5] lib: add igt_dummyload

names (Chris) Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdi

[Intel-gfx] [i-g-t PATCH v10 5/5] igt/kms_busy.c: Use new igt_spin_batch

v7: Adapt to api rename v8: Tidy up finish_fb_busy (Chris Wilson) v10: Adapt to api rename Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.jan

[Intel-gfx] [i-g-t PATCH v10 4/5] igt/kms_flip: Use new igt_spin_batch

o <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 212 +++ 1 file changed, 26 insertions(+), 186 deletions(-) diff --git a/tests/kms_flip.c b/tests/kms_flip.c index 2a9fe2e..28

[Intel-gfx] [i-g-t PATCH v10 3/5] igt/gem_wait: Use new igt_spin_batch

@ffwll.ch> Reviewed-by: Tomeu Vizoso <tomeu.viz...@collabora.com> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 124 +++ 1 file changed, 6 insertions(+), 118 deletions(-) diff --git a/

[Intel-gfx] i-g-t dummyload/spin batch v10

Mostly reviewed by Tomeu but includes last minute changes from Chris: split out the timeout function and api naming clarifications. - Abdiel ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org

[Intel-gfx] [i-g-t PATCH v8 1/5] lib: Make signal helper definitions reusable

v2: Remove definition in benchmarks/gem_syslatency.c More and more test-cases are using this. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- benchmarks/gem_syslatency.c | 4 lib/igt_aux.c | 11 --- lib/igt_aux.h

[Intel-gfx] [i-g-t PATCH v9 1/3] igt/gem_wait: Use new igt_spin_batch

Tomeu Vizoso <to...@tomeuvizoso.net> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 126 --- 1 file changed, 7 insertions(+), 119 deletions(-) diff --git a/tests/gem_wait.c b/tests/gem_wait.c index

Re: [Intel-gfx] [i-g-t PATCH v8 3/5] igt/gem_wait: Use new igt_spin_batch

On 11/21/2016 01:33 PM, Tomeu Vizoso wrote: > On 21 November 2016 at 10:25, Abdiel Janulgue > <abdiel.janul...@linux.intel.com> wrote: >> v7: Adapt to api rename >> v8: Restore sanitycheck wait on the recursive batch and >> avoid using C99 locals (Chris Wils

[Intel-gfx] [i-g-t PATCH v8 5/5] igt/kms_busy.c: Use new igt_spin_batch

v7: Adapt to api rename v8: Tidy up finish_fb_busy (Chris Wilson) Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tes

[Intel-gfx] [i-g-t PATCH v8 3/5] igt/gem_wait: Use new igt_spin_batch

v7: Adapt to api rename v8: Restore sanitycheck wait on the recursive batch and avoid using C99 locals (Chris Wilson) Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> ---

[Intel-gfx] [i-g-t PATCH v8 4/5] igt/kms_flip: Use new igt_spin_batch

v7: Reuse NSEC_PER_SEC defines v8: Don't wait on the fb, it must be busy. Exit the spin batch when flip is queued (Chris Wilson) Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.

[Intel-gfx] [i-g-t PATCH v9 2/5] lib: add igt_dummyload

gt; Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- .../intel-gpu-tools/intel-gpu-tools-docs.xml | 1

[Intel-gfx] [i-g-t PATCH v7 1/5] lib: Make signal helper definitions reusable

More and more test-cases are using this. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_aux.c | 11 --- lib/igt_aux.h | 10 ++ lib/igt_core.c | 3 --- tests/drv_hangman.c | 1 - 4 files changed, 10 insertions(+), 15 del

[Intel-gfx] i-g-t dummyload/spin batch v8

Changes: * lib/dummyload: Add a helper to terminate recursive batch manually. * gem_wait: Restore sanitycheck wait on the recursive batch and avoid using C99 locals. * kms_flip: Don't wait on the fb - should be busy. Exit the spin batch when flip is queued * kms_busy: Tidy up finish_fb_busy

Re: [Intel-gfx] [i-g-t PATCH v7 5/5] igt/kms_busy.c: Use new igt_spin_batch

On 17.11.2016 12:27, Chris Wilson wrote: > On Wed, Nov 16, 2016 at 11:18:04PM +0200, Abdiel Janulgue wrote: >> @@ -179,7 +111,8 @@ static void flip_to_fb(igt_display_t *dpy, int pipe, >> } >> igt_assert_f(nanosleep(, NULL) == -1, >>

[Intel-gfx] [i-g-t PATCH v8 2/5] lib: add igt_dummyload

intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- .../intel-gpu-tools/intel-gpu-tools-docs.xml | 1 + lib/Makefile.sources

Re: [Intel-gfx] [i-g-t PATCH v7 2/5] lib: add igt_dummyload

On 16.11.2016 15:56, Chris Wilson wrote: > On Wed, Nov 16, 2016 at 11:18:01PM +0200, Abdiel Janulgue wrote: >> A lot of igt testcases need some GPU workload to make sure a race >> window is big enough. Unfortunately having a fixed amount of >> workload leads to spurious tes

[Intel-gfx] [i-g-t PATCH v7 4/5] igt/kms_flip: Use new igt_spin_batch

v7: Reuse NSEC_PER_SEC defines Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 188 ++- 1 file cha

[Intel-gfx] [i-g-t PATCH v7 3/5] igt/gem_wait: Use new igt_spin_batch

v7: Adapt to api rename Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 129 +++ 1 file changed, 7

[Intel-gfx] [i-g-t PATCH v7 2/5] lib: add igt_dummyload

respectively (Tomeu Vizoso). Fix error in dependency relocation handling in HSW causing tests to fail. Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: to...@tomeuvizoso.net Sig

[Intel-gfx] [i-g-t PATCH v7 1/5] lib: Make signal helper definitions reusable

More and more test-cases are using this. Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/igt_aux.c | 11 --- lib/igt_aux.h | 10 ++ lib/igt_core.c | 3 --- tests/drv_hangman.c | 1 - 4 files changed, 10 insertions(+), 15 del

[Intel-gfx] i-g-t dummyload/spin batch v7

* Fix for Haswell in generating a dummy reloc * code cleanups / api name clarifications - Abdiel ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: [Intel-gfx] [i-g-t PATCH v6 1/4] lib: add igt_dummyload

On 15.11.2016 12:59, Tomeu Vizoso wrote: > On 14 November 2016 at 19:24, Abdiel Janulgue > <abdiel.janul...@linux.intel.com> wrote: >> A lot of igt testcases need some GPU workload to make sure a race >> window is big enough. Unfortunately having a fixed amount of >&

Re: [Intel-gfx] [i-g-t PATCH v6 4/4] igt/kms_busy.c: Use new igt_spin_batch

and a cover letter as the one generated by > git-send-email would be good to have, even if with very succinct > contents. Forgot to include the subject in git-send-email. Sorry about that... > > Thanks, > > Tomeu > > On 14 November 2016 at 19:24, Abdiel Janulgue >

[Intel-gfx] [i-g-t PATCH v6 4/4] igt/kms_busy.c: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_busy.c | 75 +++- 1 file changed, 4 insertions(+), 71 deleti

[Intel-gfx] (no subject)

On 11.11.2016 18:16, Daniel Vetter wrote: > On Fri, Nov 11, 2016 at 07:41:10PM +0200, Abdiel Janulgue wrote: >> A lot of igt testcases need some GPU workload to make sure a race >> window is big enough. Unfortunately having a fixed amount of >> workload leads to spurious tes

[Intel-gfx] [i-g-t PATCH v6 3/4] igt/kms_flip: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 185 ++- 1 file changed, 4 insertions(+), 181 deleti

[Intel-gfx] [i-g-t PATCH v6 1/4] lib: add igt_dummyload

of batchbuffer start on ILK causing test failure v5: Convert kms_busy to use this api v6: Add this library to docs Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Signed-off-by: Abdiel Janu

[Intel-gfx] [i-g-t PATCH v6 2/4] igt/gem_wait: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 125 --- 1 file changed, 7 insertions(+), 118 deleti

[Intel-gfx] [PATCH i-g-t v5 4/4] igt/kms_busy.c: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_busy.c | 75 +++- 1 file changed, 4 insertions(+), 71 deleti

[Intel-gfx] i-g-t dummyload/spin batch v5

v5: Add kms_busy to satisfy rule of 3. - Abdiel [PATCH 1/4] lib: add igt_dummyload [PATCH 2/4] igt/gem_wait: Use new igt_spin_batch [PATCH 3/4] igt/kms_flip: Use new igt_spin_batch [PATCH 4/4] igt/kms_busy.c: Use new igt_spin_batch ___ Intel-gfx

[Intel-gfx] [PATCH i-g-t v5 1/4] lib: add igt_dummyload

of batchbuffer start on ILK causing test failure Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/Makefile.so

[Intel-gfx] [PATCH i-g-t v5 3/4] igt/kms_flip: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 185 ++- 1 file changed, 4 insertions(+), 181 deleti

[Intel-gfx] [PATCH i-g-t v5 2/4] igt/gem_wait: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 125 --- 1 file changed, 7 insertions(+), 118 deleti

Re: [Intel-gfx] [PATCH v4 1/3] lib: add igt_dummyload

By the way, series was tested on CI with no regressions. - Abdiel On 11/07/2016 09:48 AM, Abdiel Janulgue wrote: > A lot of igt testcases need some GPU workload to make sure a race > window is big enough. Unfortunately having a fixed amount of > workload leads to spurious test

[Intel-gfx] [PATCH v4 2/3] igt/gem_wait: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 125 --- 1 file changed, 7 insertions(+), 118 deleti

[Intel-gfx] [PATCH v4 1/3] lib: add igt_dummyload

of batchbuffer start on ILK causing test failure Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com> Cc: Chris Wilson <ch...@chris-wilson.co.uk> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- lib/Makefile.so

[Intel-gfx] [PATCH v4 3/3] igt/kms_flip: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 185 ++- 1 file changed, 4 insertions(+), 181 deleti

Re: [Intel-gfx] [PATCH v3 1/3] lib: add igt_dummyload

On 11/03/2016 04:38 PM, Ville Syrjälä wrote: > On Thu, Nov 03, 2016 at 11:40:36AM +0200, Abdiel Janulgue wrote: >> A lot of igt testcases need some GPU workload to make sure a race >> window is big enough. Unfortunately having a fixed amount of >> workload leads to s

[Intel-gfx] [PATCH v3 3/3] igt/kms_flip: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/kms_flip.c | 185 ++- 1 file changed, 4 insertions(+), 181 deleti

[Intel-gfx] [PATCH v3 2/3] igt/gem_wait: Use new igt_spin_batch

Cc: Chris Wilson <ch...@chris-wilson.co.uk> Cc: Daniel Vetter <daniel.vet...@ffwll.ch> Signed-off-by: Abdiel Janulgue <abdiel.janul...@linux.intel.com> --- tests/gem_wait.c | 125 --- 1 file changed, 7 insertions(+), 118 deleti

<    1   2   3   >