[Intel-gfx] HDMI output on Gigabyte BRiX i7-5500

2016-11-22 Thread Chris Lovett
It has Intel HD 5500 graphics, the mini display port works, but the larger HDMI output port produces nothing. I know my HDMI cable and monitor are good because my Raspberry Pi has no problem showing up on it. The /var/log/Xorg.0.log shows that it has discovered this: - Integrated Graphics Chipset

Re: [Intel-gfx] [PATCH 1/4] drm/i915/lspcon: Ensure AUX CH is awake while in DP Sleep state

2016-11-22 Thread Sharma, Shashank
Reviewed-by: Shashank Sharma Regards Shashank On 11/22/2016 12:45 AM, Imre Deak wrote: Some LSPCON adaptors won't properly wake up in response to an AUX request after the adaptor was placed to a DP Sink Sleep state (via writing 0x2 to DP_SET_POWER). Based on the DP 1.4 specification 5.2.5, the s

Re: [Intel-gfx] [PATCH 2/4] drm/i915/lspcon: Add dp_to_lspcon helper()

2016-11-22 Thread Sharma, Shashank
Regards Shashank On 11/22/2016 12:45 AM, Imre Deak wrote: We need to get to LSPCON in the next patch, so factor out the helper for it. While at it also remove the redundant GEN9 check. Cc: Shashank Sharma Signed-off-by: Imre Deak --- drivers/gpu/drm/i915/intel_dp.c | 7 +++ drivers/

Re: [Intel-gfx] [PATCH 2/4] drm/i915/lspcon: Add dp_to_lspcon helper()

2016-11-22 Thread Imre Deak
On ti, 2016-11-22 at 15:49 +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 11/22/2016 12:45 AM, Imre Deak wrote: > > We need to get to LSPCON in the next patch, so factor out the helper for > > it. While at it also remove the redundant GEN9 check. > > > > Cc: Shashank Sharma > >

[Intel-gfx] [PATCH] drm/i915: Moving scaler numbers to runtime init

2016-11-22 Thread Nabendu Maiti
Numbers of scalers initialization is moved to runtime init for platform specific initialization. Signed-off-by: Nabendu Maiti --- drivers/gpu/drm/i915/i915_debugfs.c | 2 +- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_device_info.c | 3 +++ drivers/gpu/drm/

Re: [Intel-gfx] [PATCH] drm/i915: Moving scaler numbers to runtime init

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 04:26:18PM +0530, Nabendu Maiti wrote: > Numbers of scalers initialization is moved to runtime init for platform > specific initialization. > > Signed-off-by: Nabendu Maiti > --- > drivers/gpu/drm/i915/i915_debugfs.c | 2 +- > drivers/gpu/drm/i915/i915_drv.h

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Brian Starkey
Hi Gustavo, A little late to the party here, but I was blocked by our internal contributions process... I didn't see these end up in my checkout yet though, so I guess they aren't picked up yet. On Mon, Nov 14, 2016 at 06:59:21PM +0900, Gustavo Padovan wrote: From: Gustavo Padovan Add suppor

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 10:53:51AM +, Brian Starkey wrote: > Hi Gustavo, > > A little late to the party here, but I was blocked by our internal > contributions process... > > I didn't see these end up in my checkout yet though, so I guess they > aren't picked up yet. > > On Mon, Nov 14, 2016

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Moving scaler numbers to runtime init

2016-11-22 Thread Patchwork
== Series Details == Series: drm/i915: Moving scaler numbers to runtime init URL : https://patchwork.freedesktop.org/series/15726/ State : success == Summary == Series 15726v1 drm/i915: Moving scaler numbers to runtime init https://patchwork.freedesktop.org/api/1.0/series/15726/revisions/1/mbo

Re: [Intel-gfx] [PATCH 06/12] lib/igt_kms: Add support for the IN_FENCE_FD property

2016-11-22 Thread Brian Starkey
Hi, On Mon, Nov 14, 2016 at 06:59:20PM +0900, Gustavo Padovan wrote: From: 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

[Intel-gfx] [PATCH] drm/i915: Moving no of scalers initialization to runtime init

2016-11-22 Thread Nabendu Maiti
Number of scalers initialization is moved to runtime init from intel_crtc_init for platform specific initialization. Signed-off-by: Nabendu Maiti --- drivers/gpu/drm/i915/i915_debugfs.c | 2 +- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_device_info.c | 3 +

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Brian Starkey
On Tue, Nov 22, 2016 at 11:06:00AM +, Chris Wilson wrote: On Tue, Nov 22, 2016 at 10:53:51AM +, Brian Starkey wrote: Hi Gustavo, A little late to the party here, but I was blocked by our internal contributions process... I didn't see these end up in my checkout yet though, so I guess t

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 11:54:57AM +, Brian Starkey wrote: > On Tue, Nov 22, 2016 at 11:06:00AM +, Chris Wilson wrote: > >On Tue, Nov 22, 2016 at 10:53:51AM +, Brian Starkey wrote: > >>Hi Gustavo, > >> > >>A little late to the party here, but I was blocked by our internal > >>contributi

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Moving scaler numbers to runtime init (rev2)

