Re: [PATCH 00/17] RFC: proposed changes to the NFS mount command

[Chuck Lever <chuck.lever@oracle.com>]

[-- Attachment #1: Type: text/plain, Size: 1247 bytes --]



Christoph Hellwig wrote:
> On Mon, Jul 16, 2007 at 02:05:34PM +1000, Neil Brown wrote:
>> I've wondered why that was happening... Can't we just make a version
>> x+1 with a big string buffer so new options get put there is regular
>> string format and old options stay the way they are?
>>
>> But I guess that - long term - it is better to be just like other
>> filesystems.
> 
> Having anything but a plain string means the filesystem is a complete
> oddball.  And with the planned removal of smbfs nfs would be the only
> oddball left.
> 
>>> option that will disable building nfsmount.c and nfs4mount.c, and
>>> enable building a version of these files that will introduce support
>>> for passing mount options as a string (that work is yet to be done).
>> Does this mean that you cannot compile nfs-utils to support both old
>> and new kernels?  Is that really a good plan?
>> What is the over-all migration strategy?
> 
> Making the different argument passing variants exclusive sounds like
> a bad idea indeed.

Yeah, Neil has convinced me that mount.nfs must support binary and 
string option processing simultaneously.

The 17 refactoring patches are still probably good clean ups overall.  I 
will leave them stand on their own merits.

[-- Attachment #2: chuck.lever.vcf --]
[-- Type: text/x-vcard, Size: 290 bytes --]

begin:vcard
fn:Chuck Lever
n:Lever;Chuck
org:Oracle Corporation;Corporate Architecture: Linux Projects Group
adr:;;1015 Granger Avenue;Ann Arbor;MI;48104;USA
title:Principal Member of Staff
tel;work:+1 248 614 5091
x-mozilla-html:FALSE
url:http://oss.oracle.com/~cel
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 286 bytes --]

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

[-- Attachment #4: Type: text/plain, Size: 140 bytes --]

_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs