Re: [Intel-gfx] [PATCH v2 9/9] drm/i915/gvt: Do not allocate initial ring scan buffer

2017-09-11 Thread Wang, Zhi A
Ah... sorry. This patchset shouldn't be here. -Original Message- From: Wang, Zhi A Sent: Tuesday, September 12, 2017 8:40 AM To: intel-gfx@lists.freedesktop.org Cc: Wang, Zhi A Subject: [PATCH v2 9/9] drm/i915/gvt: Do not allocate initial ring scan buffer Theoretically, the largest bulk

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v2,1/9] drm/i915/gvt: Make elsp_dwords in the right order

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [v2,1/9] drm/i915/gvt: Make elsp_dwords in the right order URL : https://patchwork.freedesktop.org/series/30164/ State : success == Summary == Series 30164v1 series starting with [v2,1/9] drm/i915/gvt: Make elsp_dwords in the right order http

[Intel-gfx] [PATCH v2 7/9] drm/i915/gvt: Rename reserved ring buffer

2017-09-11 Thread Zhi Wang
"reserved" means reserve something from somewhere. Actually they are buffers used by command scanner. Rename it to ring_scan_buffer. v2: - Remove the usage of an extra variable. (Zhenyu) Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 16 drivers/gpu/drm/i9

[Intel-gfx] [PATCH v2 3/9] drm/i915/gvt: Move workload cache init/clean into intel_vgpu_{setup, clean}_submission()

2017-09-11 Thread Zhi Wang
Move vGPU workload cache initialization/de-initialization into intel_vgpu_{setup, clean}_submission() since they are not specific to execlist stuffs. Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/execlist.c | 15 +-- drivers/gpu/drm/i915/gvt/scheduler.c | 24 +

[Intel-gfx] [PATCH v2 2/9] drm/i915/gvt: Rename intel_vgpu_{init, clean}_gvt_context()

2017-09-11 Thread Zhi Wang
To move workload related functions into scheduler.c, an expected way is to collect all the init/clean functions related to vGPU workload submission into fewer functions. Rename intel_vgpu_{init, clean}_gvt_context() for above usage in future. Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt

[Intel-gfx] [PATCH v2 1/9] drm/i915/gvt: Make elsp_dwords in the right order

2017-09-11 Thread Zhi Wang
The context descriptors in elsp_dwords are stored in a reversed order and the definition of context descriptor is also reversed. The revesred stuff is hard to be used and might cause misunderstanding. Make them in the right oder for following code re-factoring. Tested on my SKL NUC. Signed-off-by

[Intel-gfx] [PATCH v2 9/9] drm/i915/gvt: Do not allocate initial ring scan buffer

2017-09-11 Thread Zhi Wang
Theoretically, the largest bulk of commands in the ring buffer of an engine might be the first submission, which usually contains a lot of commands to initialize the HW. After removing the initial allocation of the ring scan buffer and let krealloc() do everything we need, we still have a big chanc

[Intel-gfx] [PATCH v2 6/9] drm/i915/gvt: Fix a memory leak in cmd_parser.c

2017-09-11 Thread Zhi Wang
The pointer points to the original memory can never take the return value of krealloc(). Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/gvt/cmd_parser.c b/drivers/gpu/drm/i915/

[Intel-gfx] [PATCH v2 5/9] drm/i915/gvt: Move tlb_handle_pending into intel_vgpu_submission

2017-09-11 Thread Zhi Wang
Move tlb_handle_pending into intel_vgpu_submssion since it belongs to a part of vGPU submission stuffs Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/gvt.h | 2 +- drivers/gpu/drm/i915/gvt/handlers.c | 2 +- drivers/gpu/drm/i915/gvt/render.c| 3 ++- drivers/gpu/drm/i915/gvt/sche

[Intel-gfx] [PATCH v2 8/9] drm/i915/gvt: Move ring scan buffers into intel_vgpu_submission

2017-09-11 Thread Zhi Wang
Move ring scan buffers into intel_vgpu_submission since they belongs to a part of vGPU submission stuffs. Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/cmd_parser.c | 11 ++- drivers/gpu/drm/i915/gvt/execlist.c | 23 +-- drivers/gpu/drm/i915/gvt/gvt.h

[Intel-gfx] [PATCH v2 4/9] drm/i915/gvt: Introduce intel_vgpu_submission

2017-09-11 Thread Zhi Wang
Introduce intel_vgpu_submission to hold all members related to submission in struct intel_vgpu before. Signed-off-by: Zhi Wang --- drivers/gpu/drm/i915/gvt/execlist.c | 30 +--- drivers/gpu/drm/i915/gvt/gvt.h | 17 + drivers/gpu/drm/i915/gvt/handlers.c | 2 +- driver

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

2017-09-11 Thread Kamble, Sagar A
On 9/11/2017 11:34 PM, Michal Wajdeczko wrote: On Mon, Sep 11, 2017 at 11:32:23AM +0530, Sagar Arun Kamble wrote: From: "Kamble, Sagar A" With GuC v9, new type of Default/critical logging in GuC to enable capturing minimal important logs in production systems efficiently. This patch enables

