On Thu, Feb 24, 2011 at 04:17:22PM -0600, Eric Sandeen wrote:
> Chris Mason pointed out that some filesystems were not doing
> the right thing on fiemap, in the face of delalloc extents.
>
> Because test 225 ran with FIEMAP_FLAG_SYNC only, this didn't
> get caught. Add a runtime option, and run it both ways.
>
> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
> ---
>
> Whoops, V2: actually call w/ the sync flag.
>
> diff --git a/225 b/225
> index 6e879a4..6b203e6 100755
> --- a/225
> +++ b/225
> @@ -60,7 +60,10 @@ _cleanup()
> }
> trap "_cleanup; exit \$status" 0 1 2 3 15
>
> -echo "fiemap run without preallocation"
> +echo "fiemap run without preallocation, with sync"
> +$here/src/fiemap-tester -q -S -p 0 -r 200 $fiemapfile 2>&1 | tee $fiemaplog
> +
> +echo "fiemap run without preallocation or sync"
> $here/src/fiemap-tester -q -p 0 -r 200 $fiemapfile 2>&1 | tee $fiemaplog
>
> if grep -q "Operation not supported" $fiemaplog; then
> diff --git a/225.out b/225.out
> index 7bc9312..b4f6477 100644
> --- a/225.out
> +++ b/225.out
> @@ -1,2 +1,3 @@
> QA output created by 225
> -fiemap run without preallocation
> +fiemap run without preallocation, with sync
> +fiemap run without preallocation or sync
> diff --git a/src/fiemap-tester.c b/src/fiemap-tester.c
> index 53d7a64..6320580 100644
> --- a/src/fiemap-tester.c
> +++ b/src/fiemap-tester.c
> @@ -418,7 +418,7 @@ static int query_fiemap_count(int fd, int blocks, int
> blocksize)
> }
>
> static int
> -compare_fiemap_and_map(int fd, char *map, int blocks, int blocksize)
> +compare_fiemap_and_map(int fd, char *map, int blocks, int blocksize, int
> syncfile)
> {
> struct fiemap *fiemap;
> char *fiebuf;
> @@ -446,7 +446,7 @@ compare_fiemap_and_map(int fd, char *map, int blocks, int
> blocksize)
> last_data = i;
> }
>
> - fiemap->fm_flags = FIEMAP_FLAG_SYNC;
> + fiemap->fm_flags = syncfile ? FIEMAP_FLAG_SYNC : 0;
> fiemap->fm_extent_count = blocks_to_map;
> fiemap->fm_mapped_extents = 0;
>
> @@ -519,9 +519,10 @@ main(int argc, char **argv)
> int blocks = 0; /* the number of blocks to generate */
> int maxblocks = 0; /* max # of blocks to create */
> int prealloc = 1; /* whether or not to do preallocation */
> + int syncfile = 0; /* whether fiemap should sync file first */
> int seed = 1;
>
> - while ((opt = getopt(argc, argv, "m:r:s:p:q")) != -1) {
> + while ((opt = getopt(argc, argv, "m:r:s:p:qS")) != -1) {
> switch(opt) {
> case 'm':
> map = strdup(optarg);
> @@ -538,13 +539,16 @@ main(int argc, char **argv)
> case 'q':
> quiet = 1;
> break;
> - /* sync file before mapping */
> case 'r':
> runs = atoi(optarg);
> break;
> case 's':
> seed = atoi(optarg);
> break;
> + /* sync file before mapping */
> + case 'S':
> + syncfile = 1;
> + break;
> default:
> usage();
> }
> @@ -623,7 +627,7 @@ main(int argc, char **argv)
> exit(1);
> }
>
> - rc = compare_fiemap_and_map(fd, map, blocks, blocksize);
> + rc = compare_fiemap_and_map(fd, map, blocks, blocksize,
> syncfile);
> if (rc) {
> printf("Problem comparing fiemap and map\n");
> free(map);
>
>
How about something in the usage() function to show we have this handy option
:). Thanks,
Josef
|