diff options
author | Darrick J. Wong <darrick.wong@oracle.com> | 2017-01-09 10:38:57 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-01-12 05:39:44 -0500 |
commit | c00203386d50c1d2f2621163b9d7a533de4819fc (patch) | |
tree | 2d9aa969a1ef50bafe8905a76d376f1aa77e298a /fs | |
parent | f8b20705a383357ddca04ea648a7ac8fccd438ef (diff) |
xfs: don't cap maximum dedupe request length
commit 1bb33a98702d8360947f18a44349df75ba555d5d upstream.
After various discussions on linux-fsdevel, it has been decided that it
is not necessary to cap the length of a dedupe request, and that
correctly-written userspace client programs will be able to absorb the
change. Therefore, remove the length clamping behavior.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/xfs/xfs_file.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/fs/xfs/xfs_file.c b/fs/xfs/xfs_file.c index 6e4f7f900fea..9a5d64b5f35a 100644 --- a/fs/xfs/xfs_file.c +++ b/fs/xfs/xfs_file.c | |||
@@ -939,7 +939,6 @@ xfs_file_clone_range( | |||
939 | len, false); | 939 | len, false); |
940 | } | 940 | } |
941 | 941 | ||
942 | #define XFS_MAX_DEDUPE_LEN (16 * 1024 * 1024) | ||
943 | STATIC ssize_t | 942 | STATIC ssize_t |
944 | xfs_file_dedupe_range( | 943 | xfs_file_dedupe_range( |
945 | struct file *src_file, | 944 | struct file *src_file, |
@@ -950,14 +949,6 @@ xfs_file_dedupe_range( | |||
950 | { | 949 | { |
951 | int error; | 950 | int error; |
952 | 951 | ||
953 | /* | ||
954 | * Limit the total length we will dedupe for each operation. | ||
955 | * This is intended to bound the total time spent in this | ||
956 | * ioctl to something sane. | ||
957 | */ | ||
958 | if (len > XFS_MAX_DEDUPE_LEN) | ||
959 | len = XFS_MAX_DEDUPE_LEN; | ||
960 | |||
961 | error = xfs_reflink_remap_range(src_file, loff, dst_file, dst_loff, | 952 | error = xfs_reflink_remap_range(src_file, loff, dst_file, dst_loff, |
962 | len, true); | 953 | len, true); |
963 | if (error) | 954 | if (error) |