Re: [PATCH v5 06/13] iommu: Add iommu_copy_struct_from_full_user_array helper

2024-10-29 Thread Nicolin Chen
On Tue, Oct 29, 2024 at 08:24:52AM +, Tian, Kevin wrote:
> > From: Nicolin Chen 
> > Sent: Saturday, October 26, 2024 7:51 AM
> >
> > From: Jason Gunthorpe 
> >
> > The iommu_copy_struct_from_user_array helper can be used to copy a
> > single
> > entry from a user array which might not be efficient if the array is big.
> >
> > Add a new iommu_copy_struct_from_full_user_array to copy the entire user
> > array at once. Update the existing iommu_copy_struct_from_user_array
> > kdoc
> > accordingly.
> 
> what about:
> 
> iommu_copy_struct_from_user_array_single()
> iommu_copy_struct_from_user_array_full()

I am okay with that, yet might prefer that in a separate patch? As
I am trying to reduce the number of changes in the series since we
are likely merging three series at once :)

Thanks
Nicolin

> > Signed-off-by: Jason Gunthorpe 
> > Signed-off-by: Nicolin Chen 
> 
> Reviewed-by: Kevin Tian 



RE: [PATCH v5 06/13] iommu: Add iommu_copy_struct_from_full_user_array helper

2024-10-29 Thread Tian, Kevin
> From: Nicolin Chen 
> Sent: Saturday, October 26, 2024 7:51 AM
> 
> From: Jason Gunthorpe 
> 
> The iommu_copy_struct_from_user_array helper can be used to copy a
> single
> entry from a user array which might not be efficient if the array is big.
> 
> Add a new iommu_copy_struct_from_full_user_array to copy the entire user
> array at once. Update the existing iommu_copy_struct_from_user_array
> kdoc
> accordingly.

what about:

iommu_copy_struct_from_user_array_single()
iommu_copy_struct_from_user_array_full()

?

> 
> Signed-off-by: Jason Gunthorpe 
> Signed-off-by: Nicolin Chen 

Reviewed-by: Kevin Tian