2016-11-22 Thread Patchwork
== Series Details == Series: drm/i915: Moving scaler numbers to runtime init (rev2) URL : https://patchwork.freedesktop.org/series/15726/ State : success == Summary == Series 15726v2 drm/i915: Moving scaler numbers to runtime init https://patchwork.freedesktop.org/api/1.0/series/15726/revision

Re: [Intel-gfx] HDMI output on Gigabyte BRiX i7-5500

2016-11-22 Thread Jani Nikula
On Mon, 21 Nov 2016, Chris Lovett wrote: > It has Intel HD 5500 graphics, the mini display port works, but the larger > HDMI output port produces nothing. I know my HDMI cable and monitor are > good because my Raspberry Pi has no problem showing up on it. Please file a bug at [1], add drm.debug=

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Brian Starkey
On Tue, Nov 22, 2016 at 12:10:52PM +, Chris Wilson wrote: On Tue, Nov 22, 2016 at 11:54:57AM +, Brian Starkey wrote: On Tue, Nov 22, 2016 at 11:06:00AM +, Chris Wilson wrote: >On Tue, Nov 22, 2016 at 10:53:51AM +, Brian Starkey wrote: >>Hi Gustavo, >> >>A little late to the party

[Intel-gfx] [PATCH] drm/i915: Remove all ->config dereferences from intel_hdmi

2016-11-22 Thread Maarten Lankhorst
We can now obtain the relevant crtc_state/conn_state from the relevant callbacks, which means all the ->config accesses can be removed and the code cleaned up. Also fixes the following WARN_ON, because we don't have to iterate atomic connectors any more: [ 1562.981768] [ cut here ]---

Re: [Intel-gfx] [PATCH v5 5/8] drm/i915/skl+: change WM calc to fixed point 16.16

2016-11-22 Thread Paulo Zanoni
Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu: > This patch changes Watermak calculation to fixed point calculation. > Problem with current calculation is during plane_blocks_per_line > calculation we divide intermediate blocks with min_scanlines and > takes floor of the result because o

Re: [Intel-gfx] [PATCH] drm/i915/bxt: Correct dual-link MIPI port control.

2016-11-22 Thread Ville Syrjälä
On Mon, Nov 21, 2016 at 02:24:06PM -0800, Bob Paauwe wrote: > For BXT, there is only one bit that enables/disables dual-link mode > and not different bits depending on which pipe is being used. > > Signed-off-by: Bob Paauwe > --- > drivers/gpu/drm/i915/intel_dsi.c | 5 - > 1 file changed, 4

[Intel-gfx] [PATCH igt] stubs/drm: Add drm_intel_gem_context_get_id stub

2016-11-22 Thread Robert Bragg
The plan is to use this api in i915-perf tests so this adds the corresponding stub in case libdrm was built with libdrm_intel disabled. Signed-off-by: Robert Bragg --- lib/stubs/drm/intel_bufmgr.c | 7 +++ lib/stubs/drm/intel_bufmgr.h | 2 ++ 2 files changed, 9 insertions(+) diff --git a/li

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Daniel Vetter
On Tue, Nov 22, 2016 at 12:37:47PM +, Brian Starkey wrote: > On Tue, Nov 22, 2016 at 12:10:52PM +, Chris Wilson wrote: > > On Tue, Nov 22, 2016 at 11:54:57AM +, Brian Starkey wrote: > > > On Tue, Nov 22, 2016 at 11:06:00AM +, Chris Wilson wrote: > > > >On Tue, Nov 22, 2016 at 10:53:

Re: [Intel-gfx] [PATCH igt] stubs/drm: Add drm_intel_gem_context_get_id stub

2016-11-22 Thread Matthew Auld
On 22 November 2016 at 13:02, Robert Bragg wrote: > The plan is to use this api in i915-perf tests so this adds the > corresponding stub in case libdrm was built with libdrm_intel disabled. > > Signed-off-by: Robert Bragg Reviewed-by: Matthew Auld ___

Re: [Intel-gfx] [PATCH] drm/i915: Remove all ->config dereferences from intel_hdmi

2016-11-22 Thread Ville Syrjälä
On Tue, Nov 22, 2016 at 01:38:51PM +0100, Maarten Lankhorst wrote: > We can now obtain the relevant crtc_state/conn_state from the > relevant callbacks, which means all the ->config accesses > can be removed and the code cleaned up. > > Also fixes the following WARN_ON, because we don't have to it

[Intel-gfx] [PATCH i-g-t v9 00/21] Implement sw_sync test

2016-11-22 Thread robert . foss
From: Robert Foss This series implements the sw_sync test and the lib/sw_sync helper functions for said test. The sw_sync subtests range from very basic tests of the sw_sync functionality, to stress testing and randomized tests. Changes since v1: Added "Reviewed-by: Eric Engestrom " tag lib

[Intel-gfx] [PATCH i-g-t v9 03/21] tests/sw_sync: Add subtest test_alloc_fence

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest alloc_fence that verifies that it's possible to allocate a fence on a timeline. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 16 1 file changed, 16 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c inde

[Intel-gfx] [PATCH i-g-t v9 05/21] tests/sw_sync: Add subtest test_alloc_merge_fence

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies that merging two fences works in the simples possible case. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index f

