Attachment 315289 Details for Bug 460327 – Patch to fix the problem

[patch] Patch to fix the problem

bz460327.patch (text/plain), 1.15 KB, created by Robert Peterson on 2008-08-28 19:42:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Robert Peterson

Created: 2008-08-28 19:42:24 UTC

Size: 1.15 KB

patch

obsolete

> gfs2/fsck/pass5.c |   14 ++++++++++++--
> 1 files changed, 12 insertions(+), 2 deletions(-)
>
>diff --git a/gfs2/fsck/pass5.c b/gfs2/fsck/pass5.c
>index bc4b859..030a82c 100644
>--- a/gfs2/fsck/pass5.c
>+++ b/gfs2/fsck/pass5.c
>@@ -88,8 +88,18 @@ int check_block_status(struct gfs2_sbd *sbp, char *buffer, unsigned int buflen,
> 
> 		block_status = convert_mark(&q, count);
> 
>-		if (rg_status != block_status) {
>-			const char *blockstatus[] = {"Free", "Data", "Invalid", "inode"};
>+		/* If one node opens a file and another node deletes it, we
>+		   may be left with a block that appears to be "unlinked" in
>+		   the bitmap, but nothing links to it. This is a valid case
>+		   and should be cleaned up by the file system eventually.
>+		   So we ignore it. */
>+		if (rg_status == GFS2_BLKST_UNLINKED &&
>+		    block_status == GFS2_BLKST_FREE) {
>+			log_warn("Unlinked block found at block %"
>+				 PRIu64" (0x%" PRIx64 "), left unchanged.\n",
>+				 block, block);
>+		} else if (rg_status != block_status) {
>+			const char *blockstatus[] = {"Free", "Data", "Unlinked", "inode"};
> 
> 			log_err("Ondisk and fsck bitmaps differ at"
> 					" block %"PRIu64" (0x%" PRIx64 ") \n", block, block);

Actions: View | Diff

Attachments on bug 460327: 315289