[Intel-gfx] [PATCH] drm/i915: Split i915_gem_timeline into individual timelines

2018-04-23 Thread Chris Wilson
We need to move to a more flexible timeline that doesn't assume one fence context per engine, and so allow for a single timeline to be used across a combination of engines. This means that preallocating a fence context per engine is now a hindrance, and so we want to introduce the singular timeline

[Intel-gfx] [PATCH igt] test/gem_exec_schedule: Check each engine is an independent timeline

2018-04-23 Thread Chris Wilson
In the existing ABI, each engine operates its own timeline (fence.context) and so should execute independently of any other. If we install a blocker on all other engines, that should not affect execution on the local engine. v2: Move the requirements checks from the fixture to subtest so that the

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2) URL : https://patchwork.freedesktop.org/series/42139/ State : warning == Summary == $ dim checkpatch origin/drm-tip 84a91bc19416 drm/i915: Stop tracking timeline->inflight_seqno

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/kbl: Add KBL GT2 sku (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: drm/i915/kbl: Add KBL GT2 sku (rev2) URL : https://patchwork.freedesktop.org/series/42144/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4081_full -> Patchwork_8779_full = == Summary - FAILURE == Serious unknown changes coming with Patchwork_

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2) URL : https://patchwork.freedesktop.org/series/42139/ State : warning == Summary == $ dim sparse origin/drm-tip Commit: drm/i915: Stop tracking timeline->inflight_seqnos -O:driv

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: series starting with [v2,1/6] drm/i915: Stop tracking timeline->inflight_seqnos (rev2) URL : https://patchwork.freedesktop.org/series/42139/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4082 -> Patchwork_8780 = == Summary - FAILURE == Serio

Re: [Intel-gfx] [PATCH v3 2/4] drm/i915/psr: Prevent PSR exit when a non-pipe related register is written

2018-04-23 Thread Souza, Jose
On Fri, 2018-04-20 at 15:57 -0700, Rodrigo Vivi wrote: > On Fri, Apr 20, 2018 at 03:27:56PM -0700, José Roberto de Souza > wrote: > > Any write in any display register was causing HW to exit PSR, > > masking it to allow more power savings. Writes to pipe related > > registers will still cause HW to

[Intel-gfx] [PATCH] drm/i915: Build request info on stack before printk

2018-04-23 Thread Chris Wilson
printk unhelpfully inserts a '\n' between consecutive calls, and since our drm_printf wrapper may be emitting info a seq_file instead, KERN_CONT is not an option. To work with any drm_printf destination, we need to build up the output into a temporary buf on the stack and then feed the complete lin

[Intel-gfx] [PATCH] drm/i915: Build request info on stack before printk

2018-04-23 Thread Chris Wilson
printk unhelpfully inserts a '\n' between consecutive calls, and since our drm_printf wrapper may be emitting info a seq_file instead, KERN_CONT is not an option. To work with any drm_printf destination, we need to build up the output into a temporary buf on the stack and then feed the complete lin

[Intel-gfx] [PATCH] drm/i915: Don't dump umpteen thousand requests

2018-04-23 Thread Chris Wilson
If we have more than a few, possibly several thousand request in the queue, don't show the central portion, just the first few and the last being executed and/or queued. The first few should be enough to help identify a problem in execution, and most often comparing the first/last in the queue is e

Re: [Intel-gfx] [PATCH] drm/i915: Build request info on stack before printk

2018-04-23 Thread Chris Wilson
Quoting Chris Wilson (2018-04-24 02:08:39) > printk unhelpfully inserts a '\n' between consecutive calls, and since > our drm_printf wrapper may be emitting info a seq_file instead, > KERN_CONT is not an option. To work with any drm_printf destination, we > need to build up the output into a tempor

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Build request info on stack before printk (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: drm/i915: Build request info on stack before printk (rev2) URL : https://patchwork.freedesktop.org/series/42150/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4082 -> Patchwork_8781 = == Summary - SUCCESS == No regressions found. External

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Don't dump umpteen thousand requests

2018-04-23 Thread Patchwork
== Series Details == Series: drm/i915: Don't dump umpteen thousand requests URL : https://patchwork.freedesktop.org/series/42151/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4082 -> Patchwork_8782 = == Summary - FAILURE == Serious unknown changes coming with Patchwork_

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Build request info on stack before printk (rev2)

2018-04-23 Thread Patchwork
== Series Details == Series: drm/i915: Build request info on stack before printk (rev2) URL : https://patchwork.freedesktop.org/series/42150/ State : success == Summary == = CI Bug Log - changes from CI_DRM_4082_full -> Patchwork_8781_full = == Summary - WARNING == Minor unknown changes co

[Intel-gfx] [PATCH 2/3] drm/i915/dp: Fix sink-crc reads.

2018-04-23 Thread Dhinakaran Pandiyan
Sink crc is calculated by the sink for static frames irrespective of what the driver sets in TEST_SINK_START dpcd. Since PSR is the only use case for sink crc, we don't really need the sink_crc_{start, stop} code. The second problem with the current implementation is vblank waits. Enabling vblank

[Intel-gfx] [PATCH 3/3] drm/i915/psr: Move sink-crc to intel_psr.c

2018-04-23 Thread Dhinakaran Pandiyan
With sink-crc now being relevant only for PSR static frames, move the code to intel_psr.c and rename the function. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915/i915_debugfs.c | 4 ++-- drivers/gpu/drm/i915/intel_dp.c | 35 --- drivers/gpu/drm/i

[Intel-gfx] [PATCH 1/3] drm/i915/dp: Check if the sink crc we read is 6 bytes.

2018-04-23 Thread Dhinakaran Pandiyan
We attempt to read 6 bytes, make sure we have. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915/intel_dp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 62f82c4298ac..7dcc874b7d8f 100644 ---

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915/dp: Check if the sink crc we read is 6 bytes.

2018-04-23 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/dp: Check if the sink crc we read is 6 bytes. URL : https://patchwork.freedesktop.org/series/42154/ State : failure == Summary == = CI Bug Log - changes from CI_DRM_4082 -> Patchwork_8783 = == Summary - FAILURE == Serious un

Re: [Intel-gfx] [PATCH v11 07/11] drm: Add helper functions to handle aspect-ratio flag bits

2018-04-23 Thread Nautiyal, Ankit K
On 4/23/2018 3:52 PM, Jani Nikula wrote: On Mon, 23 Apr 2018, "Nautiyal, Ankit K" wrote: On 4/20/2018 7:42 PM, Ville Syrjälä wrote: On Fri, Apr 20, 2018 at 07:01:47PM +0530, Nautiyal, Ankit K wrote: From: Ankit Nautiyal +bool +drm_mode_aspect_ratio_allowed(const struct drm_file *file_priv,

Re: [Intel-gfx] [PATCH v11 07/11] drm: Add helper functions to handle aspect-ratio flag bits

2018-04-23 Thread Nautiyal, Ankit K
On 4/23/2018 3:43 PM, Ville Syrjälä wrote: On Mon, Apr 23, 2018 at 10:55:54AM +0530, Nautiyal, Ankit K wrote: On 4/20/2018 7:42 PM, Ville Syrjälä wrote: On Fri, Apr 20, 2018 at 07:01:47PM +0530, Nautiyal, Ankit K wrote: From: Ankit Nautiyal This patch adds helper functions for determining

<    1   2