Re: [PATCH v2] repair: update extent count after zapping duplicate blocks

Mark Tinguely
2012-02-02T18:19:20Z

On 02/02/12 06:39, Christoph Hellwig wrote:
> When we find a duplicate extent in an extern format inode we do not zap
> the whole inode, but just truncate it to the point where the duplicate
> extent was found. But the current code only updates di_nblocks for the
> new size, but no di_nextents/di_anextents. In most cases this isn't noticed,
> but when moving such an inode to the lost+found directoy the consistency
> check in xfs_iformat trips over it. Fix this by updating the on-disk
> extent count as part of the inode repair.

Looks good.

Reviewed-by: Mark Tinguely <tinguely@sgi.com>