[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [CI,1/5] drm/i915: Use snprintf to avoid line-break when pretty-printing engines

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915: Use snprintf to avoid line-break when pretty-printing engines URL : https://patchwork.freedesktop.org/series/35066/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Unwind i915_gem_init() failure (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Unwind i915_gem_init() failure (rev2) URL : https://patchwork.freedesktop.org/series/35060/ State : failure == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render: pass -> FAIL

Re: [Intel-gfx] [PATCH] drm/i915: Increase max texture to 16k for gen9+

2017-12-07 Thread Alex Tu
looks it's not that easy as I thought. This change is for https://bugs.freedesktop.org/show_bug.cgi?id=102508 , and the hardware configuration is * 00:02.0 VGA compatible controller [0300]: Intel Corporation Device [8086:591d] (rev 04) * Dell Thunderbolt BME dock :

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Unwind i915_gem_init() failure

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Unwind i915_gem_init() failure URL : https://patchwork.freedesktop.org/series/35060/ State : warning == Summary == Test drv_module_reload: Subgroup basic-reload: pass -> DMESG-WARN (shard-snb) fdo#102848 Test kms_flip:

[Intel-gfx] ✗ Fi.CI.BAT: failure for tests/perf: Enable i915 perf tests on CNL

2017-12-07 Thread Patchwork
== Series Details == Series: tests/perf: Enable i915 perf tests on CNL URL : https://patchwork.freedesktop.org/series/35067/ State : failure == Summary == IGT patchset build failed on latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak wait_for_rc6, yet again

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] drm/i915: Use snprintf to avoid line-break when pretty-printing engines

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915: Use snprintf to avoid line-break when pretty-printing engines URL : https://patchwork.freedesktop.org/series/35066/ State : success == Summary == Series 35066v1 series starting with [CI,1/5] drm/i915: Use snprintf to avoid

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Unwind i915_gem_init() failure (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Unwind i915_gem_init() failure (rev2) URL : https://patchwork.freedesktop.org/series/35060/ State : success == Summary == Series 35060v2 drm/i915: Unwind i915_gem_init() failure https://patchwork.freedesktop.org/api/1.0/series/35060/revisions/2/mbox/

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/5] igt_dummyload: Wrap function parameters into struct

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/5] igt_dummyload: Wrap function parameters into struct URL : https://patchwork.freedesktop.org/series/35059/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render:

[Intel-gfx] [PATCH i-g-t 6/6] tests/perf: add test config uuid for Cannonlake

2017-12-07 Thread Lionel Landwerlin
This will enable running the tests on Cannonlake. Signed-off-by: Lionel Landwerlin --- tests/perf.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/perf.c b/tests/perf.c index cc0c21a1..99bcdc72 100644 --- a/tests/perf.c +++ b/tests/perf.c @@ -1082,6

[Intel-gfx] [PATCH i-g-t 4/6] tests/perf: query CS timestamp frequency if available

2017-12-07 Thread Lionel Landwerlin
On Cannonlake+ the CS timestamp frequency might vary from one part to another. We have a new param to query this from the kernel (which reads the value from registers). Signed-off-by: Lionel Landwerlin --- tests/perf.c | 31 +-- 1 file

[Intel-gfx] [PATCH i-g-t 2/6] tests/perf: drop copied i915 defines/structs

2017-12-07 Thread Lionel Landwerlin
Now that we have drm uapi headers in tree, we can drop this stuff. Signed-off-by: Lionel Landwerlin --- tests/perf.c | 114 --- 1 file changed, 7 insertions(+), 107 deletions(-) diff --git a/tests/perf.c

[Intel-gfx] [PATCH i-g-t 1/6] include/drm-uapi: bump headers

2017-12-07 Thread Lionel Landwerlin
Taken from drm-next : commit 9c606cd4117a3c45e04a6616b1a0dbeb1862 Merge: c5dd52f653fa 3997eea57caf Author: Dave Airlie Date: Thu Dec 7 06:28:22 2017 +1000 Merge branch 'drm-next-4.16' of git://people.freedesktop.org/~agd5f/linux into drm-next Signed-off-by:

[Intel-gfx] [PATCH i-g-t 3/6] tests/perf: factorize max oa buffer size define

2017-12-07 Thread Lionel Landwerlin
We use this value in several places. Signed-off-by: Lionel Landwerlin --- tests/perf.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/tests/perf.c b/tests/perf.c index 08ac106a..7f988d7c 100644 --- a/tests/perf.c +++ b/tests/perf.c

