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
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
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
== 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
=
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
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
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
== 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
===
== 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
== 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
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,
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
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
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
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
> -Original Message-
> From: Jani Nikula
> Sent: Thursday, April 16, 2020 2:10 AM
> To: Paauwe, Bob J ; intel-gfx g...@lists.freedesktop.org>
> Cc: Daniel Vetter
> Subject: Re: [Intel-gfx] [PATCH 1/1] drm/i915: Adding YUV444 packed format
> support for skl+ (V15)
>
> On Wed, 08 Apr 2020
== Series Details ==
Series: series starting with [1/2] drm/i915/tgl: TBT AUX should use TC power
well ops
URL : https://patchwork.freedesktop.org/series/75998/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8305_full -> Patchwork_17319_full
===
== Series Details ==
Series: series starting with [1/5] drm/i915/selftests: Delay spinner before
waiting for an interrupt
URL : https://patchwork.freedesktop.org/series/76052/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17337
==
== Series Details ==
Series: series starting with [1/5] drm/i915/selftests: Delay spinner before
waiting for an interrupt
URL : https://patchwork.freedesktop.org/series/76052/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.
== Series Details ==
Series: series starting with [1/5] drm/i915/selftests: Delay spinner before
waiting for an interrupt
URL : https://patchwork.freedesktop.org/series/76052/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
0a7f8f2b844b drm/i915/selftests: Delay spinner before w
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
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
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
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
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,
== Series Details ==
Series: drm/i915/selftests: Delay spinner before waiting for an interrupt
URL : https://patchwork.freedesktop.org/series/76049/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17336
Summ
== Series Details ==
Series: drm/i915/selftests: Delay spinner before waiting for an interrupt
URL : https://patchwork.freedesktop.org/series/76049/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate
Hi Chris,
> +static u64 __measure_power(int duration_ms)
> +{
> + u64 nrg, dt;
> +
> + dt = -ktime_get();
> + nrg = -st_energy_uJ();
> + msleep(5);
usleep_range?
Reviewed-by: Andi Shyti
Thanks,
Andi
___
Intel-gfx mailing list
Intel-gf
== Series Details ==
Series: drm/i915/tgl: Wa_14011059788 (rev2)
URL : https://patchwork.freedesktop.org/series/75990/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8304_full -> Patchwork_17318_full
Summary
---
**FAI
== Series Details ==
Series: drm/i915: Add missing deinitialization cases of load failure (rev2)
URL : https://patchwork.freedesktop.org/series/75987/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17335
Su
Hi Chris,
On Thu, Apr 16, 2020 at 09:33:52PM +0100, Chris Wilson wrote:
> Quoting Andi Shyti (2020-04-16 21:31:10)
> > Hi Chris,
> >
> > > For many configuration details within RC6 and RPS we are programming
> > > intervals for the internal clocks. From gen11, these clocks are
> > > configuration
Quoting Andi Shyti (2020-04-16 21:31:10)
> Hi Chris,
>
> > 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
Hi Chris,
> 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
== Series Details ==
Series: drm/i915: Add missing deinitialization cases of load failure (rev2)
URL : https://patchwork.freedesktop.org/series/75987/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicat
== Series Details ==
Series: drm/i915: Add missing deinitialization cases of load failure (rev2)
URL : https://patchwork.freedesktop.org/series/75987/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
a685f486207f drm/i915: Add missing deinitialization cases of load failure
-:17: W
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: Move gpu energy
measurement into its own little lib
URL : https://patchwork.freedesktop.org/series/76044/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17334
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
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: Move gpu energy
measurement into its own little lib
URL : https://patchwork.freedesktop.org/series/76044/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu
== Series Details ==
Series: series starting with [1/2] drm/i915/selftests: Move gpu energy
measurement into its own little lib
URL : https://patchwork.freedesktop.org/series/76044/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
04da6c2d287e drm/i915/selftests: Move gpu energy
Acked-by: Dave Airlie
On Tue, 5 Nov 2019 at 21:06, Joonas Lahtinen
wrote:
>
> Dave, ping for Acked-by here so we can merge? You already gave an
> early ack in IRC while travelling.
>
> Regards, Joonas
>
> Quoting Joonas Lahtinen (2019-08-30 13:50:53)
> > To ensure cross-driver locking compatibil
== Series Details ==
Series: drm/i915/selftests: Add request throughput measurement to perf (rev3)
URL : https://patchwork.freedesktop.org/series/73930/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17333
On Wed, 2020-04-15 at 20:36 +0100, Chris Wilson wrote:
> Quoting José Roberto de Souza (2020-04-15 20:14:08)
> > + i915_reset_error_state(i915);
>
> If you are bored, we should move this to unregister as that is the
> last
> point at which it can be accessed from userspace. Hopefully I
> rem
== Series Details ==
Series: drm/i915/selftests: Add request throughput measurement to perf (rev3)
URL : https://patchwork.freedesktop.org/series/73930/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplic
The intel_display_power_put_async() used in TC cold sequences made
easy to hit the missing deinitialization of driver in case of load
failure as seen in the stack trace bellow.
intel_modeset_driver_remove_noirq() had to be removed from
i915_driver_modeset_remove_noirq() as those are different
init
== Series Details ==
Series: drm/i915/selftests: Add request throughput measurement to perf (rev3)
URL : https://patchwork.freedesktop.org/series/73930/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
a685a79c5061 drm/i915/selftests: Add request throughput measurement to perf
-:9
On Tue, Apr 14, 2020 at 12:49:56PM -0700, José Roberto de Souza wrote:
> This is a expected timeout of static TC ports not conneceted, so
> not throwing warnings that would taint CI.
>
> v3:
> - moved checks to tc_phy_aux_timeout_expected()
>
> v4:
> - moved and add comments to tc_phy_aux_timeout
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
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
== Series Details ==
Series: drm: Fix page flip ioctl format check
URL : https://patchwork.freedesktop.org/series/76040/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17331
Summary
---
**SUCCESS**
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events (rev3)
URL : https://patchwork.freedesktop.org/series/76036/
State : failure
== Summary ==
Applying: drm/i915/gt: Trace RPS events
Applying: drm/i915/gt: Use the RPM config register to determine clk frequenci
On Thu, 16 Apr 2020, Alex Deucher wrote:
> On Wed, Apr 15, 2020 at 6:05 AM Jani Nikula wrote:
>>
>>
>> Alex, Harry, Christian, can you please eyeball this series and see if it
>> makes sense for you?
>>
>
> Patches 4, 5 are:
> Acked-by: Alex Deucher
> Feel free to take them through whichever tre
== Series Details ==
Series: drm: Fix page flip ioctl format check
URL : https://patchwork.freedesktop.org/series/76040/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate label
gpu/i915:layout, othe
== Series Details ==
Series: drm/i915/icl: Update forcewake firmware ranges (rev2)
URL : https://patchwork.freedesktop.org/series/75864/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17330
Summary
---
Under ideal circumstances, the driver should be able to keep the GPU
fully saturated with work. Measure how close to ideal we get under the
harshest of conditions with no user payload.
v2: Also measure throughput using only one thread.
Signed-off-by: Chris Wilson
---
.../drm/i915/selftests/i915
== Series Details ==
Series: drm/i915/icl: Update forcewake firmware ranges (rev2)
URL : https://patchwork.freedesktop.org/series/75864/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate label
gpu/i
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
On Thu, Apr 16, 2020 at 08:08:14PM +0300, Laurent Pinchart wrote:
> Hi Ville,
>
> Thank you for the patch.
>
> On Thu, Apr 16, 2020 at 08:04:20PM +0300, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > Revert back to comparing fb->format->format instead fb->format for the
> > page flip ioct
On Thu, Apr 16, 2020 at 08:20:23PM +0300, Souza, Jose wrote:
> On Thu, 2020-04-16 at 20:06 +0300, Imre Deak wrote:
> > On Thu, Apr 16, 2020 at 08:03:39PM +0300, Souza, Jose wrote:
> > > On Thu, 2020-04-16 at 19:42 +0300, Imre Deak wrote:
> > > > On Wed, Apr 15, 2020 at 12:14:08PM -0700, José Robert
== Series Details ==
Series: drm/i915/dpcd_bl: Unbreak enable_dpcd_backlight modparam (rev2)
URL : https://patchwork.freedesktop.org/series/75895/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17329
Summar
On Thu, 2020-04-16 at 20:06 +0300, Imre Deak wrote:
> On Thu, Apr 16, 2020 at 08:03:39PM +0300, Souza, Jose wrote:
> > On Thu, 2020-04-16 at 19:42 +0300, Imre Deak wrote:
> > > On Wed, Apr 15, 2020 at 12:14:08PM -0700, José Roberto de Souza
> > > wrote:
> > > > The intel_display_power_put_async() u
== Series Details ==
Series: drm/i915/dpcd_bl: Unbreak enable_dpcd_backlight modparam (rev2)
URL : https://patchwork.freedesktop.org/series/75895/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate la
Hi Ville,
Thank you for the patch.
On Thu, Apr 16, 2020 at 08:04:20PM +0300, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> Revert back to comparing fb->format->format instead fb->format for the
> page flip ioctl. This check was originally only here to disallow pixel
> format changes, but when
On Thu, Apr 16, 2020 at 08:03:39PM +0300, Souza, Jose wrote:
> On Thu, 2020-04-16 at 19:42 +0300, Imre Deak wrote:
> > On Wed, Apr 15, 2020 at 12:14:08PM -0700, José Roberto de Souza
> > wrote:
> > > The intel_display_power_put_async() used in TC cold sequences made
> > > easy to hit the missing de
From: Ville Syrjälä
Revert back to comparing fb->format->format instead fb->format for the
page flip ioctl. This check was originally only here to disallow pixel
format changes, but when we changed it to do the pointer comparison
we potentially started to reject some (but definitely not all) modi
On Thu, 2020-04-16 at 19:42 +0300, Imre Deak wrote:
> On Wed, Apr 15, 2020 at 12:14:08PM -0700, José Roberto de Souza
> wrote:
> > The intel_display_power_put_async() used in TC cold sequences made
> > easy to hit the missing deinitialization of driver in case of load
> > failure as seen in the sta
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events (rev2)
URL : https://patchwork.freedesktop.org/series/76036/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17328
Summar
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events (rev2)
URL : https://patchwork.freedesktop.org/series/76036/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate la
Some workarounds are not sticking across suspend resume cycles. The
forcewake ranges table has been updated and would reflect the hardware
appropriately.
Closes: https://gitlab.freedesktop.org/drm/intel/issues/1222
v2: Simplify the table and use 0 for some unused ranges(Matt)
Cc: Matt Roper
Sig
On Wed, Apr 15, 2020 at 12:14:08PM -0700, José Roberto de Souza wrote:
> The intel_display_power_put_async() used in TC cold sequences made
> easy to hit the missing deinitialization of driver in case of load
> failure as seen in the stack trace bellow.
>
> intel_modeset_driver_remove_noirq() had
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events (rev2)
URL : https://patchwork.freedesktop.org/series/76036/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
8557361fae6d drm/i915/gt: Trace RPS events
7a6ae457560a drm/i915/gt: Use the RPM conf
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events
URL : https://patchwork.freedesktop.org/series/76036/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17327
Summary
-
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events
URL : https://patchwork.freedesktop.org/series/76036/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate label
gp
On Wed, 2020-04-15 at 20:56 -0700, Matt Roper wrote:
> On Tue, Apr 14, 2020 at 04:04:40PM -0700, José Roberto de Souza
> wrote:
> > Right now dp.regs.dp_tp_ctl/status are only set during the encoder
> > pre_enable() hook, what is causing all reads and writes to those
> > registers to go to offset 0
== Series Details ==
Series: drm/i915: Add missing deinitialization cases of load failure
URL : https://patchwork.freedesktop.org/series/75987/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8302_full -> Patchwork_17316_full
== Series Details ==
Series: series starting with [1/2] drm/i915/gt: Trace RPS events
URL : https://patchwork.freedesktop.org/series/76036/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
2aaa5fffbe7c drm/i915/gt: Trace RPS events
1e6b7b0ed0e3 drm/i915/gt: Use the RPM config regi
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
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,
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
== Series Details ==
Series: i915 lpsp support for lpsp igt (rev9)
URL : https://patchwork.freedesktop.org/series/74648/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8302_full -> Patchwork_17315_full
Summary
---
**W
Hi,
On Thu, Apr 16, 2020 at 08:49:42AM -0400, Rodrigo Siqueira wrote:
> First of all, thank you a lot for your review and for testing this
> patchset. I'm going to prepare a new version trying to address all
> issues highlight by you, Petri, and Simon (I'm already working in a new
> version).
Tha
== Series Details ==
Series: series starting with [CI,1/2] drm/i915/selftests: Exercise basic RPS
interrupt generation
URL : https://patchwork.freedesktop.org/series/75983/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_8302_full -> Patchwork_17314_full
===
== Series Details ==
Series: drm/i915/perf: Enable application triggered OA reports (rev2)
URL : https://patchwork.freedesktop.org/series/75310/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8311 -> Patchwork_17326
Summary
== Series Details ==
Series: drm/i915/perf: Enable application triggered OA reports (rev2)
URL : https://patchwork.freedesktop.org/series/75310/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate labe
On Wed, Apr 15, 2020 at 6:05 AM Jani Nikula wrote:
>
>
> Alex, Harry, Christian, can you please eyeball this series and see if it
> makes sense for you?
>
Patches 4, 5 are:
Acked-by: Alex Deucher
Feel free to take them through whichever tree you want.
Alex
> Thanks,
> Jani.
>
>
> On Tue, 07 A
== Series Details ==
Series: drm/i915/perf: Enable application triggered OA reports (rev2)
URL : https://patchwork.freedesktop.org/series/75310/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
f12265d9a10d drm/i915/perf: Enable application triggered OA reports
-:26: WARNING:COMMI
== Series Details ==
Series: drm/i915/gt: Scrub execlists state on resume
URL : https://patchwork.freedesktop.org/series/76026/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8310 -> Patchwork_17325
Summary
---
**SUCC
Hi Maxime,
First of all, thank you a lot for your review and for testing this
patchset. I'm going to prepare a new version trying to address all
issues highlight by you, Petri, and Simon (I'm already working in a new
version).
Just a note, I run this test on top of VKMS in a virtual machine (x86)
== Series Details ==
Series: drm/i915/gt: Scrub execlists state on resume
URL : https://patchwork.freedesktop.org/series/76026/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate label
gpu/i915:layou
Quoting Jani Nikula (2020-04-15 10:48:15)
> On Wed, 15 Apr 2020, Daniel Vetter wrote:
> > On Wed, Apr 15, 2020 at 8:40 AM Jani Nikula
> > wrote:
> >>
> >> On Wed, 08 Apr 2020, Maarten Lankhorst
> >> wrote:
> >> > Hey,
> >> >
> >> > Here's a pull request to pull in the DP PHY Compliance series.
== Series Details ==
Series: SAGV support for Gen12+ (rev21)
URL : https://patchwork.freedesktop.org/series/75129/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8301_full -> Patchwork_17312_full
Summary
---
**SUCCESS
== Series Details ==
Series: drm/i915/display: Enable DP Display Audio WA (rev6)
URL : https://patchwork.freedesktop.org/series/75582/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8309 -> Patchwork_17324
Summary
---
Gen12 brought an important redesign of the OA unit, splitting it in 2
with a per context part (OAR) and a global part (OAG).
OAR deals with per context counters and implements the
MI_REPORT_PERF_COUNT command.
OAG deals with global counters and the OA buffer.
Unfortunately some of the counters a
== Series Details ==
Series: drm/i915/display: Enable DP Display Audio WA (rev6)
URL : https://patchwork.freedesktop.org/series/75582/
State : warning
== Summary ==
$ make htmldocs 2>&1 > /dev/null | grep i915
/home/cidrm/kernel/Documentation/gpu/i915.rst:610: WARNING: duplicate label
gpu/i91
== Series Details ==
Series: drm/i915/gt: Move the batch buffer pool from the engine to the gt
URL : https://patchwork.freedesktop.org/series/75979/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_8301_full -> Patchwork_17310_full
Before we resume, we reset the HW so we restart from a known good state.
However, as a part of the reset process, we drain our pending CS event
queue -- and if we are resuming that does not correspond to internal
state. On setup, we are scrubbing the CS pointers, but alas only on
setup.
Apply the
For certain DP VDSC bpp settings, hblank asserts before hblank_early,
leading to a bad audio state. Driver need to program "hblank early
enable" and "samples per line" parameters in AUDIO_CONFIG_BE
register.
This is Display Audio WA #1406928334 for 4k+VDSC usecase
applicable on DP encoders. Implem
> -Original Message-
> From: Jani Nikula
> Sent: Thursday, April 16, 2020 3:43 PM
> To: Gupta, Anshuman ; Shankar, Uma
>
> Cc: intel-gfx@lists.freedesktop.org; Vehmanen, Kai
> Subject: Re: [v5] drm/i915/display: Enable DP Display Audio WA
>
> On Thu, 16 Apr 2020, Anshuman Gupta wrot
On Thu, 16 Apr 2020, Anshuman Gupta wrote:
> IMHO with control flow this function above initilization is not
> required.
> u may remove this initilization if u agree, while pushing the patch.
I don't recommend anyone changing anything even remotely functional
while pushing. I'm fine
On 2020-04-16 at 13:17:21 +0530, Uma Shankar wrote:
> For certain DP VDSC bpp settings, hblank asserts before hblank_early,
> leading to a bad audio state. Driver need to program "hblank early
> enable" and "samples per line" parameters in AUDIO_CONFIG_BE
> register.
>
> This is Display Audio WA #
On 16-04-2020 15:19, Maarten Lankhorst wrote:
Hey,
Seems we're lacking error handling still when get fails?
Hi Maarten,
Error handling is taken care by dsb-framework, if _prepare() is failed then
dsb->cmd_buf will be null and during dsb-write() mmio-write will be used to
program registers.
1 - 100 of 115 matches
Mail list logo
