On Tue, Apr 23, 2013 at 9:01 AM, Jeff Layton <jlayton@xxxxxxxxx> wrote: > On Tue, 23 Apr 2013 08:34:14 -0400 > Scott Lovenberg <scott.lovenberg@xxxxxxxxx> wrote: > >> On Mon, Apr 22, 2013 at 11:09 AM, Jeff Layton <jlayton@xxxxxxxxx> wrote: >> >> Ping. >> >> Any comments? >> >> >> > >> > I guess I don't quite understand why we need this patch. For NFS it's >> > pretty clear that we would since ':' has always served as the delimiter >> > between the hostname and the pathname. I am ok with the idea of adding additional support that looks more like what an NFS user is comfortable with. Ideally a user should be able to type very similar mount commands for cifs/smb3 and NFSv3/v4 etc. Scott, Can you give a list of an example of these - mounts with each of the IPv6 formats that work (and what doesn't work) along with the one you propose? >> > For CIFS, not so much... IPv6 addresses typically don't have '/' or '\\' >> > characters in them, so why do we need to wrap them in []? >> > >> > Note that this syntax doesn't follow what Windows does either, AFAIK. >> > There, you need something like this since UNC paths can't contain ':' >> > characters (and maybe also '%'): >> > >> > http://www.samba.org/~idra/code/nss-ipv6literal/README.html >> > >> > ...we have no such restriction under Linux though. >> > >> >> I liked this approach for two reasons. It handles a convention for >> IPv6 formatting that is used that we don't support and allows us to >> (in the future) parse an address from the path instead of having to >> use --ip=, which I feel is a bit cumbersome. Originally I was going >> to just put in support for something like "mount.cifs >> //IPv6Address/share /mnt/path" right away, but I wanted to get this >> part in first. >> > > Again, I'm confused...this sort of syntax seems to work just fine > already: > > # mount -t cifs //fecd::1/share /mnt/cifs -o sec=none,multiuser > > Also, --ip= is not a valid option in the longopts[] array in > mount.cifs (maybe you mean -o ip=). I still don't see what benefit we > receive from allowing someone specify addresses inside of square > brackets. > > -- > Jeff Layton <jlayton@xxxxxxxxx> -- 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