[Intel-gfx] [PATCH i-g-t 5/6] tests/perf: enable testing on Coffelake GT3

2017-12-07 Thread Lionel Landwerlin
Add the test config uuid for GT3. Signed-off-by: Lionel Landwerlin --- tests/perf.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/perf.c b/tests/perf.c index ef37ceb3..cc0c21a1 100644 --- a/tests/perf.c +++ b/tests/perf.c @@ -1075,8

[Intel-gfx] [PATCH i-g-t 0/6] tests/perf: Enable i915 perf tests on CNL

2017-12-07 Thread Lionel Landwerlin
Hi, Most of the work on the kernel side landed in drm-next. Time to enable this in IGT. Cheers, Lionel Landwerlin (6): include/drm-uapi: bump headers tests/perf: drop copied i915 defines/structs tests/perf: factorize max oa buffer size define tests/perf: query CS timestamp frequency if

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915/execlists: Cache ELSP register offset (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Cache ELSP register offset (rev2) URL : https://patchwork.freedesktop.org/series/35054/ State : warning == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: pass -> SKIP (shard-snb) fdo#102365

[Intel-gfx] [CI 1/5] drm/i915: Use snprintf to avoid line-break when pretty-printing engines

2017-12-07 Thread Chris Wilson
When printing the execlist ports, we first print the ELSP header then follow it with the pretty-printed request. Since switching to drm_printer and show the output via printk, it automatically appends a newline to each call (unlike the old seq_printf output). To avoid the unwanted line break,

[Intel-gfx] [CI 4/5] drm/i915: Include the global reset count for intel_engine_dump()

2017-12-07 Thread Chris Wilson
Since a global reset affects the engine, include that along side the per-engine reset counter when pretty printing the engine state in intel_engine_dump(). Signed-off-by: Chris Wilson Cc: Mika Kuoppala Reviewed-by: Mika Kuoppala

[Intel-gfx] [CI 3/5] drm/i915: Include engine state on detecting a missed breadcrumb/seqno

2017-12-07 Thread Chris Wilson
Now that we have a common engine state pretty printer, we can use that instead of the adhoc information printed when we miss a breadcrumb. v2: Rearrange intel_engine_disarm_breadcrumbs() to avoid calling intel_engine_dump() under the rb spinlock (Mika) and to pretty-print the error state early so

[Intel-gfx] [CI 2/5] drm/i915: Make engine state pretty-printer header configurable

2017-12-07 Thread Chris Wilson
Pass in a format string (and args) to specify the header to be emitted along with the engine state when pretty-printing. This allows the header to be emitted inside the drm_printer stream, so sharing the same prefix and output characteristics (e.g. debug level and filtering). Signed-off-by: Chris

[Intel-gfx] [CI 5/5] drm/i915: Add is-wedged flag to intel_engine_dump()

2017-12-07 Thread Chris Wilson
Comparing the state tested by intel_engine_is_idle() and printed by intel_engine_dump(), the only bit not shown is whether or not the device is wedged. Add that little bit of information to the pretty printer so that if the engine fails to idle we can see why. Signed-off-by: Chris Wilson

[Intel-gfx] [PATCH v2] drm/i915: Unwind i915_gem_init() failure

2017-12-07 Thread Chris Wilson
Since Michal introduced new errors other than -EIO during i915_gem_init(), we need to actually unwind on the error path as we have to abort the module load (and we expect to do so cleanly!). As we now teardown key state and then mark the driver as wedged (on EIO), we have to be careful to not

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Drop fb reference on load_detect_pipe failure path

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Drop fb reference on load_detect_pipe failure path URL : https://patchwork.freedesktop.org/series/35057/ State : warning == Summary == Test kms_flip: Subgroup flip-vs-modeset-vs-hang: pass -> DMESG-WARN (shard-snb) Test

Re: [Intel-gfx] [PATCH] Defined NM doesn't work on KBL and uses automatic N/M.

2017-12-07 Thread Wang, Quanxian
Hi, Jani I have created a Bugzilla in free desktop for tracking. https://bugs.freedesktop.org/show_bug.cgi?id=104093 Appreciate find audio expert could help us address root cause. Thank you. Quanxian -Original Message- From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org]

Re: [Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Refactor common list iteration over GGTT vma (rev3)

2017-12-07 Thread Chris Wilson
Quoting Patchwork (2017-12-08 00:32:47) > == Series Details == > > Series: drm/i915: Refactor common list iteration over GGTT vma (rev3) > URL : https://patchwork.freedesktop.org/series/35048/ > State : warning > > == Summary == > > Test kms_pipe_crc_basic: > Subgroup read-crc-pipe-b:

Re: [Intel-gfx] [PATCH v2] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Chris Wilson
Quoting Rodrigo Vivi (2017-12-07 23:04:13) > On Thu, Dec 07, 2017 at 10:24:34PM +, Chris Wilson wrote: > > Currently on every submission, we recalculate the ELSP register offset > > for the engine, after chasing the pointers to find the iomem base. Since > > this is fixed for the lifetime of

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Refactor common list iteration over GGTT vma (rev3)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Refactor common list iteration over GGTT vma (rev3) URL : https://patchwork.freedesktop.org/series/35048/ State : warning == Summary == Test kms_pipe_crc_basic: Subgroup read-crc-pipe-b: incomplete -> PASS (shard-hsw) Test

Re: [Intel-gfx] [PATCH] drm/i915: Unwind i915_gem_init() failure

2017-12-07 Thread Chris Wilson
Quoting Chris Wilson (2017-12-07 23:56:25) > Since Michal introduced new errors other than -EIO during > i915_gem_init(), we need to actually unwind on the error path as we have > to abort the module load (and we expect to do so cleanly!). > > As we now teardown key state and then mark the driver

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Unwind i915_gem_init() failure

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Unwind i915_gem_init() failure URL : https://patchwork.freedesktop.org/series/35060/ State : success == Summary == Series 35060v1 drm/i915: Unwind i915_gem_init() failure https://patchwork.freedesktop.org/api/1.0/series/35060/revisions/1/mbox/

[Intel-gfx] ✓ Fi.CI.IGT: success for lib/i915: Update detection of GuC submission

2017-12-07 Thread Patchwork
== Series Details == Series: lib/i915: Update detection of GuC submission URL : https://patchwork.freedesktop.org/series/35055/ State : success == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render: fail -> PASS

Re: [Intel-gfx] [PATCH igt] lib/i915: Update detection of GuC submission

2017-12-07 Thread Chris Wilson
Quoting Michal Wajdeczko (2017-12-07 23:30:54) > On Thu, 07 Dec 2017 22:02:59 +0100, Chris Wilson > wrote: > > > Recently the kernel has switched to using a combined i915.enable_guc > > rather than multiple i915.enable_guc_submission parameters. > > > > Signed-off-by:

[Intel-gfx] [PATCH] drm/i915: Unwind i915_gem_init() failure

2017-12-07 Thread Chris Wilson
Since Michal introduced new errors other than -EIO during i915_gem_init(), we need to actually unwind on the error path as we have to abort the module load (and we expect to do so cleanly!). As we now teardown key state and then mark the driver as wedged (on EIO), we have to be careful to not

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Cache ELSP register offset URL : https://patchwork.freedesktop.org/series/35054/ State : warning == Summary == Test kms_flip: Subgroup flip-vs-modeset-vs-hang: pass -> DMESG-WARN (shard-snb) Test kms_cursor_crc:

Re: [Intel-gfx] [PATCH igt] lib/i915: Update detection of GuC submission

2017-12-07 Thread Michal Wajdeczko
On Thu, 07 Dec 2017 22:02:59 +0100, Chris Wilson wrote: Recently the kernel has switched to using a combined i915.enable_guc rather than multiple i915.enable_guc_submission parameters. Signed-off-by: Chris Wilson Cc: Michal Wajdeczko

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/5] igt_dummyload: Wrap function parameters into struct

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/5] igt_dummyload: Wrap function parameters into struct URL : https://patchwork.freedesktop.org/series/35059/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73

