lkml.org 
[lkml]   [2008]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 00/13] vfs: add helpers to check r/o bind mounts
In message <20080424113950.818688067@szeredi.hu>, Miklos Szeredi writes:
> Patches 1-3 are cleanups/bugfixes preceding the main series. The
> description is in 4/13. This is against latest git.
>
> <rant>
>
> R/O bind mounts patches have been reviewed numerous times. Still a
> relatively large number of places remained where checking for R/O
> mounts was missed.
>
> Then I did this series, which basically guarantees, that that cannot
> happen. Al rejected it, and rather fixed some of the remaining
> places. He still missed several, which sort of proves my point.
>
> I think it's totally pointless to continue trying to fix the symptoms
> instead of getting at the root of the problem.
>
> I know that VFS interfaces are a sensitive question, but it would be
> nice it we could have some sanity back in this discussion.

And I'll reiterate my desire (see http://lkml.org/lkml/2008/4/8/394) for
stackable file systems to know as little as possible about vfsmounts (or
their derivatives ala struct path). If I have to call helpers which require
vfsmounts, but similar objects don't get passed to me somehow, then I have
to maintain vfsmounts on my own.

Sure, unionfs already does some of it now (and some of it can be recoded
more cleanly), but I'd like to get rid of much of it if I could -- these new
path_* helpers make me have to maintain vfsmounts even moreso than before.

> </rant>
>
> Thanks,
> Miklos

Erez.


\
 
 \ /
  Last update: 2008-04-24 18:55    [W:0.434 / U:0.392 seconds]
©2003-2011 Jasper Spaans. Advertise on this site