Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Jun 2012 02:24:26 +0000
From:      "Bjoern A. Zeeb" <bzeeb-lists@lists.zabbadoz.net>
To:        Konstantin Belousov <kib@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r236976 - head/sbin/fsck_ffs
Message-ID:  <59C39E5A-C63D-490C-B260-E00A4D90E213@lists.zabbadoz.net>
In-Reply-To: <201206122137.q5CLbReK077123@svn.freebsd.org>
References:  <201206122137.q5CLbReK077123@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On 12. Jun 2012, at 21:37 , Konstantin Belousov wrote:

> Author: kib
> Date: Tue Jun 12 21:37:27 2012
> New Revision: 236976
> URL: http://svn.freebsd.org/changeset/base/236976
>=20
> Log:
>  For incompleted block allocations or frees, the inode block count =
usage
>  must be recalculated. The blk_check pass of suj checker explicitely =
marks
>  inodes which owned such blocks as needing block count adjustment. But
>  ino_adjblks() is only called by cg_trunc pass, which is performed =
before
>  blk_check. As result, the block use count for such inodes is left =
wrong.
>  This causes full fsck run after journaled run to still find =
inconsistencies
>  like 'INCORRECT BLOCK COUNT I=3D14557 (328 should be 0)' in phase 1.
>=20
>  Fix this issue by running additional adj_blk pass after blk_check, =
which
>  updates the field.
>=20
>  Reviewed by:	jeff, mckusick

Thanks a lot!

/bz

--=20
Bjoern A. Zeeb                                 You have to have visions!
   It does not matter how good you are. It matters what good you do!




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?59C39E5A-C63D-490C-B260-E00A4D90E213>