Re: [Intel-gfx] [PATCH v2] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Chris Wilson
Quoting Rodrigo Vivi (2017-12-07 23:04:13) > On Thu, Dec 07, 2017 at 10:24:34PM +, Chris Wilson wrote: > > Currently on every submission, we recalculate the ELSP register offset > > for the engine, after chasing the pointers to find the iomem base. Since > > this is fixed for the lifetime of

Re: [Intel-gfx] [PATCH v2] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Rodrigo Vivi
On Thu, Dec 07, 2017 at 10:24:34PM +, Chris Wilson wrote: > Currently on every submission, we recalculate the ELSP register offset > for the engine, after chasing the pointers to find the iomem base. Since > this is fixed for the lifetime of the driver record the offset in the > execlists

[Intel-gfx] ✗ Fi.CI.IGT: failure for lib/igt_sysfs: Let igt_sysfs_read|write return -errno (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: lib/igt_sysfs: Let igt_sysfs_read|write return -errno (rev2) URL : https://patchwork.freedesktop.org/series/34989/ State : failure == Summary == Test kms_pipe_crc_basic: Subgroup read-crc-pipe-b: incomplete -> PASS (shard-hsw) Test

[Intel-gfx] [PATCH i-g-t 5/5] igt_hang_ctx: Add preemptible parameter

2017-12-07 Thread Antonio Argenziano
This patch adds a parameter to make the hanging context conditionally pre-emptible. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_gt.c | 2 +- lib/igt_gt.h | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff

[Intel-gfx] [PATCH i-g-t 3/5] lib/igt_hang_ctx: Use recursive batch to hang ctx.

2017-12-07 Thread Antonio Argenziano
To hang a context we were effectively reimplementing a spinning batch and never stopping it. This patch reuses the recursive batch from igt_dummyload to hang a context. Cc: Chris Wilson Signed-off-by: Antonio Argenziano ---

[Intel-gfx] [PATCH i-g-t 4/5] igt_dummyload: Add preemptible parameter to recursive batch

2017-12-07 Thread Antonio Argenziano
This patch adds a parameter that allows to make the spinning batch pre-emptible by conditionally adding an arbitration point to the spinning loop. From RFC: - Implicitly initialize struct members to zero. (Chris) Cc: Chris Wilson Signed-off-by: Antonio

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/6] meson: Don't install headers

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/6] meson: Don't install headers URL : https://patchwork.freedesktop.org/series/35033/ State : failure == Summary == Test kms_flip: Subgroup flip-vs-panning: incomplete -> PASS (shard-hsw) Test

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/3] lib/igt_kms: Drop all stale events on first commit.

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/3] lib/igt_kms: Drop all stale events on first commit. URL : https://patchwork.freedesktop.org/series/35032/ State : failure == Summary == Test drv_suspend: Subgroup sysfs-reader-hibernate: fail -> SKIP

