Re: [Intel-xe] [PATCH 2/3] drm/gpuva: Add drm_gpuva_for_each_op_reverse

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Aug 16, 2023 at 09:31:47PM -0700, Matthew Brost wrote:
> Xe will make use of this, add helper to walk op list in reverse.

I believe this deserves a better commit message explaining why this
is useful and not only that we are going to use it.

> 
> Signed-off-by: Matthew Brost <matthew.brost@intel.com>
> ---
>  include/drm/drm_gpuva_mgr.h | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/include/drm/drm_gpuva_mgr.h b/include/drm/drm_gpuva_mgr.h
> index c6678b1dc5d2..455eef90db21 100644
> --- a/include/drm/drm_gpuva_mgr.h
> +++ b/include/drm/drm_gpuva_mgr.h
> @@ -616,6 +616,16 @@ struct drm_gpuva_ops {
>  #define drm_gpuva_for_each_op_from_reverse(op, ops) \
>  	list_for_each_entry_from_reverse(op, &(ops)->list, entry)
>  
> +/**
> + * drm_gpuva_for_each_op_reverse - iterator to walk over &drm_gpuva_ops in reverse
> + * @op: &drm_gpuva_op to assign in each iteration step
> + * @ops: &drm_gpuva_ops to walk
> + *
> + * This iterator walks over all ops within a given list of operations in reverse
> + */
> +#define drm_gpuva_for_each_op_reverse(op, ops) \
> +	list_for_each_entry_reverse(op, &(ops)->list, entry)
> +
>  /**
>   * drm_gpuva_first_op - returns the first &drm_gpuva_op from &drm_gpuva_ops
>   * @ops: the &drm_gpuva_ops to get the fist &drm_gpuva_op from
> -- 
> 2.34.1
>