]> git.karo-electronics.de Git - linux-beck.git/commit
GFS2: Add atomic_open support
authorSteven Whitehouse <swhiteho@redhat.com>
Fri, 14 Jun 2013 10:17:15 +0000 (11:17 +0100)
committerSteven Whitehouse <swhiteho@redhat.com>
Fri, 14 Jun 2013 10:17:15 +0000 (11:17 +0100)
commit6d4ade986f9c8df31e68fd30643997f79cc5a5f8
treef0758a7a9b008d0bd3665234a5074e3cf6f4d455
parent5a00f3cc978be45b9d2597851bedaa40630bc597
GFS2: Add atomic_open support

I've restricted atomic_open to only operate on regular files, although
I still don't understand why atomic_open should not be possible also for
directories on GFS2. That can always be added in later though, if it
makes sense.

The ->atomic_open function can be passed negative dentries, which
in most cases means either ENOENT (->lookup) or a call to d_instantiate
(->create). In the GFS2 case though, we need to actually perform the
look up, since we do not know whether there has been a new inode created
on another node. The look up calls d_splice_alias which then tries to
rehash the dentry - so the solution here is to simply check for that
in d_splice_alias. The same issue is likely to affect any other cluster
filesystem implementing ->atomic_open

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: "J. Bruce Fields" <bfields fieldses org>
Cc: Jeff Layton <jlayton@redhat.com>
fs/dcache.c
fs/gfs2/file.c
fs/gfs2/inode.c
fs/gfs2/inode.h