lkml.org 
[lkml]   [2008]   [Apr]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch 01/10] vfs: add path_create() and path_mknod()
From
Date
> > R/o bind mounts require operations which modify the filesystem to be
> > wrapped in mnt_want_write()/mnt_drop_write(). Create helpers which do
> > this, so callers won't need to bother, and more importantly, cannot
> > forget! Call these path_*, analogous to vfs_*. Where there are no
> > callers of vfs_* left, make them static.
> >
> > This series is a cleanup, as well as fixing several places (mostly in
> > nfsd) where mnt_{want,drop}_write() were missing.
> >
> > It will also help with merging You Know What(*) security module, which
> > needs to know the path within the namespace, and not just within the
> > filesystem. These helpers will allow the security hooks to be in a
> > common place, and need not be repeated in all callers.
>
> Rot.
>
> Places in nfsd must be fixed, LSM hooks *moved* *to* *callers*. And
> really, by that point I absolutely do not give a damn for these clowns.
> "Help with merging" implies that they can't be arsed to do _anything_
> with their code. Just as they could not be arsed to react to any
> comments (including civil ones) in any manner except "wait for a month
> and repost without changes". Sod them.

Al, calm down. I've been asked to help with merging this code, and if
there are concerns with it, I'll help fix them. If you had bad
experience with it in the past, I'm sorry. But let that not make this
any harder that in need to be.

> And no, "make static where all (two of) current callers have vfsmount"
> is non-starter. path_...() is (at most) a convenience helper, not a
> fundamental interface - simply because new callers should not be
> forced into inventing a fake vfsmount just to use that.

It is an interface with at least 3 callers at the moment: syscalls,
nfsd and ecryptfs and unionfs in the future. It is an interface
because all external accesses to the filesystem *are* done through the
namespace and not directly on filesystem internals.

Such direct access would be conceivable, but I don't think we should
base the design on theoretically possible uses. If those uses appear,
we can change the interface to fit that.

This "move everything to callers" thing is wrong because it just
results in bloat and bugs, none of which is desirable in the kernel.

Miklos


\
 
 \ /
  Last update: 2008-04-02 23:15    [W:0.111 / U:1.264 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site