[Intel-gfx] [lkp-robot] [drm/fb] a583bc678d: WARNING:at_kernel/workqueue.c:#flush_workqueue

2017-09-16 Thread kernel test robot
FYI, we noticed the following commit: commit: a583bc678d0a5e54844d3380e242dd9bb5fb0b00 ("drm/fb-helper: Allocate a drm_file") url: https://github.com/0day-ci/linux/commits/Noralf-Tr-nnes/drm-fb-helper-Use-drm_file-to-get-a-dumb-framebuffer/20170914-132818 base: git://people.freedesktop.org/~air

[Intel-gfx] [PATCH 4/4] drm/i915/execlists: Unwind incomplete requests on resets

2017-09-16 Thread Chris Wilson
Given the mechanism to unwind and replay requests (designed to support preemption), we have an alternative to the current method of resubmitting the ELSP upon reset. Resubmitting ELSP turns out to be more complicated than expected, due to having to handle lost context-switch interrupts and so guess

[Intel-gfx] [PATCH 1/4] drm/i915/execlists: Kick start request processing after a reset

2017-09-16 Thread Chris Wilson
During a reset, we may skip over completed requests and lost context-switch interrupts. Following the reset, we may then may end up with no active requests in the ELSP (and so do not resubmit to restart the engine), but have a queue of requests ready for execution. This is unlikely, it requires the

[Intel-gfx] [PATCH 2/4] drm/i915/execlists: Move insert_request()

2017-09-16 Thread Chris Wilson
Move insert_request() earlier to avoid a forward declaration in a later patch. Signed-off-by: Chris Wilson Reviewed-by: Michał Winiarski --- drivers/gpu/drm/i915/intel_lrc.c | 128 +++ 1 file changed, 64 insertions(+), 64 deletions(-) diff --git a/drivers/gp

[Intel-gfx] [PATCH 3/4] drm/i915/execlists: Split insert_request()

2017-09-16 Thread Chris Wilson
In the next patch we will want to reinsert a request not at the end of the priority queue, but at the front. Here we split insert_request() into two, the first function retrieves the priority list (for reuse for unsubmit later) and a wrapper function to insert at the end of that list and to schedul

Re: [Intel-gfx] [PATCH 0/8] drm/fb-helper: Use drm_file to get a dumb framebuffer

2017-09-16 Thread Noralf Trønnes
Den 16.09.2017 14.37, skrev Noralf Trønnes: Den 15.09.2017 18.37, skrev Noralf Trønnes: Den 15.09.2017 00.29, skrev Laurent Pinchart: Hi Noralf, On Wednesday, 13 September 2017 18:19:22 EEST Noralf Trønnes wrote: Den 13.09.2017 07.09, skrev Laurent Pinchart: On Monday, 11 September 2017 1

Re: [Intel-gfx] [PATCH 0/8] drm/fb-helper: Use drm_file to get a dumb framebuffer

2017-09-16 Thread Noralf Trønnes
Den 15.09.2017 18.37, skrev Noralf Trønnes: Den 15.09.2017 00.29, skrev Laurent Pinchart: Hi Noralf, On Wednesday, 13 September 2017 18:19:22 EEST Noralf Trønnes wrote: Den 13.09.2017 07.09, skrev Laurent Pinchart: On Monday, 11 September 2017 17:31:54 EEST Noralf Trønnes wrote: Hi, I wan

Re: [Intel-gfx] [PATCH] Implement Limited Video Range

2017-09-16 Thread Peter Frühberger
Thanks for your comments, I tried to implement a "flag driven version" below. On Fri, Nov 11, 2016 at 2:57 PM, Ville Syrjälä < ville.syrj...@linux.intel.com> wrote: > On Fri, Nov 11, 2016 at 09:53:35AM +0100, Peter Frühberger wrote: > > Hi, > > > > I was implementing this suggestion today and I t

Re: [Intel-gfx] [PATCH 01/11] drm/i915/guc: Pass intel_guc to intel_guc_suspend/resume instead of drm_i915_private

2017-09-16 Thread Michal Wajdeczko
On Fri, 15 Sep 2017 23:06:19 +0200, Sagar Arun Kamble wrote: Also put the declaration under the i915_guc_submission section in intel_uc.h. Cc: Michal Wajdeczko Cc: Michał Winiarski Signed-off-by: Sagar Arun Kamble --- drivers/gpu/drm/i915/i915_drv.c| 6 +++--- drivers/gpu/dr

Re: [Intel-gfx] [PATCH 05/11] drm/i915: Reorganize HuC authentication

2017-09-16 Thread Michal Wajdeczko
On Fri, 15 Sep 2017 23:06:23 +0200, Sagar Arun Kamble wrote: Prepared intel_auth_huc to separate HuC specific functionality from GuC send action. Created new header intel_huc.h to group HuC specific declarations. Cc: Michal Wajdeczko Cc: Michał Winiarski Signed-off-by: Sagar Arun Kamble -

Re: [Intel-gfx] [PATCH 11/11] drm/i915/guc: Remove i915_guc_log_unregister

2017-09-16 Thread Michal Wajdeczko
On Fri, 15 Sep 2017 23:06:29 +0200, Sagar Arun Kamble wrote: Functionality needed to disable GuC interrupts and cleanup the runtime/relay data structures is already covered in the unload path via intel_guc_fini_hw and intel_guc_cleanup hence remove i915_guc_log_unregister v2: Removed the fun

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/execlists: Kick start request processing after a reset

2017-09-16 Thread Patchwork
== Series Details == Series: drm/i915/execlists: Kick start request processing after a reset URL : https://patchwork.freedesktop.org/series/30465/ State : success == Summary == Series 30465v1 drm/i915/execlists: Kick start request processing after a reset https://patchwork.freedesktop.org/api/

Re: [Intel-gfx] [PATCH 10/11] drm/i915/guc: Enable default/critical logging in GuC by default from GuC v9

2017-09-16 Thread Michal Wajdeczko
On Fri, 15 Sep 2017 23:06:28 +0200, Sagar Arun Kamble wrote: With GuC v9, new type of Default/critical logging in GuC to enable capturing minimal important logs in production systems efficiently. This patch enables this logging in GuC by default always. It should be noted that streaming suppo

[Intel-gfx] [PATCH] drm/i915/execlists: Kick start request processing after a reset

2017-09-16 Thread Chris Wilson
During a reset, we may skip over completed requests and lost context-switch interrupts. Following the reset, we may then may end up with no active requests in the ELSP (and so do not resubmit to restart the engine), but have a queue of requests ready for execution. This is unlikely, it requires the