[Intel-gfx] ✗ Fi.CI.IGT: warning for tests/kms_color: Rename pipe tests to standard notation

2017-12-07 Thread Patchwork
== Series Details == Series: tests/kms_color: Rename pipe tests to standard notation URL : https://patchwork.freedesktop.org/series/35031/ State : warning == Summary == Test kms_pipe_crc_basic: Subgroup read-crc-pipe-b: incomplete -> PASS (shard-hsw) Test

[Intel-gfx] [PATCH i-g-t 2/5] igt_hang_ctx: Wrap hang_ctx parameters into struct

2017-12-07 Thread Antonio Argenziano
Cleanup the interface to create a hang on a context by folding its parameters into a struct. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 25 - lib/igt_gt.c| 41

[Intel-gfx] [PATCH i-g-t 1/5] igt_dummyload: Wrap function parameters into struct

2017-12-07 Thread Antonio Argenziano
Cleanup the interface for creating dummyloads by folding its parameters into a struct. Cc: Chris Wilson Signed-off-by: Antonio Argenziano --- lib/igt_dummyload.c | 19 +++ lib/igt_dummyload.h | 16

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/execlists: Cache ELSP register offset (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Cache ELSP register offset (rev2) URL : https://patchwork.freedesktop.org/series/35054/ State : success == Summary == Series 35054v2 drm/i915/execlists: Cache ELSP register offset

Re: [Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Drop fb reference on load_detect_pipe failure path

2017-12-07 Thread Chris Wilson
Quoting Patchwork (2017-12-07 22:20:31) > == Series Details == > > Series: drm/i915: Drop fb reference on load_detect_pipe failure path > URL : https://patchwork.freedesktop.org/series/35057/ > State : success > > == Summary == > > Series 35057v1 drm/i915: Drop fb reference on

[Intel-gfx] [PATCH v2] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Chris Wilson
Currently on every submission, we recalculate the ELSP register offset for the engine, after chasing the pointers to find the iomem base. Since this is fixed for the lifetime of the driver record the offset in the execlists struct. In practice the difference is negligible, it just happens to

Re: [Intel-gfx] [PATCH] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Chris Wilson
Quoting Michel Thierry (2017-12-07 22:19:05) > On 07/12/17 12:45, Chris Wilson wrote: > > Currently on every submission, we recalculate the ELSP register offset > > for the engine, after chasing the pointers to find the iomem base. Since > > this is fixed for the lifetime of the driver record the

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Drop fb reference on load_detect_pipe failure path

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Drop fb reference on load_detect_pipe failure path URL : https://patchwork.freedesktop.org/series/35057/ State : success == Summary == Series 35057v1 drm/i915: Drop fb reference on load_detect_pipe failure path

Re: [Intel-gfx] [PATCH] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Michel Thierry
On 07/12/17 12:45, Chris Wilson wrote: Currently on every submission, we recalculate the ELSP register offset for the engine, after chasing the pointers to find the iomem base. Since this is fixed for the lifetime of the driver record the offset in the execlists struct. In practice the

[Intel-gfx] linux-next: Signed-off-by missing for commit in the drm-intel-fixes tree

2017-12-07 Thread Stephen Rothwell
Hi all, Commit 365ad5df9caa ("drm/i915/gvt: Export intel_gvt_render_mmio_to_ring_id()") is missing a Signed-off-by from its committer. -- Cheers, Stephen Rothwell ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org

[Intel-gfx] ✗ Fi.CI.IGT: failure for tools: Cannonlake port clock programming

2017-12-07 Thread Patchwork
== Series Details == Series: tools: Cannonlake port clock programming URL : https://patchwork.freedesktop.org/series/35028/ State : failure == Summary == Test gem_tiled_swapping: Subgroup non-threaded: incomplete -> PASS (shard-hsw) fdo#104009 Test

Re: [Intel-gfx] [PATCH] drm/i915: Drop fb reference on load_detect_pipe failure path

2017-12-07 Thread Chris Wilson
Quoting Chris Wilson (2017-12-07 22:00:25) > When intel_modeset_setup_plane_state() fails drop the local framebuffer > reference before jumping to the error, otherwise we leak the framebuffer. > > Signed-off-by: Chris Wilson > Cc: Maarten Lankhorst

[Intel-gfx] ✓ Fi.CI.IGT: success for lib: Print other clients when DRM_SET_MASTER fails

2017-12-07 Thread Patchwork
== Series Details == Series: lib: Print other clients when DRM_SET_MASTER fails URL : https://patchwork.freedesktop.org/series/35023/ State : success == Summary == Test drv_module_reload: Subgroup basic-reload: dmesg-warn -> PASS (shard-snb) fdo#102848

Re: [Intel-gfx] [RFC i-g-t 3/5] lib/igt_hang_ctx: Use dummyload batch to hang ctx.

2017-12-07 Thread Antonio Argenziano
On 07/12/17 03:05, Chris Wilson wrote: Quoting Antonio Argenziano (2017-12-06 23:03:44) To hang a context we were effectively reimplementing a spinning batch and never stopping it. This patch reuses the recursive batch from igt_dummyload to hang a context. Cc: Chris Wilson

[Intel-gfx] ✗ Fi.CI.IGT: failure for igt/gem_mocs_settings: Force use of master device

2017-12-07 Thread Patchwork
== Series Details == Series: igt/gem_mocs_settings: Force use of master device URL : https://patchwork.freedesktop.org/series/35019/ State : failure == Summary == Test drv_suspend: Subgroup forcewake: skip -> PASS (shard-hsw) Subgroup sysfs-reader:

[Intel-gfx] [PATCH] drm/i915: Drop fb reference on load_detect_pipe failure path

2017-12-07 Thread Chris Wilson
When intel_modeset_setup_plane_state() fails drop the local framebuffer reference before jumping to the error, otherwise we leak the framebuffer. Signed-off-by: Chris Wilson Cc: Maarten Lankhorst Cc: Ville Syrjälä

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Refactor common list iteration over GGTT vma URL : https://patchwork.freedesktop.org/series/35048/ State : failure == Summary == Test drv_suspend: Subgroup forcewake: pass -> SKIP (shard-snb) skip

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Support creating a proxy object that links to other object's pages

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Support creating a proxy object that links to other object's pages URL : https://patchwork.freedesktop.org/series/35042/ State : warning == Summary == Test kms_setmode: Subgroup basic: fail -> PASS (shard-hsw)

[Intel-gfx] ✓ Fi.CI.IGT: success for drm: More debug info for fb leaks in mode_config_cleanup

2017-12-07 Thread Patchwork
== Series Details == Series: drm: More debug info for fb leaks in mode_config_cleanup URL : https://patchwork.freedesktop.org/series/35038/ State : success == Summary == Test drv_suspend: Subgroup forcewake: skip -> PASS (shard-hsw) Test kms_cursor_crc:

[Intel-gfx] ✗ Fi.CI.IGT: warning for intel/atomic: Stop updating legacy fb parameters (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: intel/atomic: Stop updating legacy fb parameters (rev2) URL : https://patchwork.freedesktop.org/series/34924/ State : warning == Summary == Test kms_cursor_crc: Subgroup cursor-64x64-suspend: skip -> PASS (shard-snb) fdo#102365

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Refactor common list iteration over GGTT vma (rev3)

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Refactor common list iteration over GGTT vma (rev3) URL : https://patchwork.freedesktop.org/series/35048/ State : success == Summary == Series 35048v3 drm/i915: Refactor common list iteration over GGTT vma

[Intel-gfx] [PULL] drm-intel-next

2017-12-07 Thread Rodrigo Vivi
Hi Dave, Here goes drm-intel-next-2017-12-01. As Gustavo already pointed out, when applying this and drm-misc-next you will get the build failure. And the fix up is: - enum plane plane; + enum i9xx_plane_id plane; Changes on this tag: - Init clock gate fix (Ville) - Execlists event

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/i915: Update detection of GuC submission

2017-12-07 Thread Patchwork
== Series Details == Series: lib/i915: Update detection of GuC submission URL : https://patchwork.freedesktop.org/series/35055/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak wait_for_rc6, yet

[Intel-gfx] [PATCH v3] drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Chris Wilson
In quite a few places, we have a list iteration over the vma on an object that only want to inspect GGTT vma. By construction, these are placed at the start of the list, so we have copied that knowledge into many callsites. Pull that knowledge back to i915_vma.h and provide a for_each_ggtt_vma()

[Intel-gfx] [PATCH v2] drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Chris Wilson
In quite a few places, we have a list iteration over the vma on an object that only want to inspect GGTT vma. By construction, these are placed at the start of the list, so we have copied that knowledge into many callsites. Pull that knowledge back to i915_vma.h and provide a for_each_ggtt_vma()

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Cache ELSP register offset URL : https://patchwork.freedesktop.org/series/35054/ State : success == Summary == Series 35054v1 drm/i915/execlists: Cache ELSP register offset

[Intel-gfx] [PATCH igt] lib/i915: Update detection of GuC submission

2017-12-07 Thread Chris Wilson
Recently the kernel has switched to using a combined i915.enable_guc rather than multiple i915.enable_guc_submission parameters. Signed-off-by: Chris Wilson Cc: Michal Wajdeczko --- lib/i915/gem_submission.c | 5 + 1 file changed, 5

Re: [Intel-gfx] [PATCH i-g-t v2] lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 04:59:36PM +, Chris Wilson wrote: > Quoting Michal Wajdeczko (2017-12-07 16:52:46) > > In some cases debugfs or sysfs may return errors that we > > want to check. Return -errno from helper functions to make > > asserts easier. > > > > v2: don't forget about EOF ret=0

Re: [Intel-gfx] [PATCH] kthread: finer-grained lockdep/cross-release completion

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 08:57:09PM +0100, Peter Zijlstra wrote: > On Thu, Dec 07, 2017 at 03:58:28PM +0100, Daniel Vetter wrote: > > > [ 85.069417] gem_exec_captur/2810 is trying to acquire lock: > > [ 85.069419] ((completion)>parked){+.+.}, at: [] > > kthread_park+0x3d/0x50 > > [

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: add FTRACE dependency for DRM_I915_TRACE_GEM

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: add FTRACE dependency for DRM_I915_TRACE_GEM URL : https://patchwork.freedesktop.org/series/35035/ State : success == Summary == Test kms_cursor_crc: Subgroup cursor-256x256-suspend: incomplete -> SKIP (shard-hsw) fdo#103375

[Intel-gfx] [PATCH] drm/i915/execlists: Cache ELSP register offset

2017-12-07 Thread Chris Wilson
Currently on every submission, we recalculate the ELSP register offset for the engine, after chasing the pointers to find the iomem base. Since this is fixed for the lifetime of the driver record the offset in the execlists struct. In practice the difference is negligible, it just happens to

Re: [Intel-gfx] [PATCH] kthread: finer-grained lockdep/cross-release completion

2017-12-07 Thread Peter Zijlstra
On Thu, Dec 07, 2017 at 03:58:28PM +0100, Daniel Vetter wrote: > [ 85.069417] gem_exec_captur/2810 is trying to acquire lock: > [ 85.069419] ((completion)>parked){+.+.}, at: [] > kthread_park+0x3d/0x50 > [ 85.069426] >but task is already holding lock: > [ 85.069428]

[Intel-gfx] ✗ Fi.CI.IGT: warning for tests/kms_vblank: Add test to ensure DRM_CAP_CRTC_IN_VBLANK_EVENT works correctly (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: tests/kms_vblank: Add test to ensure DRM_CAP_CRTC_IN_VBLANK_EVENT works correctly (rev2) URL : https://patchwork.freedesktop.org/series/34299/ State : warning == Summary == Test kms_cursor_crc: Subgroup cursor-256x256-suspend: pass

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/igt_sysfs: Let igt_sysfs_read|write return -errno (rev2)

2017-12-07 Thread Patchwork
== Series Details == Series: lib/igt_sysfs: Let igt_sysfs_read|write return -errno (rev2) URL : https://patchwork.freedesktop.org/series/34989/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak

Re: [Intel-gfx] [PATCH i-g-t] overlay: fix debugfs path when debugfs mounted on path '/debug'

2017-12-07 Thread Chris Wilson
Quoting changbin...@intel.com (2017-12-07 05:56:43) > From: Changbin Du > > It mistakenly set debugfs root path to "/debug/dri", so correct it. > > Signed-off-by: Changbin Du Reviewed-by: Chris Wilson -Chris

Re: [Intel-gfx] [PATCH 4/5] drm/i915: Introduce a non-blocking power domain for vblank interrupts

2017-12-07 Thread Pandiyan, Dhinakaran
On Wed, 2017-12-06 at 15:54 -0800, Rodrigo Vivi wrote: > On Wed, Dec 06, 2017 at 10:47:40PM +, Dhinakaran Pandiyan wrote: > > When DC states are enabled and PSR is active, the hardware enters DC5/DC6 > > states resulting in frame counter resets. The frame counter resets mess > > up the

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/6] meson: Don't install headers

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/6] meson: Don't install headers URL : https://patchwork.freedesktop.org/series/35033/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled.

2017-12-07 Thread Pandiyan, Dhinakaran
On Wed, 2017-12-06 at 23:16 +, Patchwork wrote: > == Series Details == > > Series: series starting with [1/5] drm/vblank: Do not update vblank counts if > vblanks are already disabled. > URL : https://patchwork.freedesktop.org/series/34996/ > State : failure > > == Summary == > >

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled.

2017-12-07 Thread Pandiyan, Dhinakaran
On Wed, 2017-12-06 at 23:16 +, Patchwork wrote: > == Series Details == > > Series: series starting with [1/5] drm/vblank: Do not update vblank counts if > vblanks are already disabled. > URL : https://patchwork.freedesktop.org/series/34996/ > State : failure > > == Summary == > >

Re: [Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/5] drm/vblank: Do not update vblank counts if vblanks are already disabled.

2017-12-07 Thread Pandiyan, Dhinakaran
On Wed, 2017-12-06 at 23:16 +, Patchwork wrote: > == Series Details == > > Series: series starting with [1/5] drm/vblank: Do not update vblank counts if > vblanks are already disabled. > URL : https://patchwork.freedesktop.org/series/34996/ > State : failure > > == Summary == > >

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] lib/draw: Use more typical form for computing swizzle addresses

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/2] lib/draw: Use more typical form for computing swizzle addresses URL : https://patchwork.freedesktop.org/series/35017/ State : success == Summary == Test perf: Subgroup blocking: pass -> FAIL

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/3] lib/igt_kms: Drop all stale events on first commit.

