Re: [PATCH] ext4: Fix fallocate to update the file size in each transaction.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux