Return-Path: Received: from fieldses.org ([174.143.236.118]:58967 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751718Ab0HSASy (ORCPT ); Wed, 18 Aug 2010 20:18:54 -0400 Date: Wed, 18 Aug 2010 20:16:47 -0400 From: "J. Bruce Fields" To: Bian Naimeng Cc: Trond Myklebust , linux-nfs@vger.kernel.org, Christoph Hellwig Subject: Re: [PATCH] Server should allow offset larger than LLONG_MAX at commit procedure Message-ID: <20100819001647.GD18367@fieldses.org> References: <4C6B9538.2020608@cn.fujitsu.com> Content-Type: text/plain; charset=utf-8 In-Reply-To: <4C6B9538.2020608@cn.fujitsu.com> Sender: linux-nfs-owner@vger.kernel.org List-ID: MIME-Version: 1.0 On Wed, Aug 18, 2010 at 04:09:28PM +0800, Bian Naimeng wrote: > When offset larger than LLONG_MAX, it's better to sync all the data of file > than return nfserr_inval. I believe the current behavior is correct. See http://marc.info/?l=linux-nfs&m=128200558207974&w=2 for a pynfs-side fix. --b. > > Signed-off-by: Bian Naimeng > > --- > fs/nfsd/vfs.c | 14 ++++++++------ > 1 files changed, 8 insertions(+), 6 deletions(-) > > diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c > index 96360a8..f67fe31 100644 > --- a/fs/nfsd/vfs.c > +++ b/fs/nfsd/vfs.c > @@ -1174,20 +1174,23 @@ nfsd_commit(struct svc_rqst *rqstp, struct svc_fh *fhp, > { > struct file *file; > loff_t end = LLONG_MAX; > - __be32 err = nfserr_inval; > + __be32 err; > + > + if ((u64)count > ‾(u64)offset) > + return nfserr_inval; > > if (offset < 0) > - goto out; > - if (count != 0) { > + offset = 0; > + else if (count != 0) { > end = offset + (loff_t)count - 1; > if (end < offset) > - goto out; > + end = LLONG_MAX; > } > > err = nfsd_open(rqstp, fhp, S_IFREG, > NFSD_MAY_WRITE|NFSD_MAY_NOT_BREAK_LEASE, &file); > if (err) > - goto out; > + return err; > if (EX_ISSYNC(fhp->fh_export)) { > int err2 = vfs_fsync_range(file, offset, end, 0); > > @@ -1198,7 +1201,6 @@ nfsd_commit(struct svc_rqst *rqstp, struct svc_fh *fhp, > } > > nfsd_close(file); > -out: > return err; > } > #endif /* CONFIG_NFSD_V3 */ > -- > 1.6.5.2 > > > > > -- > Regards > Bian Naimeng >