summaryrefslogtreecommitdiff
path: root/drivers/gpu/drm/i915/i915_gem_timeline.c
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2016-11-14 23:40:56 +0300
committerChris Wilson <chris@chris-wilson.co.uk>2016-11-15 00:00:20 +0300
commit556b748710586afa24a696ba81fe89dc7f41cebd (patch)
treee82704c7986a2206fa66d0386fd6b54ec45f855c /drivers/gpu/drm/i915/i915_gem_timeline.c
parente3c566dfe4df07e007c9795356c63d5635664e5e (diff)
downloadlinux-556b748710586afa24a696ba81fe89dc7f41cebd.tar.xz
drm/i915: Give each sw_fence its own lockclass
Localise the static struct lock_class_key to the caller of i915_sw_fence_init() so that we create a lock_class instance for each unique sw_fence rather than all sw_fences sharing the same lock_class. This eliminate some lockdep false positive when using fences from within fence callbacks. For the relatively small number of fences currently in use [2], this adds 160 bytes of unused text/code when lockdep is disabled. This seems quite high, but fully reducing it via ifdeffery is also quite ugly. Removing the #fence strings saves 72 bytes with just a single #ifdef. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20161114204105.29171-1-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_timeline.c')
0 files changed, 0 insertions, 0 deletions