Re: [Intel-gfx] [PATCH 6/6] drm/i915/guc: Grab RPM wakelock while disabling GuC interrupts

2017-09-11 Thread Kamble, Sagar A
On 9/11/2017 11:04 PM, Michal Wajdeczko wrote: On Mon, Sep 11, 2017 at 11:32:24AM +0530, Sagar Arun Kamble wrote: Disabling GuC interrupts involves access to GuC IRQ control registers hence ensure device is RPM awake. Cc: Michal Wajdeczko Signed-off-by: Sagar Arun Kamble --- drivers/gpu/d

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/bxt: set min brightness from VBT

2017-09-11 Thread Patchwork
== Series Details == Series: drm/i915/bxt: set min brightness from VBT URL : https://patchwork.freedesktop.org/series/30163/ State : success == Summary == Test kms_setmode: Subgroup basic: pass -> FAIL (shard-hsw) fdo#99912 Test kms_cursor_legacy: Su

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/bxt: set min brightness from VBT

2017-09-11 Thread Patchwork
== Series Details == Series: drm/i915/bxt: set min brightness from VBT URL : https://patchwork.freedesktop.org/series/30163/ State : success == Summary == Series 30163v1 drm/i915/bxt: set min brightness from VBT https://patchwork.freedesktop.org/api/1.0/series/30163/revisions/1/mbox/ Test cha

[Intel-gfx] [PATCH] drm/i915/bxt: set min brightness from VBT