2017-12-07 Thread Patchwork
== Series Details == Series: series starting with [1/3] lib/igt_kms: Drop all stale events on first commit. URL : https://patchwork.freedesktop.org/series/35032/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73

[Intel-gfx] ✗ Fi.CI.IGT: warning for make stolen resource centric (rev6)

2017-12-07 Thread Patchwork
== Series Details == Series: make stolen resource centric (rev6) URL : https://patchwork.freedesktop.org/series/34256/ State : warning == Summary == Test kms_frontbuffer_tracking: Subgroup fbc-1p-offscren-pri-shrfb-draw-render: fail -> PASS (shard-snb)

[Intel-gfx] ✓ Fi.CI.BAT: success for tests/kms_color: Rename pipe tests to standard notation

2017-12-07 Thread Patchwork
== Series Details == Series: tests/kms_color: Rename pipe tests to standard notation URL : https://patchwork.freedesktop.org/series/35031/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak

Re: [Intel-gfx] [PATCH igt] lib: Print other clients when DRM_SET_MASTER fails

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 10:09:33AM +, Chris Wilson wrote: > It looks like there are some rogue processes running in CI that prevent > DRM_MASTER from being obtained. Dump the list of clients on failure to > make it more obvious what is being left behind. > > References:

Re: [Intel-gfx] [PATCH i-g-t 3/6] meson: Install benchmarks to $libexec/intel-gpu-tools/benchmarks

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 06:32:58PM +0200, Ville Syrjälä wrote: > On Thu, Dec 07, 2017 at 03:40:28PM +0200, Petri Latvala wrote: > > Signed-off-by: Petri Latvala > > Cc: Daniel Vetter > > --- > > benchmarks/meson.build | 4 > > 1 file

