== Series Details ==
Series: drm/i915/psr: Print PSR_STATUS when PSR idle wait times out.
URL : https://patchwork.freedesktop.org/series/47262/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4547 -> Patchwork_9773 =
== Summary - FAILURE ==
Serious unknown changes coming
Hi Lee,
Today's linux-next merge of the mfd tree got a conflict in:
drivers/gpu/drm/i915/intel_display.h
between commit:
6075546f57f8 ("drm/i915/icl: store the port type for TC ports")
from the drm-intel tree and commit:
9c229127aee2 ("drm/i915: hdmi: add CEC notifier to intel_hdmi")
Knowing the status of the PSR HW state machine is useful for debug,
especially since we are seeing errors with PSR2 in CI.
Cc: Rodrigo Vivi
Signed-off-by: Dhinakaran Pandiyan
---
drivers/gpu/drm/i915/intel_drv.h| 3 ++-
drivers/gpu/drm/i915/intel_psr.c| 9 ++---
== Series Details ==
Series: series starting with [v2,1/2] drm/i915/mst: Do not retrain new links
URL : https://patchwork.freedesktop.org/series/46797/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4547 -> Patchwork_9772 =
== Summary - FAILURE ==
Serious unknown changes
On Wed, 2018-07-25 at 20:28 +, Patchwork wrote:
> == Series Details ==
>
> Series: series starting with [v2,1/2] drm/i915/mst: Do not retrain
> new links
> URL : https://patchwork.freedesktop.org/series/46797/
> State : failure
>
> == Summary ==
>
> = CI Bug Log - changes from
== Series Details ==
Series: drm/i915/icl: don't set CNL_DDI_CLOCk_REG_ACCESS_ON anymore
URL : https://patchwork.freedesktop.org/series/47259/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4547 -> Patchwork_9771 =
== Summary - FAILURE ==
Serious unknown changes coming
The new recommendation from the spec is to simply not set this bit
anymore. Not setting the bit would prevent some hangs that our driver
manages to avoid since commit c8af5274c3cb ("drm/i915: enable the
pipe/transcoder/planes later on HSW+"), and the theoretical downside
of not setting the bit
== Series Details ==
Series: series starting with [1/2] drm/i915/icl: Add TBT checks for PLL
calculations
URL : https://patchwork.freedesktop.org/series/47248/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4547 -> Patchwork_9770 =
== Summary - FAILURE ==
Serious
Em Qua, 2018-07-25 às 14:28 -0700, Anusha Srivatsa escreveu:
> For a TBT sequence, we need to set the IO type to TBT
> in DDI_AUX_CTL.
>
> Cc: Paulo Zanoni
> Signed-off-by: Anusha Srivatsa
> ---
> drivers/gpu/drm/i915/i915_reg.h | 1 +
> drivers/gpu/drm/i915/intel_dp.c | 34
Em Qua, 2018-07-25 às 14:28 -0700, Anusha Srivatsa escreveu:
> Add missing TBT check in the Pll calculation.
>
> v2: do not use a auxiliary function to check if status is
> TBT or not. (Paulo)
>
> Cc: Paulo Zanoni
> Cc: Lucas De Marchi
> Signed-off-by: Anusha Srivatsa
> ---
>
On Wed, Jul 25, 2018 at 02:21:24PM -0400, Felix Miata wrote:
> Rodrigo Vivi composed on 2018-07-25 09:20 (UTC-0700):
>
> > On Wed, Jul 25, 2018 at 03:37:52AM -0400, Felix Miata wrote:
>
> >> Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video
> >> outputs):
> >>
The aim of this test is to combine gem_linear_blits, gem_tiled_blits etc
into one test runner that covers investigation into HW alignment issues
as well as driver boundaries (relocs, access, thrashing).
Signed-off-by: Chris Wilson
Cc: Katarzyna Dec
---
tests/Makefile.sources | 1 +
On Wed, 2018-07-25 at 14:21 -0400, Felix Miata wrote:
> Rodrigo Vivi composed on 2018-07-25 09:20 (UTC-0700):
>
> >
> > On Wed, Jul 25, 2018 at 03:37:52AM -0400, Felix Miata wrote:
> >
> > >
> > > Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical
> > > video outputs):
> > >
For a TBT sequence, we need to set the IO type to TBT
in DDI_AUX_CTL.
Cc: Paulo Zanoni
Signed-off-by: Anusha Srivatsa
---
drivers/gpu/drm/i915/i915_reg.h | 1 +
drivers/gpu/drm/i915/intel_dp.c | 34 +-
2 files changed, 26 insertions(+), 9 deletions(-)
diff
Add missing TBT check in the Pll calculation.
v2: do not use a auxiliary function to check if status is
TBT or not. (Paulo)
Cc: Paulo Zanoni
Cc: Lucas De Marchi
Signed-off-by: Anusha Srivatsa
---
drivers/gpu/drm/i915/intel_dpll_mgr.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
== Series Details ==
Series: drm/i915/icl: implement icl_digital_port_connected() (rev3)
URL : https://patchwork.freedesktop.org/series/47151/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4542 -> Patchwork_9769 =
== Summary - FAILURE ==
Serious unknown changes coming
Quoting Patchwork (2018-07-25 21:19:41)
> == Series Details ==
That wasn't a particularly successful rebase:
<7>[2.811183] [drm:intel_uc_fw_upload [i915]] GuC fw load
i915/kbl_guc_ver9_39.bin
<7>[2.811208] [drm:intel_uc_fw_upload [i915]] GuC fw load PENDING
<7>[2.812408]
== Series Details ==
Series: series starting with [v2,1/2] drm/i915/mst: Do not retrain new links
URL : https://patchwork.freedesktop.org/series/46797/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4541_full -> Patchwork_9767_full =
== Summary - FAILURE ==
Serious
== Series Details ==
Series: drm/i915/icl: implement icl_digital_port_connected() (rev3)
URL : https://patchwork.freedesktop.org/series/47151/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915/icl: implement icl_digital_port_connected()
Okay!
Commit: drm/i915/icl:
== Series Details ==
Series: drm/i915/icl: implement icl_digital_port_connected() (rev3)
URL : https://patchwork.freedesktop.org/series/47151/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
82dbbab3500e drm/i915/icl: implement icl_digital_port_connected()
-:7:
== Series Details ==
Series: drm/i915/guc: Use correct error code for GuC initialization failure
URL : https://patchwork.freedesktop.org/series/47241/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4542 -> Patchwork_9768 =
== Summary - FAILURE ==
Serious unknown changes
Em Qua, 2018-07-25 às 09:20 -0700, Rodrigo Vivi escreveu:
> On Wed, Jul 25, 2018 at 03:37:52AM -0400, Felix Miata wrote:
> > Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video
> > outputs):
> > https://www.asus.com/us/Motherboards/PRIME-B250M-C-CSM/
> >
> > G4600 HD Graphics 630
On Wed, Jul 25, 2018 at 10:59:32AM -0700, Paulo Zanoni wrote:
> Em Qua, 2018-07-25 às 10:27 -0700, Paulo Zanoni escreveu:
> > Em Ter, 2018-07-24 às 18:19 -0700, Lucas De Marchi escreveu:
> > > On Tue, Jul 24, 2018 at 05:28:09PM -0700, Paulo Zanoni wrote:
> > > > Do like the other functions and
Do like the other functions and check for the status bits. The "Hot Plug
Detection" page from our documentation says we can't just use the ISR bits on
the CPU side (North Display, which has the TC and TBT modes), so use the correct
register: DFLEXDPSP, TC Live State field.
v2: Rebase.
v3:
-
From: Michal Wajdeczko
Since commit 6ca9a2beb54a ("drm/i915: Unwind i915_gem_init() failure")
we believed that we correctly handle all errors encountered during
GuC initialization, including special one that indicates request to
run driver with disabled GPU submission (-EIO).
Unfortunately
== Series Details ==
Series: series starting with [v2,1/2] drm/i915/mst: Do not retrain new links
URL : https://patchwork.freedesktop.org/series/46797/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4541 -> Patchwork_9767 =
== Summary - SUCCESS ==
No regressions found.
Rodrigo Vivi composed on 2018-07-25 09:20 (UTC-0700):
> On Wed, Jul 25, 2018 at 03:37:52AM -0400, Felix Miata wrote:
>> Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video outputs):
>> https://www.asus.com/us/Motherboards/PRIME-B250M-C-CSM/
>> G4600 HD Graphics 630 CPU
>> Server
On Thu, 2018-07-19 at 16:37 -0700, Rodrigo Vivi wrote:
> On Thu, Jul 19, 2018 at 11:51:40AM -0700, Dhinakaran Pandiyan wrote:
> >
> > On Wed, 2018-07-18 at 22:43 -0700, Rodrigo Vivi wrote:
> > >
> > > On Wed, Jul 18, 2018 at 10:19:43AM -0700, Dhinakaran Pandiyan
> > > wrote:
> > > >
> > > >
>
Em Qua, 2018-07-25 às 10:27 -0700, Paulo Zanoni escreveu:
> Em Ter, 2018-07-24 às 18:19 -0700, Lucas De Marchi escreveu:
> > On Tue, Jul 24, 2018 at 05:28:09PM -0700, Paulo Zanoni wrote:
> > > Do like the other functions and check for the status bits. The
> > > "Hot
> > > Plug Detection" page from
On Wed, 25 Jul 2018 12:56:55 +0200, Jakub Bartmiński
wrote:
Missing commit message ... at minimum just repeat commit title
v4:
Move the injection inside the WOPCM init.
Signed-off-by: Jakub Bartmiński
Cc: Chris Wilson
Cc: Michał Winiarski
Cc: Michal Wajdeczko
---
On Wed, 25 Jul 2018 12:56:54 +0200, Jakub Bartmiński
wrote:
Since ggtt_offset_bias is now stored in ggtt.pin_bias, it is duplicated
inside i915_gem_context, and can instead be accessed directly from ggtt.
I'm tempted to merge this patch with previous one (without renames)
v3:
Added a
On Wed, 25 Jul 2018 12:56:53 +0200, Jakub Bartmiński
wrote:
Removing the pin bias from GuC allows us to not check for GuC every time
we pin a context, which fixes the assertion error on unresolved GuC
platform default in mock contexts selftest.
It also seems that we were using uninitialized
Em Ter, 2018-07-24 às 18:19 -0700, Lucas De Marchi escreveu:
> On Tue, Jul 24, 2018 at 05:28:09PM -0700, Paulo Zanoni wrote:
> > Do like the other functions and check for the status bits. The "Hot
> > Plug Detection" page from our documentation says we can't just use
> > the
> > ISR bits on the
== Series Details ==
Series: drm/i915: Mark up object tiling-and-stride getters as const
URL : https://patchwork.freedesktop.org/series/47230/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4540_full -> Patchwork_9765_full =
== Summary - SUCCESS ==
No regressions found.
On Wed, Jul 25, 2018 at 09:52:25AM -0700, Rodrigo Vivi wrote:
> On Tue, Jul 24, 2018 at 05:28:11PM -0700, Paulo Zanoni wrote:
> > From: Animesh Manna
> >
> > In ICL, Flexible IO Adapter (FIA) muxes data and clocks of USB 3.1,
> > tbt and display controller. In DP alt mode FIA configure the
> >
Em Qua, 2018-07-25 às 09:52 -0700, Rodrigo Vivi escreveu:
> On Tue, Jul 24, 2018 at 05:28:11PM -0700, Paulo Zanoni wrote:
> > From: Animesh Manna
> >
> > In ICL, Flexible IO Adapter (FIA) muxes data and clocks of USB 3.1,
> > tbt and display controller. In DP alt mode FIA configure the
> >
On Wed, Jul 25, 2018 at 09:41:18AM -0700, Rodrigo Vivi wrote:
> On Tue, Jul 24, 2018 at 05:28:10PM -0700, Paulo Zanoni wrote:
> > The type is detected based on the live status bits. Once detected,
> > it's not supposed to be changed, so we have some sanity checks for
> > that.
> >
> > v2: Rebase.
On Tue, Jul 24, 2018 at 05:17:53PM -0700, Marc Herbert wrote:
> Reviewed-by: Marc Herbert
pushed, thanks.
>
> Thx Nathan, I think this helps. I'm still curious how training normally
> converges much faster than the total number of possibilities but - unlike
> this latest clarification below -
Em Ter, 2018-07-24 às 18:19 -0700, Lucas De Marchi escreveu:
> On Tue, Jul 24, 2018 at 05:28:09PM -0700, Paulo Zanoni wrote:
> > Do like the other functions and check for the status bits. The "Hot
> > Plug Detection" page from our documentation says we can't just use
> > the
> > ISR bits on the
On Wed, Jul 25, 2018 at 09:52:44AM -0700, Dhinakaran Pandiyan wrote:
> On Wed, 2018-07-25 at 09:12 -0700, Rodrigo Vivi wrote:
> > On Wed, Jul 25, 2018 at 12:22:28AM -0700, Dhinakaran Pandiyan wrote:
> > >
> > > We have merged several fixes, re-written some tests and improved
> > > debug
> > >
On Tue, Jul 24, 2018 at 05:28:11PM -0700, Paulo Zanoni wrote:
> From: Animesh Manna
>
> In ICL, Flexible IO Adapter (FIA) muxes data and clocks of USB 3.1,
> tbt and display controller. In DP alt mode FIA configure the
> number of lanes and will be used apart from DPCD read to calculate max
>
On Tue, Jul 24, 2018 at 05:28:10PM -0700, Paulo Zanoni wrote:
> The type is detected based on the live status bits. Once detected,
> it's not supposed to be changed, so we have some sanity checks for
> that.
>
> v2: Rebase.
>
> Cc: Animesh Manna
> Signed-off-by: Rodrigo Vivi
> Signed-off-by:
On Wed, 2018-07-25 at 09:12 -0700, Rodrigo Vivi wrote:
> On Wed, Jul 25, 2018 at 12:22:28AM -0700, Dhinakaran Pandiyan wrote:
> >
> > We have merged several fixes, re-written some tests and improved
> > debug
> > capability in the past several months, so this is a good time to
> > give PSR1
> >
== Series Details ==
Series: drm/i915: Mark up object tiling-and-stride getters as const
URL : https://patchwork.freedesktop.org/series/47230/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4540 -> Patchwork_9765 =
== Summary - WARNING ==
Minor unknown changes coming
== Series Details ==
Series: drm/i915: Protect guc_fini_wq() against module load abort (rev3)
URL : https://patchwork.freedesktop.org/series/47127/
State : failure
== Summary ==
Applying: drm/i915: Protect guc_fini_wq() against module load abort
error: corrupt patch at line 8
error: could not
On Wed, Jul 25, 2018 at 03:37:52AM -0400, Felix Miata wrote:
> Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video outputs):
> https://www.asus.com/us/Motherboards/PRIME-B250M-C-CSM/
>
> G4600 HD Graphics 630 CPU
>
> Server 1.20.0 (openSUSE Tumbleweed)
>
> kernel 4.17.4
>
> Xorg
On Tue, 24 Jul 2018 16:19:36 +0200, Chris Wilson
wrote:
Prevent
[ 397.873143] general protection fault: [#1] PREEMPT SMP PTI
[ 397.873154] CPU: 4 PID: 4799 Comm: drv_module_relo Tainted: G
U4.18.0-rc6-CI-CI_DRM_4534+ #1
[ 397.873162] Hardware name: Micro-Star
On Wed, Jul 25, 2018 at 12:22:28AM -0700, Dhinakaran Pandiyan wrote:
> We have merged several fixes, re-written some tests and improved debug
> capability in the past several months, so this is a good time to give PSR1
> another try. PSR1 has not been tested on HSW and BDW recently, so let's
>
For that little bit of defense against a tired programmer.
Signed-off-by: Chris Wilson
---
drivers/gpu/drm/i915/i915_gem.c| 4 ++--
drivers/gpu/drm/i915/i915_gem_object.h | 10 +-
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_gem.c
== Series Details ==
Series: mm, oom: distinguish blockable mode for mmu notifiers (rev9)
URL : https://patchwork.freedesktop.org/series/45263/
State : failure
== Summary ==
Applying: mm, oom: distinguish blockable mode for mmu notifiers
error: sha1 information is lacking or useless
On Tue, 24 Jul 2018, Michal Hocko wrote:
> oom_reap_task_mm should return false when __oom_reap_task_mm return
> false. This is what my patch did but it seems this changed by
> http://www.ozlabs.org/~akpm/mmotm/broken-out/mm-oom-remove-oom_lock-from-oom_reaper.patch
> so that one should be fixed.
== Series Details ==
Series: drm/i915/icl: implement icl_digital_port_connected() (rev2)
URL : https://patchwork.freedesktop.org/series/47151/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4539_full -> Patchwork_9762_full =
== Summary - WARNING ==
Minor unknown changes
== Series Details ==
Series: series starting with [v5,1/5] drm/i915/guc: Avoid wasting memory on
incorrect GuC pin bias
URL : https://patchwork.freedesktop.org/series/47201/
State : failure
== Summary ==
= CI Bug Log - changes from CI_DRM_4539 -> Patchwork_9763 =
== Summary - FAILURE ==
Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video outputs):
https://www.asus.com/us/Motherboards/PRIME-B250M-C-CSM/
G4600 HD Graphics 630 CPU
Server 1.20.0 (openSUSE Tumbleweed)
kernel 4.17.4
Xorg driver: modesetting
Xrandr Actual Cable
Reports Connectors
== Series Details ==
Series: series starting with [v5,1/5] drm/i915/guc: Avoid wasting memory on
incorrect GuC pin bias
URL : https://patchwork.freedesktop.org/series/47201/
State : warning
== Summary ==
$ dim sparse origin/drm-tip
Commit: drm/i915/guc: Avoid wasting memory on incorrect GuC
== Series Details ==
Series: drm/i915/icl: implement icl_digital_port_connected() (rev2)
URL : https://patchwork.freedesktop.org/series/47151/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4539 -> Patchwork_9762 =
== Summary - SUCCESS ==
No regressions found.
Since ggtt_offset_bias is now stored in ggtt.pin_bias, it is duplicated
inside i915_gem_context, and can instead be accessed directly from ggtt.
v3:
Added a helper function to retrieve the ggtt.pin_bias from the vma.
v4:
Moved the helper function to the previous patch in the series.
Dropped the
From: Michal Wajdeczko
Signed-off-by: Michal Wajdeczko
---
drivers/gpu/drm/i915/i915_params.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_params.h
b/drivers/gpu/drm/i915/i915_params.h
index aebe0469ddaa..3e4e128237ac 100644
---
v4:
Move the injection inside the WOPCM init.
Signed-off-by: Jakub Bartmiński
Cc: Chris Wilson
Cc: Michał Winiarski
Cc: Michal Wajdeczko
---
drivers/gpu/drm/i915/intel_wopcm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/i915/intel_wopcm.c
It would appear that the calculated GuC pin bias was larger than it should
be, as the GuC address space does NOT contain the "HW contexts RSVD" part
of the WOPCM. Thus, the GuC pin bias is simply the GuC WOPCM size.
v5:
Clarify the diagram to better represent the GuC address space.
Since we now
Removing the pin bias from GuC allows us to not check for GuC every time
we pin a context, which fixes the assertion error on unresolved GuC
platform default in mock contexts selftest.
It also seems that we were using uninitialized WOPCM variables when
setting the GuC pin bias. The pin bias has
== Series Details ==
Series: drm/i915/psr: Enable PSR1 by default on gen9+ platforms
URL : https://patchwork.freedesktop.org/series/47188/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4538_full -> Patchwork_9761_full =
== Summary - WARNING ==
Minor unknown changes
On Mon, Jul 23, 2018 at 09:07:36PM +0100, Chris Wilson wrote:
> Exercise O(N^2) behaviour in reading the error state, and push it to the
> extreme.
>
> Reported-by: Jason Ekstrand
> Signed-off-by: Chris Wilson
LGTM,
Reviewed-by: Katarzyna Dec
Kasia :)
== Series Details ==
Series: drm/i915/psr: Enable PSR1 by default on gen9+ platforms
URL : https://patchwork.freedesktop.org/series/47188/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4538 -> Patchwork_9761 =
== Summary - SUCCESS ==
No regressions found.
External
On Mon, Jul 23, 2018 at 09:07:34PM +0100, Chris Wilson wrote:
> Modernise the test to use igt's ioctl library as opposed to the
> antiquated libdrm_intel.
>
> Signed-off-by: Chris Wilson
LGTM,
Reviewed-by: Katarzyna Dec
Kasia :)
___
Intel-gfx mailing
Felix Miata composed on 2018-07-25 03:37 (UTC-0400):
> Asus B250 LGA 1151 (Kaby Lake) Motherboard (4 total physical video outputs):
> https://www.asus.com/us/Motherboards/PRIME-B250M-C-CSM/
> G4600 HD Graphics 630 CPU
> Server 1.20.0 (openSUSE Tumbleweed)
> kernel 4.17.4
> Xorg driver:
We have merged several fixes, re-written some tests and improved debug
capability in the past several months, so this is a good time to give PSR1
another try. PSR1 has not been tested on HSW and BDW recently, so let's
enable only on gen9+ now.
Cc: Rodigo Vivi
Cc: José Roberto de Souza
Quoting Rodrigo Vivi (2018-07-23 23:23:32)
> On Mon, Jul 23, 2018 at 03:53:35PM +0100, Chris Wilson wrote:
> > If we already wedged, i915_gem_set_wedged() becomes a complicated no-op.
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=107343
> > Signed-off-by: Chris Wilson
>
>
On Tue 24-07-18 12:53:07, Andrew Morton wrote:
[...]
> > On top of that the proposed cleanup looks as follows:
> >
>
> Looks good to me. Seems a bit strange that we omit the pr_info()
> output if the mm was partially reaped - people would still want to know
> this? Not very important though.
On Tue 24-07-18 14:07:49, David Rientjes wrote:
[...]
> mm/oom_kill.c: clean up oom_reap_task_mm() fix
>
> indicate reaping has been partially skipped so we can expect future skips
> or another start before finish.
But we are not skipping. This is essentially the same case as mmap_sem
trylock
70 matches
Mail list logo
