<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:mike@fireburn.co.uk" title="Mike Lothian <mike@fireburn.co.uk>"> <span class="fn">Mike Lothian</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [skl][bisected] GPU Hangs on nightly"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98282">bug 98282</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;">Summary</td>
           <td>[skl] GPU Hangs on nightly
           </td>
           <td>[skl][bisected] GPU Hangs on nightly
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [skl][bisected] GPU Hangs on nightly"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98282#c10">Comment # 10</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [skl][bisected] GPU Hangs on nightly"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=98282">bug 98282</a>
              from <span class="vcard"><a class="email" href="mailto:mike@fireburn.co.uk" title="Mike Lothian <mike@fireburn.co.uk>"> <span class="fn">Mike Lothian</span></a>
</span></b>
        <pre>I've bisected this as well as I can and got:

axion intel # git bisect good
d209b9c3cd281e4543e1150d173388b6d8f29a42 is the first bad commit
commit d209b9c3cd281e4543e1150d173388b6d8f29a42
Author: Michał Winiarski <<a href="mailto:michal.winiarski@intel.com">michal.winiarski@intel.com</a>>
Date:   Thu Oct 13 14:02:41 2016 +0200

    drm/i915/gtt: Split gen8_ppgtt_clear_pte_range

    Let's use more top-down approach, where each gen8_ppgtt_clear_* function
    is responsible for clearing the struct passed as an argument and calling
    relevant clear_range functions on lower-level tables.
    Doing this rather than operating on PTE ranges makes the implementation
    of shrinking page tables quite simple.

    v2: Drop min when calculating num_entries, no negation in 48b ppgtt
    check, no newlines in vars block (Joonas)

    Cc: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
    Cc: Joonas Lahtinen <<a href="mailto:joonas.lahtinen@linux.intel.com">joonas.lahtinen@linux.intel.com</a>>
    Cc: Michel Thierry <<a href="mailto:michel.thierry@intel.com">michel.thierry@intel.com</a>>
    Cc: Mika Kuoppala <<a href="mailto:mika.kuoppala@intel.com">mika.kuoppala@intel.com</a>>
    Signed-off-by: Michał Winiarski <<a href="mailto:michal.winiarski@intel.com">michal.winiarski@intel.com</a>>
    Signed-off-by: Chris Wilson <<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>>
    Link:
<a href="http://patchwork.freedesktop.org/patch/msgid/1476360162-24062-2-git-send-email-michal.winiarski@intel.com">http://patchwork.freedesktop.org/patch/msgid/1476360162-24062-2-git-send-email-michal.winiarski@intel.com</a>

:040000 040000 3d4b545bc817fdcb2b262dd33d3f95cc2f7612b2
4ef823c1499311cc8d4374ca20133e24ebbd323a M      drivers


As this bug doesn't always manifest itself I'm not 100% sure this is the right
one

axion intel # git bisect log
git bisect start 'drivers/gpu/drm/i915/'
# bad: [0fb1abf5eac2230894a7352e36022066f88a9b19] drm-intel-nightly:
2016y-10m-27d-14h-09m-00s UTC integration manifest
git bisect bad 0fb1abf5eac2230894a7352e36022066f88a9b19
# good: [07d9a380680d1c0eb51ef87ff2eab5c994949e69] Linux 4.9-rc2
git bisect good 07d9a380680d1c0eb51ef87ff2eab5c994949e69
# bad: [a62163e97bafbc072093ee5645873227f33a43ee] drm/i915/gen9: Make
skl_wm_level per-plane
git bisect bad a62163e97bafbc072093ee5645873227f33a43ee
# good: [cdb324bde5700725f04172bbeb6ef0bbbb6886c3] drm/i915: Show bounds of
active request in the ring on GPU hang
git bisect good cdb324bde5700725f04172bbeb6ef0bbbb6886c3
# good: [45353ce59b3ec606e0a35386ac04210b1656e829] drm/i915: Treat a
framebuffer reference as an active reference whilst shrinking
git bisect good 45353ce59b3ec606e0a35386ac04210b1656e829
# good: [fd6b8f43c9e9a3adc384423a1d3dfeefd38655ea] drm/i915: Make IS_IVYBRIDGE
only take dev_priv                                                              
git bisect good fd6b8f43c9e9a3adc384423a1d3dfeefd38655ea                        
# good: [11a914c28679f19d7daf4218c698ac6c3e184e1a] drm/i915: Make IS_VALLEYVIEW
only take dev_priv                                                              
git bisect good 11a914c28679f19d7daf4218c698ac6c3e184e1a                        
# bad: [d209b9c3cd281e4543e1150d173388b6d8f29a42] drm/i915/gtt: Split
gen8_ppgtt_clear_pte_range                                                      
git bisect bad d209b9c3cd281e4543e1150d173388b6d8f29a42                         
# good: [5db9401983ac7bf9ddc45de54c53ccfa31d21774] drm/i915: Make IS_GEN macros
only take dev_priv                                                              
git bisect good 5db9401983ac7bf9ddc45de54c53ccfa31d21774                        
# good: [4fb84d991ef2172d425234391d7215978345f6cd] drm/i915: Remove unused
"valid" parameter from pte_encode                                               
git bisect good 4fb84d991ef2172d425234391d7215978345f6cd                        
# first bad commit: [d209b9c3cd281e4543e1150d173388b6d8f29a42] drm/i915/gtt:
Split gen8_ppgtt_clear_pte_range 

I originally had d209b9c3cd281e4543e1150d173388b6d8f29a42 down as good but it
froze up when it was compiling the next kernel

The commit doesn't revert cleanly, so I've not tested without this commit</pre>
        </div>
      </p>


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

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