Re: [Intel-gfx] [PATCH i-g-t 1/6] meson: Don't install headers

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 03:40:26PM +0200, Petri Latvala wrote: > Until we can at least check for a matching ABI, the only supported way > of building is having the headers from the source checkout. > > Signed-off-by: Petri Latvala > Cc: Daniel Vetter

Re: [Intel-gfx] [PATCH] drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 04:27:17PM +, Chris Wilson wrote: > In quite a few places, we have a list iteration over the vma on an > object that only want to inspect GGTT vma. By construction, these are > placed at the start of the list, so we have copied that knowledge into > many callsites. Pull

Re: [Intel-gfx] [PATCH] drm: More debug info for fb leaks in mode_config_cleanup

2017-12-07 Thread Daniel Vetter
On Thu, Dec 07, 2017 at 04:52:04PM +0100, Noralf Trønnes wrote: > > Den 07.12.2017 15.49, skrev Daniel Vetter: > > We're spotting this very rarely in CI, but have no idea. Let's add > > more debug info about what's going on here. > > > > References:

[Intel-gfx] ✓ Fi.CI.BAT: success for tools: Cannonlake port clock programming

2017-12-07 Thread Patchwork
== Series Details == Series: tools: Cannonlake port clock programming URL : https://patchwork.freedesktop.org/series/35028/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak wait_for_rc6, yet again

