lkml.org 
[lkml]   [2012]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 01/13] vfs: add i_op->open()
On Wed, 15 Aug 2012 13:21:50 -0400 "J. Bruce Fields" <bfields@fieldses.org>
wrote:

> On Wed, Aug 15, 2012 at 05:48:08PM +0200, Miklos Szeredi wrote:
> > From: Miklos Szeredi <mszeredi@suse.cz>
> >
> > Add a new inode operation i_op->open(). This is for stacked
>
> Shouldn't that "->open()" be "->dentry_open()" ?
>
> --b.
>
> > filesystems that want to return a struct file from a different
> > filesystem.
> >
> > Signed-off-by: Miklos Szeredi <mszeredi@suse.cz>
> > ---
> > Documentation/filesystems/Locking | 2 ++
> > Documentation/filesystems/vfs.txt | 7 +++++++
> > fs/namei.c | 9 ++++++---
> > fs/open.c | 23 +++++++++++++++++++++--
> > include/linux/fs.h | 2 ++
> > 5 files changed, 38 insertions(+), 5 deletions(-)
> >
> > diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking
> > index 0f103e3..d222b6a 100644
> > --- a/Documentation/filesystems/Locking
> > +++ b/Documentation/filesystems/Locking
> > @@ -64,6 +64,7 @@ prototypes:
> > int (*atomic_open)(struct inode *, struct dentry *,
> > struct file *, unsigned open_flag,
> > umode_t create_mode, int *opened);
> > + int (*dentry_open)(struct dentry *, struct file *, const struct cred *);
> >
> > locking rules:
> > all may block
> > @@ -92,6 +93,7 @@ removexattr: yes
> > fiemap: no
> > update_time: no
> > atomic_open: yes
> > +open: no
> >

and that last line should be
+dentry_open: no
??

NeilBrown
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2012-08-15 23:45    [W:0.118 / U:1.936 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site