X-Git-Url: https://mattmccutchen.net/rsync/rsync-patches.git/blobdiff_plain/fe6407b5a9f04c88f74ea95e959aaa38334e41a6..f264662f71d65a8addc50d3ee0372d82c903bf2a:/fsync.diff

diff --git a/fsync.diff b/fsync.diff
index d0ec598..7d53eda 100644
--- a/fsync.diff
+++ b/fsync.diff
@@ -1,85 +1,102 @@
 This patch from Sami Farin lets you specify --fsync if you want fsync()
 to be called on every file we write.
 
---- options.c	6 May 2004 21:08:01 -0000	1.148
-+++ options.c	15 May 2004 20:20:27 -0000
-@@ -37,6 +37,7 @@ int make_backups = 0;
-  **/
- int whole_file = -1;
- 
-+int do_fsync = 0;
- int archive_mode = 0;
+To use this patch, run these commands for a successful build:
+
+    patch -p1 <patches/fsync.diff
+    ./configure                         (optional if already run)
+    make
+
+--- old/options.c
++++ new/options.c
+@@ -45,6 +45,7 @@ int append_mode = 0;
+ int keep_dirlinks = 0;
+ int copy_dirlinks = 0;
  int copy_links = 0;
++int do_fsync = 0;
  int preserve_links = 0;
-@@ -230,6 +231,7 @@ void usage(enum logcode F)
-   rprintf(F," -b, --backup                make backups (see --suffix & --backup-dir)\n");
-   rprintf(F,"     --backup-dir            make backups into this directory\n");
-   rprintf(F,"     --suffix=SUFFIX         backup suffix (default %s w/o --backup-dir)\n",BACKUP_SUFFIX);
+ int preserve_hard_links = 0;
+ int preserve_acls = 0;
+@@ -375,6 +376,7 @@ void usage(enum logcode F)
+   rprintf(F,"     --partial-dir=DIR       put a partially transferred file into DIR\n");
+   rprintf(F,"     --delay-updates         put all updated files into place at transfer's end\n");
+   rprintf(F," -m, --prune-empty-dirs      prune empty directory chains from the file-list\n");
 +  rprintf(F,"     --fsync                 fsync every written file\n");
-   rprintf(F," -u, --update                update only (don't overwrite newer files)\n");
-   rprintf(F," -l, --links                 copy symlinks as symlinks\n");
-   rprintf(F," -L, --copy-links            copy the referent of all symlinks\n");
-@@ -332,6 +334,7 @@ static struct poptOption long_options[] 
-   {"safe-links",       0,  POPT_ARG_NONE,   &safe_symlinks, 0, 0, 0 },
-   {"help",            'h', POPT_ARG_NONE,   0,              'h', 0, 0 },
-   {"backup",          'b', POPT_ARG_NONE,   &make_backups, 0, 0, 0 },
+   rprintf(F,"     --numeric-ids           don't map uid/gid values by user/group name\n");
+   rprintf(F,"     --timeout=TIME          set I/O timeout in seconds\n");
+   rprintf(F," -I, --ignore-times          don't skip files that match in size and mod-time\n");
+@@ -600,6 +602,7 @@ static struct poptOption long_options[] 
+   {"no-numeric-ids",   0,  POPT_ARG_VAL,    &numeric_ids, 0, 0, 0 },
+   {"timeout",          0,  POPT_ARG_INT,    &io_timeout, 0, 0, 0 },
+   {"no-timeout",       0,  POPT_ARG_VAL,    &io_timeout, 0, 0, 0 },
 +  {"fsync",            0,  POPT_ARG_NONE,   &do_fsync, 0, 0, 0 },
-   {"dry-run",         'n', POPT_ARG_NONE,   &dry_run, 0, 0, 0 },
-   {"sparse",          'S', POPT_ARG_NONE,   &sparse_files, 0, 0, 0 },
-   {"cvs-exclude",     'C', POPT_ARG_NONE,   &cvs_exclude, 0, 0, 0 },
-@@ -943,6 +946,9 @@ void server_options(char **args,int *arg
- 		args[ac++] = "--temp-dir";
- 		args[ac++] = tmpdir;
- 	}
-+
-+	if (do_fsync && am_sender)
-+		args[ac++] = "--fsync";
+   {"rsh",             'e', POPT_ARG_STRING, &shell_cmd, 0, 0, 0 },
+   {"rsync-path",       0,  POPT_ARG_STRING, &rsync_path, 0, 0, 0 },
+   {"temp-dir",        'T', POPT_ARG_STRING, &tmpdir, 0, 0, 0 },
+@@ -1934,6 +1937,9 @@ void server_options(char **args, int *ar
+ 			args[ac++] = tmpdir;
+ 		}
  
