<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 19, 2014 at 8:40 AM, Daniel Vetter <span dir="ltr"><<a href="mailto:daniel@ffwll.ch" target="_blank">daniel@ffwll.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Sep 18, 2014 at 11:42:44AM -0700, Rodrigo Vivi wrote:<br>
> On Thu, Sep 18, 2014 at 4:39 AM, Daniel Vetter <<a href="mailto:daniel@ffwll.ch">daniel@ffwll.ch</a>> wrote:<br>
><br>
> > On Thu, Sep 18, 2014 at 07:28:48AM +0100, Chris Wilson wrote:<br>
> > > On Wed, Sep 17, 2014 at 04:59:20PM -0400, Rodrigo Vivi wrote:<br>
> > > > If it wasn't never enabled by kernel parameter or platform default<br>
> > > > we can avoid reading registers so many times in vain<br>
> > ><br>
> > > Nak.<br>
> ><br>
> > Well I've merged this for now to reduce fbc impact.<br>
> ><br>
><br>
> Uhm, unfortunatelly I'm afraid Chris was right.<br>
> Paulo also nacked it. Because it just helps when it was explicitly disabled<br>
> by setting i915.enable_fbc=0 while the default is -1.<br>
><br>
> I though about returning on <= 0, but Paulo is afraid that when enabling<br>
> back for some platform people would forget to fix this part here and I<br>
> agree.<br>
<br>
</span>Well I guess I should have read mails before pushing out a new -next ;-)<br>
So this is now baked in. Should I revert or can we just fix up on top?<br></blockquote><div><br></div><div>No problem, I can fix this on top.</div><div><br></div><div>But what do you prefer:</div><div><br></div><div>1. <=0 return and changing parameter permission from 600 to 400</div><div>2. dev_priv->fbc_enabled </div><div>3. or check if fbc_no_reason was ever set? Although I don't like the fbc_no_reason and would like to clean it up in the future. But anyway, any option here is temporary until a proper rework for cleanup and real fix.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">-Daniel<br>
--<br>
Daniel Vetter<br>
Software Engineer, Intel Corporation<br>
<a href="tel:%2B41%20%280%29%2079%20365%2057%2048" value="+41793655748">+41 (0) 79 365 57 48</a> - <a href="http://blog.ffwll.ch" target="_blank">http://blog.ffwll.ch</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div></div>