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 <paulo.r.zanoni@intel.com>
Reviewed-by: Niranjana Vishwanathapura <niranjana.vishwanathapura@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

diff --git a/drivers/gpu/drm/xe/xe_vm.c b/drivers/gpu/drm/xe/xe_vm.c
index ae5578a3e1219761d7bc018d90d5028b89317849..b61ed51b503da2e96e7ad2e2bbe5efda65ff5d23 100644 (file)
--- 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);