summaryrefslogtreecommitdiffstats
path: root/fs/gfs2/glops.c
diff options
context:
space:
mode:
authorSteven Whitehouse <swhiteho@redhat.com>2006-09-04 11:41:31 -0400
committerSteven Whitehouse <swhiteho@redhat.com>2006-09-04 11:41:31 -0400
commit75d3b817a0b48425da921052955cc58f20bbab52 (patch)
tree7fb2e9bf18d794a590f43397ee56f3ce917a2233 /fs/gfs2/glops.c
parent31e77ac55f18db0ec1c724840927562ef3093ef6 (diff)
downloadop-kernel-dev-75d3b817a0b48425da921052955cc58f20bbab52.zip
op-kernel-dev-75d3b817a0b48425da921052955cc58f20bbab52.tar.gz
[GFS2] Tidy up bmap/inode code
As per Jan Engelhardt's third set of comments, this make various code style changes and moves the structures from format.h into super.c, which was the only place that format.h was actually used. Cc: Jan Engelhardt <jengelh@linux01.gwdg.de> Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/gfs2/glops.c')
-rw-r--r--fs/gfs2/glops.c12
1 files changed, 5 insertions, 7 deletions
diff --git a/fs/gfs2/glops.c b/fs/gfs2/glops.c
index 8e1d8ee..d3e7b08 100644
--- a/fs/gfs2/glops.c
+++ b/fs/gfs2/glops.c
@@ -234,7 +234,6 @@ static void inode_go_sync(struct gfs2_glock *gl, int flags)
if (flags & DIO_RELEASE)
gfs2_ail_empty_gl(gl);
}
-
}
/**
@@ -324,12 +323,11 @@ static void inode_go_unlock(struct gfs2_holder *gh)
struct gfs2_glock *gl = gh->gh_gl;
struct gfs2_inode *ip = gl->gl_object;
- if (ip) {
- if (test_bit(GLF_DIRTY, &gl->gl_flags))
- gfs2_inode_attr_in(ip);
-
- gfs2_meta_cache_flush(ip);
- }
+ if (ip == NULL)
+ return;
+ if (test_bit(GLF_DIRTY, &gl->gl_flags))
+ gfs2_inode_attr_in(ip);
+ gfs2_meta_cache_flush(ip);
}
/**
OpenPOWER on IntegriCloud