On Mar 11, 2008 11:25 +0530, Aneesh Kumar K.V wrote: > + -x: run test after preallocating the area (1|2) 2 to not update size.\n\ The issue with preallocating just a single area in fsx is that this isn't much different than starting the test with a single large write or truncate. What would be a lot more useful is to have fsx continually do fallocate requests of variable sizes during the test. This would be quite similar to a "write" operation, except that it wouldn't change any existing data and holes would still read back as zero. The only difference is in the "do not update size" test any fallocate at the end of the file would not increase the file size. This would exercise the fallocate code a LOT because it would put unwritten extents in the middle of the file, map a single fallocate to multiple discontiguous holes of the file (not overwriting existing allocated data in the middle of the file), verify fallocate of an overlapping unwritten extent works, etc. > +#define FALLOC_FL_KEEP_SIZE 0x01 > +#ifdef __i386__ > +#define __NR_fallocate 324 > + loff_t offset, maxlen; > + offset = 0; > + maxlen = maxfilelen; > + if (prealloc == 1) { > + if (syscall(__NR_fallocate, fd, 0, offset, maxlen) < 0) { > + prterr(fname); > + exit(97); > + } > + } else if (prealloc == 2) { > + if (syscall(__NR_fallocate, fd, FALLOC_FL_KEEP_SIZE, offset, maxlen) < 0) { > + prterr(fname); > + exit(97); > + } > + } > +#elif defined (__powerpc__) > +#define __NR_fallocate 309 > + /* Work only with 32 bit user space */ > + unsigned int maxlen = maxfilelen; > + if (prealloc == 1) { > + if (syscall(__NR_fallocate, fd, 0, 0, 0, 0, maxlen) < 0) { > + prterr(fname); > + exit(97); > + } > + } else if (prealloc == 2) { > + if (syscall(__NR_fallocate, fd, FALLOC_FL_KEEP_SIZE, 0, 0, 0, maxlen) < 0) { > + prterr(fname); > + exit(97); > + } > + } > +#endif This should be moved into a separate "do_fallocate()" wrapper function that takes normal fallocate parameters fd, offset, len. In the future it would just call "fallocate()" directly. Cheers, Andreas -- Andreas Dilger Sr. Staff Engineer, Lustre Group Sun Microsystems of Canada, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html