From: Chris Wilson <chris@chris-wilson.co.uk>
To: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: ✗ Fi.CI.BAT: warning for drm/i915: Name the anonymous per-engine context struct (rev2)
Date: Tue, 22 Mar 2016 10:07:56 +0000 [thread overview]
Message-ID: <20160322100756.GT21717@nuc-i3427.alporthouse.com> (raw)
In-Reply-To: <56F11615.20301@linux.intel.com>
On Tue, Mar 22, 2016 at 09:53:25AM +0000, Tvrtko Ursulin wrote:
>
>
> On 21/03/16 11:22, Patchwork wrote:
> >== Series Details ==
> >
> >Series: drm/i915: Name the anonymous per-engine context struct (rev2)
> >URL : https://patchwork.freedesktop.org/series/4635/
> >State : warning
> >
> >== Summary ==
> >
> >Series 4635v2 drm/i915: Name the anonymous per-engine context struct
> >http://patchwork.freedesktop.org/api/1.0/series/4635/revisions/2/mbox/
> >
> >Test gem_ringfill:
> > Subgroup basic-default-s3:
> > dmesg-warn -> PASS (bsw-nuc-2)
> >Test kms_flip:
> > Subgroup basic-flip-vs-dpms:
> > pass -> DMESG-WARN (ilk-hp8440p) UNSTABLE
>
> Sporadic ILK pipe underruns:
> https://bugs.freedesktop.org/show_bug.cgi?id=93787
>
> > Subgroup basic-flip-vs-wf_vblank:
> > fail -> PASS (byt-nuc)
> > Subgroup basic-plain-flip:
> > pass -> DMESG-WARN (hsw-brixbox)
>
> Unrelated intermittent device suspended while HW access:
> https://bugs.freedesktop.org/show_bug.cgi?id=94349
>
> >Test kms_force_connector_basic:
> > Subgroup force-connector-state:
> > pass -> SKIP (ilk-hp8440p)
> >Test kms_pipe_crc_basic:
> > Subgroup nonblocking-crc-pipe-a:
> > dmesg-warn -> PASS (snb-x220t)
> > Subgroup nonblocking-crc-pipe-a-frame-sequence:
> > pass -> DMESG-WARN (snb-dellxps)
>
> Same.
>
> > Subgroup suspend-read-crc-pipe-c:
> > pass -> DMESG-WARN (bsw-nuc-2)
>
>
> Unrelated lockdep splat: https://bugs.freedesktop.org/show_bug.cgi?id=94350
Is it really the same one? There should be another lockdep chain that
isn't in bugzilla...
> > incomplete -> PASS (hsw-gt2)
> >Test pm_rpm:
> > Subgroup basic-pci-d3-state:
> > fail -> DMESG-FAIL (snb-x220t)
>
> Device suspended while HW access again.
>
> > dmesg-warn -> PASS (snb-dellxps)
> >
> >bdw-nuci7 total:194 pass:182 dwarn:0 dfail:0 fail:0 skip:12
> >bdw-ultra total:194 pass:173 dwarn:0 dfail:0 fail:0 skip:21
> >bsw-nuc-2 total:194 pass:156 dwarn:1 dfail:0 fail:0 skip:37
> >byt-nuc total:194 pass:159 dwarn:0 dfail:0 fail:0 skip:35
> >hsw-brixbox total:194 pass:171 dwarn:1 dfail:0 fail:0 skip:22
> >hsw-gt2 total:194 pass:176 dwarn:1 dfail:0 fail:0 skip:17
> >ilk-hp8440p total:194 pass:129 dwarn:1 dfail:0 fail:0 skip:64
> >ivb-t430s total:194 pass:169 dwarn:0 dfail:0 fail:0 skip:25
> >skl-i7k-2 total:194 pass:171 dwarn:0 dfail:0 fail:0 skip:23
> >snb-dellxps total:194 pass:159 dwarn:1 dfail:0 fail:0 skip:34
> >snb-x220t total:194 pass:160 dwarn:0 dfail:1 fail:0 skip:33
> >
> >Results at /archive/results/CI_IGT_test/Patchwork_1649/
> >
> >e7a7673e9840fe8b50a5a2894c75565ec7858a00 drm-intel-nightly: 2016y-03m-19d-10h-09m-53s UTC integration manifest
> >a1c5f9b1e8b9cbfdab0fb71ccf7a5a0838b56069 drm/i915: Name the anonymous per-engine context struct
>
> So looking good.
>
> Chris, r-b on v2? It was just a revert of a hunk which changed one
> instance of ctx->i915->dev->struct_mutex to
> engine->dev->struct_mutex which the CI reminded me is not allowed in
> some places.
That one again! One day we will get engine init/fini sorted. Yes,
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2016-03-22 10:08 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-18 15:00 [PATCH] drm/i915: Name the anonymous per-engine context struct Tvrtko Ursulin
2016-03-18 15:14 ` Chris Wilson
2016-03-18 17:26 ` [PATCH v2] " Tvrtko Ursulin
2016-03-21 15:23 ` Dave Gordon
2016-03-22 9:28 ` Chris Wilson
2016-03-22 11:20 ` Dave Gordon
2016-03-22 11:47 ` Tvrtko Ursulin
2016-03-22 11:48 ` [PATCH v3 1/2] drm/i915: name " Dave Gordon
2016-03-22 11:48 ` [PATCH v3 2/2] drm/i915: tidy up a few more references to engine[] Dave Gordon
2016-03-21 11:22 ` ✗ Fi.CI.BAT: warning for drm/i915: Name the anonymous per-engine context struct (rev2) Patchwork
2016-03-22 9:53 ` Tvrtko Ursulin
2016-03-22 10:07 ` Chris Wilson [this message]
2016-03-22 10:27 ` Tvrtko Ursulin
2016-03-22 10:32 ` Chris Wilson
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=20160322100756.GT21717@nuc-i3427.alporthouse.com \
--to=chris@chris-wilson.co.uk \
--cc=intel-gfx@lists.freedesktop.org \
--cc=tvrtko.ursulin@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.