Re: [PATCH] cifs: use standard token parser for mount options

[Scott Lovenberg <scott.lovenberg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On 3/14/2012 3:58 PM, Sachin Prabhu wrote:
> ----- Original Message -----
>> I don't mind reviewing this, but it does appear to change behaviour
>> significantly from existing code.  Does this fix a reported bug?
>>
>> The linux cifs kernel driver, like some other
>> network clients, allows partial option matching to ease
>> usability for users of other platforms (e.g. "pass" or
>> "passwd" or "password" are intentionally all accepted
>> since we match on the first four bytes of that keyword).
>> Is there a way to get that to work with your patchset.
>> I worry that some users will be confused that their
>> mount options won't work if this is taken.
>>
> The reason I was looking at this code was because I was helping Jeff include in the sloppy mount option for cifs. The current if/elseif block is difficult to read through. There are also hidden gotchas like ordering the if blocks correctly if in case a mount option is also a substring in a different mount option. The decision to use the standard token parser was simply for consistency and to make the code easier to understand.
>
> Multiple versions of mount options can be accepted for the same mount option. For example, in the patch sent earlier, for the password, we accept the following mount options to pass the password.
>
>
> +       { Opt_pass, "pass=%s" },
> +       { Opt_pass, "password=%s" },
>
> So the user can use pass or password and they will both be accepted as the mount option to set the password.
> We can add more options if I have missed any.
>
> The only difference I see is that the options matching is case sensitive and therefore will reject upper case mount options.
>
> Also, in my opinion the number of mount options is large at the moment and we should consider deprecating a few of the mount options.
>
> Sachin Prabhu
>

Would this patchset be a reasonable compromise? This is the one I was 
talking about in my last email. 
http://samba.2283325.n4.nabble.com/Add-token-parsing-for-kernel-CIFS-mount-options-td2517375.html

Also, some mount options are synonyms for other mount options, they can 
be combined.

Which were you thinking about deprecating?  RHEL/SLED probably aren't 
going to like shipping a kernel with deprecated mount options.  That's 
more of a political problem, but a problem nonetheless.