From patchwork Sat Oct 24 23:17:16 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Peng Tao X-Patchwork-Id: 7480591 Return-Path: X-Original-To: patchwork-linux-nfs@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 183389F731 for ; Sat, 24 Oct 2015 15:30:05 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 056CF206EF for ; Sat, 24 Oct 2015 15:30:04 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id C3B7C207DB for ; Sat, 24 Oct 2015 15:30:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752814AbbJXP3y (ORCPT ); Sat, 24 Oct 2015 11:29:54 -0400 Received: from mail-pa0-f43.google.com ([209.85.220.43]:36499 "EHLO mail-pa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752768AbbJXP3v (ORCPT ); Sat, 24 Oct 2015 11:29:51 -0400 Received: by pacfv9 with SMTP id fv9so151053487pac.3 for ; Sat, 24 Oct 2015 08:29:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=primarydata_com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=B3JJbCl/bH75dKhcpCUsP6uK/jLdWxrlrfU07ogoAHI=; b=RTMMV9gAVHOM49yKHvY3nmZxRS0D212qppI2Tndr8aQ0Ctzhx2mFF6pWaxVtWhvrAn 904o+RYzDFTXCO1YQrKRDTTA6/4b4BnTe46oXub6gMLTLSUQxVpbUIwDxhEYdH+nX13d m/DdJBp/+zVcbXqmEW2FBPR0NA6DQAFLohgjCGMkxHSk3Vt00afv7A9VajlK02FJKTVq QZfwjCPhxxW7pjf2kEnbKHlpPZWcTb9VfVGmL2GWS64su0xgL0lVMKf2WnxnoiVCdaZz SaJUJc2jSC9yZMvuFYH+5FE+d44qryKUYfCyZ5mgu1A+X55iQ9Aomroul7feEYzOB/0W K1gw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=B3JJbCl/bH75dKhcpCUsP6uK/jLdWxrlrfU07ogoAHI=; b=gB0YvRY6Sz66Lomg/DprNZaqj4A+cOV9YPfL/cwan97wKCnFHh1wyUez41RDFoBAIH jJXVm/a0WZ4SXlZpjSv2xMbf41cVaP46vnhLIMQhKKOpqvSwYV656Wvy5JtF6t7kUJS4 1n7fJzJYmFliFFJ+s5r6QAsMwixgDKZumg1J7NpE6At/o8LvtBD7TNVZWC1yHRMtqvav 5FbiqjveDzyyCSXSSKJpDy1WjTHlyIReFkS5RuSPt5gDv0tGZR2IqIBLXpSEYe+VRMDF bsQmXJXPzLqm2MVYfbnwLDQ+CBn/UxbJx8PNZejsTqzGgG7MkW/QA9NMXGHOnvq/zZSj Mjfg== X-Gm-Message-State: ALoCoQlfQ2by8MGviSQKq/FyzcUR1YXOoMmY3iiF5DoaiIxx1d1xgNzpFnKUtsUpoDpeQTQ/PXgh X-Received: by 10.68.252.194 with SMTP id zu2mr11881400pbc.58.1445700590868; Sat, 24 Oct 2015 08:29:50 -0700 (PDT) Received: from lear.localdomain (ec2-54-65-164-9.ap-northeast-1.compute.amazonaws.com. [54.65.164.9]) by smtp.gmail.com with ESMTPSA id nj1sm24764918pbc.2.2015.10.24.08.29.43 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Sat, 24 Oct 2015 08:29:50 -0700 (PDT) From: Peng Tao To: linux-fsdevel@vger.kernel.org Cc: Trond Myklebust , Anna Schumaker , Christoph Hellwig , Zach Brown , Darren Hart , Jeff Layton , bfields@fieldses.org, "Darrick J. Wong" , viro@zeniv.linux.org.uk, linux-nfs@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-cifs@vger.kernel.org, Steve French , Anna Schumaker , Christoph Hellwig , Peng Tao Subject: [PATCH 9/9] NFSD: Implement the CLONE call Date: Sun, 25 Oct 2015 07:17:16 +0800 Message-Id: <1445728636-10109-10-git-send-email-tao.peng@primarydata.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1445728636-10109-1-git-send-email-tao.peng@primarydata.com> References: <1445728636-10109-1-git-send-email-tao.peng@primarydata.com> Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Spam-Status: No, score=-4.8 required=5.0 tests=BAYES_00, DATE_IN_FUTURE_06_12, DKIM_SIGNED, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Anna Schumaker I can simply call vfs_file_clone_range() and have the vfs do the right thing for the filesystem being exported. Signed-off-by: Anna Schumaker [hch: change to implement the CLONE op instead of COPY] Signed-off-by: Christoph Hellwig Signed-off-by: Peng Tao --- fs/nfsd/nfs4proc.c | 80 ++++++++++++++++++++++++++++++++++++++++++++++++++++ fs/nfsd/nfs4xdr.c | 21 ++++++++++++++ fs/nfsd/vfs.c | 18 ++++++++++++ fs/nfsd/vfs.h | 1 + fs/nfsd/xdr4.h | 10 +++++++ include/linux/nfs4.h | 4 +-- 6 files changed, 132 insertions(+), 2 deletions(-) diff --git a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c index eab6d6d..d1c5a87 100644 --- a/fs/nfsd/nfs4proc.c +++ b/fs/nfsd/nfs4proc.c @@ -1012,6 +1012,80 @@ nfsd4_write(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, } static __be32 +nfsd4_verify_clone(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, + struct nfsd4_clone *clone, struct file **src, + struct file **dst) +{ + struct inode *src_ino, *dst_ino; + __be32 status; + + status = nfs4_preprocess_stateid_op(rqstp, cstate, &cstate->save_fh, + &clone->cl_src_stateid, RD_STATE, + src, NULL); + if (status) { + dprintk("NFSD: %s: couldn't process src stateid!\n", __func__); + return status; + } + + status = nfs4_preprocess_stateid_op(rqstp, cstate, &cstate->current_fh, + &clone->cl_dst_stateid, WR_STATE, + dst, NULL); + if (status) { + dprintk("NFSD: %s: couldn't process dst stateid!\n", __func__); + fput(*src); + } + + /* a few extra check to make sure we send back proper errors per RFC */ + src_ino = file_inode(*src); + dst_ino = file_inode(*dst); + + if (S_ISDIR(src_ino->i_mode) || S_ISDIR(dst_ino->i_mode)) { + status = nfserr_wrong_type; + goto out_fput; + } + + if (src_ino == dst_ino) { + status = nfserr_inval; + goto out_fput; + } + + if (!(*src)->f_op || !(*src)->f_op->copy_file_range) { + status = nfserr_notsupp; + goto out_fput; + } +out: + return status; +out_fput: + fput(*src); + fput(*dst); + goto out; +} + +static __be32 +nfsd4_clone(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, + struct nfsd4_clone *clone) +{ + int ret; + __be32 status; + struct file *src = NULL, *dst = NULL; + + status = nfsd4_verify_clone(rqstp, cstate, clone, &src, &dst); + if (status) + return status; + + ret = nfsd4_clone_range(src, dst, clone->cl_src_pos, + clone->cl_count, clone->cl_dst_pos); + if (ret < 0) + status = nfserrno(ret); + else + status = nfs_ok; + + fput(src); + fput(dst); + return status; +} + +static __be32 nfsd4_fallocate(struct svc_rqst *rqstp, struct nfsd4_compound_state *cstate, struct nfsd4_fallocate *fallocate, int flags) { @@ -2277,6 +2351,12 @@ static struct nfsd4_operation nfsd4_ops[] = { .op_name = "OP_DEALLOCATE", .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize, }, + [OP_CLONE] = { + .op_func = (nfsd4op_func)nfsd4_clone, + .op_flags = OP_MODIFIES_SOMETHING | OP_CACHEME, + .op_name = "OP_CLONE", + .op_rsize_bop = (nfsd4op_rsize)nfsd4_only_status_rsize, + }, [OP_SEEK] = { .op_func = (nfsd4op_func)nfsd4_seek, .op_name = "OP_SEEK", diff --git a/fs/nfsd/nfs4xdr.c b/fs/nfsd/nfs4xdr.c index 51c9e9c..924416f 100644 --- a/fs/nfsd/nfs4xdr.c +++ b/fs/nfsd/nfs4xdr.c @@ -1675,6 +1675,25 @@ nfsd4_decode_fallocate(struct nfsd4_compoundargs *argp, } static __be32 +nfsd4_decode_clone(struct nfsd4_compoundargs *argp, struct nfsd4_clone *clone) +{ + DECODE_HEAD; + + status = nfsd4_decode_stateid(argp, &clone->cl_src_stateid); + if (status) + return status; + status = nfsd4_decode_stateid(argp, &clone->cl_dst_stateid); + if (status) + return status; + + READ_BUF(8 + 8 + 8); + p = xdr_decode_hyper(p, &clone->cl_src_pos); + p = xdr_decode_hyper(p, &clone->cl_dst_pos); + p = xdr_decode_hyper(p, &clone->cl_count); + DECODE_TAIL; +} + +static __be32 nfsd4_decode_seek(struct nfsd4_compoundargs *argp, struct nfsd4_seek *seek) { DECODE_HEAD; @@ -1785,6 +1804,7 @@ static nfsd4_dec nfsd4_dec_ops[] = { [OP_READ_PLUS] = (nfsd4_dec)nfsd4_decode_notsupp, [OP_SEEK] = (nfsd4_dec)nfsd4_decode_seek, [OP_WRITE_SAME] = (nfsd4_dec)nfsd4_decode_notsupp, + [OP_CLONE] = (nfsd4_dec)nfsd4_decode_clone, }; static inline bool @@ -4292,6 +4312,7 @@ static nfsd4_enc nfsd4_enc_ops[] = { [OP_READ_PLUS] = (nfsd4_enc)nfsd4_encode_noop, [OP_SEEK] = (nfsd4_enc)nfsd4_encode_seek, [OP_WRITE_SAME] = (nfsd4_enc)nfsd4_encode_noop, + [OP_CLONE] = (nfsd4_enc)nfsd4_encode_noop, }; /* diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index 45c0497..9ae4992 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c @@ -36,6 +36,7 @@ #endif /* CONFIG_NFSD_V3 */ #ifdef CONFIG_NFSD_V4 +#include "../internal.h" #include "acl.h" #include "idmap.h" #endif /* CONFIG_NFSD_V4 */ @@ -498,6 +499,23 @@ __be32 nfsd4_set_nfs4_label(struct svc_rqst *rqstp, struct svc_fh *fhp, } #endif +int nfsd4_clone_range(struct file *src, struct file *dst, u64 src_pos, + u64 count, u64 dst_pos) +{ + struct inode *src_ino = file_inode(src); + int rc; + + /* vfs_copy_file_range thinks count == 0 is nothing to clone, + * while in NFS it means from src_pos to eof + */ + if (count == 0) + count = i_size_read(src_ino) - src_pos; + rc = vfs_copy_file_range(src, src_pos, dst, dst_pos, + count, COPY_FILE_CLONE_ONLY); + + return rc < 0 ? rc : 0; +} + __be32 nfsd4_vfs_fallocate(struct svc_rqst *rqstp, struct svc_fh *fhp, struct file *file, loff_t offset, loff_t len, int flags) diff --git a/fs/nfsd/vfs.h b/fs/nfsd/vfs.h index fee2451..604830c 100644 --- a/fs/nfsd/vfs.h +++ b/fs/nfsd/vfs.h @@ -56,6 +56,7 @@ __be32 nfsd4_set_nfs4_label(struct svc_rqst *, struct svc_fh *, struct xdr_netobj *); __be32 nfsd4_vfs_fallocate(struct svc_rqst *, struct svc_fh *, struct file *, loff_t, loff_t, int); +int nfsd4_clone_range(struct file *, struct file *, u64, u64, u64); #endif /* CONFIG_NFSD_V4 */ __be32 nfsd_create(struct svc_rqst *, struct svc_fh *, char *name, int len, struct iattr *attrs, diff --git a/fs/nfsd/xdr4.h b/fs/nfsd/xdr4.h index 9f99100..3d70712 100644 --- a/fs/nfsd/xdr4.h +++ b/fs/nfsd/xdr4.h @@ -491,6 +491,15 @@ struct nfsd4_fallocate { u64 falloc_length; }; +struct nfsd4_clone { + /* request */ + stateid_t cl_src_stateid; + stateid_t cl_dst_stateid; + u64 cl_src_pos; + u64 cl_dst_pos; + u64 cl_count; +}; + struct nfsd4_seek { /* request */ stateid_t seek_stateid; @@ -555,6 +564,7 @@ struct nfsd4_op { /* NFSv4.2 */ struct nfsd4_fallocate allocate; struct nfsd4_fallocate deallocate; + struct nfsd4_clone clone; struct nfsd4_seek seek; } u; struct nfs4_replay * replay; diff --git a/include/linux/nfs4.h b/include/linux/nfs4.h index e7e7853..43aeabd 100644 --- a/include/linux/nfs4.h +++ b/include/linux/nfs4.h @@ -139,10 +139,10 @@ enum nfs_opnum4 { Needs to be updated if more operations are defined in future.*/ #define FIRST_NFS4_OP OP_ACCESS -#define LAST_NFS4_OP OP_WRITE_SAME #define LAST_NFS40_OP OP_RELEASE_LOCKOWNER #define LAST_NFS41_OP OP_RECLAIM_COMPLETE -#define LAST_NFS42_OP OP_WRITE_SAME +#define LAST_NFS42_OP OP_CLONE +#define LAST_NFS4_OP LAST_NFS42_OP enum nfsstat4 { NFS4_OK = 0,