[Intel-gfx] [PATCH i-g-t v9 11/21] tests/sw_sync: Add subtest test_sync_random_merge

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies that creating many timelines and merging random fences from each timeline with eachother results in merged fences that are fully functional. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 73

[Intel-gfx] [PATCH i-g-t v9 13/21] tests/sw_sync: Add subtest test_sync_multi_producer_single_consumer

2016-11-22 Thread robert . foss
From: Robert Foss This subtest runs a single consumer thread and multiple producer thread that are synchronized using multiple timelines. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 139 1 file changed,

[Intel-gfx] [PATCH i-g-t v9 08/21] tests/sw_sync: Add subtest test_sync_merge_same

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies merging a fence with itself does not fail. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index 96dcbff..ada12

[Intel-gfx] [PATCH i-g-t v9 14/21] tests/sw_sync: Add subtest test_sync_expired_merge

2016-11-22 Thread robert . foss
From: Rafael Antognolli This test creates an already expired fence, then creates a merged fence out of that expired one (passed twice to the merge operation), and finally closes the merged fence. It shows that if the refcounts are wrong on the original expired fence, it might get freed while stil

[Intel-gfx] [PATCH i-g-t v9 02/21] test/sw_sync: Add sw_sync test

2016-11-22 Thread robert . foss
From: Robert Foss Add initial tests for sw_sync. Signed-off-by: Robert Foss Signed-off-by: Gustavo Padovan Reviewed-by: Eric Engestrom --- tests/Makefile.sources | 1 + tests/sw_sync.c| 54 ++ 2 files changed, 55 insertions(+) create

[Intel-gfx] [PATCH i-g-t v9 10/21] tests/sw_sync: Add subtest test_sync_multi_consumer_producer

2016-11-22 Thread robert . foss
From: Robert Foss This test verifies that stressing the kernel by creating multiple consumer/producer threads that wait on a single timeline to be incremented by another conumer/producer thread does not fail. And that the order amongst the threads is maintained. Signed-off-by: Robert Foss Revie

[Intel-gfx] [PATCH i-g-t v9 15/21] tests/sw_sync: Add subtest test_timeline_closed

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies that the fences of a timeline are not signalled when a timelne is closed. Signed-off-by: Robert Foss --- tests/sw_sync.c | 17 + 1 file changed, 17 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index b5ea692..1d2c921 10064

[Intel-gfx] [PATCH i-g-t v9 12/21] tests/sw_sync: Add subtest test_sync_multi_timeline_wait

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies that waiting, timing out on a wait and that counting fences in various states works. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 66 + 1 file changed, 66 insertions

[Intel-gfx] [PATCH i-g-t v9 04/21] tests/sw_sync: Add subtest test_alloc_fence_invalid_timeline

2016-11-22 Thread robert . foss
From: Robert Foss This subtests tests that creating fences on negative timelines fail. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index 8e7764a..f9d4fe9 100644 --- a/te

[Intel-gfx] [PATCH i-g-t v9 17/21] lib/sw_sync: Add igt_require_sw_sync to enable skipping on no sw_sync support

2016-11-22 Thread robert . foss
From: Robert Foss Add igt_require_sw_sync to provide tests to skip if sw_sync support isn't available on the host machine. Signed-off-by: Robert Foss --- lib/sw_sync.c | 22 ++ lib/sw_sync.h | 1 + 2 files changed, 23 insertions(+) diff --git a/lib/sw_sync.c b/lib/sw_sync

[Intel-gfx] [PATCH i-g-t v9 18/21] tests/sw_sync: Add igt_require check for sw_sync feature

2016-11-22 Thread robert . foss
From: Robert Foss Make sure that this test is skipped if the sw_sync feature is missing from the host system. Signed-off-by: Robert Foss --- lib/sw_sync.c | 1 + tests/sw_sync.c | 3 +++ 2 files changed, 4 insertions(+) diff --git a/lib/sw_sync.c b/lib/sw_sync.c index 38fe670..ac793ad 10064

[Intel-gfx] [PATCH i-g-t v9 06/21] tests/sw_sync: Add subtest test_sync_busy

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies that waiting on fences works properly. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 51 +++ 1 file changed, 51 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c

[Intel-gfx] [PATCH i-g-t v9 19/21] tests/sw_sync: Add subtest test_sync_merge_invalid

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest test_sync_merge_invalid that tests merging invalid fences. Signed-off-by: Robert Foss --- tests/sw_sync.c | 41 + 1 file changed, 41 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index 562b71f..3a67877 100644

[Intel-gfx] [PATCH i-g-t v9 01/21] lib/sw_sync: Add helper functions for managing synchronization primitives

2016-11-22 Thread robert . foss
From: Robert Foss Base functions to help testing the Sync File Framework (explicit fencing mechanism ported from Android). These functions allow you to create, use and destroy timelines and fences. Signed-off-by: Robert Foss Signed-off-by: Gustavo Padovan Reviewed-by: Eric Engestrom --- lib/

[Intel-gfx] [PATCH i-g-t v9 09/21] tests/sw_sync: Add subtest test_sync_multi_consumer

