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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 27, 2014 at 11:17:55AM -0400, bjschuma@xxxxxxxxxx wrote:
> From: Anna Schumaker <Anna.Schumaker@xxxxxxxxxx>
> 
> These patches add server support for the NFS v4.2 operation SEEK.  The first
> two patches are error cleanups and infrastructure setup to prepare for the
> addition of new operations.
> 
> My biggest concern is that these patches freeze the v4.2 opcodes to their
> current values, which makes any future spec changes to add or remove
> operations more difficult to accomidate.

I don't think it's so bad--worst case we end up with a hole in the list
of operation numbers, but I don't think that's a big deal.

So I'd drop the warning in the config text.

Anyway, the code's simple enough.  Assuming the client patches are
ready, I'll be inclined to merge (revised versions of) these for 3.18.

--b.

> 
> Questions?  Comments?  Thoughts?
> 
> Anna
> 
> 
> Anna Schumaker (3):
>   NFSD: Update error codes
>   NFSD: Create nfs v4.2 decode ops
>   NFSD: Implement SEEK
> 
>  fs/nfsd/Kconfig      | 12 +++++++++
>  fs/nfsd/nfs4proc.c   | 47 +++++++++++++++++++++++++++++++++
>  fs/nfsd/nfs4xdr.c    | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  fs/nfsd/nfsd.h       |  2 +-
>  fs/nfsd/xdr4.h       | 14 ++++++++++
>  include/linux/nfs4.h | 26 ++++++++++++++++---
>  6 files changed, 170 insertions(+), 4 deletions(-)
> 
> -- 
> 2.1.0
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux