Return-Path: linux-nfs-owner@vger.kernel.org Received: from mx12.netapp.com ([216.240.18.77]:6587 "EHLO mx12.netapp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932820Ab3GVTiu (ORCPT ); Mon, 22 Jul 2013 15:38:50 -0400 Message-ID: <51ED8A47.7040506@netapp.com> Date: Mon, 22 Jul 2013 15:38:47 -0400 From: Bryan Schumaker MIME-Version: 1.0 To: "J. Bruce Fields" CC: , Subject: Re: [RFC 0/5] NFS Server Side Copy References: <1374267830-30154-1-git-send-email-bjschuma@netapp.com> <20130722185336.GC10109@fieldses.org> In-Reply-To: <20130722185336.GC10109@fieldses.org> Content-Type: text/plain; charset="ISO-8859-1" Sender: linux-nfs-owner@vger.kernel.org List-ID: On 07/22/2013 02:53 PM, J. Bruce Fields wrote: > On Fri, Jul 19, 2013 at 05:03:45PM -0400, bjschuma@netapp.com wrote: >> From: Bryan Schumaker >> >> These patches build on Zach Brown's copyfile patches to add server side >> copy to both the NFS client and the NFS server. >> >> The first patch improves on the copyfile syscall to make it usable on my >> machine and also includes notes on other potential problems that I've >> found. The remaining patches first implement a sync copy, then expand to >> async. >> >> My testing was done on a server exporting an ext4 filesystem exporting an >> ext4 filesystem. I compared copying using the cp command to copying with >> the copyfile system call. > > Were these tests using the full series of patches? (So, using the > asynchronous mechanism?) Yes. Want me to re-run them without it? - Bryan > > --b. > >> >> >> File size: 512 MB >> cp: 4.244 seconds >> copyfile: 0.961 seconds >> >> File size: 1024 MB >> cp: 9.091 seconds >> copyfile: 1.919 seconds >> >> File size: 1536 MB >> cp: 15.291 seconds >> copyfile: 6.016 seconds >> >> >> Repeating these tests on a btrfs exported filesystem supporting the copyfile >> system call drops the time for copyfile to about 0.01 seconds. >> >> Feel free to send me any questions, comments or other thoughts! >> >> - Bryan >> >> Bryan Schumaker (5): >> Improve on the copyfile systemcall >> NFSD: Implement the COPY call >> NFS: Add COPY nfs operation >> NFSD: Defer copying >> NFS: Change copy to support async servers >> >> fs/copy_range.c | 10 +++- >> fs/nfs/callback.h | 13 ++++ >> fs/nfs/callback_proc.c | 9 +++ >> fs/nfs/callback_xdr.c | 54 ++++++++++++++++- >> fs/nfs/inode.c | 2 + >> fs/nfs/nfs4_fs.h | 7 +++ >> fs/nfs/nfs4file.c | 101 +++++++++++++++++++++++++++++++ >> fs/nfs/nfs4proc.c | 16 +++++ >> fs/nfs/nfs4xdr.c | 150 +++++++++++++++++++++++++++++++++++++++++++++++ >> fs/nfsd/nfs4callback.c | 136 ++++++++++++++++++++++++++++++++++++++++++ >> fs/nfsd/nfs4proc.c | 104 ++++++++++++++++++++++++++++++-- >> fs/nfsd/nfs4state.c | 15 ++++- >> fs/nfsd/nfs4xdr.c | 121 +++++++++++++++++++++++++++++++++++++- >> fs/nfsd/state.h | 23 +++++++- >> fs/nfsd/vfs.c | 9 +++ >> fs/nfsd/vfs.h | 1 + >> fs/nfsd/xdr4.h | 24 ++++++++ >> fs/nfsd/xdr4cb.h | 9 +++ >> include/linux/nfs4.h | 14 ++++- >> include/linux/nfs_xdr.h | 33 +++++++++++ >> include/linux/syscalls.h | 1 + >> 21 files changed, 836 insertions(+), 16 deletions(-) >> >> -- >> 1.8.3.3 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html