On Wed, May 27, 2015 at 10:36:24AM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@xxxxxxxxxx>
>
> This results in BMBT corruption, as seen by this test:
....
> - if ((temp = (align_alen % extsz))) {
> - align_alen += extsz - temp;
> - }
> + while (align_alen > MAXEXTLEN)
> + align_alen -= extsz;
> + ASSERT(align_alen <= MAXEXTLEN);
FYI, this bit is the only difference in this version - it fixes the
test case that Brian pointed out that still didn't work properly.
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
