xfstests 258: Test xfs fs creation with fs size close to 4 TB

Alex Elder aelder at sgi.com
Mon Sep 26 07:18:46 CDT 2011


On Mon, 2011-09-26 at 07:44 -0400, Christoph Hellwig wrote:
> This one actually fails for me in a 32-bit userspace VM:

This is my fault, and you busted me doing something I
should not have.

It doesn't matter in this case, but there is a window in
which "testfile" is undefined but gets referenced in
_cleanup() (which could get called due to interrupt).
I "trivially" moved the definition to fix that oversight
and as a result broke the test.

I won't do that any more without taking proper credit
when I commit.

Sorry Boris (and all).

Your fix looks fine, Christoph.

Reviewed-by: Alex Elder <aelder at sgi.com>





More information about the xfs mailing list