svn commit: r236976 - head/sbin/fsck_ffs

[Konstantin Belousov]

On Sat, Jun 16, 2012 at 11:41:20AM +0200, Tijl Coosemans wrote:
> On 12-06-2012 23: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
> > 
> > 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=14557 (328 should be 0)' in phase 1.
> >   
> >   Fix this issue by running additional adj_blk pass after blk_check, which
> >   updates the field.
> >   
> >   Reviewed by:	jeff, mckusick
> >   MFC after:	1 week
> 
> Thanks! This probably fixes http://www.freebsd.org/cgi/query-pr.cgi?pr=165950

There is more in the PR then just 'INCORRECT BLOCK COUNT' inconsistency.
Somebody will look at the remaining issues eventually.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20120616/92d3b7af/attachment.pgp