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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to