← Back to team overview

kernel-packages team mailing list archive

[Bug 1319656] [NEW] [i915_bdw] Add a patch to fix crashes

 

Public bug reported:

We need a new patch from bdw-backports branch to fix a number of
crashers in the i915_bdw driver:

commit 17ef4e5850c6fc48a3af2ca89aea19fec7298ace
Author: Ben Widawsky <benjamin.widawsky@xxxxxxxxx>
Date:   Fri May 9 15:09:26 2014 -0700

    drm/i915/bdw: Use MMIO for loading PDPs
    
    This is a partial-functional backport of:
    commit eeb9488e751a0a6401e7516a893efaf9d1f77fb5
    Author: Ben Widawsky <ben@xxxxxxxxxxxx>
    Date:   Fri Dec 6 14:11:10 2013 -0800
    
        drm/i915: Extract mm switching to function
    
    In that commit, the new mm switching function introduced the ability to
    either load the page tables via the ring, or via MMIO. The issue we have
    is during reset, we do not have the ability to use the rings. Trying to
    do so will fail, and prevent our Aliasing PPGTT from coming back. In
    current upstream, we use the MMIO on reset, and LRI for everything else.
    
    Here however, we always use the MMIO (in spite of what docs say, I've
    used this for a long time, and it's never *not* worked.). During
    development, we were using MMIO, and it was changed toward the end on
    code review. We do not have enough of the other code in place to be as
    smart as we are for current upstream.
    
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77587
    Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx>

** Affects: linux (Ubuntu)
     Importance: High
         Status: Fix Released

** Affects: linux (Ubuntu Trusty)
     Importance: High
     Assignee: Timo Aaltonen (tjaalton)
         Status: In Progress

** Changed in: linux (Ubuntu)
   Importance: Undecided => High

** Changed in: linux (Ubuntu)
       Status: New => In Progress

** Changed in: linux (Ubuntu)
     Assignee: (unassigned) => Timo Aaltonen (tjaalton)

** Also affects: linux (Ubuntu Trusty)
   Importance: Undecided
       Status: New

** Changed in: linux (Ubuntu Trusty)
   Importance: Undecided => High

** Changed in: linux (Ubuntu Trusty)
       Status: New => In Progress

** Changed in: linux (Ubuntu Trusty)
     Assignee: (unassigned) => Timo Aaltonen (tjaalton)

-- 
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1319656

Title:
  [i915_bdw] Add a patch to fix crashes

Status in “linux” package in Ubuntu:
  Fix Released
Status in “linux” source package in Trusty:
  In Progress

Bug description:
  We need a new patch from bdw-backports branch to fix a number of
  crashers in the i915_bdw driver:

  commit 17ef4e5850c6fc48a3af2ca89aea19fec7298ace
  Author: Ben Widawsky <benjamin.widawsky@xxxxxxxxx>
  Date:   Fri May 9 15:09:26 2014 -0700

      drm/i915/bdw: Use MMIO for loading PDPs
      
      This is a partial-functional backport of:
      commit eeb9488e751a0a6401e7516a893efaf9d1f77fb5
      Author: Ben Widawsky <ben@xxxxxxxxxxxx>
      Date:   Fri Dec 6 14:11:10 2013 -0800
      
          drm/i915: Extract mm switching to function
      
      In that commit, the new mm switching function introduced the ability to
      either load the page tables via the ring, or via MMIO. The issue we have
      is during reset, we do not have the ability to use the rings. Trying to
      do so will fail, and prevent our Aliasing PPGTT from coming back. In
      current upstream, we use the MMIO on reset, and LRI for everything else.
      
      Here however, we always use the MMIO (in spite of what docs say, I've
      used this for a long time, and it's never *not* worked.). During
      development, we were using MMIO, and it was changed toward the end on
      code review. We do not have enough of the other code in place to be as
      smart as we are for current upstream.
      
      Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77587
      Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx>

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1319656/+subscriptions


Follow ups

References