Comment 29 for bug 597075

Revision history for this message
Kees Cook (kees) wrote :

So, this didn't work either, so I went back and did an unrestricted bisect across the entire rc1 to rc2 span. It resulted in:

kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect start v2.6.35-rc2 v2.6.35-rc1
Bisecting: 373 revisions left to test after this (roughly 9 steps)
[b1413357d924792e2e332dcb6b712a7fb2a5fb25] fbdev: fix frame buffer devices menu
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect bad
Bisecting: 179 revisions left to test after this (roughly 8 steps)
[aef4b9aaae1decc775778903922bd0075cce7a88] Merge branch 'next' of git://git.kernel.org/pub/scm/linux/kernel/git/benh/powerpc
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect bad
Bisecting: 111 revisions left to test after this (roughly 7 steps)
[64ffc9ff424c65adcffe7d590018cc75e2d5d42a] kbuild: Revert part of e8d400a to res
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect good
Bisecting: 63 revisions left to test after this (roughly 6 steps)
[08a66859e69264f3223560d06b88e80c1a6a6387] FS-Cache: Remove unneeded null checks
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect good
Bisecting: 31 revisions left to test after this (roughly 5 steps)
[85cd4612fdab4e837d7eea048a697c75d0477d3b] drm/i915: Check error code whilst moving buffer to GTT domain.
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect bad
Bisecting: 15 revisions left to test after this (roughly 4 steps)
[9962c9252e46eda7058067cbe73bdf1ed74b0d37] drm/i915/dp: Only enable enhanced framing if the sink supports it
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect bad
Bisecting: 7 revisions left to test after this (roughly 3 steps)
[f41275e893191eeb7a88e431d594e167adbd5234] drm/i915: Convert more trace events to DEFINE_EVENT
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect good
Bisecting: 3 revisions left to test after this (roughly 2 steps)
[7648fa99eb77a2e1a90b7beaa420e07d819b9c11] drm/i915: add power monitoring support
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect good
Bisecting: 1 revision left to test after this (roughly 1 step)
[f1befe71fa7a79ab733011b045639d8d809924ad] agp/intel: Restrict GTT mapping to valid range on i915 and i945
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect bad
Bisecting: 0 revisions left to test after this (roughly 0 steps)
[9a7e8492d17394a81d5534abf90b5b2ada7ea3c0] drm/i915: Storage class should be before const qualifier
kees@tyler:~/lp597075/bisect/linux-2.6$ git bisect good
f1befe71fa7a79ab733011b045639d8d809924ad is the first bad commit
commit f1befe71fa7a79ab733011b045639d8d809924ad
Author: Chris Wilson <email address hidden>
Date: Tue May 18 12:24:51 2010 +0100

    agp/intel: Restrict GTT mapping to valid range on i915 and i945

    References:

      Bug 15733 - Crash when accessing nonexistent GTT entries in i915
      https://bugzilla.kernel.org/show_bug.cgi?id=15733

    On G33 and above, the size of the GTT space is determined by the GMCH
    control register. Prior to this revision, the size is determined by the
    size of the aperture. So we must careful to map and fill the appropriate
    range depending on chipset.

    Signed-off-by: Chris Wilson <email address hidden>
    Signed-off-by: Eric Anholt <email address hidden>

:040000 040000 a640ccd942bab2077babf162ec79cfa9dfb166c3 8472a2b84c676e3c714e8c1f8392255b1959ae83 M drivers

Building -6 maverick kernel with this reverted to test now.