Re: [PATCH 3.16 012/114] drm/i915: Exit cherryview_irq_handler() after one pass

From: Ben Hutchings
Date: Tue Jun 14 2016 - 08:49:07 EST


On Tue, 2016-06-14 at 15:08 +0300, Ville SyrjÃlà wrote:
> On Tue, Jun 14, 2016 at 12:37:34PM +0100, Ben Hutchings wrote:
> > On Tue, 2016-06-14 at 13:47 +0300, Ville SyrjÃlà wrote:
> > > On Mon, Jun 13, 2016 at 07:36:37PM +0100, Ben Hutchings wrote:
> > > > 3.16.36-rc1 review patch.ÂÂIf anyone has any objections, please let me know.
> > >
> > > Do not backport this one. It'll break things.
> >
> > But this has not been re-reverted in mainline, has it? ÂIs it that
> > 3.16-stable would need more changes backported to make this work, or is
> > mainline currently broken on Cherryview hardware?
>
> No, as of 4.7 we have a proper fix, but it's a bit too big to backport
> (see [1]). I think 4.6.x is still busted, but Greg said he'd revert
> this broken patch there, so it should get fixed eventually.
>
> OTOH CHV wasn't even officially supported until maybe 4.1, so whatever
> you do in 3.16 shouldn't really matter.

OK, I've dropped this.

> It's a bit tedious having to block the same patch from different stable
> trees over and over again. It would be nice it there would be some kind
> of stable blacklist you guys could share so that we wouldn't have to
> repeat this dance with every stable maintainer...

Sorry about that; I do try to look for things like that on the stable
list but it's too high volume for me to keep up with most of the time.

Ben.

--
Ben Hutchings
We get into the habit of living before acquiring the habit of thinking.
ÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ- Albert
Camus

Attachment: signature.asc
Description: This is a digitally signed message part