2016-11-22 Thread robert . foss
From: Robert Foss This subtest verifies the access ordering of multiple consumer threads. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 103 1 file changed, 103 insertions(+) diff --git a/tests/sw_sync.c

[Intel-gfx] [PATCH i-g-t v9 16/21] tests/sw_sync: Add subtest test_timeline_closed_signaled

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest test_timeline_closed_signaled that verifies that a signaled fence stays signaled after its timeline has been closed. Signed-off-by: Robert Foss --- tests/sw_sync.c | 18 ++ 1 file changed, 18 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_s

[Intel-gfx] [PATCH i-g-t v9 21/21] tests/sw_sync: Add subtest test_sync_busy_unixsocket

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest test_sync_busy_fork which increments the timeline in a forked child process, where the timeline fd has been sent through a UNIX socket. Signed-off-by: Robert Foss --- tests/sw_sync.c | 103 1 file changed, 1

[Intel-gfx] [PATCH i-g-t v9 07/21] tests/sw_sync: Add subtest test_sync_merge

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest test_sync_merge that tests merging fences and the validity of the resulting merged fence. Signed-off-by: Robert Foss Reviewed-by: Eric Engestrom --- tests/sw_sync.c | 67 + 1 file changed, 67 insertions(+)

Re: [Intel-gfx] [PATCH v9 01/11] drm/i915: Add i915 perf infrastructure

2016-11-22 Thread Daniel Vetter
On Wed, Nov 09, 2016 at 08:00:06PM +, Matthew Auld wrote: > On 7 November 2016 at 19:49, Robert Bragg wrote: > > Adds base i915 perf infrastructure for Gen performance metrics. > > > > This adds a DRM_IOCTL_I915_PERF_OPEN ioctl that takes an array of uint64 > > properties to configure a stream

[Intel-gfx] [PATCH i-g-t v9 20/21] tests/sw_sync: Add subtest test_sync_busy_fork

2016-11-22 Thread robert . foss
From: Robert Foss Add subtest test_sync_busy_fork which increments the timeline in a forked child process. Signed-off-by: Robert Foss --- tests/sw_sync.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/tests/sw_sync.c b/tests/sw_sync.c index 3a67877

Re: [Intel-gfx] [PATCH v9 01/11] drm/i915: Add i915 perf infrastructure

2016-11-22 Thread Daniel Vetter
On Tue, Nov 22, 2016 at 02:29:18PM +0100, Daniel Vetter wrote: > On Wed, Nov 09, 2016 at 08:00:06PM +, Matthew Auld wrote: > > On 7 November 2016 at 19:49, Robert Bragg wrote: > > > Adds base i915 perf infrastructure for Gen performance metrics. > > > > > > This adds a DRM_IOCTL_I915_PERF_OPEN

Re: [Intel-gfx] [PATCH v5 4/8] drm/i915/bxt: Enable IPC support

