[Cluster-devel] [GFS2 PATCH] GFS2: Combine func meta_prep_new with its only caller

Bob Peterson <[email protected]>
Newsgroups com.redhat.cluster-devel
Message-ID <[email protected]>
Hi,

Before this patch function meta_prep_new was only called in one
place, function gfs2_meta_new which is only a couple lines long.
This patch combines them for readability.

Signed-off-by: Bob Peterson <[email protected]>
---
diff --git a/fs/gfs2/meta_io.c b/fs/gfs2/meta_io.c
index 663ffc1..bbd9411 100644
--- a/fs/gfs2/meta_io.c
+++ b/fs/gfs2/meta_io.c
@@ -158,18 +158,6 @@ struct buffer_head *gfs2_getbuf(struct gfs2_glock *gl, u64 blkno, int create)
 	return bh;
 }
 
-static void meta_prep_new(struct buffer_head *bh)
-{
-	struct gfs2_meta_header *mh = (struct gfs2_meta_header *)bh->b_data;
-
-	lock_buffer(bh);
-	clear_buffer_dirty(bh);
-	set_buffer_uptodate(bh);
-	unlock_buffer(bh);
-
-	mh->mh_magic = cpu_to_be32(GFS2_MAGIC);
-}
-
 /**
  * gfs2_meta_new - Get a block
  * @gl: The glock associated with this block
@@ -180,9 +168,15 @@ static void meta_prep_new(struct buffer_head *bh)
 
 struct buffer_head *gfs2_meta_new(struct gfs2_glock *gl, u64 blkno)
 {
-	struct buffer_head *bh;
-	bh = gfs2_getbuf(gl, blkno, CREATE);
-	meta_prep_new(bh);
+	struct buffer_head *bh = gfs2_getbuf(gl, blkno, CREATE);
+	struct gfs2_meta_header *mh = (struct gfs2_meta_header *)bh->b_data;
+
+	lock_buffer(bh);
+	clear_buffer_dirty(bh);
+	set_buffer_uptodate(bh);
+	unlock_buffer(bh);
+
+	mh->mh_magic = cpu_to_be32(GFS2_MAGIC);
 	return bh;
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.