Re: [PATCH 2/3] vfs: dedupe: rationalize args

From: Matthew Wilcox
Date: Mon May 07 2018 - 07:17:01 EST


On Mon, May 07, 2018 at 10:21:07AM +0200, Miklos Szeredi wrote:
> @@ -1738,7 +1738,7 @@ struct file_operations {
> loff_t, size_t, unsigned int);
> int (*clone_file_range)(struct file *, loff_t, struct file *, loff_t,
> u64);
> - s64 (*dedupe_file_range)(struct file *, u64, u64, struct file *,
> + s64 (*dedupe_file_range)(struct file *, loff_t, struct file *, loff_t,
> u64);

Please name the parameters here ...

+ loff_t (*dedupe_file_range)(struct file *src, loff_t src_off,
+ struct file *dst, loff_t dst_off, loff_t len);