RE: [PATCH] xfs: Wrapped journal record corruption on read at recovery

["Alex Elder" <aelder@xxxxxxx>]

Alex Elder wrote:
> Alex Elder wrote:
>> Andy Poling wrote:
>>> On Wed, 14 Oct 2009, Christoph Hellwig wrote:
>>>>> It seems like the more elegant approach would be to set offset before the
>>>>> first read, and then update it if the first read takes place (in case it 
>>>>> was
>>>>> unaligned).  That also gets rid of bufaddr, and seems like it might read 
>>>>> ...
> 
> 
> Andy, can you tell me the log sector
> size of a file system that exhibits
> this failure condition?
> 
> You can just send the output of:
>     xfs_info /dev/<whatever>
> if you like.  I have a suspicion
> that there may still be a problem,
> even with your proposed fix, and
> would like to rule that possibility
> out.  Basically, if log sectsz is
> is > 1024 your fix is probably OK.


Nevermind, I think all is well with your patch.  I'll respond to
your other posting with my positive review shortly.

                                        -Alex

> Thanks.
> 
>                                       -Alex
> 
> _______________________________________________
> xfs mailing list
> xfs@xxxxxxxxxxx
> http://oss.sgi.com/mailman/listinfo/xfs