drm/i915: Move user fault tracking to a separate list

Browse source

We want to decouple RPM and struct_mutex, but currently RPM has to walk
the list of bound objects and remove userspace mmapping before we
suspend (otherwise userspace may continue to access the GTT whilst it is
powered down). This currently requires the struct_mutex to walk the
bound_list, but if we move that to a separate list and lock we can take
the first step towards removing the struct_mutex.

v2: Split runtime suspend unmapping vs regular unmapping, to make the
locking (and barriers) clearer. Add the object to the userfault_list
prior to inserting the first PTE, the race between add/revoke depends
upon struct_mutex for regular unmappings and rpm for runtime-suspend.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> #v1
Link: http://patchwork.freedesktop.org/patch/msgid/20161024124218.18252-1-chris@chris-wilson.co.uk

...

This commit is contained in:

Chris Wilson 2016-10-24 13:42:14 +01:00

parent 352cb4ef2d

commit 275f039db5

5 changed files with 49 additions and 19 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

2

drivers/gpu/drm/i915/i915_debugfs.c

View file

20

drivers/gpu/drm/i915/i915_drv.h

View file

42

drivers/gpu/drm/i915/i915_gem.c

View file

2

drivers/gpu/drm/i915/i915_gem_evict.c

View file

2

drivers/gpu/drm/i915/i915_gem_fence.c

View file