* [PATCH] fideduperange.2: fix the discussion of maximum sizes
@ 2016-11-23 4:22 Darrick J. Wong
2016-11-23 4:39 ` Omar Sandoval
0 siblings, 1 reply; 4+ messages in thread
From: Darrick J. Wong @ 2016-11-23 4:22 UTC (permalink / raw)
To: mtk.manpages; +Cc: linux-fsdevel, linux-api, linux-xfs, linux-btrfs, linux-man
Fix the discussion of the limitations on the dest_count and src_length
parameters to the fideduperange ioctl to reflect what's actually in the
kernel.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
man2/ioctl_fideduperange.2 | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/man2/ioctl_fideduperange.2 b/man2/ioctl_fideduperange.2
index 2112d10..59f0ea9 100644
--- a/man2/ioctl_fideduperange.2
+++ b/man2/ioctl_fideduperange.2
@@ -95,7 +95,7 @@ struct file_dedupe_range_info {
.in
Each deduplication operation targets
-.IR length
+.IR src_length
bytes in file descriptor
.IR dest_fd
at offset
@@ -108,8 +108,14 @@ During the call,
must be open for reading and
.IR dest_fd
must be open for writing.
-For any call to this ioctl, there may not be more than 65,536
-requests attached; each request may not exceed 16MiB.
+The combined size of the struct
+.IR file_dedupe_range
+and the struct
+.IR file_dedupe_range_info
+array must not exceed the system page size.
+The maximum size of
+.IR src_length
+is filesystem dependent and is typically 16MiB.
By convention, the storage used by
.IR src_fd
is mapped into
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] fideduperange.2: fix the discussion of maximum sizes
@ 2016-11-23 4:39 ` Omar Sandoval
0 siblings, 0 replies; 4+ messages in thread
From: Omar Sandoval @ 2016-11-23 4:39 UTC (permalink / raw)
To: Darrick J. Wong
Cc: mtk.manpages, linux-fsdevel, linux-api, linux-xfs, linux-btrfs,
linux-man
On Tue, Nov 22, 2016 at 08:22:22PM -0800, Darrick J. Wong wrote:
> Fix the discussion of the limitations on the dest_count and src_length
> parameters to the fideduperange ioctl to reflect what's actually in the
> kernel.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> ---
> man2/ioctl_fideduperange.2 | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/man2/ioctl_fideduperange.2 b/man2/ioctl_fideduperange.2
> index 2112d10..59f0ea9 100644
> --- a/man2/ioctl_fideduperange.2
> +++ b/man2/ioctl_fideduperange.2
> @@ -95,7 +95,7 @@ struct file_dedupe_range_info {
> .in
>
> Each deduplication operation targets
> -.IR length
> +.IR src_length
> bytes in file descriptor
> .IR dest_fd
> at offset
> @@ -108,8 +108,14 @@ During the call,
> must be open for reading and
> .IR dest_fd
> must be open for writing.
> -For any call to this ioctl, there may not be more than 65,536
> -requests attached; each request may not exceed 16MiB.
> +The combined size of the struct
> +.IR file_dedupe_range
> +and the struct
> +.IR file_dedupe_range_info
> +array must not exceed the system page size.
> +The maximum size of
> +.IR src_length
> +is filesystem dependent and is typically 16MiB.
> By convention, the storage used by
> .IR src_fd
> is mapped into
Might be worth it to clarify that the file_dedupe_range_info limit is a
hard limit that will result in an error if exceeded and that the
src_length limit is silently enforced. These semantics suck...
--
Omar
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fideduperange.2: fix the discussion of maximum sizes
@ 2016-11-23 4:39 ` Omar Sandoval
0 siblings, 0 replies; 4+ messages in thread
From: Omar Sandoval @ 2016-11-23 4:39 UTC (permalink / raw)
To: Darrick J. Wong
Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w,
linux-fsdevel-u79uwXL29TY76Z2rM5mHXA,
linux-api-u79uwXL29TY76Z2rM5mHXA,
linux-xfs-u79uwXL29TY76Z2rM5mHXA,
linux-btrfs-u79uwXL29TY76Z2rM5mHXA,
linux-man-u79uwXL29TY76Z2rM5mHXA
On Tue, Nov 22, 2016 at 08:22:22PM -0800, Darrick J. Wong wrote:
> Fix the discussion of the limitations on the dest_count and src_length
> parameters to the fideduperange ioctl to reflect what's actually in the
> kernel.
>
> Signed-off-by: Darrick J. Wong <darrick.wong-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
> ---
> man2/ioctl_fideduperange.2 | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/man2/ioctl_fideduperange.2 b/man2/ioctl_fideduperange.2
> index 2112d10..59f0ea9 100644
> --- a/man2/ioctl_fideduperange.2
> +++ b/man2/ioctl_fideduperange.2
> @@ -95,7 +95,7 @@ struct file_dedupe_range_info {
> .in
>
> Each deduplication operation targets
> -.IR length
> +.IR src_length
> bytes in file descriptor
> .IR dest_fd
> at offset
> @@ -108,8 +108,14 @@ During the call,
> must be open for reading and
> .IR dest_fd
> must be open for writing.
> -For any call to this ioctl, there may not be more than 65,536
> -requests attached; each request may not exceed 16MiB.
> +The combined size of the struct
> +.IR file_dedupe_range
> +and the struct
> +.IR file_dedupe_range_info
> +array must not exceed the system page size.
> +The maximum size of
> +.IR src_length
> +is filesystem dependent and is typically 16MiB.
> By convention, the storage used by
> .IR src_fd
> is mapped into
Might be worth it to clarify that the file_dedupe_range_info limit is a
hard limit that will result in an error if exceeded and that the
src_length limit is silently enforced. These semantics suck...
--
Omar
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fideduperange.2: fix the discussion of maximum sizes
2016-11-23 4:39 ` Omar Sandoval
(?)
@ 2016-11-23 4:47 ` Darrick J. Wong
-1 siblings, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2016-11-23 4:47 UTC (permalink / raw)
To: Omar Sandoval
Cc: mtk.manpages, linux-fsdevel, linux-api, linux-xfs, linux-btrfs,
linux-man
On Tue, Nov 22, 2016 at 08:39:18PM -0800, Omar Sandoval wrote:
> On Tue, Nov 22, 2016 at 08:22:22PM -0800, Darrick J. Wong wrote:
> > Fix the discussion of the limitations on the dest_count and src_length
> > parameters to the fideduperange ioctl to reflect what's actually in the
> > kernel.
> >
> > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
> > ---
> > man2/ioctl_fideduperange.2 | 12 +++++++++---
> > 1 file changed, 9 insertions(+), 3 deletions(-)
> >
> > diff --git a/man2/ioctl_fideduperange.2 b/man2/ioctl_fideduperange.2
> > index 2112d10..59f0ea9 100644
> > --- a/man2/ioctl_fideduperange.2
> > +++ b/man2/ioctl_fideduperange.2
> > @@ -95,7 +95,7 @@ struct file_dedupe_range_info {
> > .in
> >
> > Each deduplication operation targets
> > -.IR length
> > +.IR src_length
> > bytes in file descriptor
> > .IR dest_fd
> > at offset
> > @@ -108,8 +108,14 @@ During the call,
> > must be open for reading and
> > .IR dest_fd
> > must be open for writing.
> > -For any call to this ioctl, there may not be more than 65,536
> > -requests attached; each request may not exceed 16MiB.
> > +The combined size of the struct
> > +.IR file_dedupe_range
> > +and the struct
> > +.IR file_dedupe_range_info
> > +array must not exceed the system page size.
> > +The maximum size of
> > +.IR src_length
> > +is filesystem dependent and is typically 16MiB.
> > By convention, the storage used by
> > .IR src_fd
> > is mapped into
>
> Might be worth it to clarify that the file_dedupe_range_info limit is a
> hard limit that will result in an error if exceeded and that the
> src_length limit is silently enforced. These semantics suck...
Yeah... :(
Anyway I'll post a v2 patch.
--D
>
> --
> Omar
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-11-23 4:47 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-23 4:22 [PATCH] fideduperange.2: fix the discussion of maximum sizes Darrick J. Wong
2016-11-23 4:39 ` Omar Sandoval
2016-11-23 4:39 ` Omar Sandoval
2016-11-23 4:47 ` Darrick J. Wong
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.