- 	if (compare_dest && am_sender) {
- 		/* the server only needs this option if it is not the sender,
---- receiver.c	15 May 2004 19:31:10 -0000	1.78
-+++ receiver.c	15 May 2004 20:20:27 -0000
-@@ -46,6 +46,7 @@ extern int module_id;
- extern int ignore_errors;
- extern int orig_umask;
- extern int keep_partial;
++		if (do_fsync)
++			args[ac++] = "--fsync";
++
+ 		if (basis_dir[0]) {
+ 			/* the server only needs this option if it is not the sender,
+ 			 *   and it may be an older version that doesn't know this
+--- old/receiver.c
++++ new/receiver.c
+@@ -39,6 +39,7 @@ extern int relative_paths;
+ extern int preserve_hard_links;
+ extern int preserve_perms;
+ extern int preserve_xattrs;
 +extern int do_fsync;
+ extern int basis_dir_cnt;
+ extern int make_backups;
+ extern int cleanup_got_literal;
+@@ -261,6 +262,12 @@ static int receive_data(int f_in, char *
+ 		exit_cleanup(RERR_FILEIO);
+ 	}
  
- static void delete_one(char *fn, int is_dir)
- {
-@@ -266,6 +267,12 @@ static int receive_data(int f_in,struct 
- 
- 	if (fd != -1 && offset > 0 && sparse_end(fd) != 0) {
- 		rsyserr(FERROR, errno, "write failed on %s",
++	if (do_fsync && fd != -1 && fsync(fd) != 0) {
++		rsyserr(FERROR, errno, "fsync failed on %s",
 +			full_fname(fname));
 +		exit_cleanup(RERR_FILEIO);
 +	}
 +
-+	if (do_fsync && (fd != -1) && (fsync(fd) != 0)) {
-+		rsyserr(FERROR, errno, "fsync failed on %s",
- 			full_fname(fname));
- 		exit_cleanup(RERR_FILEIO);
- 	}
---- util.c	15 May 2004 19:31:10 -0000	1.143
-+++ util.c	15 May 2004 20:20:28 -0000
-@@ -29,6 +29,7 @@
+ 	sum_len = sum_end(file_sum1);
  
- extern int verbose;
- extern struct exclude_list_struct server_exclude_list;
-+extern int do_fsync;
+ 	if (mapbuf)
+--- old/t_stub.c
++++ new/t_stub.c
+@@ -21,6 +21,7 @@
  
- int sanitize_paths = 0;
+ #include "rsync.h"
  
-@@ -291,6 +292,12 @@ int copy_file(char *source, char *dest, 
++int do_fsync = 0;
+ int modify_window = 0;
+ int module_id = -1;
+ int relative_paths = 0;
+--- old/util.c
++++ new/util.c
+@@ -25,6 +25,7 @@
+ extern int verbose;
+ extern int dry_run;
+ extern int module_id;
++extern int do_fsync;
+ extern int modify_window;
+ extern int relative_paths;
+ extern int human_readable;
+@@ -310,6 +311,13 @@ int copy_file(const char *source, const 
+ 			full_fname(source));
+ 	}
  
- 	if (close(ofd) < 0) {
- 		rsyserr(FERROR, errno, "close failed on %s",
++	if (do_fsync && fsync(ofd) < 0) {
++		rsyserr(FERROR, errno, "fsync failed on %s",
 +			full_fname(dest));
++		close(ofd);
 +		return -1;
 +	}
 +
-+	if (do_fsync && fsync(ofd) < 0) {
-+		rsyserr(FERROR, errno, "fsync failed on %s",
+ 	if (close(ofd) < 0) {
+ 		rsyserr(FERROR, errno, "close failed on %s",
  			full_fname(dest));
- 		return -1;
- 	}