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

"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

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

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

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

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 ---

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

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.

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

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

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

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 +

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

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

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

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 ++-

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

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 +--

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

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

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

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

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

== 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:

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

== 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

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

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

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

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

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

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

== 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

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

== 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.

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

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

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

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

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

== 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

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

== 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

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

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

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

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 >

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

== 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

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.

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

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

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

== 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

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

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ä >

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

== 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

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

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

== 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

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

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;

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

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 > ---

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

> -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] [PATCH i-g-t 08/12] meson: Build and install intel_l3_parity

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

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

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 +++

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

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

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

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

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

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

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

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

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

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

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

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(-)

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

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 ---

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

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 *

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

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

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

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 ---

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

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

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

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

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

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

== 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

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

== 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

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

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

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

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

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

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

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

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

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

== 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

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

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

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

== 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

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:

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

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

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

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

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

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

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

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

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

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(-)

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

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

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

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

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

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

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

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 +

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

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

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

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

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

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 +

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

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

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

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 >

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

== 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

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

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

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

== 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

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

== 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

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

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(-)

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

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

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

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

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

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 ---

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

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

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

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

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

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 +++

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

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

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

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.

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

--- 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 +++

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

== 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

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

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

== 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

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

== 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

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

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

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

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

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

>-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

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

== 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

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

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:

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

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

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

== 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 : success == Summary == Series 30108v1 series starting with [01/11] drm/i915: Mark the userptr invalidate

Re: [Intel-gfx] [PATCH 2/2] drm/i915/guc: Remove obsolete comments and remove unused variable

Quoting Michał Winiarski (2017-09-11 14:10:00) > Originally removed in: > c1adab970348 ("drm/i915/guc: Remove failed doorbell stat from debugfs") > f1448a62a103 ("drm/i915/guc: Remove last submission result from debugfs") > > Were accidentaly restored in: > 925344ccc91d ("BackMerge tag

  1   2   >