Re: [Intel-gfx] [PATCH i-g-t v1] lib/drmtest: Add comment explaining DRIVER_ANY excluding DRIVER_VGEM

2017-01-30 Thread Eric Engestrom
On Monday, 2017-01-30 10:12:28 -0500, Robert Foss wrote: > Signed-off-by: Robert Foss > --- > lib/drmtest.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/lib/drmtest.h b/lib/drmtest.h > index 19d4bd19..c9c019c0 100644 > --- a/lib/drmtest.h > +++ b/lib/drmtest.h > @@ -42,8 +42,1

Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Acquire P-Unit access when modifying P-Unit settings

2017-01-30 Thread Ville Syrjälä
On Mon, Jan 30, 2017 at 04:27:58PM +0100, Hans de Goede wrote: > Hi, > > On 30-01-17 16:11, Ville Syrjälä wrote: > > On Mon, Jan 30, 2017 at 04:02:19PM +0100, Hans de Goede wrote: > >> Hi, > >> > >> On 30-01-17 14:10, Ville Syrjälä wrote: > >>> On Sat, Jan 28, 2017 at 06:18:45PM +0100, Hans de Goe

Re: [Intel-gfx] [PATCH v2 5/5] drm/i915/glk: Enable pipe CSC

2017-01-30 Thread Ville Syrjälä
On Thu, Jan 26, 2017 at 01:24:25PM +0200, Ander Conselvan de Oliveira wrote: > Now that the pre-csc degamma table is set up correctly in Geminilake, > pipe CSC can be enabled without causing a black screen. > > v2: Rebase. > Signed-off-by: Ander Conselvan de Oliveira > Reviewed-by: Ville Syrjäl

Re: [Intel-gfx] [PATCH v2 13/13] drm/i915: Acquire P-Unit access when modifying P-Unit settings

2017-01-30 Thread Hans de Goede
Hi, On 30-01-17 16:38, Ville Syrjälä wrote: On Mon, Jan 30, 2017 at 04:27:58PM +0100, Hans de Goede wrote: Hi, On 30-01-17 16:11, Ville Syrjälä wrote: On Mon, Jan 30, 2017 at 04:02:19PM +0100, Hans de Goede wrote: Hi, On 30-01-17 14:10, Ville Syrjälä wrote: On Sat, Jan 28, 2017 at 06:18:45

Re: [Intel-gfx] [PATCH] drm: Don't race connector registration

2017-01-30 Thread Dave Hansen
On 01/30/2017 01:12 AM, Daniel Vetter wrote: > On Thu, Jan 26, 2017 at 12:34:29PM -0800, Dave Hansen wrote: ... >> And, yeah, I think it just gets the connected status wrong. The >> connector is still there. > > Hm, I thought I replied here but I didn't: > - Is this just after boot (and then the

[Intel-gfx] [PATCH] drm/i915: Drop unused engine parameter from i915_gem_validate_context

2017-01-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem_execbuffer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c index b7b0cf7c50d2..ba021e66469f 100

[Intel-gfx] ✗ Fi.CI.BAT: warning for drm/i915: Record more information about the hanging contexts (rev2)

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915: Record more information about the hanging contexts (rev2) URL : https://patchwork.freedesktop.org/series/18662/ State : warning == Summary == Series 18662v2 drm/i915: Record more information about the hanging contexts https://patchwork.freedesktop.org/api

[Intel-gfx] [PATCH] drm/atomic: Fix double kfree on crtc_state->event

2017-01-30 Thread Daniel Vetter
This is a bug Maarten reported, with the following slab debug backtrace: [IGT] kms_rotation_crc: starting subtest primary-rotation-180 = BUG kmalloc-128 (Tainted: G U ): Object already free

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/3] drm/i915: Make intel_detect_preproduction_hw easier to extend

2017-01-30 Thread Patchwork
== Series Details == Series: series starting with [v2,1/3] drm/i915: Make intel_detect_preproduction_hw easier to extend URL : https://patchwork.freedesktop.org/series/18758/ State : success == Summary == Series 18758v1 Series without cover letter https://patchwork.freedesktop.org/api/1.0/ser

[Intel-gfx] [PATCH v2 2/8] drm/i915/tracepoints: Adjust i915_gem_ring_dispatch

2017-01-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Rename it to i915_gem_request_queue and fix the logged info equivalent to the i915_gem_request even class. Also moved it a bit further apart from the i915_gem_request_add tracepoint since they otherwise provide similar information too close in time. v2: Remove sw fence singa

[Intel-gfx] [PATCH v2 3/8] drm/i915/tracepoints: Tidy i915_gem_request_wait_begin

