[Intel-gfx] [PATCH -next] drm/i915: Fix non static symbol warning

2016-09-16 Thread Wei Yongjun
From: Wei Yongjun Fixes the following sparse warning: drivers/gpu/drm/i915/intel_dp.c:1527:5: warning: symbol 'intel_dp_compute_bpp' was not declared. Should it be static? Signed-off-by: Wei Yongjun --- drivers/gpu/drm/i915/intel_dp.c | 4 ++--

Re: [Intel-gfx] [PATCH v5] drm/i915/dp: DP audio API changes for MST

2016-09-16 Thread Pandiyan, Dhinakaran
Ville who had R-B'd the previous version of the patch, confirmed this version "looks all right" via IRC. On Fri, 2016-09-02 at 10:53 -0700, Dhinakaran Pandiyan wrote: > DP MST provides the capability to send multiple video and audio streams > through a single port. This requires the API's

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v7,1/6] drm/i915: Fallback to lower link rate and lane count during link training (rev2)

2016-09-16 Thread Patchwork
== Series Details == Series: series starting with [v7,1/6] drm/i915: Fallback to lower link rate and lane count during link training (rev2) URL : https://patchwork.freedesktop.org/series/12534/ State : success == Summary == Series 12534v2 Series without cover letter

[Intel-gfx] [PATCH 2/2] drm/i915/execlists: Reset RING registers upon resume

2016-09-16 Thread Chris Wilson
There is a disparity in the context image saved to disk and our own bookkeeping - that is we presume the RING_HEAD and RING_TAIL match our stored ce->ring->tail value. However, as we emit WA_TAIL_DWORDS into the ring but may not tell the GPU about them, the GPU may be lagging behind our

[Intel-gfx] [PATCH 1/2] drm/i915: Only shrink the unbound objects during freeze

2016-09-16 Thread Chris Wilson
At the point of creating the hibernation image, the runtime power manage core is disabled - and using the rpm functions triggers a warn. i915_gem_shrink_all() tries to unbind objects, which requires device access and so tries to how an rpm reference triggering a warning: [ 44.235420]

[Intel-gfx] [PATCH v7 1/6] drm/i915: Fallback to lower link rate and lane count during link training

2016-09-16 Thread Manasi Navare
According to the DisplayPort Spec, in case of Clock Recovery failure the link training sequence should fall back to the lower link rate followed by lower lane count until CR succeeds. On CR success, the sequence proceeds with Channel EQ. In case of Channel EQ failures, it should fallback to lower

Re: [Intel-gfx] [PATCH] drm/i915: Restore current RPS state after reset

2016-09-16 Thread Mika Kuoppala
Chris Wilson writes: > Following commit 821ed7df6e2a ("drm/i915: Update reset path to fix > incomplete requests") we no longer mark the context as lost on reset as > we keep the requests (and contexts) alive. However, RPS remains reset > and we need to restore the

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/skl: drop pre-production stepping workarounds

2016-09-16 Thread Patchwork
== Series Details == Series: drm/i915/skl: drop pre-production stepping workarounds URL : https://patchwork.freedesktop.org/series/12579/ State : failure == Summary == Series 12579v1 drm/i915/skl: drop pre-production stepping workarounds

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Restore current RPS state after reset

2016-09-16 Thread Patchwork
== Series Details == Series: drm/i915: Restore current RPS state after reset URL : https://patchwork.freedesktop.org/series/12575/ State : warning == Summary == Series 12575v1 drm/i915: Restore current RPS state after reset

[Intel-gfx] [PATCH 3/5] drm/i915/skl: drop workarounds for D0 revision

2016-09-16 Thread Jani Nikula
Pre-production hardware is not supported. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_lrc.c| 10 -- drivers/gpu/drm/i915/intel_pm.c | 3 +-- drivers/gpu/drm/i915/intel_ringbuffer.c | 12 ++-- 3 files changed, 7 insertions(+),

[Intel-gfx] [PATCH 5/5] drm/i915/skl: drop workarounds for F0 revision

2016-09-16 Thread Jani Nikula
Pre-production hardware is not supported. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_ringbuffer.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c index

[Intel-gfx] [PATCH 4/5] drm/i915/skl: drop workarounds for E0 revision