2017-09-11 Thread Lee, Shawn C
Min brightness value from vbt was missing for BXT platform. This setting have to refer backlight ic spec to restrict min backlight output. Without this restriction, driver would allow to configure lower brightness value and violate backlight ic requirement. Fixes: 0fb890c01349 ("drm/i915/bxt: BLC

Re: [Intel-gfx] [RFC 04/11] drm/i915/pmu: Expose a PMU interface for perf queries

2017-09-11 Thread Rogozhkin, Dmitry V
On Mon, 2017-09-11 at 16:25 +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > From: Chris Wilson > From: Tvrtko Ursulin > From: Dmitry Rogozhkin > > The first goal is to be able to measure GPU (and invidual ring) busyness > without having to poll registers from userspace. (Which not onl

Re: [Intel-gfx] [RFC v3 00/11] i915 PMU and engine busy stats

2017-09-11 Thread Rogozhkin, Dmitry V
Hi, Just tried v3 series. perf-stat works fine. From the IGT tests which I wrote for i915 PMU (https://patchwork.freedesktop.org/series/29313/) all pass (assuming pmu.enabled will be exposed in debugfs) except cpu_online subtest. And this is pretty interesting - see details below. Ok, be prepa

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.

2017-09-11 Thread Patchwork
== Series Details == Series: drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine. URL : https://patchwork.freedesktop.org/series/30161/ State : success == Summary == Test gem_eio: Subgroup in-flight: fail -> PASS (shard-hsw) fdo#102616 Test kms_cu

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.

2017-09-11 Thread Patchwork
== Series Details == Series: drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine. URL : https://patchwork.freedesktop.org/series/30161/ State : success == Summary == Series 30161v1 drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine. https://patchwork.freedesktop.org/api/

[Intel-gfx] [PATCH] drm/i915: Refresh VLV/CHV PSR comments on HW PSR_state machine.

2017-09-11 Thread Rodrigo Vivi
DK had pointed out a comment there was hard to understand, so I tried to read back again and I couldn't understand that as well. So let me re-phrase that in a way that anyone can understand later, even myself. Also fixed the comment block style. Cc: Dhinakaran Pandiyan Signed-off-by: Rodrigo Viv

Re: [Intel-gfx] [PATCH 11/11] drm/i915/psr: Add enable_source vfunc.

2017-09-11 Thread Vivi, Rodrigo
On Fri, 2017-09-08 at 01:43 +, Pandiyan, Dhinakaran wrote: > On Thu, 2017-09-07 at 16:00 -0700, Rodrigo Vivi wrote: > > Continue on VLV PSR split with vfunc, let's also create one > > for enabling source. > > > > Also since we are touching *_enable_source functions let's > > fix a comment with

[Intel-gfx] ✗ Fi.CI.IGT: failure for meson: Build more tools and install more stuff

2017-09-11 Thread Patchwork
== Series Details == Series: meson: Build more tools and install more stuff URL : https://patchwork.freedesktop.org/series/30155/ State : failure == Summary == Test kms_cursor_legacy: Subgroup flip-vs-cursor-varying-size: pass -> FAIL (shard-hsw) Test drv_su

[Intel-gfx] ✗ Fi.CI.IGT: failure for kms_rotation_crc: 90 degree flip test is not a stress test (rev5)

2017-09-11 Thread Patchwork
== Series Details == Series: kms_rotation_crc: 90 degree flip test is not a stress test (rev5) URL : https://patchwork.freedesktop.org/series/28308/ State : failure == Summary == Test gem_exec_capture: Subgroup capture-render: pass -> INCOMPLETE (shard-hsw) Test g

Re: [Intel-gfx] [PATCH 1/2] drm/i915/tracepoints: Don't compile-out low-level tracepoints

2017-09-11 Thread Rogozhkin, Dmitry V
On Mon, 2017-09-11 at 20:52 +0100, Chris Wilson wrote: > Quoting Tvrtko Ursulin (2017-09-11 16:34:08) > > > > On 11/09/2017 14:09, Michał Winiarski wrote: > > > There's no reason to hide those tracepoints. > > > Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option. > > > > No numbe

Re: [Intel-gfx] [PATCH 1/2] drm/i915/tracepoints: Don't compile-out low-level tracepoints

2017-09-11 Thread Chris Wilson
Quoting Tvrtko Ursulin (2017-09-11 16:34:08) > > On 11/09/2017 14:09, Michał Winiarski wrote: > > There's no reason to hide those tracepoints. > > Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option. > > No numbers from (micro-)bechmarks showing how small the impact of doing > th

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids URL : https://patchwork.freedesktop.org/series/30151/ State : success == Summary == Test kms_atomic_transition: Subgroup plane-all-transition-nonblocking: fail ->

Re: [Intel-gfx] [PATCH v2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-11 Thread Pandiyan, Dhinakaran
On Mon, 2017-09-11 at 16:33 +0300, Ville Syrjälä wrote: > On Wed, Sep 06, 2017 at 05:14:58PM -0700, Dhinakaran Pandiyan wrote: > > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow > > the source to reqest any node in a mst path or a whole path to be > > powered down or up. Th

Re: [Intel-gfx] [PATCH] drm/doc: Update todo.rst

2017-09-11 Thread Sean Paul
On Mon, Sep 11, 2017 at 08:51:51AM +0200, Daniel Vetter wrote: > We're using this for outreachy, unfortunately someone already tried to > look at a task that was done already :-( > > Update them all. > > Cc: Sean Paul > Signed-off-by: Daniel Vetter Applied to -misc-next Thanks! Sean > --- >

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/fb-helper: Use drm_file to get a dumb framebuffer

2017-09-11 Thread Patchwork
== Series Details == Series: drm/fb-helper: Use drm_file to get a dumb framebuffer URL : https://patchwork.freedesktop.org/series/30136/ State : success == Summary == Test kms_setmode: Subgroup basic: fail -> PASS (shard-hsw) fdo#99912 Test kms_atomic_transi

Re: [Intel-gfx] [PATCH] drm/i915: Enable scanline read for gen9 dsi

2017-09-11 Thread Ville Syrjälä
On Mon, Sep 11, 2017 at 01:19:15PM +, Shankar, Uma wrote: > > > >-Original Message- > >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of > >Daniel Vetter > >Sent: Saturday, September 9, 2017 1:15 AM > >To: Ville Syrjälä > >Cc: intel-gfx@lists.freedesktop.o

[Intel-gfx] ✓ Fi.CI.BAT: success for meson: Build more tools and install more stuff

2017-09-11 Thread Patchwork
== Series Details == Series: meson: Build more tools and install more stuff URL : https://patchwork.freedesktop.org/series/30155/ State : success == Summary == IGT patchset tested on top of latest successful build f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(alignment,everything):

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

2017-09-11 Thread Michal Wajdeczko
On Mon, Sep 11, 2017 at 11:32:23AM +0530, Sagar Arun Kamble wrote: > From: "Kamble, Sagar A" > > 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

[Intel-gfx] ✓ Fi.CI.BAT: success for kms_rotation_crc: 90 degree flip test is not a stress test (rev5)

2017-09-11 Thread Patchwork
== Series Details == Series: kms_rotation_crc: 90 degree flip test is not a stress test (rev5) URL : https://patchwork.freedesktop.org/series/28308/ State : success == Summary == IGT patchset tested on top of latest successful build f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(align

Re: [Intel-gfx] [PATCH] drm/i915: Enable scanline read for gen9 dsi

2017-09-11 Thread Ville Syrjälä
On Mon, Sep 11, 2017 at 01:04:18PM +, Shankar, Uma wrote: > > > >-Original Message- > >From: Ville Syrjälä [mailto:ville.syrj...@linux.intel.com] > >Sent: Friday, September 8, 2017 8:18 PM > >To: Srinivas, Vidya > >Cc: intel-gfx@lists.freedesktop.org; Kahola, Mika ; > >Kamath, Sunil

Re: [Intel-gfx] [PATCH 6/6] drm/i915/guc: Grab RPM wakelock while disabling GuC interrupts

2017-09-11 Thread Michal Wajdeczko
On Mon, Sep 11, 2017 at 11:32:24AM +0530, Sagar Arun Kamble wrote: > Disabling GuC interrupts involves access to GuC IRQ control registers > hence ensure device is RPM awake. > > Cc: Michal Wajdeczko > Signed-off-by: Sagar Arun Kamble > --- > drivers/gpu/drm/i915/intel_guc_log.c | 2 ++ > 1 fil

Re: [Intel-gfx] [PATCH] drm/i915: Disable DRRS when PSR is enabled

2017-09-11 Thread Sripada, Radhakrishna
> -Original Message- > From: Daniel Vetter [mailto:daniel.vet...@ffwll.ch] On Behalf Of Daniel > Vetter > Sent: Monday, September 4, 2017 1:14 AM > To: Sripada, Radhakrishna > Cc: Vivi, Rodrigo ; Nikula, Jani > ; intel-gfx@lists.freedesktop.org; Pandiyan, > Dhinakaran ; > nicholas.stom..

[Intel-gfx] [PATCH i-g-t 08/12] meson: Build and install intel_l3_parity

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tools/meson.build | 5 + 1 file changed, 5 insertions(+) diff --git a/tools/meson.build b/tools/meson.build index b6052a0c71c3..66dd697973c3 100644 --- a/tools/meson.build +++ b/tools/meson.build @@ -57,6 +57,11 @@ endforeach pkgdatad

[Intel-gfx] [PATCH i-g-t 12/12] meson: Install intel_gpu_abrt

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tools/meson.build | 4 1 file changed, 4 insertions(+) diff --git a/tools/meson.build b/tools/meson.build index 0e64621d1d45..84b3467a6d72 100644 --- a/tools/meson.build +++ b/tools/meson.build @@ -75,6 +75,10 @@ executable('intel_reg',

[Intel-gfx] [PATCH i-g-t 11/12] meson: Rename and install intel-gpu-overlay

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Autotools called the binary 'intel-gpu-overlay', let's have meson use that name as well. Also install it. Signed-off-by: Ville Syrjälä --- overlay/meson.build | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/overlay/meson.build b/overlay/meson.build

[Intel-gfx] [PATCH i-g-t 05/12] meson: Install intel_reg register definitions

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä FIXME: install_subdir exclude_files knob seems to require meson 0.42+. Is that acceptable? Signed-off-by: Ville Syrjälä --- tools/meson.build | 5 + 1 file changed, 5 insertions(+) diff --git a/tools/meson.build b/tools/meson.build index 9d08a0d1f986..b6052a0c71c3 1006

[Intel-gfx] [PATCH i-g-t 07/12] meson: Install test image files

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tests/meson.build | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tests/meson.build b/tests/meson.build index 37c4c4c24fa5..df315de2713c 100644 --- a/tests/meson.build +++ b/tests/meson.build @@ -301,3 +301,10 @@ foreach prog : han

[Intel-gfx] [PATCH i-g-t 06/12] meson: Install test-list.txt

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tests/generate_testlist.sh | 9 ++--- tests/meson.build | 9 - 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/tests/generate_testlist.sh b/tests/generate_testlist.sh index 6ea78655daca..a0d53e9ff095 10075

[Intel-gfx] [PATCH i-g-t 09/12] meson: Build and install intel_dp_compliance

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tools/meson.build | 5 + 1 file changed, 5 insertions(+) diff --git a/tools/meson.build b/tools/meson.build index 66dd697973c3..bf24602396f1 100644 --- a/tools/meson.build +++ b/tools/meson.build @@ -57,6 +57,11 @@ endforeach pkgdatad

[Intel-gfx] [PATCH i-g-t 10/12] meson: Drop the 'lib' prefix from intel_aubdump.so

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Autotools didn't add the 'lib' prefix to the intel_aubdump.so filename. Coerce meson to match that. Signed-off-by: Ville Syrjälä --- tools/meson.build | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/meson.build b/tools/meson.build index bf24602

[Intel-gfx] [PATCH i-g-t 04/12] meson: Build and install intel_reg

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tools/meson.build | 10 ++ 1 file changed, 10 insertions(+) diff --git a/tools/meson.build b/tools/meson.build index 3b6d6376d2d6..9d08a0d1f986 100644 --- a/tools/meson.build +++ b/tools/meson.build @@ -55,6 +55,16 @@ foreach prog :

[Intel-gfx] [PATCH i-g-t 00/12] meson: Build more tools and install more stuff

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä This series makes meson install a bunch of missing tools, and tries to unify a bit more what meson vs. autotools installs. Notable differences left after this series: * asm/disasm missing from meson * intel_aubdump script missing from meson * some benchmarks/tests missing fro

[Intel-gfx] [PATCH i-g-t 01/12] meson: Install tests into $libexecdir

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä autotools installs the tests into $libexecdir. Make meson do the same. Signed-off-by: Ville Syrjälä --- tests/meson.build | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/meson.build b/tests/meson.build index 4dd5a9c9d4c7..1f98f2a02bb9 10

[Intel-gfx] [PATCH i-g-t 02/12] meson: Install tools

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä Signed-off-by: Ville Syrjälä --- tools/meson.build | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/meson.build b/tools/meson.build index d2d4410e2130..3b6d6376d2d6 100644 --- a/tools/meson.build +++ b/tools/meson.build @@ -51,7 +51,8 @@ endif f

[Intel-gfx] [PATCH i-g-t 03/12] tools/intel_reg: s/PKGDATADIR/IGT_DATADIR/

2017-09-11 Thread Ville Syrjala
From: Ville Syrjälä We use IGT_DATADIR to refer to $pkgdatadir elsewhere, so let's have intel_reg follow suit. Signed-off-by: Ville Syrjälä --- tools/Makefile.am | 2 +- tools/intel_reg.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/Makefile.am b/tools/Makefile.

[Intel-gfx] [PATCH i-g-t v4] kms_rotation_crc: 90 degree flip test is not a stress test

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin To the best of my recollection the page flipping test was added simply to start exercising page flips with 90/270 rotation. There is no need to do 60 flips which can take quite some time, because we do 60 flips against each pipe and each fb geometry. Also, calling this a st

Re: [Intel-gfx] [PATCH 1/2] drm/i915/kbl: Remove unused Kabylake pci ids

2017-09-11 Thread Rodrigo Vivi
On Mon, Sep 11, 2017 at 04:11:33PM +, Anuj Phogat wrote: > See Mesa commits: ebc5ccf and b2dae9f I believe we need to be in sync between multiple gfx stack components, but I don't believe we should remove ids. In the past we had cases where we noticed a product group using a listed id to do

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK URL : https://patchwork.freedesktop.org/series/30124/ State : success == Summary == Test perf: Subgroup blocking: fail -> PASS (shard-hsw) fdo#102252 fdo#102252 ht

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids URL : https://patchwork.freedesktop.org/series/30151/ State : success == Summary == Series 30151v1 series starting with [1/2] drm/i915/kbl: Remove unused Kabylake pci ids https://patchwork.fre

[Intel-gfx] [PATCH libdrm 2/2] intel: Change a KBL pci id to GT2 from GT1.5

2017-09-11 Thread Anuj Phogat
See Mesa commit 9c588ff Cc: Matt Turner Cc: Rodrigo Vivi Signed-off-by: Anuj Phogat --- intel/intel_chipset.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/intel/intel_chipset.h b/intel/intel_chipset.h index 77a9ca6..6bd8ae2 100644 --- a/intel/intel_chipset.h +++ b/

[Intel-gfx] [PATCH libdrm 1/2] intel: Remove unused Kabylake pci ids

2017-09-11 Thread Anuj Phogat
These PCI IDs are not used in any Kabylake SKUs. See Mesa commits: ebc5ccf and b2dae9f Cc: Matt Turner Cc: Rodrigo Vivi Signed-off-by: Anuj Phogat --- intel/intel_chipset.h | 26 -- 1 file changed, 4 insertions(+), 22 deletions(-) diff --git a/intel/intel_chipset.h b/i

[Intel-gfx] [PATCH 1/2] drm/i915/kbl: Remove unused Kabylake pci ids

2017-09-11 Thread Anuj Phogat
See Mesa commits: ebc5ccf and b2dae9f Cc: Matt Turner Cc: Rodrigo Vivi Signed-off-by: Anuj Phogat --- drivers/gpu/drm/i915/i915_pci.c | 1 - include/drm/i915_pciids.h | 15 ++- 2 files changed, 2 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_pci.c b/

[Intel-gfx] [PATCH 2/2] drm/i915/kbl: Change a KBL pci id to GT2 from GT1.5

2017-09-11 Thread Anuj Phogat
See Mesa commit 9c588ff Cc: Matt Turner Cc: Rodrigo Vivi Signed-off-by: Anuj Phogat --- include/drm/i915_pciids.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/drm/i915_pciids.h b/include/drm/i915_pciids.h index a1bf90e..1c29063 100644 --- a/include/drm/i915_pciid

[Intel-gfx] ✗ Fi.CI.IGT: warning for series starting with [01/11] drm/i915: Mark the userptr invalidate workqueue as WQ_MEM_RECLAIM

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [01/11] drm/i915: Mark the userptr invalidate workqueue as WQ_MEM_RECLAIM URL : https://patchwork.freedesktop.org/series/30108/ State : warning == Summary == Test perf: Subgroup polling: fail -> PASS (shard-

Re: [Intel-gfx] [PATCH 1/2] drm/i915/guc: Add GuC Load time to debugfs

2017-09-11 Thread Tvrtko Ursulin
On 07/09/2017 18:07, Srivatsa, Anusha wrote: -Original Message- From: Tvrtko Ursulin [mailto:tvrtko.ursu...@linux.intel.com] Sent: Thursday, September 7, 2017 1:49 AM To: Srivatsa, Anusha ; intel- g...@lists.freedesktop.org Cc: Sundaresan, Sujaritha Subject: Re: [Intel-gfx] [PATCH 1/2]

[Intel-gfx] ✗ Fi.CI.BAT: warning for i915 PMU and engine busy stats (rev3)

2017-09-11 Thread Patchwork
== Series Details == Series: i915 PMU and engine busy stats (rev3) URL : https://patchwork.freedesktop.org/series/27488/ State : warning == Summary == Series 27488v3 i915 PMU and engine busy stats https://patchwork.freedesktop.org/api/1.0/series/27488/revisions/3/mbox/ Test chamelium:

Re: [Intel-gfx] [PATCH v6 1/2] drm/i915: Introduce INTEL_GEN_MASK

2017-09-11 Thread Tvrtko Ursulin
On 11/09/2017 13:32, Joonas Lahtinen wrote: Split INTEL_GEN_MASK out of IS_GEN macro, and make it usable within static declarations (unlike combound statements). Typo -> compound to fixup when merging. Regards, Tvrtko Signed-off-by: Joonas Lahtinen Cc: Jani Nikula Cc: Chris Wilson Revi

Re: [Intel-gfx] [PATCH 1/2] drm/i915/tracepoints: Don't compile-out low-level tracepoints

2017-09-11 Thread Tvrtko Ursulin
On 11/09/2017 14:09, Michał Winiarski wrote: There's no reason to hide those tracepoints. Let's also remove the DRM_I915_LOW_LEVEL_TRACEPOINTS Kconfig option. No numbers from (micro-)bechmarks showing how small the impact of doing this is? I thought John was compiling this data. It will be ju

[Intel-gfx] [RFC 10/11] drm/i915: Export engine stats API to other users

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Other kernel users might want to look at total GPU busyness in order to implement things like package power distribution algorithms more efficiently. Signed-off-by: Tvrtko Ursulin Cc: Ben Widawsky Cc: Ben Widawsky --- drivers/gpu/drm/i915/intel_engine_cs.c | 3 +++ 1 fil

[Intel-gfx] [RFC 11/11] drm/i915: Gate engine stats collection with a static key

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin This reduces the cost of the software engine busyness tracking to a single no-op instruction when there are no listeners. v2: Rebase and some comments. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_pmu.c | 54 +++-- drivers/gpu/dr

[Intel-gfx] [RFC 08/11] drm/i915: Export engine busy stats in debugfs

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Export the stats added in the previous patch in debugfs. Number of active clients reading this data is tracked and the static key is only enabled whilst there are some. Userspace is intended to keep the file descriptor open, seeking to the beginning of the file periodically

[Intel-gfx] [RFC 06/11] drm/i915: Wrap context schedule notification

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin No functional change just something which will be handy in the following patch. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/intel_lrc.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/driv

[Intel-gfx] [RFC 09/11] drm/i915/pmu: Wire up engine busy stats to PMU

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin We can use engine busy stats instead of the MMIO sampling timer for better efficiency. As minimum this saves period * num_engines / sec mmio reads, and in a better case, when only engine busy samplers are active, it enables us to not kick off the sampling timer at all. v2:

[Intel-gfx] [RFC 01/11] drm/i915: Convert intel_rc6_residency_us to ns

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Will be used for exposing the PMU counters. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_drv.h | 8 +++- drivers/gpu/drm/i915/intel_pm.c | 23 +-- 2 files changed, 16 insertions(+), 15 deletions(-) diff --git a/drivers/gpu/drm/i915/

[Intel-gfx] [RFC 07/11] drm/i915: Engine busy time tracking

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Track total time requests have been executing on the hardware. We add new kernel API to allow software tracking of time GPU engines are spending executing requests. Both per-engine and global API is added with the latter also being exported for use by external users. v2:

[Intel-gfx] [RFC 03/11] drm/i915: Extract intel_get_cagf

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Code to be shared between debugfs and the PMU implementation. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_debugfs.c | 8 +--- drivers/gpu/drm/i915/i915_drv.h | 1 + drivers/gpu/drm/i915/intel_pm.c | 14 ++ 3 files changed, 16 inser

[Intel-gfx] [RFC 04/11] drm/i915/pmu: Expose a PMU interface for perf queries

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin From: Chris Wilson From: Tvrtko Ursulin From: Dmitry Rogozhkin The first goal is to be able to measure GPU (and invidual ring) busyness without having to poll registers from userspace. (Which not only incurs holding the forcewake lock indefinitely, perturbing the system,

[Intel-gfx] [RFC 05/11] drm/i915/pmu: Suspend sampling when GPU is idle

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin If only a subset of events is enabled we can afford to suspend the sampling timer when the GPU is idle and so save some cycles and power. v2: Rebase and limit timer even more. v3: Rebase. v4: Rebase. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_drv.h

[Intel-gfx] [RFC 02/11] drm/i915: Add intel_energy_uJ

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Extract code from i915_energy_uJ (debugfs) so it can be used by other callers in future patches. v2: Rebase. Signed-off-by: Tvrtko Ursulin --- drivers/gpu/drm/i915/i915_debugfs.c | 17 + drivers/gpu/drm/i915/i915_drv.h | 2 ++ drivers/gpu/drm/i915/int

[Intel-gfx] [RFC v3 00/11] i915 PMU and engine busy stats

2017-09-11 Thread Tvrtko Ursulin
From: Tvrtko Ursulin Third spin of the i915 PMU series. In this version I integrated some of the changes Dmitry made after clarifying the perf API usage questions with Peter Zijlstra. Some other changes I integrated in spirit but changed the implementation, and then finally there are some new fi

Re: [Intel-gfx] [PATCH 2/2] dim: sign pull-request tags if the GPG key is configured

2017-09-11 Thread Sean Paul
On Mon, Sep 11, 2017 at 9:16 AM, Jani Nikula wrote: > Add new environment variable $DIM_GPG_KEYID for configuring the GPG key > ID of the key to use for signing tags for pull requests. The tags will > be signed if the key ID is set, otherwise annotated tags will be used > like before. > > The drm-

[Intel-gfx] ✗ Fi.CI.BAT: failure for igt/sw_sync: Fix up close(timeline) tests for unsignaled fences

2017-09-11 Thread Patchwork
== Series Details == Series: igt/sw_sync: Fix up close(timeline) tests for unsignaled fences URL : https://patchwork.freedesktop.org/series/30126/ State : failure == Summary == IGT patchset tested on top of latest successful build f9e0154630766b63617c64255a68e5129e233a4b igt/gem_evict_(alignme

Re: [Intel-gfx] [PATCH 1/2] drm/i915/guc: Submit GuC workitems containing coalesced requests

2017-09-11 Thread Chris Wilson
Quoting Michał Winiarski (2017-09-11 15:17:31) > To create an upper bound on number of GuC workitems, we need to change > the way that requests are being submitted. Rather than submitting each > request as an individual workitem, we can do coalescing in a similar way > we're handlig execlist submis

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/fb-helper: Use drm_file to get a dumb framebuffer

2017-09-11 Thread Patchwork
== Series Details == Series: drm/fb-helper: Use drm_file to get a dumb framebuffer URL : https://patchwork.freedesktop.org/series/30136/ State : success == Summary == Series 30136v1 drm/fb-helper: Use drm_file to get a dumb framebuffer https://patchwork.freedesktop.org/api/1.0/series/30136/rev

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/2] drm/i915/guc: Submit GuC workitems containing coalesced requests

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/guc: Submit GuC workitems containing coalesced requests URL : https://patchwork.freedesktop.org/series/30135/ State : failure == Summary == Series 30135v1 series starting with [1/2] drm/i915/guc: Submit GuC workitems containing

[Intel-gfx] [PATCH 7/8] drm/tinydrm: Use drm_fbdev_cma_init()

2017-09-11 Thread Noralf Trønnes
Since the cma library now uses drm_framebuffer_create_dumb(), drm_fbdev_cma_init() will provide a framebuffer with the dirty callback set. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/tinydrm/core/tinydrm-core.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/driver

[Intel-gfx] [PATCH 6/8] drm/fb-cma-helper: Drop unnecessary fbdev buffer offset

2017-09-11 Thread Noralf Trønnes
Drop the unnecessary fbdev buffer offset, it can only be zero since drm_fb_helper_fill_var() sets xoffset and yoffset to zero; Cc: Laurent Pinchart Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_cma_helper.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/d

[Intel-gfx] [PATCH 8/8] drm/fb-cma-helper: Remove drm_fbdev_cma_init_with_funcs()

2017-09-11 Thread Noralf Trønnes
No users left so remove drm_fbdev_cma_init_with_funcs(). Also remove example fbdev code in docs, there's nothing special now. Cc: Laurent Pinchart Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_cma_helper.c | 59 ++--- 1 file changed, 3 insertions(+), 5

[Intel-gfx] [PATCH 5/8] drm/fb-cma-helper: Use drm_framebuffer_create_dumb()

2017-09-11 Thread Noralf Trønnes
Use drm_framebuffer_create_dumb() to create a framebuffer for fbdev. No need for error cleanup in drm_fbdev_cma_create(), since the one in drm_fbdev_cma_init_with_funcs() will do fine. Cc: Laurent Pinchart Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_cma_helper.c | 44 +-

[Intel-gfx] [PATCH 4/8] drm/fb-helper: Allocate a drm_file

2017-09-11 Thread Noralf Trønnes
Allocate a drm_file so drivers can use drm_framebuffer_create_dumb() to create a buffer. The framebuffer is reaped automtically in drm_fb_helper_fini(). Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_fb_helper.c | 22 -- include/drm/drm_fb_helper.h | 9 +

[Intel-gfx] [PATCH 3/8] drm/auth: Export drm_dropmaster_ioctl()

2017-09-11 Thread Noralf Trønnes
Export drm_dropmaster_ioctl() so in-kernel drm_file_alloc() users can drop master. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_auth.c | 1 + drivers/gpu/drm/drm_internal.h | 2 -- include/drm/drm_auth.h | 2 ++ 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/

[Intel-gfx] [PATCH 2/8] drm/framebuffer: Add drm_framebuffer_create_dumb()

2017-09-11 Thread Noralf Trønnes
drm_framebuffer_create_dumb() uses the dumb buffer API to create a buffer that is attached to a framebuffer. Useful for fbdev emulation. Signed-off-by: Noralf Trønnes --- drivers/gpu/drm/drm_framebuffer.c | 61 +++ include/drm/drm_framebuffer.h | 4 +++ 2

[Intel-gfx] [PATCH 1/8] drm: provide management functions for drm_file

2017-09-11 Thread Noralf Trønnes
From: David Herrmann Rather than doing drm_file allocation/destruction right in the fops, lets provide separate helpers. This decouples drm_file management from the still-mandatory drm-fops. It prepares for use of drm_file without the fops, both by possible separate fops implementations and APIs

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

2017-09-11 Thread Noralf Trønnes
Hi, I want to start out by saying that this patchset is low priority for me and if no one has interest or time to review this, that is just fine. I was in the flow and just typed it out. This patchset adds a way for fbdev emulation code to create a framebuffer that is backed by a dumb buffer. drm

[Intel-gfx] [PATCH 2/2] HAX Enable GuC Submission for CI

2017-09-11 Thread Michał Winiarski
--- drivers/gpu/drm/i915/i915_params.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_params.c b/drivers/gpu/drm/i915/i915_params.c index 8ab003dca113..c9d72f1b8383 100644 --- a/drivers/gpu/drm/i915/i915_params.c +++ b/drivers/gpu/drm/i915/i915_p

[Intel-gfx] ✗ Fi.CI.BAT: warning for series starting with [1/2] drm/i915/tracepoints: Don't compile-out low-level tracepoints

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915/tracepoints: Don't compile-out low-level tracepoints URL : https://patchwork.freedesktop.org/series/30129/ State : warning == Summary == Series 30129v1 series starting with [1/2] drm/i915/tracepoints: Don't compile-out low-leve

[Intel-gfx] [PATCH 1/2] drm/i915/guc: Submit GuC workitems containing coalesced requests

2017-09-11 Thread Michał Winiarski
To create an upper bound on number of GuC workitems, we need to change the way that requests are being submitted. Rather than submitting each request as an individual workitem, we can do coalescing in a similar way we're handlig execlist submission ports. We also need to stop pretending that we're

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK URL : https://patchwork.freedesktop.org/series/30124/ State : success == Summary == Series 30124v1 series starting with [v6,1/2] drm/i915: Introduce INTEL_GEN_MASK https://patchwork.freedesktop.org/ap

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [v5,1/2] drm/i915: Introduce INTEL_GEN_MASK

2017-09-11 Thread Patchwork
== Series Details == Series: series starting with [v5,1/2] drm/i915: Introduce INTEL_GEN_MASK URL : https://patchwork.freedesktop.org/series/30113/ State : success == Summary == Test perf: Subgroup blocking: pass -> FAIL (shard-hsw) fdo#102252 Test kms_setmo

Re: [Intel-gfx] [PATCH v2] drm/dp/mst: Sideband message transaction to power up/down nodes

2017-09-11 Thread Ville Syrjälä
On Wed, Sep 06, 2017 at 05:14:58PM -0700, Dhinakaran Pandiyan wrote: > The POWER_DOWN_PHY and POWER_UP_PHY sideband message transactions allow > the source to reqest any node in a mst path or a whole path to be > powered down or up. This allows drivers to target a specific sink in the > MST topolog

Re: [Intel-gfx] [PATCH 1/2] dim: make dim tag-next generate annotated tags

2017-09-11 Thread Ewelina Musial
On Mon, Sep 11, 2017 at 04:15:59PM +0300, Jani Nikula wrote: > Presumably an oversight while converting update-next-continue to use > annotated tags. > > Cc: Daniel Vetter > Signed-off-by: Jani Nikula Reviewed-by: Ewelina Musial > --- > dim | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(

Re: [Intel-gfx] [PATCH] drm/i915: Enable scanline read for gen9 dsi

2017-09-11 Thread Shankar, Uma
>-Original Message- >From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of >Daniel Vetter >Sent: Saturday, September 9, 2017 1:15 AM >To: Ville Syrjälä >Cc: intel-gfx@lists.freedesktop.org; Srinivas, Vidya >Subject: Re: [Intel-gfx] [PATCH] drm/i915: Enable scanli

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915: Skylake plane update/disable unifications [v3]

2017-09-11 Thread Patchwork
== Series Details == Series: drm/i915: Skylake plane update/disable unifications [v3] URL : https://patchwork.freedesktop.org/series/30122/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrele

[Intel-gfx] [PATCH 2/2] dim: sign pull-request tags if the GPG key is configured

2017-09-11 Thread Jani Nikula
Add new environment variable $DIM_GPG_KEYID for configuring the GPG key ID of the key to use for signing tags for pull requests. The tags will be signed if the key ID is set, otherwise annotated tags will be used like before. The drm-intel-testing tags are left unsigned, at least for now. Cc: Dan

[Intel-gfx] [PATCH 1/2] dim: make dim tag-next generate annotated tags

2017-09-11 Thread Jani Nikula
Presumably an oversight while converting update-next-continue to use annotated tags. Cc: Daniel Vetter Signed-off-by: Jani Nikula --- dim | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dim b/dim index dbaeb1ec944d..23dfccb5513d 100755 --- a/dim +++ b/dim @@ -1589,7 +1589,7

  1   2   >