drm/i915: Track rcu_head for our idle worker
authorChris Wilson <chris@chris-wilson.co.uk>
Thu, 8 Nov 2018 09:21:01 +0000 (09:21 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 9 Nov 2018 12:02:09 +0000 (12:02 +0000)
While our little rcu worker might be able to be replaced now by the
dedicated rcu_work, in the meantime we should mark up the rcu_head for
correct debugobjects tracking.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20181108092101.27598-2-chris@chris-wilson.co.uk
drivers/gpu/drm/i915/i915_gem.c

index 5f69b9aadae8fb7bdffaf56ca988dd3b938a5e68..7d9457915704d76dae3b8055b185d0c3321dbc00 100644 (file)
@@ -3557,6 +3557,8 @@ static void __sleep_rcu(struct rcu_head *rcu)
        struct sleep_rcu_work *s = container_of(rcu, typeof(*s), rcu);
        struct drm_i915_private *i915 = s->i915;
 
+       destroy_rcu_head(&s->rcu);
+
        if (same_epoch(i915, s->epoch)) {
                INIT_WORK(&s->work, __sleep_work);
                queue_work(i915->wq, &s->work);
@@ -3673,6 +3675,7 @@ out_rearm:
        if (same_epoch(dev_priv, epoch)) {
                struct sleep_rcu_work *s = kmalloc(sizeof(*s), GFP_KERNEL);
                if (s) {
+                       init_rcu_head(&s->rcu);
                        s->i915 = dev_priv;
                        s->epoch = epoch;
                        call_rcu(&s->rcu, __sleep_rcu);