Rename vfs operation posix_fallocate to just fallocate and add the vfs_fallocate_mode parameter.

It turns out we need the fallocate operations to be able to both
allocate and extend filesize, and to allocate and not extend
filesize, and posix_fallocate can only do the former. So by defining
the vfs op as posix_fallocate we lose the opportunity to use any
underlying syscalls (like Linux fallocate) that can do the latter
as well.

We don't currently use the non-extending filesize call, but now
I've changed the vfs op definition we can in the future. For the
moment simply map the fallocate op onto posix_fallocate for the
VFS_FALLOCATE_EXTEND_SIZE case and return ENOSYS for the
VFS_FALLOCATE_KEEP_SIZE case.

Jeremy.

Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat Dec 18 08:59:27 CET 2010 on sn-devel-104

1 files changed, 17 insertions, 9 deletions

diff --git a/source3/modules/vfs_default.c b/source3/modules/vfs_default.c
index 66dd5a912d..e08d48ff5f 100644
--- a/source3/modules/vfs_default.c
+++ b/source3/modules/vfs_default.c
@@ -848,12 +848,13 @@ static int strict_allocate_ftruncate(vfs_handle_struct *handle, files_struct *fs
 
 	space_to_write = len - pst->st_ex_size;
 
-	/* for allocation try posix_fallocate first. This can fail on some
+	/* for allocation try fallocate first. This can fail on some
 	   platforms e.g. when the filesystem doesn't support it and no
 	   emulation is being done by the libc (like on AIX with JFS1). In that
-	   case we do our own emulation. posix_fallocate implementations can
+	   case we do our own emulation. fallocate implementations can
 	   return ENOTSUP or EINVAL in cases like that. */
-	ret = SMB_VFS_POSIX_FALLOCATE(fsp, pst->st_ex_size, space_to_write);
+	ret = SMB_VFS_FALLOCATE(fsp, VFS_FALLOCATE_EXTEND_SIZE,
+				pst->st_ex_size, space_to_write);
 	if (ret == ENOSPC) {
 		errno = ENOSPC;
 		return -1;
@@ -861,7 +862,7 @@ static int strict_allocate_ftruncate(vfs_handle_struct *handle, files_struct *fs
 	if (ret == 0) {
 		return 0;
 	}
-	DEBUG(10,("strict_allocate_ftruncate: SMB_VFS_POSIX_FALLOCATE failed with "
+	DEBUG(10,("strict_allocate_ftruncate: SMB_VFS_FALLOCATE failed with "
 		"error %d. Falling back to slow manual allocation\n", ret));
 
 	/* available disk space is enough or not? */
@@ -953,16 +954,23 @@ static int vfswrap_ftruncate(vfs_handle_struct *handle, files_struct *fsp, SMB_O
 	return result;
 }
 
-static int vfswrap_posix_fallocate(vfs_handle_struct *handle,
+static int vfswrap_fallocate(vfs_handle_struct *handle,
 			files_struct *fsp,
+			enum vfs_fallocate_mode mode,
 			SMB_OFF_T offset,
 			SMB_OFF_T len)
 {
 	int result;
 
-	START_PROFILE(syscall_posix_fallocate);
-	result = sys_posix_fallocate(fsp->fh->fd, offset, len);
-	END_PROFILE(syscall_posix_fallocate);
+	START_PROFILE(syscall_fallocate);
+	if (mode == VFS_FALLOCATE_EXTEND_SIZE) {
+		result = sys_posix_fallocate(fsp->fh->fd, offset, len);
+	} else {
+		/* TODO - implement call into Linux fallocate call. */
+		errno = ENOSYS;
+		result = -1;
+	}
+	END_PROFILE(syscall_fallocate);
 	return result;
 }
 
@@ -1651,7 +1659,7 @@ static struct vfs_fn_pointers vfs_default_fns = {
 	.getwd = vfswrap_getwd,
 	.ntimes = vfswrap_ntimes,
 	.ftruncate = vfswrap_ftruncate,
-	.posix_fallocate = vfswrap_posix_fallocate,
+	.fallocate = vfswrap_fallocate,
 	.lock = vfswrap_lock,
 	.kernel_flock = vfswrap_kernel_flock,
 	.linux_setlease = vfswrap_linux_setlease,