Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors

To:	Dave Chinner <david@xxxxxxxxxxxxx>
Subject:	Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors
From:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date:	Tue, 29 Apr 2014 11:18:47 -0700
Cc:	xfs@xxxxxxxxxxx
Delivered-to:	xfs@xxxxxxxxxxx
In-reply-to:	<1398719099-19194-10-git-send-email-david@xxxxxxxxxxxxx>
References:	<1398719099-19194-1-git-send-email-david@xxxxxxxxxxxxx> <1398719099-19194-10-git-send-email-david@xxxxxxxxxxxxx>
User-agent:	Mutt/1.5.21 (2010-09-15)

On Tue, Apr 29, 2014 at 07:04:59AM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
> 
> Currently the attribute code will not detect and correct errors in
> the attribute tree. It also fails to validate the CRCs and headers
> on remote attribute blocks. Ensure that all the attribute blocks are
> CRC checked and that the processing functions understand the correct
> block formats for decoding.
> 
> Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>

Looks good,

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

<Prev in Thread]	Current Thread	[Next in Thread>
[PATCH 2/9] db: verify buffer on type change, (continued) [PATCH 2/9] db: verify buffer on type change, Dave Chinner [PATCH 5/9] repair: detect CRC errors in AG headers, Dave Chinner Re: [PATCH 5/9] repair: detect CRC errors in AG headers, Brian Foster Re: [PATCH 5/9] repair: detect CRC errors in AG headers, Christoph Hellwig [PATCH 6/9] repair: report AG btree verifier errors, Dave Chinner [PATCH 4/9] repair: detect and correct CRC errors in directory blocks, Dave Chinner [PATCH 1/9] db: don't claim unchecked CRCs are correct, Dave Chinner [PATCH 9/9] repair: detect and handle attribute tree CRC errors, Dave Chinner Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors, Brian Foster Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors, Dave Chinner Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors, Christoph Hellwig <= [PATCH 8/9] repair: handle remote symlink CRC errors, Dave Chinner Re: [PATCH 8/9] repair: handle remote symlink CRC errors, Brian Foster Re: [PATCH 8/9] repair: handle remote symlink CRC errors, Christoph Hellwig [PATCH 3/9] repair: ensure prefetched buffers have CRCs validated, Dave Chinner Re: [PATCH 3/9] repair: ensure prefetched buffers have CRCs validated, Brian Foster Re: [PATCH 3/9] repair: ensure prefetched buffers have CRCs validated, Christoph Hellwig [PATCH 7/9] repair: remove more dirv1 leftovers, Dave Chinner

Previous by Date:	Re: [PATCH 8/9] repair: handle remote symlink CRC errors, Christoph Hellwig
Next by Date:	Re: [PATCH] xfs: test for shut down fs in xfs_dir_fsync(), Dave Chinner
Previous by Thread:	Re: [PATCH 9/9] repair: detect and handle attribute tree CRC errors, Dave Chinner
Next by Thread:	[PATCH 8/9] repair: handle remote symlink CRC errors, Dave Chinner
Indexes:	[Date] [Thread] [Top] [All Lists]