fsync.diff

   1 This patch from Sami Farin lets you specify --fsync if you want fsync()
   2 to be called on every file we write.
   3
   4 --- options.c   17 Apr 2004 17:07:23 -0000      1.147
   5 +++ options.c   27 Apr 2004 20:05:03 -0000
   6 @@ -37,6 +37,7 @@ int make_backups = 0;
   7   **/
   8  int whole_file = -1;
   9
  10 +int do_fsync = 0;
  11  int archive_mode = 0;
  12  int copy_links = 0;
  13  int preserve_links = 0;
  14 @@ -230,6 +231,7 @@ void usage(enum logcode F)
  15    rprintf(F," -b, --backup                make backups (see --suffix & --backup-dir)\n");
  16    rprintf(F,"     --backup-dir            make backups into this directory\n");
  17    rprintf(F,"     --suffix=SUFFIX         backup suffix (default %s w/o --backup-dir)\n",BACKUP_SUFFIX);
  18 +  rprintf(F,"     --fsync                 fsync every written file\n");
  19    rprintf(F," -u, --update                update only (don't overwrite newer files)\n");
  20    rprintf(F," -l, --links                 copy symlinks as symlinks\n");
  21    rprintf(F," -L, --copy-links            copy the referent of all symlinks\n");
  22 @@ -332,6 +334,7 @@ static struct poptOption long_options[]
  23    {"safe-links",       0,  POPT_ARG_NONE,   &safe_symlinks, 0, 0, 0 },
  24    {"help",            'h', POPT_ARG_NONE,   0,              'h', 0, 0 },
  25    {"backup",          'b', POPT_ARG_NONE,   &make_backups, 0, 0, 0 },
  26 +  {"fsync",            0,  POPT_ARG_NONE,   &do_fsync, 0, 0, 0 },
  27    {"dry-run",         'n', POPT_ARG_NONE,   &dry_run, 0, 0, 0 },
  28    {"sparse",          'S', POPT_ARG_NONE,   &sparse_files, 0, 0, 0 },
  29    {"cvs-exclude",     'C', POPT_ARG_NONE,   &cvs_exclude, 0, 0, 0 },
  30 @@ -943,6 +946,9 @@ void server_options(char **args,int *arg
  31                 args[ac++] = "--temp-dir";
  32                 args[ac++] = tmpdir;
  33         }
  34 +
  35 +       if (do_fsync && am_sender)
  36 +               args[ac++] = "--fsync";
  37
  38         if (compare_dest && am_sender) {
  39                 /* the server only needs this option if it is not the sender,
  40 --- receiver.c  27 Apr 2004 19:51:33 -0000      1.76
  41 +++ receiver.c  27 Apr 2004 20:05:03 -0000
  42 @@ -45,6 +45,7 @@ extern int cleanup_got_literal;
  43  extern int module_id;
  44  extern int ignore_errors;
  45  extern int orig_umask;
  46 +extern int do_fsync;
  47
  48  static void delete_one(char *fn, int is_dir)
  49  {
  50 @@ -264,6 +265,12 @@ static int receive_data(int f_in,struct
  51
  52         if (fd != -1 && offset > 0 && sparse_end(fd) != 0) {
  53                 rprintf(FERROR, "write failed on %s: %s\n",
  54 +                       full_fname(fname), strerror(errno));
  55 +               exit_cleanup(RERR_FILEIO);
  56 +       }
  57 +
  58 +       if (do_fsync && (fd != -1) && (fsync(fd) != 0)) {
  59 +               rprintf(FERROR, "fsync failed on %s: %s\n",
  60                         full_fname(fname), strerror(errno));
  61                 exit_cleanup(RERR_FILEIO);
  62         }
  63 --- util.c      27 Apr 2004 19:59:37 -0000      1.141
  64 +++ util.c      27 Apr 2004 20:05:04 -0000
  65 @@ -29,6 +29,7 @@
  66
  67  extern int verbose;
  68  extern struct exclude_list_struct server_exclude_list;
  69 +extern int do_fsync;
  70
  71  int sanitize_paths = 0;
  72
  73 @@ -296,6 +297,12 @@ int copy_file(char *source, char *dest,
  74
  75         if (close(ofd) < 0) {
  76                 rprintf(FERROR, "close failed on %s: %s\n",
  77 +                       full_fname(dest), strerror(errno));
  78 +               return -1;
  79 +       }
  80 +
  81 +       if (do_fsync && fsync(ofd) < 0) {
  82 +               rprintf(FERROR, "fsync failed on %s: %s\n",
  83                         full_fname(dest), strerror(errno));
  84                 return -1;
  85         }