diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-04 19:27:15 +0300 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-04 19:34:40 +0300 |
commit | af3302b90775ca3389c93ab31458d696e8a8fa60 (patch) | |
tree | a0c007511b8311a05da3799b96cd4cc9e9a9b2ad /include/drm/i915_pciids.h | |
parent | 61ad992875639b0ac5866321e2edcba573f5f3a2 (diff) | |
download | linux-af3302b90775ca3389c93ab31458d696e8a8fa60.tar.xz |
Revert "drm/i915: Extend LRC pinning to cover GPU context writeback"
This reverts commit 6d65ba943a2d1e4292a07ca7ddb6c5138b9efa5d.
Mika Kuoppala traced down a use-after-free crash in module unload to
this commit, because ring->last_context is leaked beyond when the
context gets destroyed. Mika submitted a quick fix to patch that up in
the context destruction code, but that's too much of a hack.
The right fix is instead for the ring to hold a full reference onto
it's last context, like we do for legacy contexts.
Since this is causing a regression in BAT it gets reverted before we
can close this.
Cc: Nick Hoath <nicholas.hoath@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: David Gordon <david.s.gordon@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Alex Dai <yu.dai@intel.com>
Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93248
Acked-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'include/drm/i915_pciids.h')
0 files changed, 0 insertions, 0 deletions