2016-09-16 Thread Jani Nikula
Pre-production hardware is not supported. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_lrc.c| 5 ++--- drivers/gpu/drm/i915/intel_ringbuffer.c | 14 -- 2 files changed, 2 insertions(+), 17 deletions(-) diff --git

[Intel-gfx] [PATCH 2/5] drm/i915/skl: drop workarounds for C0 revision

2016-09-16 Thread Jani Nikula
Pre-production hardware is not supported. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_guc_loader.c | 3 +-- drivers/gpu/drm/i915/intel_ringbuffer.c | 5 ++--- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git

[Intel-gfx] [PATCH 1/5] drm/i915/skl: drop workarounds for A0 and B0 revisions

2016-09-16 Thread Jani Nikula
Pre-production hardware is not supported. Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dp.c | 4 drivers/gpu/drm/i915/intel_dp_link_training.c | 3 --- drivers/gpu/drm/i915/intel_guc_loader.c | 5 ++---

[Intel-gfx] [PATCH 0/5] drm/i915/skl: drop pre-production stepping workarounds

2016-09-16 Thread Jani Nikula
Only production steppings are supported, drop workarounds for anything else. The series is split by revision so we can bikeshed if there are steppings some people still need to use for some reason. BR, Jani. Jani Nikula (5): drm/i915/skl: drop workarounds for A0 and B0 revisions

[Intel-gfx] [PATCH] drm/i915: Restore current RPS state after reset

2016-09-16 Thread Chris Wilson
Following commit 821ed7df6e2a ("drm/i915: Update reset path to fix incomplete requests") we no longer mark the context as lost on reset as we keep the requests (and contexts) alive. However, RPS remains reset and we need to restore the current state to match the in-flight requests. Signed-off-by:

Re: [Intel-gfx] [PATCH 05/18] drm/i915: Move GEM activity tracking into a common struct reservation_object

2016-09-16 Thread Chris Wilson
On Wed, Sep 14, 2016 at 07:52:37AM +0100, Chris Wilson wrote: > In preparation to support many distinct timelines, we need to expand the > activity tracking on the GEM object to handle more than just a request > per engine. We already use the struct reservation_object on the dma-buf > to handle

Re: [Intel-gfx] [PATCH v3 4/9] drm/i915: Decode system memory bandwidth

2016-09-16 Thread Mahesh Kumar
On Friday 16 September 2016 01:32 PM, Pandiyan, Dhinakaran wrote: On Fri, 2016-09-09 at 13:31 +0530, Kumar, Mahesh wrote: From: Mahesh Kumar This patch adds support to decode system memory bandwidth which will be used for arbitrated display memory percentage

[Intel-gfx] ✓ Fi.CI.BAT: success for drm: fix implicit declaration build error on ia64

2016-09-16 Thread Patchwork
== Series Details == Series: drm: fix implicit declaration build error on ia64 URL : https://patchwork.freedesktop.org/series/12562/ State : success == Summary == Series 12562v1 drm: fix implicit declaration build error on ia64

Re: [Intel-gfx] [PATCH] drm: Fix DisplayPort branch device ID

2016-09-16 Thread Jani Nikula
On Fri, 16 Sep 2016, Mika Kahola wrote: > Fix missing parameter description for DisplayPort branch device ID. > This fixes warning of "No description found for parameter 'id[6]'" when > creating documentation by 'make htmldocs'. > > Signed-off-by: Mika Kahola

Re: [Intel-gfx] [PATCH i-g-t v5 02/13] tests/sw_sync: Add sw_sync test

2016-09-16 Thread Chris Wilson
On Thu, Sep 15, 2016 at 05:05:47PM -0400, Robert Foss wrote: > This is a solid improvement, igt_debugfs_mount() does not however > seem to be available on upstream/master. Done. igt_debugfs_mount() is now available. -Chris -- Chris Wilson, Intel Open Source Technology Centre

[Intel-gfx] [PATCH] drm: Fix DisplayPort branch device ID

2016-09-16 Thread Mika Kahola
Fix missing parameter description for DisplayPort branch device ID. This fixes warning of "No description found for parameter 'id[6]'" when creating documentation by 'make htmldocs'. Signed-off-by: Mika Kahola --- drivers/gpu/drm/drm_dp_helper.c | 1 + 1 file changed, 1

