From 5a00f3cc978be45b9d2597851bedaa40630bc597 Mon Sep 17 00:00:00 2001 From: Steven Whitehouse Date: Tue, 11 Jun 2013 13:45:29 +0100 Subject: GFS2: Only do one directory search on create Creation of a new inode requires a directory search in order to ensure that we are not trying to create an inode with the same name as an existing one. This was hidden away inside the create_ok() function. In the case that there was an existing inode, and a lookup can be substituted for a create (which is the case with regular files when the O_EXCL flag is not in use) then we were doing a second lookup in order to return the inode. This patch merges these two lookups into one. This can be done by passing a flag to gfs2_dir_search() to tell it to just return -EEXIST in the cases where we don't actually want to look up the inode. Signed-off-by: Steven Whitehouse --- fs/gfs2/dir.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'fs/gfs2/dir.h') diff --git a/fs/gfs2/dir.h b/fs/gfs2/dir.h index 98c960beab35..d3f273870b49 100644 --- a/fs/gfs2/dir.h +++ b/fs/gfs2/dir.h @@ -18,7 +18,8 @@ struct gfs2_inode; struct gfs2_inum; extern struct inode *gfs2_dir_search(struct inode *dir, - const struct qstr *filename); + const struct qstr *filename, + bool fail_on_exist); extern int gfs2_dir_check(struct inode *dir, const struct qstr *filename, const struct gfs2_inode *ip); extern int gfs2_dir_add(struct inode *inode, const struct qstr *filename, -- cgit v1.2.3-59-g8ed1b