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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.