Attachment 291028 Details for Bug 338651 – Possible fix

[patch] Possible fix

gronk (text/plain), 1018 bytes, created by Robert Peterson on 2008-01-08 00:30:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Peterson

Created: 2008-01-08 00:30:12 UTC

Size: 1018 bytes

patch

obsolete

>--- ../../../../kernel-2.6.18-63/linux-2.6.18.i686/fs/gfs2/lops.c	2008-01-07 12:14:56.000000000 -0600
>+++ lops.c	2008-01-07 16:55:35.000000000 -0600
>@@ -386,6 +386,7 @@ static void revoke_lo_before_commit(stru
> 		return;
> 
> 	bh = gfs2_get_log_desc(sdp, GFS2_LOG_DESC_REVOKE);
>+	gfs2_log_lock(sdp);
> 	ld = bh_log_desc(bh);
> 	ld->ld_length = cpu_to_be32(gfs2_struct2blk(sdp, sdp->sd_log_num_revoke,
> 						    sizeof(u64)));
>@@ -398,9 +399,11 @@ static void revoke_lo_before_commit(stru
> 		sdp->sd_log_num_revoke--;
> 
> 		if (offset + sizeof(u64) > sdp->sd_sb.sb_bsize) {
>+			gfs2_log_unlock(sdp);
> 			submit_bh(WRITE, bh);
> 
> 			bh = gfs2_log_get_buf(sdp);
>+			gfs2_log_lock(sdp);
> 			mh = (struct gfs2_meta_header *)bh->b_data;
> 			mh->mh_magic = cpu_to_be32(GFS2_MAGIC);
> 			mh->mh_type = cpu_to_be32(GFS2_METATYPE_LB);
>@@ -414,6 +417,7 @@ static void revoke_lo_before_commit(stru
> 		offset += sizeof(u64);
> 	}
> 	gfs2_assert_withdraw(sdp, !sdp->sd_log_num_revoke);
>+	gfs2_log_unlock(sdp);
> 
> 	submit_bh(WRITE, bh);
> }

Actions: View | Diff

Attachments on bug 338651: 231441 | 291028