Attachment 147922 Details for Bug 222308 – Final patch to fix the problem

[patch] Final patch to fix the problem

222308.patch.try4 (text/plain), 7.06 KB, created by Robert Peterson on 2007-02-12 18:53:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Peterson

Created: 2007-02-12 18:53:06 UTC

Size: 7.06 KB

patch

obsolete

>Index: fs_ops.c
>===================================================================
>RCS file: /cvs/cluster/cluster/gfs2/libgfs2/fs_ops.c,v
>retrieving revision 1.4
>diff -w -u -p -p -u -r1.4 fs_ops.c
>--- fs_ops.c	12 Jun 2006 20:38:27 -0000	1.4
>+++ fs_ops.c	12 Feb 2007 18:45:07 -0000
>@@ -209,7 +209,7 @@ static void unstuff_dinode(struct gfs2_i
> 	ip->i_di.di_height = 1;
> }
> 
>-static unsigned int calc_tree_height(struct gfs2_inode *ip, uint64_t size)
>+unsigned int calc_tree_height(struct gfs2_inode *ip, uint64_t size)
> {
> 	struct gfs2_sbd *sdp = ip->i_sbd;
> 	uint64_t *arr;
>@@ -233,7 +233,7 @@ static unsigned int calc_tree_height(str
> 	return height;
> }
> 
>-static void build_height(struct gfs2_inode *ip, int height)
>+void build_height(struct gfs2_inode *ip, int height)
> {
> 	struct gfs2_sbd *sdp = ip->i_sbd;
> 	struct gfs2_buffer_head *bh;
>@@ -335,7 +335,7 @@ static void lookup_block(struct gfs2_ino
> }
> 
> void block_map(struct gfs2_inode *ip, uint64_t lblock, int *new,
>-			   uint64_t *dblock, uint32_t *extlen)
>+			   uint64_t *dblock, uint32_t *extlen, int prealloc)
> {
> 	struct gfs2_sbd *sdp = ip->i_sbd;
> 	struct gfs2_buffer_head *bh;
>@@ -395,6 +395,7 @@ void block_map(struct gfs2_inode *ip, ui
> 			bh = bread(sdp, *dblock);
> 	}
> 
>+	if (!prealloc)
> 	lookup_block(ip, bh, end_of_metadata, mp, create, new, dblock);
> 
> 	if (extlen && *dblock) {
>@@ -480,7 +481,7 @@ int gfs2_readi(struct gfs2_inode *ip, vo
> 			amount = sdp->bsize - o;
> 
> 		if (!extlen)
>-			block_map(ip, lblock, &not_new, &dblock, &extlen);
>+			block_map(ip, lblock, &not_new, &dblock, &extlen, FALSE);
> 
> 		if (dblock) {
> 			bh = bread(sdp, dblock);
>@@ -552,7 +553,7 @@ int gfs2_writei(struct gfs2_inode *ip, v
> 
> 		if (!extlen) {
> 			new = TRUE;
>-			block_map(ip, lblock, &new, &dblock, &extlen);
>+			block_map(ip, lblock, &new, &dblock, &extlen, FALSE);
> 		}
> 
> 		if (new) {
>@@ -583,7 +584,8 @@ int gfs2_writei(struct gfs2_inode *ip, v
> 	return copied;
> }
> 
>-struct gfs2_buffer_head *get_file_buf(struct gfs2_inode *ip, uint64_t lbn)
>+struct gfs2_buffer_head *get_file_buf(struct gfs2_inode *ip, uint64_t lbn,
>+									  int prealloc)
> {
> 	struct gfs2_sbd *sdp = ip->i_sbd;
> 	uint64_t dbn;
>@@ -592,11 +594,11 @@ struct gfs2_buffer_head *get_file_buf(st
> 	if (inode_is_stuffed(ip))
> 		unstuff_dinode(ip);
> 
>-	block_map(ip, lbn, &new, &dbn, NULL);
>+	block_map(ip, lbn, &new, &dbn, NULL, prealloc);
> 	if (!dbn)
> 		die("get_file_buf\n");
> 
>-	if (new &&
>+	if (!prealloc && new &&
> 	    ip->i_di.di_size < (lbn + 1) << sdp->bsize_shift)
> 		ip->i_di.di_size = (lbn + 1) << sdp->bsize_shift;
> 
>Index: libgfs2.h
>===================================================================
>RCS file: /cvs/cluster/cluster/gfs2/libgfs2/libgfs2.h,v
>retrieving revision 1.9
>diff -w -u -p -p -u -r1.9 libgfs2.h
>--- libgfs2.h	23 Jan 2007 19:23:06 -0000	1.9
>+++ libgfs2.h	12 Feb 2007 18:45:07 -0000
>@@ -391,7 +391,8 @@ int gfs2_readi(struct gfs2_inode *ip, vo
> 			   uint64_t offset, unsigned int size);
> int gfs2_writei(struct gfs2_inode *ip, void *buf,
> 				uint64_t offset, unsigned int size);
>-struct gfs2_buffer_head *get_file_buf(struct gfs2_inode *ip, uint64_t lbn);
>+struct gfs2_buffer_head *get_file_buf(struct gfs2_inode *ip, uint64_t lbn,
>+									  int prealloc);
> struct gfs2_buffer_head *init_dinode(struct gfs2_sbd *sdp,
> 									 struct gfs2_inum *inum,
> 									 unsigned int mode, uint32_t flags,
>@@ -408,7 +409,7 @@ void dir_add(struct gfs2_inode *dip, cha
> int gfs2_dirent_del(struct gfs2_inode *dip, struct gfs2_buffer_head *bh,
> 					const char *filename, int filename_len);
> void block_map(struct gfs2_inode *ip, uint64_t lblock, int *new,
>-			   uint64_t *dblock, uint32_t *extlen);
>+			   uint64_t *dblock, uint32_t *extlen, int prealloc);
> void gfs2_get_leaf_nr(struct gfs2_inode *dip, uint32_t index,
> 					  uint64_t *leaf_out);
> int gfs2_freedi(struct gfs2_sbd *sdp, uint64_t block);
>@@ -418,6 +419,10 @@ int gfs2_dirent_first(struct gfs2_inode 
> 					  struct gfs2_dirent **dent);
> int gfs2_dirent_next(struct gfs2_inode *dip, struct gfs2_buffer_head *bh,
> 					 struct gfs2_dirent **dent);
>+void build_height(struct gfs2_inode *ip, int height);
>+unsigned int calc_tree_height(struct gfs2_inode *ip, uint64_t size);
>+void write_journal(struct gfs2_sbd *sdp, struct gfs2_inode *ip, unsigned int j,
>+				   unsigned int blocks);
> 
> /**
>  * device_size - figure out a device's size
>Index: structures.c
>===================================================================
>RCS file: /cvs/cluster/cluster/gfs2/libgfs2/structures.c,v
>retrieving revision 1.6
>diff -w -u -p -p -u -r1.6 structures.c
>--- structures.c	22 Nov 2006 18:39:38 -0000	1.6
>+++ structures.c	12 Feb 2007 18:45:07 -0000
>@@ -83,20 +83,19 @@ build_sb(struct gfs2_sbd *sdp)
> 	}
> }
> 
>-void build_journal(struct gfs2_inode *jindex, unsigned int j)
>+void write_journal(struct gfs2_sbd *sdp, struct gfs2_inode *ip, unsigned int j,
>+				   unsigned int blocks)
> {
>-	struct gfs2_sbd *sdp = jindex->i_sbd;
>-	char name[256];
>-	struct gfs2_inode *ip;
> 	struct gfs2_log_header lh;
>-	unsigned int blocks = sdp->jsize << 20 >> sdp->bsize_shift;
> 	unsigned int x;
> 	uint64_t seq = RANDOM(blocks);
> 	uint32_t hash;
>+	unsigned int height;
> 
>-	sprintf(name, "journal%u", j);
>-	ip = createi(jindex, name, S_IFREG | 0600,
>-		     GFS2_DIF_SYSTEM);
>+	/* Build the height up so our journal blocks will be contiguous and */
>+	/* not broken up by indirect block pages.                           */
>+	height = calc_tree_height(ip, (blocks + 1) * sdp->bsize);
>+	build_height(ip, height);
> 
> 	memset(&lh, 0, sizeof(struct gfs2_log_header));
> 	lh.lh_header.mh_magic = GFS2_MAGIC;
>@@ -105,9 +104,15 @@ void build_journal(struct gfs2_inode *ji
> 	lh.lh_flags = GFS2_LOG_HEAD_UNMOUNT;
> 
> 	for (x = 0; x < blocks; x++) {
>-		struct gfs2_buffer_head *bh = get_file_buf(ip, ip->i_di.di_size >> sdp->bsize_shift);
>+		struct gfs2_buffer_head *bh = get_file_buf(ip, x, TRUE);
> 		if (!bh)
>-			die("build_journals\n");
>+			die("write_journal\n");
>+		brelse(bh, updated);
>+	}
>+	for (x = 0; x < blocks; x++) {
>+		struct gfs2_buffer_head *bh = get_file_buf(ip, x, FALSE);
>+		if (!bh)
>+			die("write_journal\n");
> 
> 		lh.lh_sequence = seq;
> 		lh.lh_blkno = x;
>@@ -125,8 +130,6 @@ void build_journal(struct gfs2_inode *ji
> 		printf("\nJournal %u:\n", j);
> 		gfs2_dinode_print(&ip->i_di);
> 	}
>-
>-	inode_put(ip, updated);
> }
> 
> void
>@@ -138,8 +141,15 @@ build_jindex(struct gfs2_sbd *sdp)
> 	jindex = createi(sdp->master_dir, "jindex", S_IFDIR | 0700,
> 			 GFS2_DIF_SYSTEM);
> 
>-	for (j = 0; j < sdp->md.journals; j++)
>-		build_journal(jindex, j);
>+	for (j = 0; j < sdp->md.journals; j++) {
>+		char name[256];
>+		struct gfs2_inode *ip;
>+
>+		sprintf(name, "journal%u", j);
>+		ip = createi(jindex, name, S_IFREG | 0600, GFS2_DIF_SYSTEM);
>+		write_journal(sdp, ip, j, sdp->jsize << 20 >> sdp->bsize_shift);
>+		inode_put(ip, updated);
>+	}
> 
> 	if (sdp->debug) {
> 		printf("\nJindex:\n");
>@@ -209,7 +219,7 @@ build_quota_change(struct gfs2_inode *pe
> 		     GFS2_DIF_SYSTEM);
> 
> 	for (x = 0; x < blocks; x++) {
>-		struct gfs2_buffer_head *bh = get_file_buf(ip, ip->i_di.di_size >> sdp->bsize_shift);
>+		struct gfs2_buffer_head *bh = get_file_buf(ip, ip->i_di.di_size >> sdp->bsize_shift, FALSE);
> 		if (!bh)
> 			die("build_quota_change\n");
>

Actions: View | Diff

Attachments on bug 222308: 147527 | 147620 | 147922