Skip to content

Commit 84603ed

Browse files
Thomas Hellströmgregkh
authored andcommitted
drm/xe/vm: Clear the scratch_pt pointer on error
commit 2b55ddf upstream. Avoid triggering a dereference of an error pointer on cleanup in xe_vm_free_scratch() by clearing any scratch_pt error pointer. Signed-off-by: Thomas Hellström <thomas.hellstrom@linux.intel.com> Fixes: 06951c2 ("drm/xe: Use NULL PTEs as scratch PTEs") Cc: Brian Welty <brian.welty@intel.com> Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Lucas De Marchi <lucas.demarchi@intel.com> Cc: <stable@vger.kernel.org> # v6.8+ Reviewed-by: Matthew Brost <matthew.brost@intel.com> Link: https://lore.kernel.org/r/20250821143045.106005-4-thomas.hellstrom@linux.intel.com (cherry picked from commit 358ee50) Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 39fc274 commit 84603ed

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

drivers/gpu/drm/xe/xe_vm.c

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1582,8 +1582,12 @@ static int xe_vm_create_scratch(struct xe_device *xe, struct xe_tile *tile,
15821582

15831583
for (i = MAX_HUGEPTE_LEVEL; i < vm->pt_root[id]->level; i++) {
15841584
vm->scratch_pt[id][i] = xe_pt_create(vm, tile, i);
1585-
if (IS_ERR(vm->scratch_pt[id][i]))
1586-
return PTR_ERR(vm->scratch_pt[id][i]);
1585+
if (IS_ERR(vm->scratch_pt[id][i])) {
1586+
int err = PTR_ERR(vm->scratch_pt[id][i]);
1587+
1588+
vm->scratch_pt[id][i] = NULL;
1589+
return err;
1590+
}
15871591

15881592
xe_pt_populate_empty(tile, vm, vm->scratch_pt[id][i]);
15891593
}

0 commit comments

Comments
 (0)