Re: [Intel-gfx] [PATCH i-g-t v5 06/13] tests/sw_sync: Add subtest test_sync_wait

2016-09-16 Thread Chris Wilson
On Thu, Sep 15, 2016 at 06:25:13PM -0400, Robert Foss wrote: > > > On 2016-09-15 04:32 PM, Chris Wilson wrote: > >On Thu, Sep 15, 2016 at 02:40:11PM -0400, robert.f...@collabora.com wrote: > >>From: Robert Foss > >> > >>This subtest verifies that waiting on fences

[Intel-gfx] ✓ Fi.CI.BAT: success for DRM: i915: Fix random GPU hang, Bug 156851

2016-09-16 Thread Patchwork
== Series Details == Series: DRM: i915: Fix random GPU hang, Bug 156851 URL : https://patchwork.freedesktop.org/series/12559/ State : success == Summary == Series 12559v1 DRM: i915: Fix random GPU hang, Bug 156851 https://patchwork.freedesktop.org/api/1.0/series/12559/revisions/1/mbox/ Test

[Intel-gfx] [PATCH] drm: fix implicit declaration build error on ia64

2016-09-16 Thread Jani Nikula
drivers/gpu/drm/drm_dp_helper.c: In function 'drm_dp_downstream_debug': >> drivers/gpu/drm/drm_dp_helper.c:551:2: error: implicit declaration of >> function 'seq_printf' [-Werror=implicit-function-declaration] seq_printf(m, "\tDP branch device present: %s\n", ^ >>

Re: [Intel-gfx] [PATCH v2 1/2] drm/i915: Cleanup instdone collection

2016-09-16 Thread Jani Nikula
On Tue, 06 Sep 2016, Mika Kuoppala wrote: > Imre Deak writes: > >> From: Ben Widawsky >> >> Consolidate the instdone logic so we can get a bit fancier. This patch also >> removes the duplicated print of

[Intel-gfx] [PATCH] DRM: i915: Fix gen8 graphics on Broadwell-U. These patches stop the random gpu hang on my XPS-13-9343, kernel version 4.8-rc5.

2016-09-16 Thread bobcao3
Signed-off-by: bobcao3 --- drivers/gpu/drm/i915/i915_gem_gtt.c | 6 drivers/gpu/drm/i915/i915_gem_stolen.c | 61 - drivers/gpu/drm/i915/i915_reg.h | 6 drivers/gpu/drm/i915/intel_ringbuffer.c | 19 +- 4 files

[Intel-gfx] [PATCH] DRM: i915: Fix random GPU hang, Bug 156851

2016-09-16 Thread Cheng Cao
Signed-off-by: Cheng Cao --- drivers/gpu/drm/i915/i915_gem_gtt.c | 6 drivers/gpu/drm/i915/i915_gem_stolen.c | 61 - drivers/gpu/drm/i915/i915_reg.h | 6 drivers/gpu/drm/i915/intel_ringbuffer.c | 20 ++- 4

Re: [Intel-gfx] [drm-intel:for-linux-next 6/12] htmldocs: drivers/gpu/drm/drm_dp_helper.c:523: warning: No description found for parameter 'id[6]'

2016-09-16 Thread Mika Kahola
I'll fix this. Cheers, Mika On Fri, 2016-09-16 at 10:53 +0300, Jani Nikula wrote: > Mika, please send in the documentation fix. > > BR, > Jani. > > > On Fri, 16 Sep 2016, kbuild test robot > wrote: > > > > tree:   git://anongit.freedesktop.org/drm-intel

Re: [Intel-gfx] [PATCH v6 1/6] drm/i915: Fallback to lower link rate and lane count during link training

2016-09-16 Thread Mika Kahola
On Thu, 2016-09-15 at 17:03 -0700, Manasi Navare wrote: > According to the DisplayPort Spec, in case of Clock Recovery failure > the link training sequence should fall back to the lower link rate > followed by lower lane count until CR succeeds. > On CR success, the sequence proceeds with Channel

Re: [Intel-gfx] [PATCH i-g-t v5 07/13] tests/sw_sync: Add subtest test_sync_merge

2016-09-16 Thread Chris Wilson
On Thu, Sep 15, 2016 at 08:27:15PM -0400, Robert Foss wrote: > > > On 2016-09-15 04:41 PM, Chris Wilson wrote: > >On Thu, Sep 15, 2016 at 02:40:12PM -0400, robert.f...@collabora.com wrote: > >>From: Robert Foss > >> > >>Add subtest test_sync_merge that tests merging

Re: [Intel-gfx] [PATCH v3 3/6] drm/i915: Change the placement of some static functions in intel_dp.c

2016-09-16 Thread Mika Kahola
Reviewed-by: Mika Kahola On Thu, 2016-09-15 at 17:04 -0700, Manasi Navare wrote: > These static helper functions are required to be used within upfront > link training related functions so they need to be placed at the top > of the file. It also changes macro dev to

Re: [Intel-gfx] [PATCH v3 4/9] drm/i915: Decode system memory bandwidth

2016-09-16 Thread Pandiyan, Dhinakaran
On Fri, 2016-09-09 at 13:31 +0530, Kumar, Mahesh wrote: > From: Mahesh Kumar > > This patch adds support to decode system memory bandwidth > which will be used for arbitrated display memory percentage > calculation in GEN9 based system. > > Signed-off-by: Mahesh Kumar

Re: [Intel-gfx] [drm-intel:for-linux-next 6/12] htmldocs: drivers/gpu/drm/drm_dp_helper.c:523: warning: No description found for parameter 'id[6]'

2016-09-16 Thread Jani Nikula
Mika, please send in the documentation fix. BR, Jani. On Fri, 16 Sep 2016, kbuild test robot wrote: > tree: git://anongit.freedesktop.org/drm-intel for-linux-next > head: 80209e5f2c42c491ec5f4a63705b4377b407587c > commit: 266d783baaf5f34a5bea3b56489f091451a89767

Re: [Intel-gfx] [PATCH 1/6] drm/i915: make intel_dp_compute_bpp static

2016-09-16 Thread Jani Nikula
On Thu, 15 Sep 2016, Jim Bride wrote: > On Thu, Sep 15, 2016 at 04:28:52PM +0300, Jani Nikula wrote: >> Fix sparse warning: >> >> drivers/gpu/drm/i915/intel_dp.c:1527:5: warning: symbol >> 'intel_dp_compute_bpp' was not declared. Should it be static? >> >> Fixes:

Re: [Intel-gfx] [PATCH 4/6] drm/i915: Code cleanup to use dev_priv and INTEL_GEN

2016-09-16 Thread Mika Kahola
Reviewed-by: Mika Kahola On Thu, 2016-09-15 at 17:04 -0700, Manasi Navare wrote: > Replace dev with dev_priv and INTEL_INFO with INTEL_GEN > > Signed-off-by: Manasi Navare > --- >  drivers/gpu/drm/i915/intel_dp.c | 14 +++--- >  1 file

Re: [Intel-gfx] 4.8-rc1: it is now common that machine needs re-run of xrandr after resume

2016-09-16 Thread Jani Nikula
On Thu, 15 Sep 2016, Martin Steigerwald wrote: > Am Mittwoch, 14. September 2016, 14:14:35 CEST schrieb Jani Nikula: >> On Wed, 14 Sep 2016, Jani Nikula wrote: >> > On Wed, 14 Sep 2016, Pavel Machek wrote: >> >> For the "sometimes

Re: [Intel-gfx] [PATCH v3 5/5] drm/i915: squash lines for simple wrapper functions

2016-09-16 Thread Masahiro Yamada
Hi Chris, 2016-09-16 15:15 GMT+09:00 Chris Wilson : > On Fri, Sep 16, 2016 at 10:40:23AM +0900, Masahiro Yamada wrote: >> Remove unneeded variables and assignments. >> >> Signed-off-by: Masahiro Yamada >> --- >> >> Changes in v3: >> -

Re: [Intel-gfx] [PATCH v3 5/5] drm/i915: squash lines for simple wrapper functions

2016-09-16 Thread Chris Wilson
On Fri, Sep 16, 2016 at 10:40:23AM +0900, Masahiro Yamada wrote: > Remove unneeded variables and assignments. > > Signed-off-by: Masahiro Yamada > --- > > Changes in v3: > - Keep the wrapper function. > Cleanup of variables and assignments only. > - Fix