[07/26] vfs: skip zero-length dedupe requests

Message ID	153965944451.1256.13419242995682379278.stgit@magnolia (mailing list archive)
State	New, archived
Headers	show Return-Path: <owner-linux-mm@kvack.org> Received-SPF: pass (google.com: domain of darrick.wong@oracle.com designates 156.151.31.86 as permitted sender) client-ip=156.151.31.86; Subject: [PATCH 07/26] vfs: skip zero-length dedupe requests From: "Darrick J. Wong" <darrick.wong@oracle.com> To: david@fromorbit.com, darrick.wong@oracle.com Cc: sandeen@redhat.com, linux-nfs@vger.kernel.org, linux-cifs@vger.kernel.org, Amir Goldstein <amir73il@gmail.com>, linux-unionfs@vger.kernel.org, linux-xfs@vger.kernel.org, linux-mm@kvack.org, linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christoph Hellwig <hch@lst.de>, ocfs2-devel@oss.oracle.com Date: Mon, 15 Oct 2018 20:10:44 -0700 Message-ID: <153965944451.1256.13419242995682379278.stgit@magnolia> In-Reply-To: <153965939489.1256.7400115244528045860.stgit@magnolia> References: <153965939489.1256.7400115244528045860.stgit@magnolia> User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Precedence: bulk
Series	fs: fixes for serious clone/dedupe problems \| expand [v5,00/26] fs: fixes for serious clone/dedupe problems [01/26] xfs: add a per-xfs trace_printk macro [02/26] vfs: vfs_clone_file_prep_inodes should return EINVAL for a clone from beyond EOF [03/26] vfs: check file ranges before cloning files [04/26] vfs: exit early from zero length remap operations [05/26] vfs: strengthen checking of file range inputs to generic_remap_checks [06/26] vfs: avoid problematic remapping requests into partial EOF block [07/26] vfs: skip zero-length dedupe requests [08/26] vfs: rename vfs_clone_file_prep to be more descriptive [09/26] vfs: rename clone_verify_area to remap_verify_area [10/26] vfs: combine the clone and dedupe into a single remap_file_range [11/26] vfs: pass remap flags to generic_remap_file_range_prep [12/26] vfs: pass remap flags to generic_remap_checks [13/26] vfs: create generic_remap_file_range_touch to update inode metadata [14/26] vfs: make remap_file_range functions take and return bytes completed [15/26] vfs: plumb remap flags through the vfs clone functions [16/26] vfs: plumb remap flags through the vfs dedupe functions [17/26] vfs: enable remap callers that can handle short operations [18/26] vfs: hide file range comparison function [19/26] vfs: clean up generic_remap_file_range_prep return value [20/26] ocfs2: truncate page cache for clone destination file before remapping [21/26] ocfs2: fix pagecache truncation prior to reflink [22/26] ocfs2: support partial clone range and dedupe range [23/26] ocfs2: remove ocfs2_reflink_remap_range [24/26] xfs: fix pagecache truncation prior to reflink [25/26] xfs: support returning partial reflink results [26/26] xfs: remove redundant remap partial EOF block checks

Message ID

153965944451.1256.13419242995682379278.stgit@magnolia (mailing list archive)

State

New, archived

Headers

Received-SPF: pass (google.com: domain of darrick.wong@oracle.com designates
 156.151.31.86 as permitted sender) client-ip=156.151.31.86;
Subject: [PATCH 07/26] vfs: skip zero-length dedupe requests
From: "Darrick J. Wong" <darrick.wong@oracle.com>
To: david@fromorbit.com, darrick.wong@oracle.com
Cc: sandeen@redhat.com, linux-nfs@vger.kernel.org, linux-cifs@vger.kernel.org,
        Amir Goldstein <amir73il@gmail.com>, linux-unionfs@vger.kernel.org,
        linux-xfs@vger.kernel.org, linux-mm@kvack.org,
        linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org,
        Christoph Hellwig <hch@lst.de>, ocfs2-devel@oss.oracle.com
Date: Mon, 15 Oct 2018 20:10:44 -0700
Message-ID: <153965944451.1256.13419242995682379278.stgit@magnolia>
In-Reply-To: <153965939489.1256.7400115244528045860.stgit@magnolia>
References: <153965939489.1256.7400115244528045860.stgit@magnolia>
User-Agent: StGit/0.17.1-dirty
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
Sender: owner-linux-mm@kvack.org
Precedence: bulk

Series

fs: fixes for serious clone/dedupe problems | expand

Commit Message

Darrick J. Wong Oct. 16, 2018, 3:10 a.m. UTC

From: Darrick J. Wong <darrick.wong@oracle.com>

Don't bother calling the filesystem for a zero-length dedupe request;
we can return zero and exit.

Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Amir Goldstein <amir73il@gmail.com>
---
 fs/read_write.c |    5 +++++
 1 file changed, 5 insertions(+)

diff --git a/fs/read_write.c b/fs/read_write.c
index 0f0a6efdd502..f5395d8da741 100644
--- a/fs/read_write.c
+++ b/fs/read_write.c
@@ -2009,6 +2009,11 @@  int vfs_dedupe_file_range_one(struct file *src_file, loff_t src_pos,
 	if (!dst_file->f_op->dedupe_file_range)
 		goto out_drop_write;
 
+	if (len == 0) {
+		ret = 0;
+		goto out_drop_write;
+	}
+
 	ret = dst_file->f_op->dedupe_file_range(src_file, src_pos,
 						dst_file, dst_pos, len);
 out_drop_write:

[07/26] vfs: skip zero-length dedupe requests

Commit Message

Patch