Use do_ftruncate() for the added call too.
[rsync-patches.git] / fsync.diff
index 27bc773..a5b6116 100644 (file)
@@ -1,55 +1,64 @@
 This patch from Sami Farin lets you specify --fsync if you want fsync()
 to be called on every file we write.
 
---- orig/options.c     2005-01-24 01:43:10
-+++ options.c  2004-11-27 18:10:44
-@@ -38,6 +38,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
+
+based-on: d64bda1c1e79dc385f194d74f7957ce7cd118654
+diff --git a/options.c b/options.c
+--- a/options.c
++++ b/options.c
+@@ -48,6 +48,7 @@ int append_mode = 0;
  int keep_dirlinks = 0;
+ int copy_dirlinks = 0;
  int copy_links = 0;
-@@ -251,6 +252,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 do_fsync = 0;
+ int preserve_links = 0;
+ int preserve_hard_links = 0;
+ int preserve_acls = 0;
+@@ -733,6 +734,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,"     --inplace               update destination files in-place (SEE MAN PAGE)\n");
-   rprintf(F," -d, --dirs                  transfer directories without recursing\n");
-@@ -358,6 +360,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,"     --usermap=STRING        custom username mapping\n");
+   rprintf(F,"     --groupmap=STRING       custom groupname mapping\n");
+@@ -994,6 +996,7 @@ static struct poptOption long_options[] = {
+   {"no-timeout",       0,  POPT_ARG_VAL,    &io_timeout, 0, 0, 0 },
+   {"contimeout",       0,  POPT_ARG_INT,    &connect_timeout, 0, 0, 0 },
+   {"no-contimeout",    0,  POPT_ARG_VAL,    &connect_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 },
-@@ -1227,6 +1230,9 @@ void server_options(char **args,int *arg
-               args[ac++] = tmpdir;
-       }
+   {"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 },
+@@ -2620,6 +2623,9 @@ void server_options(char **args, int *argc_p)
+                       args[ac++] = tmpdir;
+               }
  
-+      if (do_fsync && am_sender)
-+              args[ac++] = "--fsync";
++              if (do_fsync)
++                      args[ac++] = "--fsync";
 +
-       if (basis_dir[0] && am_sender) {
-               /* the server only needs this option if it is not the sender,
-                *   and it may be an older version that doesn't know this
---- orig/receiver.c    2005-01-24 01:43:10
-+++ receiver.c 2004-07-16 20:13:31
-@@ -35,6 +35,7 @@ extern int keep_dirlinks;
+               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
+diff --git a/receiver.c b/receiver.c
+--- a/receiver.c
++++ b/receiver.c
+@@ -38,6 +38,7 @@ extern int relative_paths;
  extern int preserve_hard_links;
  extern int preserve_perms;
- extern int io_error;
+ extern int preserve_xattrs;
 +extern int do_fsync;
- extern char *tmpdir;
- extern char *partial_dir;
- extern char *basis_dir[];
-@@ -237,6 +238,12 @@ static int receive_data(int f_in, char *
-               exit_cleanup(RERR_FILEIO);
-       }
+ extern int basis_dir_cnt;
+ extern int make_backups;
+ extern int cleanup_got_literal;
+@@ -341,6 +342,12 @@ static int receive_data(int f_in, char *fname_r, int fd_r, OFF_T size_r,
+       if (sum_end(file_sum1) != checksum_len)
+               overflow_exit("checksum_len"); /* Impossible... */
  
 +      if (do_fsync && fd != -1 && fsync(fd) != 0) {
 +              rsyserr(FERROR, errno, "fsync failed on %s",
@@ -57,39 +66,42 @@ to be called on every file we write.
 +              exit_cleanup(RERR_FILEIO);
 +      }
 +
-       sum_end(file_sum1);
        if (mapbuf)
---- orig/t_stub.c      2004-07-29 16:08:04
-+++ t_stub.c   2004-10-09 04:03:09
-@@ -28,6 +28,7 @@
+               unmap_file(mapbuf);
  
+diff --git a/t_stub.c b/t_stub.c
+--- a/t_stub.c
++++ b/t_stub.c
+@@ -21,6 +21,7 @@
+ #include "rsync.h"
++int do_fsync = 0;
  int modify_window = 0;
  int module_id = -1;
-+int do_fsync;
- char *partial_dir;
- struct exclude_list_struct server_exclude_list;
+ int relative_paths = 0;
+diff --git a/util.c b/util.c
+--- a/util.c
++++ b/util.c
+@@ -26,6 +26,7 @@
+ #include "inums.h"
  
---- orig/util.c        2005-01-20 23:05:34
-+++ util.c     2004-07-03 20:18:13
-@@ -33,6 +33,7 @@ extern int module_id;
- extern int modify_window;
- extern char *partial_dir;
- extern struct exclude_list_struct server_exclude_list;
+ extern int module_id;
 +extern int do_fsync;
int sanitize_paths = 0;
-@@ -298,6 +299,12 @@ int copy_file(char *source, char *dest, 
-               return -1;
+ extern int modify_window;
extern int relative_paths;
+ extern int preserve_times;
+@@ -370,6 +371,13 @@ int copy_file(const char *source, const char *dest, int ofd, mode_t mode)
+                       full_fname(source));
        }
  
 +      if (do_fsync && fsync(ofd) < 0) {
 +              rsyserr(FERROR, errno, "fsync failed on %s",
 +                      full_fname(dest));
++              close(ofd);
 +              return -1;
 +      }
 +
-       return 0;
- }
+       if (close(ofd) < 0) {
+               int save_errno = errno;
+               rsyserr(FERROR_XFER, errno, "close failed on %s",