Intel-XE Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/3] drm/xe/userptr: restore invalidation list on error
@ 2025-02-14 17:05 Matthew Auld
  2025-02-14 17:05 ` [PATCH v2 2/3] drm/xe/userptr: fix EFAULT handling Matthew Auld
                   ` (17 more replies)
  0 siblings, 18 replies; 28+ messages in thread
From: Matthew Auld @ 2025-02-14 17:05 UTC (permalink / raw)
  To: intel-xe; +Cc: Matthew Brost, Thomas Hellström, stable

On error restore anything still on the pin_list back to the invalidation
list on error. For the actual pin, so long as the vma is tracked on
either list it should get picked up on the next pin, however it looks
possible for the vma to get nuked but still be present on this per vm
pin_list leading to corruption. An alternative might be then to instead
just remove the link when destroying the vma.

Fixes: ed2bdf3b264d ("drm/xe/vm: Subclass userptr vmas")
Suggested-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: <stable@vger.kernel.org> # v6.8+
---
 drivers/gpu/drm/xe/xe_vm.c | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c
index d664f2e418b2..668b0bde7822 100644
--- a/drivers/gpu/drm/xe/xe_vm.c
+++ b/drivers/gpu/drm/xe/xe_vm.c
@@ -670,12 +670,12 @@ int xe_vm_userptr_pin(struct xe_vm *vm)
 	list_for_each_entry_safe(uvma, next, &vm->userptr.invalidated,
 				 userptr.invalidate_link) {
 		list_del_init(&uvma->userptr.invalidate_link);
-		list_move_tail(&uvma->userptr.repin_link,
-			       &vm->userptr.repin_list);
+		list_add_tail(&uvma->userptr.repin_link,
+			      &vm->userptr.repin_list);
 	}
 	spin_unlock(&vm->userptr.invalidated_lock);
 
-	/* Pin and move to temporary list */
+	/* Pin and move to bind list */
 	list_for_each_entry_safe(uvma, next, &vm->userptr.repin_list,
 				 userptr.repin_link) {
 		err = xe_vma_userptr_pin_pages(uvma);
@@ -691,10 +691,10 @@ int xe_vm_userptr_pin(struct xe_vm *vm)
 			err = xe_vm_invalidate_vma(&uvma->vma);
 			xe_vm_unlock(vm);
 			if (err)
-				return err;
+				break;
 		} else {
-			if (err < 0)
-				return err;
+			if (err)
+				break;
 
 			list_del_init(&uvma->userptr.repin_link);
 			list_move_tail(&uvma->vma.combined_links.rebind,
@@ -702,7 +702,19 @@ int xe_vm_userptr_pin(struct xe_vm *vm)
 		}
 	}
 
-	return 0;
+	if (err) {
+		down_write(&vm->userptr.notifier_lock);
+		spin_lock(&vm->userptr.invalidated_lock);
+		list_for_each_entry_safe(uvma, next, &vm->userptr.repin_list,
+					 userptr.repin_link) {
+			list_del_init(&uvma->userptr.repin_link);
+			list_move_tail(&uvma->userptr.invalidate_link,
+				       &vm->userptr.invalidated);
+		}
+		spin_unlock(&vm->userptr.invalidated_lock);
+		up_write(&vm->userptr.notifier_lock);
+	}
+	return err;
 }
 
 /**
-- 
2.48.1


^ permalink raw reply related	[flat|nested] 28+ messages in thread

end of thread, other threads:[~2025-02-21 13:23 UTC | newest]

Thread overview: 28+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-14 17:05 [PATCH v2 1/3] drm/xe/userptr: restore invalidation list on error Matthew Auld
2025-02-14 17:05 ` [PATCH v2 2/3] drm/xe/userptr: fix EFAULT handling Matthew Auld
2025-02-15  1:23   ` Matthew Brost
2025-02-17  9:19     ` Matthew Auld
2025-02-14 17:05 ` [PATCH v2 3/3] drm/xe/userptr: remove tmp_evict list Matthew Auld
2025-02-14 17:35 ` ✓ CI.Patch_applied: success for series starting with [v2,1/3] drm/xe/userptr: restore invalidation list on error Patchwork
2025-02-14 17:35 ` ✓ CI.checkpatch: " Patchwork
2025-02-14 17:36 ` ✓ CI.KUnit: " Patchwork
2025-02-14 17:53 ` ✓ CI.Build: " Patchwork
2025-02-14 17:55 ` ✓ CI.Hooks: " Patchwork
2025-02-14 17:56 ` ✓ CI.checksparse: " Patchwork
2025-02-15  1:28 ` [PATCH v2 1/3] " Matthew Brost
2025-02-17  9:38   ` Matthew Auld
2025-02-18  3:58     ` Matthew Brost
2025-02-20 23:52       ` Matthew Brost
2025-02-21 11:11         ` Matthew Auld
2025-02-21 11:20           ` Thomas Hellström
2025-02-21 13:17             ` Matthew Auld
2025-02-21 13:23               ` Thomas Hellström
2025-02-17  7:46 ` ✓ CI.Patch_applied: success for series starting with [v2,1/3] drm/xe/userptr: restore invalidation list on error (rev2) Patchwork
2025-02-17  7:47 ` ✓ CI.checkpatch: " Patchwork
2025-02-17  7:48 ` ✓ CI.KUnit: " Patchwork
2025-02-17  7:51 ` ✗ Xe.CI.Full: failure for series starting with [v2,1/3] drm/xe/userptr: restore invalidation list on error Patchwork
2025-02-17  8:04 ` ✓ CI.Build: success for series starting with [v2,1/3] drm/xe/userptr: restore invalidation list on error (rev2) Patchwork
2025-02-17  8:07 ` ✓ CI.Hooks: " Patchwork
2025-02-17  8:08 ` ✓ CI.checksparse: " Patchwork
2025-02-17  8:27 ` ✓ Xe.CI.BAT: " Patchwork
2025-02-17 10:47 ` ✗ Xe.CI.Full: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox