From: Dongsu Park Subject: [PATCH v2] util-linux/fallocate: introduce an option -c to support COLLAPSE_RANGE Date: Thu, 27 Feb 2014 11:35:07 +0100 Message-ID: <1393497307-9205-1-git-send-email-dongsu.park@profitbricks.com> References: Cc: Karel Zak , linux-fsdevel@vger.kernel.org, xfs@oss.sgi.com, linux-ext4@vger.kernel.org, tytso@mit.edu, Lukas Czerner , Namjae Jeon , Ashish Sangwan , Dongsu Park To: Namjae Jeon Return-path: In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org From: Dongsu Park Introduce a new option -c (or --collapse-range) to support a new flag FALLOC_FL_COLLAPSE_RANGE for fallocate(2). It will nullify a particular range [offset, offset+len] by shifting extents beyond the range to the beginning of the hole. To test that, it's necessary to apply kernel patches in the patchset "fs: Introduce new flag (FALLOC_FL_COLLAPSE_RANGE) for fallocate" [1], as well as "[PATCH v6] ext4: Add support FALLOC_FL_COLLAPSE_RANGE for fallocate". [2] As discussed in the thread "[PATCH RESEND 3/10] ext4: Add support FALLOC_FL_COLLAPSE_RANGE for fallocate", [3] this patch to util-linux will be useful for testing collapse-range with fsstress. [1] https://lkml.org/lkml/2014/2/18/374 [2] https://lkml.org/lkml/2014/2/20/318 [3] https://lkml.org/lkml/2014/2/18/83 Cc: Karel Zak Cc: Lukas Czerner Cc: Namjae Jeon Cc: Ashish Sangwan Signed-off-by: Dongsu Park --- Changelog: v2: - update patch description by replacing xfstests with fsstress - update help message by replacing hole with space sys-utils/fallocate.1 | 7 +++++++ sys-utils/fallocate.c | 45 ++++++++++++++++++++++++++++----------------- 2 files changed, 35 insertions(+), 17 deletions(-) diff --git a/sys-utils/fallocate.1 b/sys-utils/fallocate.1 index 8a3aa4f..634c595 100644 --- a/sys-utils/fallocate.1 +++ b/sys-utils/fallocate.1 @@ -6,6 +6,7 @@ fallocate \- preallocate or deallocate space to a file .B fallocate .RB [ \-n ] .RB [ \-p ] +.RB [ \-c ] .RB [ \-o .IR offset ] .B \-l @@ -54,6 +55,12 @@ implied. .IP You can think of this as doing a "\fBcp --sparse\fP" and renaming the dest file as the original, without the need for extra disk space. +.IP "\fB\-c, \-\-collapse-range\fP" +Collapse a particular file range to nullify the hole. Extents beyond the range +[offset, offset+length] will be shifted to the beginning of hole. Hence this +command does not leave a hole, while \fI\-\-punch-hole\fP leaves a hole +instead of shifting extents. Both offset and length should be aligned to +the block size of filesystem. .IP "\fB\-o, \-\-offset\fP \fIoffset\fP Specifies the beginning offset of the allocation, in bytes. .IP "\fB\-l, \-\-length\fP \fIlength\fP diff --git a/sys-utils/fallocate.c b/sys-utils/fallocate.c index d8a74bf..3c8a8d2 100644 --- a/sys-utils/fallocate.c +++ b/sys-utils/fallocate.c @@ -39,7 +39,8 @@ #endif #if defined(HAVE_LINUX_FALLOC_H) && \ - (!defined(FALLOC_FL_KEEP_SIZE) || !defined(FALLOC_FL_PUNCH_HOLE)) + (!defined(FALLOC_FL_KEEP_SIZE) || !defined(FALLOC_FL_PUNCH_HOLE) || \ + !defined(FALLOC_FL_COLLAPSE_RANGE)) # include /* non-libc fallback for FALLOC_FL_* flags */ #endif @@ -51,6 +52,10 @@ # define FALLOC_FL_PUNCH_HOLE 2 #endif +#ifndef FALLOC_FL_COLLAPSE_RANGE +# define FALLOC_FL_COLLAPSE_RANGE 8 +#endif + #include "nls.h" #include "strutils.h" #include "c.h" @@ -66,12 +71,13 @@ static void __attribute__((__noreturn__)) usage(FILE *out) fprintf(out, _(" %s [options] \n"), program_invocation_short_name); fputs(USAGE_OPTIONS, out); - fputs(_(" -d, --dig-holes detect and dig holes\n"), out); - fputs(_(" -l, --length length of the (de)allocation, in bytes\n"), out); - fputs(_(" -n, --keep-size don't modify the length of the file\n"), out); - fputs(_(" -o, --offset offset of the (de)allocation, in bytes\n"), out); - fputs(_(" -p, --punch-hole punch holes in the file\n"), out); - fputs(_(" -v, --verbose verbose mode\n"), out); + fputs(_(" -c, --collapse-range collapse space in the file\n"), out); + fputs(_(" -d, --dig-holes detect and dig holes\n"), out); + fputs(_(" -l, --length length of the (de)allocation, in bytes\n"), out); + fputs(_(" -n, --keep-size don't modify the length of the file\n"), out); + fputs(_(" -o, --offset offset of the (de)allocation, in bytes\n"), out); + fputs(_(" -p, --punch-hole punch holes in the file\n"), out); + fputs(_(" -v, --verbose verbose mode\n"), out); fputs(USAGE_SEPARATOR, out); fputs(USAGE_HELP, out); @@ -258,15 +264,16 @@ int main(int argc, char **argv) loff_t offset = 0; static const struct option longopts[] = { - { "help", 0, 0, 'h' }, - { "version", 0, 0, 'V' }, - { "keep-size", 0, 0, 'n' }, - { "punch-hole", 0, 0, 'p' }, - { "dig-holes", 0, 0, 'd' }, - { "offset", 1, 0, 'o' }, - { "length", 1, 0, 'l' }, - { "verbose", 0, 0, 'v' }, - { NULL, 0, 0, 0 } + { "help", 0, 0, 'h' }, + { "version", 0, 0, 'V' }, + { "keep-size", 0, 0, 'n' }, + { "punch-hole", 0, 0, 'p' }, + { "collapse-range", 0, 0, 'c' }, + { "dig-holes", 0, 0, 'd' }, + { "offset", 1, 0, 'o' }, + { "length", 1, 0, 'l' }, + { "verbose", 0, 0, 'v' }, + { NULL, 0, 0, 0 } }; setlocale(LC_ALL, ""); @@ -274,7 +281,8 @@ int main(int argc, char **argv) textdomain(PACKAGE); atexit(close_stdout); - while ((c = getopt_long(argc, argv, "hvVnpdl:o:", longopts, NULL)) != -1) { + while ((c = getopt_long(argc, argv, "hvVncpdl:o:", longopts, NULL)) + != -1) { switch(c) { case 'h': usage(stdout); @@ -282,6 +290,9 @@ int main(int argc, char **argv) case 'V': printf(UTIL_LINUX_VERSION); return EXIT_SUCCESS; + case 'c': + mode |= FALLOC_FL_COLLAPSE_RANGE; + break; case 'p': mode |= FALLOC_FL_PUNCH_HOLE; /* fall through */ -- 1.8.5.3