Re: [PATCH 01/13] vfs: add i_op->open()

2012-08-16 Thread Miklos Szeredi
NeilBrown writes: > On Wed, 15 Aug 2012 13:21:50 -0400 "J. Bruce Fields" > wrote: > >> On Wed, Aug 15, 2012 at 05:48:08PM +0200, Miklos Szeredi wrote: >> > From: Miklos Szeredi >> > >> > Add a new inode operation i_op->open(). This is for stacked >> >> Shouldn't that "->open()" be "->dentry_

Re: [PATCH 01/13] vfs: add i_op->open()

2012-08-15 Thread NeilBrown
On Wed, 15 Aug 2012 13:21:50 -0400 "J. Bruce Fields" wrote: > On Wed, Aug 15, 2012 at 05:48:08PM +0200, Miklos Szeredi wrote: > > From: Miklos Szeredi > > > > Add a new inode operation i_op->open(). This is for stacked > > Shouldn't that "->open()" be "->dentry_open()" ? > > --b. > > > file

Re: [PATCH 01/13] vfs: add i_op->open()

2012-08-15 Thread J. Bruce Fields
On Wed, Aug 15, 2012 at 05:48:08PM +0200, Miklos Szeredi wrote: > From: Miklos Szeredi > > 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. > > Sign

[PATCH 01/13] vfs: add i_op->open()

2012-08-15 Thread Miklos Szeredi
From: Miklos Szeredi Add a new inode operation i_op->open(). This is for stacked filesystems that want to return a struct file from a different filesystem. Signed-off-by: Miklos Szeredi --- Documentation/filesystems/Locking |2 ++ Documentation/filesystems/vfs.txt |7 +++ fs/namei