[Intel-gfx] ✓ Fi.CI.BAT: success for lib: Print other clients when DRM_SET_MASTER fails

2017-12-07 Thread Patchwork
== Series Details == Series: lib: Print other clients when DRM_SET_MASTER fails URL : https://patchwork.freedesktop.org/series/35023/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak wait_for_rc6,

[Intel-gfx] ✓ Fi.CI.BAT: success for igt/gem_mocs_settings: Force use of master device

2017-12-07 Thread Patchwork
== Series Details == Series: igt/gem_mocs_settings: Force use of master device URL : https://patchwork.freedesktop.org/series/35019/ State : success == Summary == IGT patchset tested on top of latest successful build 2fc64acf8a4465d5eab3d6cfec9b3c1b5df30d73 igt/perf_pmu: Tweak wait_for_rc6,

Re: [Intel-gfx] [PATCH] drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Ville Syrjälä
On Thu, Dec 07, 2017 at 04:46:37PM +, Chris Wilson wrote: > Quoting Ville Syrjälä (2017-12-07 16:40:49) > > On Thu, Dec 07, 2017 at 04:27:17PM +, Chris Wilson wrote: > > > +#define for_each_ggtt_vma(V, OBJ) \ > > > + list_for_each_entry(V, &(OBJ)->vma_list, obj_link) \ > >

