atimes.diff

   1 To use this patch, run these commands for a successful build:
   2
   3     patch -p1 <patches/atimes.diff
   4     ./configure                      (optional if already run)
   5     make
   6
   7 diff --git a/compat.c b/compat.c
   8 --- a/compat.c
   9 +++ b/compat.c
  10 @@ -44,6 +44,7 @@ extern int protocol_version;
  11  extern int protect_args;
  12  extern int preserve_uid;
  13  extern int preserve_gid;
  14 +extern int preserve_atimes;
  15  extern int preserve_acls;
  16  extern int preserve_xattrs;
  17  extern int need_messages_from_generator;
  18 @@ -60,7 +61,7 @@ extern iconv_t ic_send, ic_recv;
  19  #endif
  20
  21  /* These index values are for the file-list's extra-attribute array. */
  22 -int uid_ndx, gid_ndx, acls_ndx, xattrs_ndx, unsort_ndx;
  23 +int uid_ndx, gid_ndx, atimes_ndx, acls_ndx, xattrs_ndx, unsort_ndx;
  24
  25  int receiver_symlink_times = 0; /* receiver can set the time on a symlink */
  26
  27 @@ -134,6 +135,8 @@ void setup_protocol(int f_out,int f_in)
  28                 uid_ndx = ++file_extra_cnt;
  29         if (preserve_gid)
  30                 gid_ndx = ++file_extra_cnt;
  31 +       if (preserve_atimes)
  32 +               atimes_ndx = (file_extra_cnt += TIME_EXTRA_CNT);
  33         if (preserve_acls && !am_sender)
  34                 acls_ndx = ++file_extra_cnt;
  35         if (preserve_xattrs)
  36 diff --git a/flist.c b/flist.c
  37 --- a/flist.c
  38 +++ b/flist.c
  39 @@ -55,6 +55,7 @@ extern int preserve_specials;
  40  extern int uid_ndx;
  41  extern int gid_ndx;
  42  extern int eol_nulls;
  43 +extern int atimes_ndx;
  44  extern int relative_paths;
  45  extern int implied_dirs;
  46  extern int file_extra_cnt;
  47 @@ -344,7 +345,7 @@ int push_pathname(const char *dir, int len)
  48
  49  static void send_file_entry(int f, struct file_struct *file, int ndx, int first_ndx)
  50  {
  51 -       static time_t modtime;
  52 +       static time_t modtime, atime;
  53         static mode_t mode;
  54  #ifdef SUPPORT_HARD_LINKS
  55         static int64 dev;
  56 @@ -452,6 +453,13 @@ static void send_file_entry(int f, struct file_struct *file, int ndx, int first_
  57                 xflags |= XMIT_SAME_TIME;
  58         else
  59                 modtime = file->modtime;
  60 +       if (atimes_ndx && !S_ISDIR(mode)) {
  61 +               time_t file_atime = f_atime(file);
  62 +               if (file_atime == atime)
  63 +                       xflags |= XMIT_SAME_ATIME;
  64 +               else
  65 +                       atime = file_atime;
  66 +       }
  67
  68  #ifdef SUPPORT_HARD_LINKS
  69         if (tmp_dev != 0) {
  70 @@ -524,6 +532,8 @@ static void send_file_entry(int f, struct file_struct *file, int ndx, int first_
  71         }
  72         if (!(xflags & XMIT_SAME_MODE))
  73                 write_int(f, to_wire_mode(mode));
  74 +       if (atimes_ndx && !S_ISDIR(mode) && !(xflags & XMIT_SAME_ATIME))
  75 +               write_varlong(f, atime, 4);
  76         if (preserve_uid && !(xflags & XMIT_SAME_UID)) {
  77                 if (protocol_version < 30)
  78                         write_int(f, uid);
  79 @@ -610,7 +620,7 @@ static void send_file_entry(int f, struct file_struct *file, int ndx, int first_
  80  static struct file_struct *recv_file_entry(struct file_list *flist,
  81                                            int xflags, int f)
  82  {
  83 -       static int64 modtime;
  84 +       static int64 modtime, atime;
  85         static mode_t mode;
  86  #ifdef SUPPORT_HARD_LINKS
  87         static int64 dev;
  88 @@ -743,6 +753,16 @@ static struct file_struct *recv_file_entry(struct file_list *flist,
  89         }
  90         if (!(xflags & XMIT_SAME_MODE))
  91                 mode = from_wire_mode(read_int(f));
  92 +       if (atimes_ndx && !S_ISDIR(mode) && !(xflags & XMIT_SAME_ATIME)) {
  93 +               atime = read_varlong(f, 4);
  94 +#if SIZEOF_TIME_T < SIZEOF_INT64
  95 +               if (!am_generator && (int64)(time_t)atime != atime) {
  96 +                       rprintf(FERROR_XFER,
  97 +                               "Access time value of %s truncated on receiver.\n",
  98 +                               lastname);
  99 +               }
 100 +#endif
 101 +       }
 102
 103         if (chmod_modes && !S_ISLNK(mode))
 104                 mode = tweak_mode(mode, chmod_modes);
 105 @@ -873,6 +893,8 @@ static struct file_struct *recv_file_entry(struct file_list *flist,
 106                 F_GROUP(file) = gid;
 107                 file->flags |= gid_flags;
 108         }
 109 +       if (atimes_ndx)
 110 +               f_atime_set(file, (time_t)atime);
 111         if (unsort_ndx)
 112                 F_NDX(file) = flist->used + flist->ndx_start;
 113
 114 @@ -1205,6 +1227,8 @@ struct file_struct *make_file(const char *fname, struct file_list *flist,
 115                 F_OWNER(file) = st.st_uid;
 116         if (gid_ndx) /* Check gid_ndx instead of preserve_gid for del support */
 117                 F_GROUP(file) = st.st_gid;
 118 +       if (atimes_ndx)
 119 +               f_atime_set(file, st.st_atime);
 120
 121         if (basename != thisname)
 122                 file->dirname = lastdir;
 123 diff --git a/generator.c b/generator.c
 124 --- a/generator.c
 125 +++ b/generator.c
 126 @@ -21,6 +21,7 @@
 127   */
 128
 129  #include "rsync.h"
 130 +#include "ifuncs.h"
 131
 132  extern int verbose;
 133  extern int dry_run;
 134 @@ -633,6 +634,9 @@ void itemize(const char *fnamecmp, struct file_struct *file, int ndx, int statre
 135                   && (!(iflags & ITEM_XNAME_FOLLOWS) || *xname))
 136                  || (keep_time && cmp_time(file->modtime, sxp->st.st_mtime) != 0))
 137                         iflags |= ITEM_REPORT_TIME;
 138 +               if (atimes_ndx && !S_ISDIR(file->mode) && !S_ISLNK(file->mode)
 139 +                && cmp_time(f_atime(file), sxp->st.st_atime) != 0)
 140 +                       iflags |= ITEM_REPORT_ATIME;
 141  #if !defined HAVE_LCHMOD && !defined HAVE_SETATTRLIST
 142                 if (S_ISLNK(file->mode)) {
 143                         ;
 144 @@ -987,6 +991,8 @@ static int try_dests_reg(struct file_struct *file, char *fname, int ndx,
 145                 if (link_dest) {
 146                         if (!hard_link_one(file, fname, cmpbuf, 1))
 147                                 goto try_a_copy;
 148 +                       if (atimes_ndx)
 149 +                               set_file_attrs(fname, file, sxp, NULL, 0);
 150                         if (preserve_hard_links && F_IS_HLINKED(file))
 151                                 finish_hard_link(file, fname, ndx, &sxp->st, itemizing, code, j);
 152                         if (!maybe_ATTRS_REPORT && (verbose > 1 || stdout_format_has_i > 1)) {
 153 @@ -1173,6 +1179,7 @@ static int try_dests_non(struct file_struct *file, char *fname, int ndx,
 154  static void list_file_entry(struct file_struct *f)
 155  {
 156         char permbuf[PERMSTRING_SIZE];
 157 +       time_t atime = atimes_ndx ? f_atime(f) : 0;
 158         double len;
 159
 160         if (!F_IS_ACTIVE(f)) {
 161 @@ -1187,14 +1194,16 @@ static void list_file_entry(struct file_struct *f)
 162
 163  #ifdef SUPPORT_LINKS
 164         if (preserve_links && S_ISLNK(f->mode)) {
 165 -               rprintf(FINFO, "%s %11.0f %s %s -> %s\n",
 166 +               rprintf(FINFO, "%s %11.0f %s %s %s -> %s\n",
 167                         permbuf, len, timestring(f->modtime),
 168 +                       atimes_ndx ? timestring(atime) : "",
 169                         f_name(f, NULL), F_SYMLINK(f));
 170         } else
 171  #endif
 172         {
 173 -               rprintf(FINFO, "%s %11.0f %s %s\n",
 174 +               rprintf(FINFO, "%s %11.0f %s %s %s\n",
 175                         permbuf, len, timestring(f->modtime),
 176 +                       atimes_ndx ? timestring(atime) : "",
 177                         f_name(f, NULL));
 178         }
 179  }
 180 @@ -1950,7 +1959,7 @@ static void touch_up_dirs(struct file_list *flist, int ndx)
 181                 if (!(file->mode & S_IWUSR))
 182                         do_chmod(fname, file->mode);
 183                 if (need_retouch_dir_times)
 184 -                       set_modtime(fname, file->modtime, file->mode);
 185 +                       set_times(fname, file->modtime, file->modtime, file->mode);
 186                 if (allowed_lull && !(counter % lull_mod))
 187                         maybe_send_keepalive();
 188                 else if (!(counter & 0xFF))
 189 diff --git a/ifuncs.h b/ifuncs.h
 190 --- a/ifuncs.h
 191 +++ b/ifuncs.h
 192 @@ -57,6 +57,28 @@ from_wire_mode(int mode)
 193         return mode;
 194  }
 195
 196 +static inline time_t
 197 +f_atime(struct file_struct *fp)
 198 +{
 199 +#if SIZEOF_TIME_T > 4
 200 +       time_t atime;
 201 +       memcpy(&atime, &REQ_EXTRA(fp, atimes_ndx)->unum, SIZEOF_TIME_T);
 202 +       return atime;
 203 +#else
 204 +       return REQ_EXTRA(fp, atimes_ndx)->unum;
 205 +#endif
 206 +}
 207 +
 208 +static inline void
 209 +f_atime_set(struct file_struct *fp, time_t atime)
 210 +{
 211 +#if SIZEOF_TIME_T > 4
 212 +       memcpy(&REQ_EXTRA(fp, atimes_ndx)->unum, &atime, SIZEOF_TIME_T);
 213 +#else
 214 +       REQ_EXTRA(fp, atimes_ndx)->unum = (uint32)atime;
 215 +#endif
 216 +}
 217 +
 218  static inline int
 219  isDigit(const char *ptr)
 220  {
 221 diff --git a/log.c b/log.c
 222 --- a/log.c
 223 +++ b/log.c
 224 @@ -644,7 +644,8 @@ static void log_formatted(enum logcode code, const char *format, const char *op,
 225                         c[5] = !(iflags & ITEM_REPORT_PERMS) ? '.' : 'p';
 226                         c[6] = !(iflags & ITEM_REPORT_OWNER) ? '.' : 'o';
 227                         c[7] = !(iflags & ITEM_REPORT_GROUP) ? '.' : 'g';
 228 -                       c[8] = !(iflags & ITEM_REPORT_ATIME) ? '.' : 'u';
 229 +                       c[8] = !(iflags & ITEM_REPORT_ATIME) ? '.'
 230 +                            : S_ISLNK(file->mode) ? 'U' : 'u';
 231                         c[9] = !(iflags & ITEM_REPORT_ACL) ? '.' : 'a';
 232                         c[10] = !(iflags & ITEM_REPORT_XATTR) ? '.' : 'x';
 233                         c[11] = '\0';
 234 diff --git a/options.c b/options.c
 235 --- a/options.c
 236 +++ b/options.c
 237 @@ -58,6 +58,7 @@ int preserve_specials = 0;
 238  int preserve_uid = 0;
 239  int preserve_gid = 0;
 240  int preserve_times = 0;
 241 +int preserve_atimes = 0;
 242  int update_only = 0;
 243  int cvs_exclude = 0;
 244  int dry_run = 0;
 245 @@ -352,6 +353,7 @@ void usage(enum logcode F)
 246    rprintf(F," -D                          same as --devices --specials\n");
 247    rprintf(F," -t, --times                 preserve modification times\n");
 248    rprintf(F," -O, --omit-dir-times        omit directories from --times\n");
 249 +  rprintf(F," -U, --atimes                preserve access (last-used) times\n");
 250    rprintf(F,"     --super                 receiver attempts super-user activities\n");
 251  #ifdef SUPPORT_XATTRS
 252    rprintf(F,"     --fake-super            store/recover privileged attrs using xattrs\n");
 253 @@ -487,6 +489,9 @@ static struct poptOption long_options[] = {
 254    {"times",           't', POPT_ARG_VAL,    &preserve_times, 2, 0, 0 },
 255    {"no-times",         0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
 256    {"no-t",             0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
 257 +  {"atimes",          'U', POPT_ARG_VAL,    &preserve_atimes, 1, 0, 0 },
 258 +  {"no-atimes",        0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
 259 +  {"no-U",             0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
 260    {"omit-dir-times",  'O', POPT_ARG_VAL,    &omit_dir_times, 1, 0, 0 },
 261    {"no-omit-dir-times",0,  POPT_ARG_VAL,    &omit_dir_times, 0, 0, 0 },
 262    {"no-O",             0,  POPT_ARG_VAL,    &omit_dir_times, 0, 0, 0 },
 263 @@ -1747,6 +1752,8 @@ void server_options(char **args, int *argc_p)
 264                 argstr[x++] = 'D';
 265         if (preserve_times)
 266                 argstr[x++] = 't';
 267 +       if (preserve_atimes)
 268 +               argstr[x++] = 'U';
 269         if (preserve_perms)
 270                 argstr[x++] = 'p';
 271         else if (preserve_executability && am_sender)
 272 diff --git a/rsync.c b/rsync.c
 273 --- a/rsync.c
 274 +++ b/rsync.c
 275 @@ -345,6 +345,7 @@ int set_file_attrs(const char *fname, struct file_struct *file, stat_x *sxp,
 276         int updated = 0;
 277         stat_x sx2;
 278         int change_uid, change_gid;
 279 +       time_t atime, mtime;
 280         mode_t new_mode = file->mode;
 281         int inherit;
 282
 283 @@ -388,20 +389,39 @@ int set_file_attrs(const char *fname, struct file_struct *file, stat_x *sxp,
 284                 set_xattr(fname, file, fnamecmp, sxp);
 285  #endif
 286
 287 +       /* This code must be the first update in the function due to
 288 +        * how it uses the "updated" variable. */
 289         if (!preserve_times || (S_ISDIR(sxp->st.st_mode) && preserve_times == 1))
 290                 flags |= ATTRS_SKIP_MTIME;
 291 +       if (!atimes_ndx || S_ISDIR(sxp->st.st_mode))
 292 +               flags |= ATTRS_SKIP_ATIME;
 293         if (!(flags & ATTRS_SKIP_MTIME)
 294             && cmp_time(sxp->st.st_mtime, file->modtime) != 0) {
 295 -               int ret = set_modtime(fname, file->modtime, sxp->st.st_mode);
 296 +               mtime = file->modtime;
 297 +               updated = 1;
 298 +       } else
 299 +               mtime = sxp->st.st_mtime;
 300 +       if (!(flags & ATTRS_SKIP_ATIME)) {
 301 +               time_t file_atime = f_atime(file);
 302 +               if (cmp_time(sxp->st.st_atime, file_atime) != 0) {
 303 +                       atime = file_atime;
 304 +                       updated = 1;
 305 +               } else
 306 +                       atime = sxp->st.st_atime;
 307 +       } else
 308 +               atime = sxp->st.st_atime;
 309 +       if (updated) {
 310 +               int ret = set_times(fname, mtime, atime, sxp->st.st_mode);
 311                 if (ret < 0) {
 312                         rsyserr(FERROR_XFER, errno, "failed to set times on %s",
 313                                 full_fname(fname));
 314                         goto cleanup;
 315                 }
 316 -               if (ret == 0) /* ret == 1 if symlink could not be set */
 317 -                       updated = 1;
 318 -               else if (receiver_symlink_times)
 319 -                       file->flags |= FLAG_TIME_FAILED;
 320 +               if (ret > 0) { /* ret == 1 if symlink could not be set */
 321 +                       updated = 0;
 322 +                       if (receiver_symlink_times)
 323 +                               file->flags |= FLAG_TIME_FAILED;
 324 +               }
 325         }
 326
 327         change_uid = am_root && uid_ndx && sxp->st.st_uid != (uid_t)F_OWNER(file);
 328 @@ -531,7 +551,7 @@ int finish_transfer(const char *fname, const char *fnametmp,
 329
 330         /* Change permissions before putting the file into place. */
 331         set_file_attrs(fnametmp, file, NULL, fnamecmp,
 332 -                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
 333 +                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME | ATTRS_SKIP_ATIME);
 334
 335         /* move tmp file over real file */
 336         if (verbose > 2)
 337 @@ -558,7 +578,7 @@ int finish_transfer(const char *fname, const char *fnametmp,
 338
 339    do_set_file_attrs:
 340         set_file_attrs(fnametmp, file, NULL, fnamecmp,
 341 -                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
 342 +                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME | ATTRS_SKIP_ATIME);
 343
 344         if (temp_copy_name) {
 345                 if (do_rename(fnametmp, fname) < 0) {
 346 diff --git a/rsync.h b/rsync.h
 347 --- a/rsync.h
 348 +++ b/rsync.h
 349 @@ -60,6 +60,7 @@
 350  #define XMIT_RDEV_MINOR_8_pre30 (1<<11)        /* protocols 28 - 29  */
 351  #define XMIT_GROUP_NAME_FOLLOWS (1<<11) /* protocols 30 - now */
 352  #define XMIT_HLINK_FIRST (1<<12)       /* protocols 30 - now (HLINKED files only) */
 353 +#define XMIT_SAME_ATIME (1<<13)                /* protocols ?? - now */
 354
 355  /* These flags are used in the live flist data. */
 356
 357 @@ -149,6 +150,7 @@
 358
 359  #define ATTRS_REPORT           (1<<0)
 360  #define ATTRS_SKIP_MTIME       (1<<1)
 361 +#define ATTRS_SKIP_ATIME       (1<<2)
 362
 363  #define FULL_FLUSH     1
 364  #define NORMAL_FLUSH   0
 365 @@ -620,12 +622,14 @@ extern int file_extra_cnt;
 366  extern int inc_recurse;
 367  extern int uid_ndx;
 368  extern int gid_ndx;
 369 +extern int atimes_ndx;
 370  extern int acls_ndx;
 371  extern int xattrs_ndx;
 372
 373  #define FILE_STRUCT_LEN (offsetof(struct file_struct, basename))
 374  #define EXTRA_LEN (sizeof (union file_extras))
 375  #define PTR_EXTRA_CNT ((sizeof (char *) + EXTRA_LEN - 1) / EXTRA_LEN)
 376 +#define TIME_EXTRA_CNT ((SIZEOF_TIME_T + EXTRA_LEN - 1) / EXTRA_LEN)
 377  #define DEV_EXTRA_CNT 2
 378  #define DIRNODE_EXTRA_CNT 3
 379  #define SUM_EXTRA_CNT ((MAX_DIGEST_LEN + EXTRA_LEN - 1) / EXTRA_LEN)
 380 diff --git a/rsync.yo b/rsync.yo
 381 --- a/rsync.yo
 382 +++ b/rsync.yo
 383 @@ -349,6 +349,7 @@ to the detailed description below for a complete description.  verb(
 384   -D                          same as --devices --specials
 385   -t, --times                 preserve modification times
 386   -O, --omit-dir-times        omit directories from --times
 387 + -U, --atimes                preserve access (use) times
 388       --super                 receiver attempts super-user activities
 389       --fake-super            store/recover privileged attrs using xattrs
 390   -S, --sparse                handle sparse files efficiently
 391 @@ -987,6 +988,12 @@ it is preserving modification times (see bf(--times)).  If NFS is sharing
 392  the directories on the receiving side, it is a good idea to use bf(-O).
 393  This option is inferred if you use bf(--backup) without bf(--backup-dir).
 394
 395 +dit(bf(-U, --atimes)) This tells rsync to set the access (use) times of the
 396 +destination files to the same value as the source files.  Note that the
 397 +reading of the source file may update the atime of the source files, so
 398 +repeated rsync runs with --atimes may be needed if you want to force the
 399 +access-time values to be 100% identical on the two systems.
 400 +
 401  dit(bf(--super)) This tells the receiving side to attempt super-user
 402  activities even if the receiving rsync wasn't run by the super-user.  These
 403  activities include: preserving users via the bf(--owner) option, preserving
 404 @@ -1689,7 +1696,10 @@ quote(itemization(
 405    sender's value (requires bf(--owner) and super-user privileges).
 406    it() A bf(g) means the group is different and is being updated to the
 407    sender's value (requires bf(--group) and the authority to set the group).
 408 -  it() The bf(u) slot is reserved for future use.
 409 +  it() A bf(u) means the access (use) time is different and is being updated to
 410 +  the sender's value (requires bf(--atimes)).  An alternate value of bf(U)
 411 +  means that the access time will be set to the transfer time, which happens
 412 +  when a symlink or directory is updated.
 413    it() The bf(a) means that the ACL information changed.
 414    it() The bf(x) means that the extended attribute information changed.
 415  ))
 416 diff --git a/testsuite/atimes.test b/testsuite/atimes.test
 417 new file mode 100644
 418 --- /dev/null
 419 +++ b/testsuite/atimes.test
 420 @@ -0,0 +1,17 @@
 421 +#! /bin/sh
 422 +
 423 +# Test rsync copying atimes
 424 +
 425 +. "$suitedir/rsync.fns"
 426 +
 427 +mkdir "$fromdir"
 428 +
 429 +touch "$fromdir/foo"
 430 +touch -a -t 200102031717.42 "$fromdir/foo"
 431 +
 432 +TLS_ARGS=--atimes
 433 +
 434 +checkit "$RSYNC -rtUgvvv \"$fromdir/\" \"$todir/\"" "$fromdir" "$todir"
 435 +
 436 +# The script would have aborted on error, so getting here means we've won.
 437 +exit 0
 438 diff --git a/testsuite/rsync.fns b/testsuite/rsync.fns
 439 --- a/testsuite/rsync.fns
 440 +++ b/testsuite/rsync.fns
 441 @@ -192,6 +192,10 @@ checkit() {
 442      # We can just write everything to stdout/stderr, because the
 443      # wrapper hides it unless there is a problem.
 444
 445 +    if test x$TLS_ARGS = x--atimes; then
 446 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
 447 +    fi
 448 +
 449      echo "Running: \"$1\""
 450      eval "$1"
 451      status=$?
 452 @@ -199,10 +203,13 @@ checkit() {
 453         failed="YES";
 454      fi
 455
 456 +    if test x$TLS_ARGS != x--atimes; then
 457 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
 458 +    fi
 459 +
 460      echo "-------------"
 461      echo "check how the directory listings compare with diff:"
 462      echo ""
 463 -    ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
 464      ( cd "$3" && rsync_ls_lR . ) > "$tmpdir/ls-to"
 465      diff $diffopt "$tmpdir/ls-from" "$tmpdir/ls-to" || failed=YES
 466
 467 diff --git a/tls.c b/tls.c
 468 --- a/tls.c
 469 +++ b/tls.c
 470 @@ -105,6 +105,8 @@ static int stat_xattr(const char *fname, STRUCT_STAT *fst)
 471
 472  #endif
 473
 474 +static int display_atimes = 0;
 475 +
 476  static void failed(char const *what, char const *where)
 477  {
 478         fprintf(stderr, PROGRAM ": %s %s: %s\n",
 479 @@ -112,12 +114,29 @@ static void failed(char const *what, char const *where)
 480         exit(1);
 481  }
 482
 483 +static void storetime(char *dest, time_t t, size_t destsize)
 484 +{
 485 +       if (t) {
 486 +               struct tm *mt = gmtime(&t);
 487 +
 488 +               snprintf(dest, destsize,
 489 +                       "%04d-%02d-%02d %02d:%02d:%02d ",
 490 +                       (int)mt->tm_year + 1900,
 491 +                       (int)mt->tm_mon + 1,
 492 +                       (int)mt->tm_mday,
 493 +                       (int)mt->tm_hour,
 494 +                       (int)mt->tm_min,
 495 +                       (int)mt->tm_sec);
 496 +       } else
 497 +               strlcpy(dest, "                    ", destsize);
 498 +}
 499 +
 500  static void list_file(const char *fname)
 501  {
 502         STRUCT_STAT buf;
 503         char permbuf[PERMSTRING_SIZE];
 504 -       struct tm *mt;
 505 -       char datebuf[50];
 506 +       char mtimebuf[50];
 507 +       char atimebuf[50];
 508         char linkbuf[4096];
 509
 510         if (do_lstat(fname, &buf) < 0)
 511 @@ -154,19 +173,11 @@ static void list_file(const char *fname)
 512
 513         permstring(permbuf, buf.st_mode);
 514
 515 -       if (buf.st_mtime) {
 516 -               mt = gmtime(&buf.st_mtime);
 517 -
 518 -               snprintf(datebuf, sizeof datebuf,
 519 -                       "%04d-%02d-%02d %02d:%02d:%02d",
 520 -                       (int)mt->tm_year + 1900,
 521 -                       (int)mt->tm_mon + 1,
 522 -                       (int)mt->tm_mday,
 523 -                       (int)mt->tm_hour,
 524 -                       (int)mt->tm_min,
 525 -                       (int)mt->tm_sec);
 526 -       } else
 527 -               strlcpy(datebuf, "                   ", sizeof datebuf);
 528 +       storetime(mtimebuf, buf.st_mtime, sizeof mtimebuf);
 529 +       if (display_atimes)
 530 +               storetime(atimebuf, S_ISDIR(buf.st_mode) ? 0 : buf.st_atime, sizeof atimebuf);
 531 +       else
 532 +               atimebuf[0] = '\0';
 533
 534         /* TODO: Perhaps escape special characters in fname? */
 535
 536 @@ -177,13 +188,14 @@ static void list_file(const char *fname)
 537                     (long)minor(buf.st_rdev));
 538         } else /* NB: use double for size since it might not fit in a long. */
 539                 printf("%12.0f", (double)buf.st_size);
 540 -       printf(" %6ld.%-6ld %6ld %s %s%s\n",
 541 +       printf(" %6ld.%-6ld %6ld %s%s%s%s\n",
 542                (long)buf.st_uid, (long)buf.st_gid, (long)buf.st_nlink,
 543 -              datebuf, fname, linkbuf);
 544 +              mtimebuf, atimebuf, fname, linkbuf);
 545  }
 546
 547  static struct poptOption long_options[] = {
 548    /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
 549 +  {"atimes",          'U', POPT_ARG_NONE,   &display_atimes, 0, 0, 0},
 550  #ifdef SUPPORT_XATTRS
 551    {"fake-super",      'f', POPT_ARG_VAL,    &am_root, -1, 0, 0 },
 552  #endif
 553 @@ -197,6 +209,7 @@ static void tls_usage(int ret)
 554    fprintf(F,"usage: " PROGRAM " [OPTIONS] FILE ...\n");
 555    fprintf(F,"Trivial file listing program for portably checking rsync\n");
 556    fprintf(F,"\nOptions:\n");
 557 +  fprintf(F," -U, --atimes                display access (last-used) times\n");
 558  #ifdef SUPPORT_XATTRS
 559    fprintf(F," -f, --fake-super            display attributes including fake-super xattrs\n");
 560  #endif
 561 diff --git a/util.c b/util.c
 562 --- a/util.c
 563 +++ b/util.c
 564 @@ -122,7 +122,7 @@ NORETURN void overflow_exit(const char *str)
 565         exit_cleanup(RERR_MALLOC);
 566  }
 567
 568 -int set_modtime(const char *fname, time_t modtime, mode_t mode)
 569 +int set_times(const char *fname, time_t modtime, time_t atime, mode_t mode)
 570  {
 571  #if !defined HAVE_LUTIMES || !defined HAVE_UTIMES
 572         if (S_ISLNK(mode))
 573 @@ -130,9 +130,13 @@ int set_modtime(const char *fname, time_t modtime, mode_t mode)
 574  #endif
 575
 576         if (verbose > 2) {
 577 -               rprintf(FINFO, "set modtime of %s to (%ld) %s",
 578 +               char mtimebuf[200];
 579 +
 580 +               strlcpy(mtimebuf, timestring(modtime), sizeof mtimebuf);
 581 +               rprintf(FINFO,
 582 +                       "set modtime, atime of %s to (%ld) %s, (%ld) %s\n",
 583                         fname, (long)modtime,
 584 -                       asctime(localtime(&modtime)));
 585 +                       mtimebuf, (long)atime, timestring(atime));
 586         }
 587
 588         if (dry_run)
 589 @@ -141,7 +145,7 @@ int set_modtime(const char *fname, time_t modtime, mode_t mode)
 590         {
 591  #ifdef HAVE_UTIMES
 592                 struct timeval t[2];
 593 -               t[0].tv_sec = time(NULL);
 594 +               t[0].tv_sec = atime;
 595                 t[0].tv_usec = 0;
 596                 t[1].tv_sec = modtime;
 597                 t[1].tv_usec = 0;
 598 @@ -155,12 +159,12 @@ int set_modtime(const char *fname, time_t modtime, mode_t mode)
 599                 return utimes(fname, t);
 600  #elif defined HAVE_STRUCT_UTIMBUF
 601                 struct utimbuf tbuf;
 602 -               tbuf.actime = time(NULL);
 603 +               tbuf.actime = atime;
 604                 tbuf.modtime = modtime;
 605                 return utime(fname,&tbuf);
 606  #elif defined HAVE_UTIME
 607                 time_t t[2];
 608 -               t[0] = time(NULL);
 609 +               t[0] = atime;
 610                 t[1] = modtime;
 611                 return utime(fname,t);
 612  #else