[Intel-gfx] [CI 1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt

2020-04-16 Thread Chris Wilson
It seems that although (perhaps because of the memory stall?) the spinner has signaled that it has started, it still takes some time to spin up to 100% utilisation of the HW. Since the test depends on the full utilisation of the HW to trigger the RPS interrupt, wait a little bit and flush the inter

[Intel-gfx] [CI 4/5] drm/i915/selftests: Move gpu energy measurement into its own little lib

2020-04-16 Thread Chris Wilson
Move the handy utility to measure the GPU energy consumption using RAPL msr into a common lib so that it can be reused easily. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/Makefile| 3 ++- drivers/gpu/drm/i915/gt/selftest_rc6.c | 25 +-- drivers/gpu/drm

[Intel-gfx] [CI 3/5] drm/i915/gt: Use the RPM config register to determine clk frequencies

2020-04-16 Thread Chris Wilson
For many configuration details within RC6 and RPS we are programming intervals for the internal clocks. From gen11, these clocks are configuration via the RPM_CONFIG and so for convenience, we would like to convert to/from more natural units (ns). Signed-off-by: Chris Wilson Cc: Andi Shyti Cc: M

[Intel-gfx] [CI 2/5] drm/i915/gt: Trace RPS events

2020-04-16 Thread Chris Wilson
Add tracek to the RPS events (interrupts, worker, enabling, threshold selection, frequency setting), so that if we have to debug reticent HW we have some traces to start from. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/intel_rps.c | 47 ++--- 1 file changed,

[Intel-gfx] [CI 5/5] drm/i915/selftests: Check power consumption at min/max frequencies

2020-04-16 Thread Chris Wilson
A basic premise of RPS is that at lower frequencies, not only do we run slower, but we save power compared to higher frequencies. For example, when idle, we set the minimum frequency just in case there is some residual current. Since the power curve should be a physical relationship, if we find no

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt

2020-04-16 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt URL : https://patchwork.freedesktop.org/series/76055/ State : warning == Summary == $ dim checkpatch origin/drm-tip 840a974dda70 drm/i915/selftests: Delay spinner befor

[Intel-gfx] ✗ Fi.CI.DOCS: warning for series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt

2020-04-16 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt URL : https://patchwork.freedesktop.org/series/76055/ State : warning == Summary == $ make htmldocs 2>&1 > /dev/null | grep i915 /home/cidrm/kernel/Documentation/gpu/i9

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt

2020-04-16 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/selftests: Delay spinner before waiting for an interrupt URL : https://patchwork.freedesktop.org/series/76055/ State : failure == Summary == CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17338 ===

[Intel-gfx] [drm-intel:topic/core-for-CI 2/20] kernel/locking/lockdep.c:2744:49: sparse: sparse: cast truncates bits from constant value (a0000 becomes 0)

2020-04-16 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel topic/core-for-CI head: d0435a9b45070b945578c093dcd363b6b73a502c commit: cbc1ad45be43de36150fd98dae644fc89a69a5a0 [2/20] lockdep: Up MAX_LOCKDEP_CHAINS reproduce: # apt-get install sparse # sparse version: v0.6.1-191-gc51a0382-dirty

Re: [Intel-gfx] [PATCH 1/2] drm/i915/tgl: TBT AUX should use TC power well ops

2020-04-16 Thread Matt Roper
On Wed, Apr 15, 2020 at 04:58:58PM -0700, Souza, Jose wrote: > On Wed, 2020-04-15 at 16:34 -0700, Matt Roper wrote: > > As on ICL, we want to use the Type-C aux handlers for the TBT aux > > wells > > to ensure the DP_AUX_CH_CTL_TBT_IO flag is set properly. > > Reviewed-by: José Roberto de Souza

Re: [Intel-gfx] improve use_mm / unuse_mm v2

2020-04-16 Thread Matthew Wilcox
On Thu, Apr 16, 2020 at 07:31:55AM +0200, Christoph Hellwig wrote: > this series improves the use_mm / unuse_mm interface by better > documenting the assumptions, and my taking the set_fs manipulations > spread over the callers into the core API. I appreciate all the work you're doing here. Do yo

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Move the batch buffer pool from the engine to the gt (rev2)

2020-04-16 Thread Patchwork
== Series Details == Series: drm/i915/gt: Move the batch buffer pool from the engine to the gt (rev2) URL : https://patchwork.freedesktop.org/series/75979/ State : success == Summary == CI Bug Log - changes from CI_DRM_8307_full -> Patchwork_17322_full =

Re: [Intel-gfx] improve use_mm / unuse_mm v2

2020-04-16 Thread Christoph Hellwig
On Thu, Apr 16, 2020 at 08:17:44PM -0700, Matthew Wilcox wrote: > On Thu, Apr 16, 2020 at 07:31:55AM +0200, Christoph Hellwig wrote: > > this series improves the use_mm / unuse_mm interface by better > > documenting the assumptions, and my taking the set_fs manipulations > > spread over the callers

Re: [Intel-gfx] [PATCH 1/1] drm/i915: Adding YUV444 packed format support for skl+ (V15)

2020-04-16 Thread Jani Nikula
On Thu, 16 Apr 2020, "Paauwe, Bob J" wrote: > Thanks Jani! What about the corresponding IGT patch? Did that get > pushed too so that this won't break CI? > > https://patchwork.freedesktop.org/series/70516/ Pushed now, thanks for the reminder. BR, Jani. -- Jani Nikula, Intel Open Source Grap

[Intel-gfx] [PATCH] drm/i915/audio: error log non-zero audio power refcount after unbind

2020-04-16 Thread Jani Nikula
We have some module unload/reload tests hitting an issue with i915 unbinding the component interface before the audio driver has properly put the power. Log an error about it for ease of debugging. (Normally this leads to a wakeref debug splat on the power well.) Cc: Kai Vehmanen Signed-off-by: J

<    1   2