2016-11-22 Thread Mahesh Kumar
Hi, On Tuesday 22 November 2016 12:16 AM, Paulo Zanoni wrote: Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu: This patch adds IPC support for platforms. This patch enables IPC only for BXT/KBL platform as for SKL recommendation is to keep is disabled. IPC (Isochronous Priority Contro

Re: [Intel-gfx] [PATCH v2] drm/i915: don't whitelist oacontrol in cmd parser

2016-11-22 Thread Daniel Vetter
On Tue, Nov 08, 2016 at 12:51:48PM +, Robert Bragg wrote: > This v2 patch bumps the command parser version so it can be referenced in > corresponding i-g-t gem_exec_parse changes. > > --- >8 --- Scissors cut everything below, not everything above, hence next time around pls switch around your

Re: [Intel-gfx] [PATCH] drm/i915: Move the release of PT page to the upper caller

2016-11-22 Thread Zhi Wang
Hi guys: Would you mind to have a quick review on this patch? :P The linux guest under GVT-g couldn't boot up without this patch in the newer kernel. Thanks, Zhi. On 11/21/16 19:44, Zhi Wang wrote: a PT page will be released if it doesn't contain any meaningful mappings during PPGTT page

Re: [Intel-gfx] [PATCH] drm/i915/dp: Add a helper function to return DP link rate index

2016-11-22 Thread Jani Nikula
On Tue, 22 Nov 2016, Manasi Navare wrote: > Common rates array is used to keep track of common rates > supported between DP sink and source. This helper function > returns the index of a specific link rate in this common rates > array. > > This is in prep with patches for DP Link training complian

[Intel-gfx] [PATCH v2 36/37] drm: Add mode_config .get_format_info() hook

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Allow drivers to return a custom drm_format_info structure for special fb layouts. We'll use this for the compression control surface in i915. v2: Fix drm_get_format_info() kernel doc (Laurent) Don't pass 'dev' to the new hook (Laurent) Cc: Laurent Pinchart Cc: Ben Wida

Re: [Intel-gfx] [PATCH v9 11/11] drm/i915: Add a kerneldoc summary for i915_perf.c

2016-11-22 Thread Daniel Vetter
On Mon, Nov 07, 2016 at 07:49:57PM +, Robert Bragg wrote: > In particular this tries to capture for posterity some of the early > challenges we had with using the core perf infrastructure in case we > ever want to revisit adapting perf for device metrics. > > Cc: Chris Wilson > Signed-off-by:

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Brian Starkey
On Tue, Nov 22, 2016 at 02:12:59PM +0100, Daniel Vetter wrote: On Tue, Nov 22, 2016 at 12:37:47PM +, Brian Starkey wrote: On Tue, Nov 22, 2016 at 12:10:52PM +, Chris Wilson wrote: > On Tue, Nov 22, 2016 at 11:54:57AM +, Brian Starkey wrote: > > On Tue, Nov 22, 2016 at 11:06:00AM +000

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Daniel Vetter
On Tue, Nov 22, 2016 at 01:50:53PM +, Brian Starkey wrote: > On Tue, Nov 22, 2016 at 02:12:59PM +0100, Daniel Vetter wrote: > > On Tue, Nov 22, 2016 at 12:37:47PM +, Brian Starkey wrote: > > > On Tue, Nov 22, 2016 at 12:10:52PM +, Chris Wilson wrote: > > > > On Tue, Nov 22, 2016 at 11:5

Re: [Intel-gfx] [PATCH v9 01/11] drm/i915: Add i915 perf infrastructure

2016-11-22 Thread Robert Bragg
On Tue, Nov 22, 2016 at 1:31 PM, Daniel Vetter wrote: > On Tue, Nov 22, 2016 at 02:29:18PM +0100, Daniel Vetter wrote: > > On Wed, Nov 09, 2016 at 08:00:06PM +, Matthew Auld wrote: > > > On 7 November 2016 at 19:49, Robert Bragg > wrote: > > > > Adds base i915 perf infrastructure for Gen per

Re: [Intel-gfx] [PATCH 07/12] lib/igt_kms: Add support for the OUT_FENCE_PTR property

2016-11-22 Thread Brian Starkey
On Tue, Nov 22, 2016 at 02:56:49PM +0100, Daniel Vetter wrote: On Tue, Nov 22, 2016 at 01:50:53PM +, Brian Starkey wrote: On Tue, Nov 22, 2016 at 02:12:59PM +0100, Daniel Vetter wrote: > On Tue, Nov 22, 2016 at 12:37:47PM +, Brian Starkey wrote: > > On Tue, Nov 22, 2016 at 12:10:52PM +00

Re: [Intel-gfx] [PATCH v2] drm/i915: Stop the machine as we install the wedged submit_request handler

2016-11-22 Thread Mika Kuoppala
Chris Wilson writes: > In order to prevent a race between the old callback submitting an > incomplete request and i915_gem_set_wedged() installing its nop handler, > we must ensure that the swap occurs when the machine is idle > (stop_machine). > > v2: move context lost from out of BKL. > > Signe

Re: [Intel-gfx] [PATCH v2] drm/i915: don't whitelist oacontrol in cmd parser

2016-11-22 Thread Robert Bragg
On Tue, Nov 22, 2016 at 1:34 PM, Daniel Vetter wrote: > On Tue, Nov 08, 2016 at 12:51:48PM +, Robert Bragg wrote: > > This v2 patch bumps the command parser version so it can be referenced in > > corresponding i-g-t gem_exec_parse changes. > > > > --- >8 --- > > Scissors cut everything below,

Re: [Intel-gfx] [PATCH v5 5/8] drm/i915/skl+: change WM calc to fixed point 16.16

2016-11-22 Thread Mahesh Kumar
On Tuesday 22 November 2016 06:12 PM, Paulo Zanoni wrote: Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu: This patch changes Watermak calculation to fixed point calculation. Problem with current calculation is during plane_blocks_per_line calculation we divide intermediate blocks wit

[Intel-gfx] [PATCH] lib/debugobjects: Export for use in modules

2016-11-22 Thread Chris Wilson
Drivers, or other modules, that use a mixture of objects (especially objects embedded within other objects) would like to take advantage of the debugobjects facilities to help catch misuse. Currently, the debugobjects interface is only available to builtin drivers and requires a set of EXPORT_SYMBO

Re: [Intel-gfx] [PATCH] drm/i915: Don't deref context->file_priv ERR_PTR upon reset

2016-11-22 Thread Mika Kuoppala
Chris Wilson writes: > When a user context is closed, it's file_priv backpointer is replaced by > ERR_PTR(-EBADF); be careful not to chase this invalid pointer after a > hang and a GPU reset. > > Signed-off-by: Chris Wilson > Fixes: b083a0870c79 ("drm/i915: Add per client max context ban limit")

[Intel-gfx] [PATCH] lib/debugobjects: Export for use in modules

2016-11-22 Thread Chris Wilson
Drivers, or other modules, that use a mixture of objects (especially objects embedded within other objects) would like to take advantage of the debugobjects facilities to help catch misuse. Currently, the debugobjects interface is only available to builtin drivers and requires a set of EXPORT_SYMBO

Re: [Intel-gfx] [PATCH 1/4] lib/scatterlist: Fix offset type in sg_alloc_table_from_pages

2016-11-22 Thread Mauro Carvalho Chehab
Em Mon, 14 Nov 2016 09:55:48 + Chris Wilson escreveu: > On Fri, Nov 11, 2016 at 08:50:17AM +, Tvrtko Ursulin wrote: > > From: Tvrtko Ursulin > > > > Scatterlist entries have an unsigned int for the offset so > > correct the sg_alloc_table_from_pages function accordingly. > > > > Since

Re: [Intel-gfx] [PATCH] drm/i915: Move the release of PT page to the upper caller

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 09:29:40PM +0800, Zhi Wang wrote: > Hi guys: > Would you mind to have a quick review on this patch? :P The > linux guest under GVT-g couldn't boot up without this patch in the > newer kernel. > > Thanks, > Zhi. > > On 11/21/16 19:44, Zhi Wang wrote: > >a PT page will b

Re: [Intel-gfx] [PATCH] drm/i915: Move the release of PT page to the upper caller

2016-11-22 Thread Michał Winiarski
On Mon, Nov 21, 2016 at 07:44:26PM +0800, Zhi Wang wrote: > a PT page will be released if it doesn't contain any meaningful mappings > during PPGTT page table shrinking. The PT entry in the upper level will > be set to a scratch entry. > > Normally this works nicely, but in virtualization world, t

[Intel-gfx] [PATCH] drm/i915: Integrate i915_sw_fence with debugobjects

2016-11-22 Thread Chris Wilson
Add the tracking required to enable debugobjects to improve error detection in BAT. The debugobject interface lets us to track the lifetime of the fences even while being embedded into larger structs, i.e. to check they are not used after they have been released. Signed-off-by: Chris Wilson ---

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/debugobjects: Export for use in modules

2016-11-22 Thread Patchwork
== Series Details == Series: lib/debugobjects: Export for use in modules URL : https://patchwork.freedesktop.org/series/15743/ State : success == Summary == Series 15743v1 lib/debugobjects: Export for use in modules https://patchwork.freedesktop.org/api/1.0/series/15743/revisions/1/mbox/ Test

Re: [Intel-gfx] [PATCH v5 4/8] drm/i915/bxt: Enable IPC support

2016-11-22 Thread Paulo Zanoni
Em Ter, 2016-11-22 às 19:05 +0530, Mahesh Kumar escreveu: > Hi, > > > On Tuesday 22 November 2016 12:16 AM, Paulo Zanoni wrote: > > > > Em Sex, 2016-11-18 às 20:39 +0530, Mahesh Kumar escreveu: > > > > > > This patch adds IPC support for platforms. This patch enables IPC > > > only for BXT/KBL

[Intel-gfx] [CI 4/4] drm/i915: Stop the machine as we install the wedged submit_request handler

2016-11-22 Thread Chris Wilson
In order to prevent a race between the old callback submitting an incomplete request and i915_gem_set_wedged() installing its nop handler, we must ensure that the swap occurs when the machine is idle (stop_machine). v2: move context lost from out of BKL. Signed-off-by: Chris Wilson Reviewed-by:

[Intel-gfx] [CI 1/4] drm/i915: Don't deref context->file_priv ERR_PTR upon reset

2016-11-22 Thread Chris Wilson
When a user context is closed, it's file_priv backpointer is replaced by ERR_PTR(-EBADF); be careful not to chase this invalid pointer after a hang and a GPU reset. Signed-off-by: Chris Wilson Fixes: b083a0870c79 ("drm/i915: Add per client max context ban limit") Cc: Mika Kuoppala Reviewed-by: M

[Intel-gfx] [CI 3/4] drm/i915: Complete requests in nop_submit_request

2016-11-22 Thread Chris Wilson
Since the submit/execute split in commit d55ac5bf97c6 ("drm/i915: Defer transfer onto execution timeline to actual hw submission") the global seqno advance was deferred until the submit_request callback. After wedging the GPU, we were installing a nop_submit_request handler (to avoid waking up the

[Intel-gfx] [CI 2/4] drm/i915: Disable hangcheck when wedged

2016-11-22 Thread Chris Wilson
If the gpu reset fails and the machine is terminally wedged, further hangchecks achieve nothing but noise. Disable them, with a corollary that we re-enable hangchecking after a successful GPU reset in case the user is artificially bringing the machine back to life through the debug interface. Sign

[Intel-gfx] ✓ Fi.CI.BAT: success for lib/debugobjects: Export for use in modules (rev2)

2016-11-22 Thread Patchwork
== Series Details == Series: lib/debugobjects: Export for use in modules (rev2) URL : https://patchwork.freedesktop.org/series/15743/ State : success == Summary == Series 15743v2 lib/debugobjects: Export for use in modules https://patchwork.freedesktop.org/api/1.0/series/15743/revisions/2/mbox

[Intel-gfx] [drm-intel:drm-intel-next-queued 6/11] undefined reference to `__udivdi3'

2016-11-22 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel drm-intel-next-queued head: 7abbd8d670bb928366aa94332a173aa3d394ebfe commit: d79651522e89c4ffa8992b48dfe449f0c583f809 [6/11] drm/i915: Enable i915 perf stream for Haswell OA unit config: i386-defconfig (attached as .config) compiler: gcc-6 (Debian

Re: [Intel-gfx] [PATCH 3/4] drm/i915/lspcon: Wait for expected LSPCON mode to settle

2016-11-22 Thread Sharma, Shashank
Regards Shashank On 11/22/2016 12:45 AM, Imre Deak wrote: Some LSPCON adaptors may return an incorrect LSPCON mode right after waking from DP Sleep state. This is the case at least for the ParadTech PS175 adaptor, both when waking because of exiting the DP Sleep to active state, or due to any

Re: [Intel-gfx] [PATCH v9 01/11] drm/i915: Add i915 perf infrastructure

2016-11-22 Thread Daniel Vetter
On Tue, Nov 22, 2016 at 02:02:38PM +, Robert Bragg wrote: > On Tue, Nov 22, 2016 at 1:31 PM, Daniel Vetter wrote: > > > On Tue, Nov 22, 2016 at 02:29:18PM +0100, Daniel Vetter wrote: > > > On Wed, Nov 09, 2016 at 08:00:06PM +, Matthew Auld wrote: > > > > On 7 November 2016 at 19:49, Rober

[Intel-gfx] [PATCH] dim: Enable --scissors by default

2016-11-22 Thread Daniel Vetter
I never knew this even was a thing, and plain git am cuts everything below the scissors line (since it matches the --- of a diffstat start). Cc: Robert Bragg Signed-off-by: Daniel Vetter --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dim b/dim index 8697eaee92cc..0b

Re: [Intel-gfx] [PATCH 4/4] drm/i915/lspcon: Remove unused force change mode parameter

2016-11-22 Thread Sharma, Shashank
The reason why I kept the force function was, that during debug phase, I saw a need to change LSPCON mode. Also, in future, if we need to provide an IOCTL/control to userspace, to change lspcon mode, force may be required to differentiate the path from user/kernel space. But in any case: Rev

Re: [Intel-gfx] [PATCH] drm/i915: Moving no of scalers initialization to runtime init

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 05:31:05PM +0530, Nabendu Maiti wrote: Try to avoid confusing contractions (i.e. no no) and give us a verb in that sentence in that Subject line. > Number of scalers initialization is moved to runtime init from > intel_crtc_init for platform specific initialization. so th

Re: [Intel-gfx] [PATCH v2] drm/i915: don't whitelist oacontrol in cmd parser

2016-11-22 Thread Daniel Vetter
On Tue, Nov 22, 2016 at 02:09:08PM +, Robert Bragg wrote: > On Tue, Nov 22, 2016 at 1:34 PM, Daniel Vetter wrote: > > > On Tue, Nov 08, 2016 at 12:51:48PM +, Robert Bragg wrote: > > > This v2 patch bumps the command parser version so it can be referenced in > > > corresponding i-g-t gem_e

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Integrate i915_sw_fence with debugobjects

2016-11-22 Thread Patchwork
== Series Details == Series: drm/i915: Integrate i915_sw_fence with debugobjects URL : https://patchwork.freedesktop.org/series/15747/ State : failure == Summary == Series 15747v1 drm/i915: Integrate i915_sw_fence with debugobjects https://patchwork.freedesktop.org/api/1.0/series/15747/revisio

Re: [Intel-gfx] [PATCH igt v4 12/13] igt/gem_exec_parse: update registers test for v >= 8

2016-11-22 Thread Chris Wilson
On Mon, Nov 14, 2016 at 08:51:21PM +, Robert Bragg wrote: > This combines some parts of the recently added store_lri test with the > registers test to be able to first load a distinguishable value before > the LRI and explicitly read back the register to determine if the > command succeeded or

Re: [Intel-gfx] [PATCH] dim: Enable --scissors by default

2016-11-22 Thread Jani Nikula
On Tue, 22 Nov 2016, Daniel Vetter wrote: > I never knew this even was a thing, and plain git am cuts everything > below the scissors line (since it matches the --- of a diffstat > start). > > Cc: Robert Bragg > Signed-off-by: Daniel Vetter > --- > dim | 2 +- > 1 file changed, 1 insertion(+),

[Intel-gfx] [PATCH 5/9] drm/i915: Use enum plane_id in VLV/CHV sprite code

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Use intel_plane->id to derive the VLV/CHV sprite register offsets instead of abusing plane->plane which is really meant to for primary planes only. v2: Convert assert_sprites_disabled() over as well v3: Rename the reg macro parameter to 'plane_id' as well (Paulo) Signed-off-

[Intel-gfx] [PATCH v2 0/9] drm/i915: Add a per-pipe plane identifier enum (v2)

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä I ended up tweaking quite a few of the patches (and even adding a new one) based on Paulo's review, so figured I'd repost the entire thing. I didn't do the _ID_ rename thing, at least not yet. I'll have to think about it more if I could come up with some nice way to deal with

[Intel-gfx] [PATCH 6/9] drm/i915: Use enum plane_id in VLV/CHV wm code

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Let's try not to abuse plane->plane for sprites on VLV/CHV and instead use plane->id. Since out watermark structures aren't entirely plane type agnostic (for now) and start indexing sprites from 0 we'll add a small helper to convert between the two bases. Signed-off-by: Vill

[Intel-gfx] [PATCH 8/9] drm/i915: Don't populate plane->plane for cursors and sprites

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä With plane->plane now purely reserved for the primary planes, let's not even populate it for cursors and sprites. Let's switch the type to enum plane as well since it's no longer being abused for anything else. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_dis

[Intel-gfx] [PATCH 9/9] drm/i915: Rename the local 'plane' variable to 'plane_id' in primary plane code

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Now we've rename the local plane id variable as 'plane_id' everywhere except the pre-SKL primary plane code. Let's do the rename there as well so that we'll free up the name 'plane' for use with struct intel_plane*. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/inte

[Intel-gfx] [PATCH 1/9] drm/i915: Add per-pipe plane identifier

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä As I told people in [1] we really should not be confusing enum plane as a per-pipe plane identifier. Looks like that happened nonetheless, so let's fix it up by splitting the two into two enums. We'll also want something we just directly pass to various register offset macros

[Intel-gfx] [PATCH 3/9] drm/i915: Use enum plane_id in SKL wm code

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Nuke skl_wm_plane_id() and just use the new intel_plane->id. v2: Convert skl_write_plane_wm() as well v3: Convert skl_pipe_wm_get_hw_state() correctly v4: Rebase due to changes in the wm code Drop the cursor FIXME from the total data rate calc (Paulo) Use the "[PLANE:

[Intel-gfx] [PATCH 2/9] drm/i915: Add crtc->plane_ids_mask

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Add a mask of which planes are available for each pipe. This doesn't quite work for old platforms with dynamic plane<->pipe assignment, but as we don't support that sort of stuff (yet) we can get away with it. The main use I have for this is the for_each_plane_id_on_crtc() ma

[Intel-gfx] [PATCH 4/9] drm/i915: Use enum plane_id in SKL plane code

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Replace the intel_plane->plane and hardcoded 0 usage in the SKL plane code with intel_plane->id. This should make the SKL "primary" and "sprite" code virtually identical, so the next logical step would likely be dropping one of the copies. Signed-off-by: Ville Syrjälä Revie

[Intel-gfx] [PATCH 7/9] drm/i915: s/plane/plane_id/ in skl+ plane register defines

2016-11-22 Thread ville . syrjala
From: Ville Syrjälä Rename the SKL plane register define 'plane' parameter to 'plane_id' to reflect the fact that you're supposed to pass in the enum plane_id rather than enum plane. Do the same for the scaler plane selector bits. Cc: Paulo Zanoni Suggested-by: Paulo Zanoni Signed-off-by: Vil

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Make GPU pages movable

2016-11-22 Thread Matthew Auld
On 9 November 2016 at 18:36, Hugh Dickins wrote: > On Wed, 9 Nov 2016, Daniel Vetter wrote: >> >> Hi all -mm folks! >> >> Any feedback on these two? It's kinda an intermediate step towards a >> full-blown gemfs, and I think useful for that. Or do we need to go >> directly to our own backing storag

Re: [Intel-gfx] [PATCH igt v4 12/13] igt/gem_exec_parse: update registers test for v >= 8

2016-11-22 Thread Chris Wilson
On Tue, Nov 22, 2016 at 03:45:59PM +, Chris Wilson wrote: > On Mon, Nov 14, 2016 at 08:51:21PM +, Robert Bragg wrote: > > This combines some parts of the recently added store_lri test with the > > registers test to be able to first load a distinguishable value before > > the LRI and explici

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/4] drm/i915: Don't deref context->file_priv ERR_PTR upon reset

2016-11-22 Thread Patchwork
== Series Details == Series: series starting with [CI,1/4] drm/i915: Don't deref context->file_priv ERR_PTR upon reset URL : https://patchwork.freedesktop.org/series/15748/ State : success == Summary == Series 15748v1 Series without cover letter https://patchwork.freedesktop.org/api/1.0/serie

Re: [Intel-gfx] [PATCH 3/4] drm/i915/lspcon: Wait for expected LSPCON mode to settle

2016-11-22 Thread Imre Deak
On Tue, 2016-11-22 at 21:06 +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 11/22/2016 12:45 AM, Imre Deak wrote: > > Some LSPCON adaptors may return an incorrect LSPCON mode right after > > waking from DP Sleep state. This is the case at least for the ParadTech > > PS175 adaptor,

[Intel-gfx] [PATCH] drm/i915/guc: Always initialize action_lock

2016-11-22 Thread Arkadiusz Hiler
Action lock is not being initialized if the GuC submission is disabled (i.e. i915.guc_submission=0). host2guc_action(), which uses the action_lock can be used for non-submission purposes, e.g. triggering HuC authentication. Moving action_lock initialization before enablement check will allow us t

Re: [Intel-gfx] [PATCH 4/4] drm/i915/lspcon: Remove unused force change mode parameter

2016-11-22 Thread Imre Deak
On Tue, 2016-11-22 at 21:09 +0530, Sharma, Shashank wrote: > The reason why I kept the force function was, that during debug phase, I  > saw a need to change LSPCON mode. > > Also, in future, if we need to provide an IOCTL/control to userspace, to  > change lspcon mode, force may be required to di

  1   2   >