diff options
| author | Paulo Zanoni <[email protected]> | 2023-09-29 10:31:03 -0700 |
|---|---|---|
| committer | Rodrigo Vivi <[email protected]> | 2023-12-21 11:42:09 -0500 |
| commit | 66aca8f04bb982b9f429fbce384beaa4badae21a (patch) | |
| tree | 7bde1ff6c1a622312232097c2b7eb7bf091c9953 | |
| parent | 5f01a35b10f3d2f55634a471c43e59e3c6f239fd (diff) | |
drm/xe/vm: use list_last_entry() to fetch last_op
I would imagine that it's more efficient to fetch ops_list->prev than
to walk the whole list forward.
Signed-off-by: Paulo Zanoni <[email protected]>
Reviewed-by: Niranjana Vishwanathapura <[email protected]>
Signed-off-by: Rodrigo Vivi <[email protected]>
| -rw-r--r-- | drivers/gpu/drm/xe/xe_vm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c index ae5578a3e121..b61ed51b503d 100644 --- a/drivers/gpu/drm/xe/xe_vm.c +++ b/drivers/gpu/drm/xe/xe_vm.c @@ -3118,8 +3118,7 @@ static int vm_bind_ioctl_ops_execute(struct xe_vm *vm, lockdep_assert_held_write(&vm->lock); - list_for_each_entry(op, ops_list, link) - last_op = op; + last_op = list_last_entry(ops_list, struct xe_vma_op, link); if (!async) { err = xe_vma_op_execute(vm, last_op); |