<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:yann.argotti@intel.com" title="yann <yann.argotti@intel.com>"> <span class="fn">yann</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [4.6 regression] FBC doesn't notice writes to scanout"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=96591">bug 96591</a>
          <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Status</td>
           <td>NEEDINFO
           </td>
           <td>RESOLVED
           </td>
         </tr>

         <tr>
           <td style="text-align:right;">Resolution</td>
           <td>---
           </td>
           <td>FIXED
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [4.6 regression] FBC doesn't notice writes to scanout"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=96591#c6">Comment # 6</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - [4.6 regression] FBC doesn't notice writes to scanout"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=96591">bug 96591</a>
              from <span class="vcard"><a class="email" href="mailto:yann.argotti@intel.com" title="yann <yann.argotti@intel.com>"> <span class="fn">yann</span></a>
</span></b>
        <pre>Thanks Vampl, 

you are right fbc default is off but on BDW this is enabled by default - see
following commit 80788a0fbbdfbb125e3fd45a640cddb582160bc7 
2016-06-20 drm/i915/fbc: sanitize i915.enable_fbc during FBC init    Paulo 

The DDX driver changes its behavior depending on the value it reads
from i915.enable_fbc, so sanitize the value in order to allow it to
know what's going on. It uses this in order to choose the defaults for
the TearFree option. Before this patch, it would read -1 and always
assume that FBC was disabled, so it wouldn't force TearFree.

v2: Extract intel_sanitize_fbc_option() (Chris).
v3: Rebase.

Signed-off-by: Paulo Zanoni <<a href="mailto:paulo.r.zanoni@intel.com">paulo.r.zanoni@intel.com</a>>
Reviewed-by: Daniel Vetter <<a href="mailto:daniel.vetter@ffwll.ch">daniel.vetter@ffwll.ch</a>>
Link:
<a href="http://patchwork.freedesktop.org/patch/msgid/1460574069-14005-1-git-send-email-paulo.r.zanoni@intel.com">http://patchwork.freedesktop.org/patch/msgid/1460574069-14005-1-git-send-email-paulo.r.zanoni@intel.com</a></pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the QA Contact for the bug.</li>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>