From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <ville.syrjala@linux.intel.com>
Subject: Re: drm/i915 4.5/4.6 stable backport request for CHV
Date: Mon, 6 Jun 2016 12:32:16 +0300
Message-ID: <20160606093216.GD4329@intel.com>
References: <1464337832-15948-1-git-send-email-ville.syrjala@linux.intel.com>
 <20160604210658.GA4068@kroah.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: QUOTED-PRINTABLE
Return-path: <stable-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <20160604210658.GA4068@kroah.com>
Sender: stable-owner@vger.kernel.org
To: Greg KH <greg@kroah.com>
Cc: intel-gfx@lists.freedesktop.org, stable@vger.kernel.org
List-Id: intel-gfx@lists.freedesktop.org

On Sat, Jun 04, 2016 at 02:06:58PM -0700, Greg KH wrote:
> On Fri, May 27, 2016 at 11:30:30AM +0300, ville.syrjala@linux.intel.c=
om wrote:
> > From: Ville Syrj=E4l=E4 <ville.syrjala@linux.intel.com>
> >=20
> > Several nasty i915 regressions affecting CHV slipped through
> > to 4.5 and 4.6.
> >=20
> > The first fix we want in 4.5 and 4.6 is
> > commit caed361d83b2 ("drm/i915: Fix watermarks for VLV/CHV")
> > It won't cherry-pick cleanly to either one, so I've included confli=
ct
> > free versions for both. This one fixes display FIFO underruns that =
can
> > lead to the screen totally blanking out.
>=20
> Now applied, thanks.
>=20
> > The other one I'd like to have in 4.6 is
> > commit 9f6151c90390 ("drm/i915: Pass the correct crtc state to .upd=
ate_plane()")
> > which avoids a totally corrupted display in some cases.
>=20
> Now applied.
>=20
> > And the third on is a bit more annoying. The regression is caused b=
y
> > commit 9dbaab56ac09 ("drm/i915: Exit cherryview_irq_handler() after=
 one pass")
> > which I though we had prevented from getting out on its own, but tu=
rns
> > out I was wrong. It basically makes the GPU unusable, so we do need=
 to
> > fix it somehow. The simple solution would be to revert it in 4.6 on=
ly.
> > The more complicated solution is to backport the proper fix, which =
more
> > or less requires the following set of commits [1], which is maybe a=
 bit
> > too much for stable. I could try to trim it a bit perhaps, but then=
 we
> > start to enter the territory of untested code which I don't particu=
larly
> > like. Let me know what you think.
> >=20
> > [1]
> >  1e1cace942ef ("drm/i915: Eliminate loop from VLV irq handler")
> >  a5e485a95c9c ("drm/i915: Clear VLV_IER around irq processing")
> >  4a0a0202b023 ("drm/i915: Clear VLV_MASTER_IER around irq processin=
g")
> >  7ce4d1f2730f ("drm/i915: Clear VLV_IIR after PIPESTAT")
> >  34c7b8a7b8b5 ("drm/i915: Set up VLV_MASTER_IER consistently")
> >  e5328c43d46e ("drm/i915: Use GEN8_MASTER_IRQ_CONTROL consistently"=
)
> >  71b8b41d5b35 ("drm/i915: Move DPINVGTT setup to vlv_display_irq_re=
set()")
> >  6b7eafc1b43d ("drm/i915: Warn if irq_mask isn't ~0 during vlv/cvh =
display irq postinstall")
> >  9ab981f22bef ("drm/i915: Use GEN5_IRQ_INIT() in vlv_display_irq_po=
stinstall()")
> >  d6c698035892 ("drm/i915: Clear display interrupt before enabling w=
hen turning on the power well")
> >  8bb613068a63 ("drm/i915: Move vlv/chv display irq code to a more l=
ogical place")
> >  9918271efc7a ("drm/i915: Skip display irq setup if display irqs ar=
en't flagged as enabled")
> >  ad22d10654ea ("drm/i915: Fix up vlv/chv display irq setup")
> >  93de68f94081 ("drm/i915: Remove "VLV magic" from irq setup")
>=20
> I think reverting that one patch for 4.6 makes more sense than adding
> all of these patches.  I'll do that if you want me to.

That works for me. Thanks.

--=20
Ville Syrj=E4l=E4
Intel OTC