[Intel-gfx] [PATCH i-g-t v2] lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-07 Thread Michal Wajdeczko
In some cases debugfs or sysfs may return errors that we want to check. Return -errno from helper functions to make asserts easier. v2: don't forget about EOF ret=0 (Chris) small re-write (Michal) Signed-off-by: Michal Wajdeczko Cc: Chris Wilson

Re: [Intel-gfx] [PATCH i-g-t v2] lib/igt_sysfs: Let igt_sysfs_read|write return -errno

2017-12-07 Thread Chris Wilson
Quoting Michal Wajdeczko (2017-12-07 16:52:46) > In some cases debugfs or sysfs may return errors that we > want to check. Return -errno from helper functions to make > asserts easier. > > v2: don't forget about EOF ret=0 (Chris) > small re-write (Michal) > > Signed-off-by: Michal Wajdeczko

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Refactor common list iteration over GGTT vma

2017-12-07 Thread Patchwork
== Series Details == Series: drm/i915: Refactor common list iteration over GGTT vma URL : https://patchwork.freedesktop.org/series/35048/ State : success == Summary == Series 35048v1 drm/i915: Refactor common list iteration over GGTT vma

Re: [Intel-gfx] [PATCH i-g-t 3/6] meson: Install benchmarks to $libexec/intel-gpu-tools/benchmarks

2017-12-07 Thread Chris Wilson
Quoting Petri Latvala (2017-12-07 13:40:28) > Signed-off-by: Petri Latvala > Cc: Daniel Vetter > --- > benchmarks/meson.build | 4 > 1 file changed, 4 insertions(+) > > diff --git a/benchmarks/meson.build b/benchmarks/meson.build > index

  1   2   >