Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 01 Aug 2012 18:15:20 +0400
From:      Andrey Zonov <andrey@zonov.org>
To:        freebsd-fs@freebsd.org
Subject:   remove check for SUJ_MAX from fsck_ffs(8)
Message-ID:  <501939F8.7070006@zonov.org>

next in thread | raw e-mail | index | archive | help
Hi,

It's possible to set any size for SU journal not less than 4Mb with 
tunefs(8), but fsck_ffs(8) reports that it could not check such FS 
because "Invalid size".  It seems that fsck_ffs should not check maximum 
size of journal.

I suggest that patch:

Index: sbin/fsck_ffs/suj.c
===================================================================
--- sbin/fsck_ffs/suj.c	(revision 238945)
+++ sbin/fsck_ffs/suj.c	(working copy)
@@ -2383,7 +2383,7 @@ suj_verifyino(union dinode *ip)
  		return (-1);
  	}

-	if (DIP(ip, di_size) < SUJ_MIN || DIP(ip, di_size) > SUJ_MAX) {
+	if (DIP(ip, di_size) < SUJ_MIN) {
  		printf("Invalid size %jd for journal inode %d\n",
  		    DIP(ip, di_size), sujino);
  		return (-1);

What do you think?

-- 
Andrey Zonov



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?501939F8.7070006>