<div dir="ltr">Hi guys,<div class="gmail_extra"><br><div class="gmail_quote">2016-06-06 11:32 GMT+02:00 Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Sat, Jun 04, 2016 at 02:06:58PM -0700, Greg KH wrote:<br>
> On Fri, May 27, 2016 at 11:30:30AM +0300, <a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a> wrote:<br>
> > From: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com">ville.syrjala@linux.intel.com</a>><br>
> ><br>
> > Several nasty i915 regressions affecting CHV slipped through<br>
> > to 4.5 and 4.6.<br>
> ><br>
> > The first fix we want in 4.5 and 4.6 is<br>
> > commit caed361d83b2 ("drm/i915: Fix watermarks for VLV/CHV")<br>
> > It won't cherry-pick cleanly to either one, so I've included conflict<br>
> > free versions for both. This one fixes display FIFO underruns that can<br>
> > lead to the screen totally blanking out.<br>
><br>
> Now applied, thanks.<br>
><br>
> > The other one I'd like to have in 4.6 is<br>
> > commit 9f6151c90390 ("drm/i915: Pass the correct crtc state to .update_plane()")<br>
> > which avoids a totally corrupted display in some cases.<br>
><br>
> Now applied.<br>
><br>
> > And the third on is a bit more annoying. The regression is caused by<br>
> > commit 9dbaab56ac09 ("drm/i915: Exit cherryview_irq_handler() after one pass")<br>
> > which I though we had prevented from getting out on its own, but turns<br>
> > out I was wrong. It basically makes the GPU unusable, so we do need to<br>
> > fix it somehow. The simple solution would be to revert it in 4.6 only.<br>
> > The more complicated solution is to backport the proper fix, which more<br>
> > or less requires the following set of commits [1], which is maybe a bit<br>
> > too much for stable. I could try to trim it a bit perhaps, but then we<br>
> > start to enter the territory of untested code which I don't particularly<br>
> > like. Let me know what you think.<br>
> ><br>
> > [1]<br>
> >  1e1cace942ef ("drm/i915: Eliminate loop from VLV irq handler")<br>
> >  a5e485a95c9c ("drm/i915: Clear VLV_IER around irq processing")<br>
> >  4a0a0202b023 ("drm/i915: Clear VLV_MASTER_IER around irq processing")<br>
> >  7ce4d1f2730f ("drm/i915: Clear VLV_IIR after PIPESTAT")<br>
> >  34c7b8a7b8b5 ("drm/i915: Set up VLV_MASTER_IER consistently")<br>
> >  e5328c43d46e ("drm/i915: Use GEN8_MASTER_IRQ_CONTROL consistently")<br>
> >  71b8b41d5b35 ("drm/i915: Move DPINVGTT setup to vlv_display_irq_reset()")<br>
> >  6b7eafc1b43d ("drm/i915: Warn if irq_mask isn't ~0 during vlv/cvh display irq postinstall")<br>
> >  9ab981f22bef ("drm/i915: Use GEN5_IRQ_INIT() in vlv_display_irq_postinstall()")<br>
> >  d6c698035892 ("drm/i915: Clear display interrupt before enabling when turning on the power well")<br>
> >  8bb613068a63 ("drm/i915: Move vlv/chv display irq code to a more logical place")<br>
> >  9918271efc7a ("drm/i915: Skip display irq setup if display irqs aren't flagged as enabled")<br>
> >  ad22d10654ea ("drm/i915: Fix up vlv/chv display irq setup")<br>
> >  93de68f94081 ("drm/i915: Remove "VLV magic" from irq setup")<br>
><br>
> I think reverting that one patch for 4.6 makes more sense than adding<br>
> all of these patches.  I'll do that if you want me to.<br>
<br>
</div></div>That works for me. Thanks.<br>
<span class="im HOEnZb"><br>
--<br>
Ville Syrjälä<br>
Intel OTC</span></blockquote><div><br></div><div>Sorry for bothering you. Will the revert make it into 4.6.3? Without it BSW on 4.6 is not really usable.</div><div><br></div><div>Thanks much in advance</div><div>Peter </div></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">                   Key-ID:     0x1A995A9B<br>                   keyserver: <a href="http://pgp.mit.edu" target="_blank">pgp.mit.edu</a><br>==============================================================<br>Fingerprint: 4606 DA19 EC2E 9A0B 0157  C81B DA07 CF63 1A99 5A9B</div>
</div></div>