2017-01-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Provide the same information as the other request event classes. v2: Pass in flags so we can properly report the blocking status. (Chris Wilson) Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_gem_request.c | 2 +- drivers/gpu/drm/i915/i915_trace.h

[Intel-gfx] [PATCH v2 6/8] drm/i915/tracepoints: Rename i915_gem_request_notify

2017-01-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin i915_gem_ring_notify is more appropriate since we do not have the request information at this point, but it is simply a signal from the engine that some request has been completed. v2: * Always trace and log if there were any waiters. * Rename to intel_engine_notify. (Ch

[Intel-gfx] [PATCH v2 7/8] drm/i915/tracepoints: Add backend level request in and out tracepoints

2017-01-30 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Two new tracepoints placed at the call sites where requests are actually passed to the GPU enable userspace to track engine utilisation. These tracepoints are only enabled when the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option is enabled. v2: Fix compilation with !CONFIG_DR

Re: [Intel-gfx] [PATCH] drm/i915: use variadic macros and arrays to choose port/pipe based registers

2017-01-30 Thread Jani Nikula
On Fri, 27 Jan 2017, Chris Wilson wrote: > On Fri, Jan 27, 2017 at 05:57:06PM +0200, Jani Nikula wrote: >> This allows the use of more than 3 ports/pipes/whatever without tricks, >> even if the register offsets are not evenly spaced. >> >> There's the risk of out of bounds access if we're not car

Re: [Intel-gfx] [PATCH v2 6/8] drm/i915/tracepoints: Rename i915_gem_request_notify

2017-01-30 Thread Chris Wilson
On Mon, Jan 30, 2017 at 06:07:29PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > i915_gem_ring_notify is more appropriate since we do not have > the request information at this point, but it is simply a > signal from the engine that some request has been completed. > > v2: > * Always

Re: [Intel-gfx] [PATCH] drm/i915: Drop unused engine parameter from i915_gem_validate_context

2017-01-30 Thread Chris Wilson
On Mon, Jan 30, 2017 at 05:08:58PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > Signed-off-by: Tvrtko Ursulin > --- > drivers/gpu/drm/i915/i915_gem_execbuffer.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c >

Re: [Intel-gfx] [PATCH] drm/i915: Drop unused engine parameter from i915_gem_validate_context

2017-01-30 Thread Chris Wilson
On Mon, Jan 30, 2017 at 08:42:59PM +, Chris Wilson wrote: > On Mon, Jan 30, 2017 at 05:08:58PM +, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > > > Signed-off-by: Tvrtko Ursulin Anyway, regardless of my jibe, Reviewed-by: Chris Wilson -Chris -- Chris Wilson, Intel Open Source Te

[Intel-gfx] ✗ Fi.CI.BAT: warning for series starting with [1/2] drm/i915: Treat stolen memory as DMA addresses (rev4)

2017-01-30 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Treat stolen memory as DMA addresses (rev4) URL : https://patchwork.freedesktop.org/series/18697/ State : warning == Summary == Series 18697v4 Series without cover letter https://patchwork.freedesktop.org/api/1.0/series/18697/r

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Drop unused engine parameter from i915_gem_validate_context

2017-01-30 Thread Patchwork
== Series Details == Series: drm/i915: Drop unused engine parameter from i915_gem_validate_context URL : https://patchwork.freedesktop.org/series/18781/ State : success == Summary == Series 18781v1 drm/i915: Drop unused engine parameter from i915_gem_validate_context https://patchwork.freedes

Re: [Intel-gfx] [Linux v4.10.0-rc1+] Still call-traces after suspend-resume (pm? i915? cpu/hotplug?)

2017-01-30 Thread Rafael J. Wysocki
On 1/24/2017 2:33 AM, Sedat Dilek wrote: On Fri, Dec 30, 2016 at 3:02 PM, Rafael J. Wysocki wrote: On Fri, Dec 30, 2016 at 12:40 PM, Sedat Dilek wrote: Hi, I have already reported this issue in [1]. One of the issue was solved. Unfortunately, it looks like there is still a different problem

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/atomic: Fix double kfree on crtc_state->event

2017-01-30 Thread Patchwork
== Series Details == Series: drm/atomic: Fix double kfree on crtc_state->event URL : https://patchwork.freedesktop.org/series/18785/ State : success == Summary == Series 18785v1 drm/atomic: Fix double kfree on crtc_state->event https://patchwork.freedesktop.org/api/1.0/series/18785/revisions/1

[Intel-gfx] ✓ Fi.CI.BAT: success for Tracepoints cleanup and improvements for requests (rev5)

2017-01-30 Thread Patchwork
== Series Details == Series: Tracepoints cleanup and improvements for requests (rev5) URL : https://patchwork.freedesktop.org/series/18676/ State : success == Summary == Series 18676v5 Tracepoints cleanup and improvements for requests https://patchwork.freedesktop.org/api/1.0/series/18676/revi

Re: [Intel-gfx] [PATCH v2 7/9] drm: Connector helper function to release atomic state

2017-01-30 Thread Pandiyan, Dhinakaran
On Wed, 2017-01-25 at 07:18 +0100, Daniel Vetter wrote: > On Tue, Jan 24, 2017 at 03:49:35PM -0800, Dhinakaran Pandiyan wrote: > > Having a ->atomic_release callback is useful to release shared resources > > that get allocated in compute_config(). > > > > Suggested-by: Daniel Vetter > > Signed-of

[Intel-gfx] [PATCH i-g-t v3 02/11] lib/igt_kms: move igt_kms_get_alt_edid() to the right place

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 4ba6316d..41acce45 100644 --- a/lib/igt_kms.c +++ b/lib

[Intel-gfx] [PATCH i-g-t v3 01/11] tests/kms_atomic_transition: use igt timeout instead of blocking

2017-01-30 Thread Robert Foss
From: Gustavo Padovan If the event never arrives we can timeout and end the test. Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c ind

[Intel-gfx] [PATCH i-g-t v3 00/11] tests/kms_atomic_transition add fence testing

2017-01-30 Thread Robert Foss
This series adds in/out fence testing to kms_atomic_transition test and makes some minor cleanups. This series is rebased ontop of the dyn_n_planes_v3 series. This series can be found here: https://git.collabora.com/cgit/user/robertfoss/intel-gpu-tools.git/log/?h=fences_$VER Changes since v1:

[Intel-gfx] [PATCH i-g-t v3 04/11] tests/kms_atomic: use global atomic properties definitions

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic.c | 123 - 1 file changed, 37 insertions(+), 86 deletions(-) diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c index e6d71c31..8df51ccd 1

[Intel-gfx] [PATCH i-g-t v3 06/11] lib/igt_kms: Add support for the IN_FENCE_FD property

2017-01-30 Thread Robert Foss
Add support dor the IN_FENCE_FD property to enable setting in fences for atomic commits. Signed-off-by: Robert Foss --- lib/igt_kms.c | 20 lib/igt_kms.h | 5 + 2 files changed, 25 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index f0e38b75..b79d2867 100644

[Intel-gfx] [PATCH i-g-t v3 03/11] lib/igt_kms: export properties names

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 6 +++--- lib/igt_kms.h | 23 +++ 2 files changed, 26 insertions(+), 3 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 41acce45..142658a6 100644 --- a/lib/igt

[Intel-gfx] [PATCH i-g-t v3 07/11] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Add support for the OUT_FENCE_PTR property to enable setting out fences for atomic commits. Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 26 +- lib/igt_kms.h | 6 +- 2 files changed, 30 insertions(+), 2 deleti

[Intel-gfx] [PATCH i-g-t v3 05/11] lib/igt_kms: Added igt_pipe_get_last_out_fence()

2017-01-30 Thread Robert Foss
Added the igt_pipe_get_last_out_fence() helper function that wraps accesses to pipe->fence_out. Signed-off-by: Robert Foss --- lib/igt_kms.c | 8 lib/igt_kms.h | 1 + 2 files changed, 9 insertions(+) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index 142658a6..f0e38b75 100644 --- a/lib/i

[Intel-gfx] [PATCH i-g-t v3 10/11] tests/kms_atomic_transition: add out_fences tests

2017-01-30 Thread Robert Foss
Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 35 ++ tests/kms_atomic_transition.c | 148 ++ 2 files changed, 169 insertions(+), 14 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_kms.c index f144

[Intel-gfx] [PATCH i-g-t v3 08/11] tests/kms_atomic: stress possible fence settings

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic.c | 187 ++--- 1 file changed, 177 insertions(+), 10 deletions(-) diff --git a/tests/kms_atomic.c b/tests/kms_atomic.c index 8df51ccd..09605e38

[Intel-gfx] [PATCH i-g-t v3 09/11] tests/kms_atomic_transition: add fencing parameter to run_transition_tests

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- tests/kms_atomic_transition.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c index 095af515..72429759 100644 --- a/t

[Intel-gfx] [PATCH i-g-t v3 11/11] tests/kms_atomic_transition: add in_fences tests

2017-01-30 Thread Robert Foss
From: Gustavo Padovan Signed-off-by: Gustavo Padovan Signed-off-by: Robert Foss --- lib/igt_kms.c | 3 +++ tests/kms_atomic_transition.c | 48 ++- 2 files changed, 32 insertions(+), 19 deletions(-) diff --git a/lib/igt_kms.c b/lib/igt_k

[Intel-gfx] [PATCH 4.9 49/66] drm/i915: Dont leak edid in intel_crt_detect_ddc()

2017-01-30 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ander Conselvan de Oliveira commit c34f078675f505c4437919bb1897b1351f16a050 upstream. In the path where intel_crt_detect_ddc() detects a CRT, if would return true without freeing the edid. Fix

[Intel-gfx] [PATCH 4.9 50/66] drm/i915: Dont init hpd polling for vlv and chv from runtime_suspend()

2017-01-30 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ander Conselvan de Oliveira commit 21d6e0bde50713922a6520ef84e5fd245b05d468 upstream. An error in the condition for avoiding the call to intel_hpd_poll_init() for valleyview and cherryview from

[Intel-gfx] [PATCH 4.9 51/66] drm/i915: Fix calculation of rotated x and y offsets for planar formats

2017-01-30 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ander Conselvan de Oliveira commit 3781bd6e7d64d5f5bea9fdee11ab9460a700c0e4 upstream. Parameters tile_size, tile_width and tile_height were passed in the wrong order to _intel_adjust_tile_offse

[Intel-gfx] [PATCH 4.9 52/66] drm/i915: Check for NULL atomic state in intel_crtc_disable_noatomic()

2017-01-30 Thread Greg Kroah-Hartman
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Ander Conselvan de Oliveira commit 6d1d427a4e24c403b4adf928d61994bdaa0ca03a upstream. In intel_crtc_disable_noatomic(), bail on a failure to allocate an atomic state to avoid a NULL pointer der

[Intel-gfx] [PATCH 4.4 17/23] drm/i915: Dont leak edid in intel_crt_detect_ddc()

2017-01-30 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Ander Conselvan de Oliveira commit c34f078675f505c4437919bb1897b1351f16a050 upstream. In the path where intel_crt_detect_ddc() detects a CRT, if would return true without freeing the edid. Fix

Re: [Intel-gfx] Broken DPMS with DisplayPort on CHV (also BYT?)

2017-01-30 Thread Takashi Iwai
On Tue, 31 Jan 2017 03:34:23 +0100, Jeffrey Cheung wrote: > > Hi Takashi > > How is the test kernel ? Just wonder when Sridhara team can test again. I already informed it in bugzilla and waiting for the test result... http://bugzilla.suse.com/show_bug.cgi?id=1019061#c35 It's found in http

[Intel-gfx] [PATCH] drm/docs: Fix documentation for drm_atomic_set_mode_for_crtc

2017-01-30 Thread Dhinakaran Pandiyan
drm_atomic_set_mode_for_crtc() modifies the .enable member of CRTC state but documentation claims otherwise, fix that. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/drm_atomic.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/drm_atomic.c b/drive

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/docs: Fix documentation for drm_atomic_set_mode_for_crtc

2017-01-30 Thread Patchwork
== Series Details == Series: drm/docs: Fix documentation for drm_atomic_set_mode_for_crtc URL : https://patchwork.freedesktop.org/series/18813/ State : success == Summary == Series 18813v1 drm/docs: Fix documentation for drm_atomic_set_mode_for_crtc https://patchwork.freedesktop.org/api/1.0/se

Re: [Intel-gfx] [PATCH] drm: Don't race connector registration

2017-01-30 Thread Daniel Vetter
On Mon, Jan 30, 2017 at 08:43:17AM -0800, Dave Hansen wrote: > On 01/30/2017 01:12 AM, Daniel Vetter wrote: > > On Thu, Jan 26, 2017 at 12:34:29PM -0800, Dave Hansen wrote: > ... > >> And, yeah, I think it just gets the connected status wrong. The > >> connector is still there. > > > > Hm, I thou

Re: [Intel-gfx] Broken DPMS with DisplayPort on CHV (also BYT?)

2017-01-30 Thread Sridhara.Venkatanara
Hi, Installed RPM, Issue still present with Single Dell E- Series monitor (Display doesn't appear ) Issue not observed with Dual Dell E-Series monitor. Sleep issue not observed. Thanks Sridhar -Original Message- From: Jeffrey Cheung [mailto:jche...@suse.com] Sent: Tuesday, January 31,

Re: [Intel-gfx] [PATCH 0/3] drm/i915: Handle hanging during nonblocking modeset correctly.

2017-01-30 Thread Daniel Vetter
On Mon, Jan 30, 2017 at 03:42:17PM +0100, Maarten Lankhorst wrote: > Op 30-01-17 om 09:17 schreef Daniel Vetter: > > On Fri, Jan 27, 2017 at 03:08:45PM +, Chris Wilson wrote: > >> On Fri, Jan 27, 2017 at 03:58:08PM +0100, Daniel Vetter wrote: > >>> On Fri, Jan 27, 2017 at 02:31:55PM +, Chri

<    1   2