Messages in this thread | | | Date | Wed, 16 Apr 2008 05:08:19 +0100 | From | Al Viro <> | 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@kernel.org> > > 04/14/2008 01:13 PM > > > > To > > Igor Mammedov <niallain@gmail.com>, Steven French/Austin/IBM@IBMUS > > cc > > linux-cifs-client@lists.samba.org, linux-kernel@vger.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...
| |