Re: [PATCH v2 0/3] NFSD: Implement SEEK

[Anna Schumaker <bjschuma@netapp.com>]

On 11/12/2013 02:45 PM, J. Bruce Fields wrote:
> On Tue, Nov 12, 2013 at 02:04:05PM -0500, Anna Schumaker wrote:
>> These patches implement just the SEEK NFS v4.2 operation.  WRITE_PLUS is
>> still under discussion with the IETF after my last series of patches, so I
>> am holding off on resubmitting until after spec discussion dies down.
>>
>> Questions?  Comments?  Thoughts?
>> Anna
>>
>> Anna Schumaker (3):
>>   NFSD: Update error codes
> 
> I don't think I got this first patch.

It's in the "patches" directory I used, but I don't see it in my gmail inbox either.  I'll send it out in a minute.

> 
>>   NFSD: Create nfs v4.2 decode ops
>>   NFSD: Implement SEEK
> 
> I'd like to be reassured the protocol is reasonably stable before we
> commit this.  I haven't been following the ietf wg discussion closely.
> 
> And this should initially be disabled by default.  So, probably either:
> 
> 	- Introduce a new NFSD_V4_SEEK option, or
> 	- Combine this and NFSD_V4_SECURITY_LABEL and this into a single
> 	  NFSD_V4_2 option.
> 
> And recommend "N" for now.
> 
> Probably the latter I guess, to be consistent with the client.  And
> because otherwise we could end up with an awful lot of config options.

Sure, I can do that easily enough.

Anna

> 
> --b.
> 
>>
>>  fs/nfsd/nfs4proc.c   | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  fs/nfsd/nfs4xdr.c    | 58 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>>  fs/nfsd/nfsd.h       |  2 +-
>>  fs/nfsd/xdr4.h       | 16 +++++++++++++++
>>  include/linux/nfs4.h | 23 ++++++++++++++++++---
>>  5 files changed, 151 insertions(+), 4 deletions(-)
>>
>> -- 
>> 1.8.4.2
>>
>> --
>> 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