linux-kernel - Re: Fw: [2.6 patch] make cifs_dfs_automount

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20080416040818.GB27459@ZenIV.linux.org.uk>
Date:	Wed, 16 Apr 2008 05:08:19 +0100
From:	Al Viro <viro@...IV.linux.org.uk>
To:	Steve French <smfrench@...il.com>
Cc:	Adrian Bunk <bunk@...nel.org>, lkml <linux-kernel@...r.kernel.org>,
	"Q (Igor Mammedov)" <qwerty0987654321@...l.ru>
Subject: Re: Fw: [2.6 patch] make cifs_dfs_automount_list static

On Tue, Apr 15, 2008 at 10:58:06PM -0500, Steve French wrote:
> Merged into cifs-2.6.git tree
> 
> > Adrian Bunk <bunk@...nel.org>
> > 04/14/2008 01:13 PM
> >
> > To
> > Igor Mammedov <niallain@...il.com>, Steven French/Austin/IBM@...US
> > cc
> > linux-cifs-client@...ts.samba.org, linux-kernel@...r.kernel.org
> > Subject
> > [2.6 patch] make cifs_dfs_automount_list static
> >
> >
> > This patch makes the needlessly global cifs_dfs_automount_list static.

This only hides the real problem, namely that logics in the user of said
list is FUBAR.

What kind of eviction policy does cifs want?  "Try to kick all such
mounts out whenever somebody does umount(2) on any mounted cifs instance"?
That's bloody odd, to put it mildly...  "Don't kick them out until
somebody tries to umount cifs instance they are mounted on"?  That's
done automatically from VFS these days (and should've been done that
way as soon as that kind of automounts had been introduced).  "Done
frequently enough"?  Then you want timer, like e.g. NFS does...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/