Quoting Abdiel Janulgue (2019-10-07 10:19:16) > @@ -312,6 +313,12 @@ vm_fault_t i915_gem_fault(struct vm_fault *vmf) > list_add(&obj->userfault_link, &i915->ggtt.userfault_list); > mutex_unlock(&i915->ggtt.vm.mutex); > > + /* Track the mmo associated with the fenced vma */ > + mutex_lock(&priv->vma_lock); > + vma->mmo = priv; > + list_move(&vma->mmo_link, &priv->vma_list); > + mutex_unlock(&priv->vma_lock); Why are we tracking the same thing twice? -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
