Re: [2.6 patch] remove smbfs

From: Steve French
Date: Wed Jan 30 2008 - 21:58:56 EST


On Jan 30, 2008 7:34 PM, Steve Langasek <vorlon@xxxxxxxxxx> wrote:
> On Thu, Jan 31, 2008 at 02:47:17AM +0200, Adrian Bunk wrote:
> > > smbfs has the unfortunate quality of momentum. A lot of users aren't
> > > aware of CIFS at all since smbfs basically does what they need it to
> > > do. Some extra warning for those users would be nice.
>
> > And many users will start whining loudly that the not deprecated driver
> > (in this case cifs) has this or that bug not before the patch to finally
> > remove the deprecated feature got applied or at least posted.
>
> > And will demand that it therefore does not get removed.
>
> We've had about 3 of these in Debian since deciding to cut it from the
> upcoming release. (The kicker for us was the samba security update that
> wasn't tested with smbfs as a client.) The key regressions of interest
> relative to smbfs seem to be:
>
> - lack of DFS support
> - lack of netbios name resolution for UNC share names

> The former seems to be a kernel issue whose resolution is in progress, and I
> think the latter would have to be addressed in the userspace mount tools?

DFS support has never been in any smb client except Windows and the
user space smbclient (but only for about a year) - it has been in
Samba server for a while. The final piece necessary for DFS support
has not been merged but is expected in the next few weeks - Christoph
and others noticed some problems during the review of the patch series
that I would like to address including rewriting CIFSDFSGetReferral.

lack of automatic netbios name resolution for the server name portion
of the UNC name was discussed in earlier posts - but can be done in
mount.cifs. More intricate techniques (probing via astat for netbios
name discovery - would require more kernel code)



--
Thanks,

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