Fwd: [PATCH] mount.cifs: remove support for "complex" usernames from mount.cifs

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

 



---------- Forwarded message ----------
From: Steve French <smfrench@xxxxxxxxx>
Date: Fri, Jan 4, 2013 at 2:04 PM
Subject: Re: [PATCH] mount.cifs: remove support for "complex"
usernames from mount.cifs
To: Scott Lovenberg <scott.lovenberg@xxxxxxxxx>
Cc: linux-cifs@xxxxxxxxxxxxxxx, Jeff Layton <jlayton@xxxxxxxxx>



On Jan 4, 2013 12:26 PM, "Scott Lovenberg" <scott.lovenberg@xxxxxxxxx> wrote:
>
> On Thu, Jan 3, 2013 at 3:04 PM, Steve French <smfrench@xxxxxxxxx> wrote:
> > I am not as worried about code cleanliness as I am about losing function,
> > especially as it is sometimes hard to figure out how many users would be
> > affected by removing function.  I mildly prefer leaving the nfs syntax
> > if it makes it easier for nfs users to use smb3, but we could make a stub
> > mount helper that parses nfs syntax and simply calls mount.cifs if you
> > prefer that approach - in any case I don't feel strongly about nfs syntax.
>
> I don't care much either way.  The reason behind removing it, IIRC,
> was that it was an undocumented feature that lead to an ambiguous
> parsing situation when you had NFS syntax with an IPv6 address as the
> server portion of the string (ie
> dead:beef::1:/shareNameWithColon:/prepath).  It was easier to remove
> the undocumented feature than to support it for this use case.
>
> How does NFS syntax make things easier for SMB3?

UNC names may be unfamiliar for some NFS users. Smb3 has many
performance advantages and other features that may eventually be of
strong interest to some Linux  NFS users


-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux