Re: [PATCH v3 0/2] NFSD: Add v4.2 SEEK support

J. Bruce Fields
2014-10-01T18:21:12Z

On Tue, Sep 30, 2014 at 01:00:24PM -0400, Anna Schumaker wrote:
> I'm testing with xfstests #285. It's an lseek specific test, and helped
> find a few edge cases during development.

I think you told me that already and I forgot. Bah.

OK, applied. Thanks!

--b.

>
> Anna
>
> On Fri, Sep 26, 2014 at 5:54 PM, J. Bruce Fields <bfields@fieldses.org>
> wrote:
>
> > On Fri, Sep 26, 2014 at 01:58:25PM -0400, Anna.Schumaker@netapp.com wrote:
> > > From: Anna Schumaker <Anna.Schumaker@Netapp.com>
> > >
> > > These patches add server support for the NFS v4.2 operation SEEK. The
> > first
> > > patch adds basic NFS v4.2 infrastructure that all future operations will
> > rely
> > > on. The second patch adds the SEEK operation.
> >
> > How are you testing these?
> >
> > It looks OK to me, so I'll likely commit in a few days if there are no
> > objections.
> >
> > --b.
> >
> > >
> > > Changes in v3:
> > > - Rework the switch statement in nfsd4_seek() so we only call
> > vfs_llseek()
> > > from one place.
> > > - Change XDR encode and decode functions to A + B notation for
> > calculating
> > > byte counts.
> > >
> > > These patches and the corresponding client patch are avaliable in the
> > [seek]
> > > branch of:
> > >
> > > git://git.linux-nfs.org/projects/anna/linux-nfs.git
> > >
> > >
> > > Questions? Comments? Thoughts?
> > >
> > > Anna
> > >
> > >
> > > Anna Schumaker (2):
> > > NFSD: Add generic v4.2 infrastructure
> > > NFSD: Implement SEEK
> > >
> > > fs/nfsd/nfs4proc.c | 49 ++++++++++++++++++++++++++++++++++++++++++
> > > fs/nfsd/nfs4xdr.c | 60
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++
> > > fs/nfsd/xdr4.h | 14 ++++++++++++
> > > include/linux/nfs4.h | 23 ++++++++++++++++++--
> > > 4 files changed, 144 insertions(+), 2 deletions(-)
> > >
> > > --
> > > 2.1.0
> > >
> > --
> > 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
> >