From: Imre Deak <imre.deak@intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>,
intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.BAT: warning for drm/i915/lspcon: Work around resume failure
Date: Mon, 24 Oct 2016 22:05:49 +0300 [thread overview]
Message-ID: <1477335949.18150.18.camel@intel.com> (raw)
In-Reply-To: <87y41dinfy.fsf@intel.com>
On Mon, 2016-10-24 at 21:58 +0300, Jani Nikula wrote:
> On Mon, 24 Oct 2016, Imre Deak <imre.deak@intel.com> wrote:
> > On ma, 2016-10-24 at 17:16 +0000, Patchwork wrote:
> > > == Series Details ==
> > >
> > > Series: drm/i915/lspcon: Work around resume failure
> > > URL : https://patchwork.freedesktop.org/series/14280/
> > > State : warning
> > >
> > > == Summary ==
> > >
> > > Series 14280v1 drm/i915/lspcon: Work around resume failure
> > > https://patchwork.freedesktop.org/api/1.0/series/14280/revisions/1/mb
> > > ox/
> > >
> > > Test drv_module_reload_basic:
> > > pass -> DMESG-WARN (fi-skl-6700hq)
> >
> > This one is a result of the previous CI run where LSPCON failed after
> > S3. After a warm-reboot LSPCON is still in a broken state, and even the
> > initial LSPCON probe will fail.
> >
> > A power cycle between CI runs would prevent the above.
>
> Err, what. So patch 8/8 doesn't fix that problem? What fixes that
> then?!?
It does fix the problem, which happens after S3. The above fail is due to the
previous CI run which left LSPCON in a broken state after an S3 test. A subsequent
warm-reset on that machine won't recover LSPCON and the load time LSPCON probe will
fail. I couldn't reproduce this problem with the workaround applied.
--Imre
>
> BR,
> Jani
>
> >
> > > Test gem_exec_suspend:
> > > Subgroup basic-s3:
> > > dmesg-warn -> PASS (fi-skl-6700hq)
> > > Test kms_pipe_crc_basic:
> > > Subgroup suspend-read-crc-pipe-a:
> > > dmesg-warn -> PASS (fi-skl-6700hq)
> > > Subgroup suspend-read-crc-pipe-b:
> > > dmesg-warn -> PASS (fi-skl-6700hq)
> > > Subgroup suspend-read-crc-pipe-c:
> > > dmesg-warn -> PASS (fi-skl-6700hq)
> > >
> > > fi-bdw-
> > > 5557u total:246 pass:231 dwarn:0 dfail:0 fail:0 skip:15
> > > fi-bsw-
> > > n3050 total:246 pass:204 dwarn:0 dfail:0 fail:0 skip:42
> > > fi-bxt-
> > > t5700 total:246 pass:216 dwarn:0 dfail:0 fail:0 skip:30
> > > fi-byt-
> > > j1900 total:246 pass:215 dwarn:0 dfail:0 fail:0 skip:31
> > > fi-byt-
> > > n2820 total:246 pass:211 dwarn:0 dfail:0 fail:0 skip:35
> > > fi-hsw-
> > > 4770 total:246 pass:224 dwarn:0 dfail:0 fail:0 skip:22
> > > fi-hsw-
> > > 4770r total:246 pass:224 dwarn:0 dfail:0 fail:0 skip:22
> > > fi-ilk-
> > > 650 total:246 pass:185 dwarn:0 dfail:0 fail:1 skip:60
> > > fi-ivb-
> > > 3520m total:246 pass:221 dwarn:0 dfail:0 fail:0 skip:25
> > > fi-ivb-
> > > 3770 total:246 pass:221 dwarn:0 dfail:0 fail:0 skip:25
> > > fi-skl-
> > > 6260u total:246 pass:232 dwarn:0 dfail:0 fail:0 skip:14
> > > fi-skl-
> > > 6700hq total:246 pass:222 dwarn:1 dfail:0 fail:0 skip:23
> > > fi-skl-
> > > 6700k total:246 pass:222 dwarn:1 dfail:0 fail:0 skip:23
> > > fi-skl-
> > > 6770hq total:246 pass:232 dwarn:0 dfail:0 fail:0 skip:14
> > > fi-snb-
> > > 2520m total:246 pass:210 dwarn:0 dfail:0 fail:0 skip:36
> > > fi-snb-
> > > 2600 total:246 pass:209 dwarn:0 dfail:0 fail:0 skip:37
> > >
> > > Results at /Patchwork_2801/
> > >
> > > 102441087e47f4892b88c4f6e308e3edf89eeda1 drm-intel-nightly: 2016y-
> > > 10m-24d-14h-28m-17s UTC integration manifest
> > > eaa672c drm/i915/lspcon: Add workaround for resuming in PCON mode
> > > eb04604 drm/i915/lspcon: Get DDC adapter via container_of() instead
> > > of cached ptr
> > > 0ca993d drm/i915/dp: Read DP descriptor for eDP and LSPCON too
> > > 0144d102 drm/i915/lspcon: Fail LSPCON probe if the start of DPCD
> > > can't be read
> > > 7568e07 drm/i915/dp: Print full branch/sink descriptor
> > > 3b4f6e9 drm/i915/dp: Print only sink or branch specific OUI based on
> > > dev type
> > > b1439bc drm/i915/dp: Remove debug dependency of DPCD SW/HW revision
> > > read
> > > 2beca1a drm/dp: Factor out helper to distinguish between branch and
> > > sink devices
> > >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
> --
> Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2016-10-24 19:05 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-24 16:33 [PATCH v2 0/8] drm/i915/lspcon: Work around resume failure Imre Deak
2016-10-24 16:33 ` [PATCH v2 1/8] drm/dp: Factor out helper to distinguish between branch and sink devices Imre Deak
2016-10-24 17:10 ` Jani Nikula
2016-10-25 6:54 ` [Intel-gfx] " Daniel Vetter
2016-10-25 7:46 ` Jani Nikula
2016-10-25 8:12 ` Daniel Vetter
2016-10-24 16:33 ` [PATCH v2 2/8] drm/i915/dp: Remove debug dependency of DPCD SW/HW revision read Imre Deak
2016-10-24 17:11 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 3/8] drm/i915/dp: Print only sink or branch specific OUI based on dev type Imre Deak
2016-10-24 17:12 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 4/8] drm/i915/dp: Print full branch/sink descriptor Imre Deak
2016-10-24 18:14 ` Jani Nikula
2016-10-24 18:56 ` Imre Deak
2016-10-24 19:10 ` Jani Nikula
2016-10-24 19:35 ` Imre Deak
2016-10-25 9:28 ` Jani Nikula
2016-10-25 10:38 ` Imre Deak
2016-10-25 10:55 ` Jani Nikula
2016-10-25 13:12 ` [PATCH v3 " Imre Deak
2016-10-25 15:07 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 5/8] drm/i915/lspcon: Fail LSPCON probe if the start of DPCD can't be read Imre Deak
2016-10-24 18:48 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 6/8] drm/i915/dp: Read DP descriptor for eDP and LSPCON too Imre Deak
2016-10-24 18:49 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 7/8] drm/i915/lspcon: Get DDC adapter via container_of() instead of cached ptr Imre Deak
2016-10-24 18:53 ` Jani Nikula
2016-10-24 16:33 ` [PATCH v2 8/8] drm/i915/lspcon: Add workaround for resuming in PCON mode Imre Deak
2016-10-25 15:12 ` Jani Nikula
2016-10-24 17:16 ` ✗ Fi.CI.BAT: warning for drm/i915/lspcon: Work around resume failure Patchwork
2016-10-24 17:25 ` Imre Deak
2016-10-24 18:58 ` Jani Nikula
2016-10-24 19:05 ` Imre Deak [this message]
2016-10-25 5:47 ` Saarinen, Jani
2016-10-25 5:46 ` Saarinen, Jani
2016-10-25 14:46 ` ✗ Fi.CI.BAT: warning for drm/i915/lspcon: Work around resume failure (rev2) Patchwork
2016-10-26 15:53 ` Imre Deak
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1477335949.18150.18.camel@intel.com \
--to=imre.deak@intel.com \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jani.nikula@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).