[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/bxt: Fix off-by-one error in Broxton PLL IDs (rev3)

[Takashi Iwai]

On Fri, 18 Mar 2016 09:12:53 +0100,
Tomi Sarvela wrote:
> 
> On Friday 18 March 2016 09:00:13 Takashi Iwai wrote:
> > On Fri, 18 Mar 2016 08:36:27 +0100,
> > 
> > Do you have the commit of the last working kernel?
> > 
> > > The unit tests that hang quite reliably on first run are
> > > igt at gem_ringfill_basic-default-s3
> > > igt at drv_module_reload_basic
> > > igt at kms_pipe_crc_basic@suspend-read-crc-pipe-[abc]
> > 
> > Where can I get these?
> 
> IGT is intel-gpu-tools, hardware unittests using piglit as a base. CI runs BAT 
> (Basic Acceptance Tests), a set of IGT tests: around 200 tests, runs under 10 
> minutes. Can be run with ${IGT_PATH}/scripts/run-tests.sh -t basic
> 
> git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> 
> http://anongit.freedesktop.org/git/piglit.git
> 
> Latest working drm-intel commit was CI_DRM_1139:
> 3e5ecc8c5ff80cb1fb635ce1cf16b7cd4cfb1979 2016-03-14_09-06-45 drm-intel-nightly: 
> 2016y-03m-14d-09h-06m-00s UTC integration manifest

There were a few different kernel warnings, so this date is possibly a
red herring.  One is the kernel warning from pin2port(), and this has
nothing to do with Skylake.  It's being addressed now.

The problem I can't follow is the PM refcount unbalance found on
Skylake.  It'd be helpful if you can give the exact *minimal*
reproducer.  The warning shows the result, and it means that something
wrong did happen before that.  So, we need the procedure until that
point, not the point of the outcome.


thanks,

Takashi