fb471063e67b20116911dc36d0192ba9941e2240
[rsync-patches.git] / 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 based-on: 63f91976112b8b2118cc17eb5fc8142175566f4f
8 diff --git a/compat.c b/compat.c
9 --- a/compat.c
10 +++ b/compat.c
11 @@ -46,6 +46,7 @@ extern int protocol_version;
12  extern int protect_args;
13  extern int preserve_uid;
14  extern int preserve_gid;
15 +extern int preserve_atimes;
16  extern int preserve_acls;
17  extern int preserve_xattrs;
18  extern int need_messages_from_generator;
19 @@ -63,7 +64,7 @@ extern char *iconv_opt;
20  #endif
21  
22  /* These index values are for the file-list's extra-attribute array. */
23 -int uid_ndx, gid_ndx, acls_ndx, xattrs_ndx, unsort_ndx;
24 +int uid_ndx, gid_ndx, atimes_ndx, acls_ndx, xattrs_ndx, unsort_ndx;
25  
26  int receiver_symlink_times = 0; /* receiver can set the time on a symlink */
27  int sender_symlink_iconv = 0;  /* sender should convert symlink content */
28 @@ -140,6 +141,8 @@ void setup_protocol(int f_out,int f_in)
29                 uid_ndx = ++file_extra_cnt;
30         if (preserve_gid)
31                 gid_ndx = ++file_extra_cnt;
32 +       if (preserve_atimes)
33 +               atimes_ndx = (file_extra_cnt += TIME_EXTRA_CNT);
34         if (preserve_acls && !am_sender)
35                 acls_ndx = ++file_extra_cnt;
36         if (preserve_xattrs)
37 diff --git a/flist.c b/flist.c
38 --- a/flist.c
39 +++ b/flist.c
40 @@ -53,6 +53,7 @@ extern int preserve_specials;
41  extern int delete_during;
42  extern int missing_args;
43  extern int eol_nulls;
44 +extern int atimes_ndx;
45  extern int relative_paths;
46  extern int implied_dirs;
47  extern int ignore_perishable;
48 @@ -397,7 +398,7 @@ static void send_file_entry(int f, const char *fname, struct file_struct *file,
49  #endif
50                             int ndx, int first_ndx)
51  {
52 -       static time_t modtime;
53 +       static time_t modtime, atime;
54         static mode_t mode;
55  #ifdef SUPPORT_HARD_LINKS
56         static int64 dev;
57 @@ -497,6 +498,13 @@ static void send_file_entry(int f, const char *fname, struct file_struct *file,
58                 modtime = file->modtime;
59         if (NSEC_BUMP(file) && protocol_version >= 31)
60                 xflags |= XMIT_MOD_NSEC;
61 +       if (atimes_ndx && !S_ISDIR(mode)) {
62 +               time_t file_atime = f_atime(file);
63 +               if (file_atime == atime)
64 +                       xflags |= XMIT_SAME_ATIME;
65 +               else
66 +                       atime = file_atime;
67 +       }
68  
69  #ifdef SUPPORT_HARD_LINKS
70         if (tmp_dev != -1) {
71 @@ -583,6 +591,8 @@ static void send_file_entry(int f, const char *fname, struct file_struct *file,
72                 write_varint(f, F_MOD_NSEC(file));
73         if (!(xflags & XMIT_SAME_MODE))
74                 write_int(f, to_wire_mode(mode));
75 +       if (atimes_ndx && !S_ISDIR(mode) && !(xflags & XMIT_SAME_ATIME))
76 +               write_varlong(f, atime, 4);
77         if (preserve_uid && !(xflags & XMIT_SAME_UID)) {
78                 if (protocol_version < 30)
79                         write_int(f, uid);
80 @@ -670,7 +680,7 @@ static void send_file_entry(int f, const char *fname, struct file_struct *file,
81  
82  static struct file_struct *recv_file_entry(int f, struct file_list *flist, int xflags)
83  {
84 -       static int64 modtime;
85 +       static int64 modtime, atime;
86         static mode_t mode;
87  #ifdef SUPPORT_HARD_LINKS
88         static int64 dev;
89 @@ -814,6 +824,16 @@ static struct file_struct *recv_file_entry(int f, struct file_list *flist, int x
90                 modtime_nsec = 0;
91         if (!(xflags & XMIT_SAME_MODE))
92                 mode = from_wire_mode(read_int(f));
93 +       if (atimes_ndx && !S_ISDIR(mode) && !(xflags & XMIT_SAME_ATIME)) {
94 +               atime = read_varlong(f, 4);
95 +#if SIZEOF_TIME_T < SIZEOF_INT64
96 +               if (!am_generator && (int64)(time_t)atime != atime) {
97 +                       rprintf(FERROR_XFER,
98 +                               "Access time value of %s truncated on receiver.\n",
99 +                               lastname);
100 +               }
101 +#endif
102 +       }
103  
104         if (chmod_modes && !S_ISLNK(mode) && mode)
105                 mode = tweak_mode(mode, chmod_modes);
106 @@ -974,6 +994,8 @@ static struct file_struct *recv_file_entry(int f, struct file_list *flist, int x
107                 F_GROUP(file) = gid;
108                 file->flags |= gid_flags;
109         }
110 +       if (atimes_ndx)
111 +               f_atime_set(file, (time_t)atime);
112         if (unsort_ndx)
113                 F_NDX(file) = flist->used + flist->ndx_start;
114  
115 @@ -1371,6 +1393,8 @@ struct file_struct *make_file(const char *fname, struct file_list *flist,
116                 F_GROUP(file) = st.st_gid;
117         if (am_generator && st.st_uid == our_uid)
118                 file->flags |= FLAG_OWNED_BY_US;
119 +       if (atimes_ndx)
120 +               f_atime_set(file, st.st_atime);
121  
122         if (basename != thisname)
123                 file->dirname = lastdir;
124 diff --git a/generator.c b/generator.c
125 --- a/generator.c
126 +++ b/generator.c
127 @@ -496,6 +496,9 @@ void itemize(const char *fnamecmp, struct file_struct *file, int ndx, int statre
128                  : iflags & (ITEM_TRANSFER|ITEM_LOCAL_CHANGE) && !(iflags & ITEM_MATCHED)
129                   && (!(iflags & ITEM_XNAME_FOLLOWS) || *xname))
130                         iflags |= ITEM_REPORT_TIME;
131 +               if (atimes_ndx && !S_ISDIR(file->mode) && !S_ISLNK(file->mode)
132 +                && cmp_time(f_atime(file), sxp->st.st_atime) != 0)
133 +                       iflags |= ITEM_REPORT_ATIME;
134  #if !defined HAVE_LCHMOD && !defined HAVE_SETATTRLIST
135                 if (S_ISLNK(file->mode)) {
136                         ;
137 @@ -907,6 +910,8 @@ static int try_dests_reg(struct file_struct *file, char *fname, int ndx,
138                 if (link_dest) {
139                         if (!hard_link_one(file, fname, cmpbuf, 1))
140                                 goto try_a_copy;
141 +                       if (atimes_ndx)
142 +                               set_file_attrs(fname, file, sxp, NULL, 0);
143                         if (preserve_hard_links && F_IS_HLINKED(file))
144                                 finish_hard_link(file, fname, ndx, &sxp->st, itemizing, code, j);
145                         if (!maybe_ATTRS_REPORT && (INFO_GTE(NAME, 2) || stdout_format_has_i > 1)) {
146 @@ -1109,6 +1114,7 @@ static int try_dests_non(struct file_struct *file, char *fname, int ndx,
147  static void list_file_entry(struct file_struct *f)
148  {
149         char permbuf[PERMSTRING_SIZE];
150 +       time_t atime = atimes_ndx ? f_atime(f) : 0;
151         int64 len;
152         int colwidth = human_readable ? 14 : 11;
153  
154 @@ -1124,10 +1130,12 @@ static void list_file_entry(struct file_struct *f)
155  
156  #ifdef SUPPORT_LINKS
157         if (preserve_links && S_ISLNK(f->mode)) {
158 -               rprintf(FINFO, "%s %*s %s %s -> %s\n",
159 +               rprintf(FINFO, "%s %*s %s%s%s %s -> %s\n",
160                         permbuf, colwidth, human_num(len),
161 -                       timestring(f->modtime), f_name(f, NULL),
162 -                       F_SYMLINK(f));
163 +                       timestring(f->modtime),
164 +                       atimes_ndx ? " " : "",
165 +                       atimes_ndx ? timestring(atime) : "",
166 +                       f_name(f, NULL), F_SYMLINK(f));
167         } else
168  #endif
169         if (missing_args == 2 && f->mode == 0) {
170 @@ -1135,9 +1143,12 @@ static void list_file_entry(struct file_struct *f)
171                         colwidth + 31, "*missing",
172                         f_name(f, NULL));
173         } else {
174 -               rprintf(FINFO, "%s %*s %s %s\n",
175 +               rprintf(FINFO, "%s %*s %s%s%s %s\n",
176                         permbuf, colwidth, human_num(len),
177 -                       timestring(f->modtime), f_name(f, NULL));
178 +                       timestring(f->modtime),
179 +                       atimes_ndx ? " " : "",
180 +                       atimes_ndx ? timestring(atime) : "",
181 +                       f_name(f, NULL));
182         }
183  }
184  
185 @@ -2034,7 +2045,7 @@ static void touch_up_dirs(struct file_list *flist, int ndx)
186                         STRUCT_STAT st;
187                         if (link_stat(fname, &st, 0) == 0
188                          && cmp_time(st.st_mtime, file->modtime) != 0)
189 -                               set_modtime(fname, file->modtime, F_MOD_NSEC(file), file->mode);
190 +                               set_times(fname, file->modtime, F_MOD_NSEC(file), file->modtime, file->mode);
191                 }
192                 if (counter >= loopchk_limit) {
193                         if (allowed_lull)
194 diff --git a/ifuncs.h b/ifuncs.h
195 --- a/ifuncs.h
196 +++ b/ifuncs.h
197 @@ -43,6 +43,28 @@ free_xbuf(xbuf *xb)
198         memset(xb, 0, sizeof (xbuf));
199  }
200  
201 +static inline time_t
202 +f_atime(struct file_struct *fp)
203 +{
204 +#if SIZEOF_TIME_T > 4
205 +       time_t atime;
206 +       memcpy(&atime, &REQ_EXTRA(fp, atimes_ndx)->unum, SIZEOF_TIME_T);
207 +       return atime;
208 +#else
209 +       return REQ_EXTRA(fp, atimes_ndx)->unum;
210 +#endif
211 +}
212 +
213 +static inline void
214 +f_atime_set(struct file_struct *fp, time_t atime)
215 +{
216 +#if SIZEOF_TIME_T > 4
217 +       memcpy(&REQ_EXTRA(fp, atimes_ndx)->unum, &atime, SIZEOF_TIME_T);
218 +#else
219 +       REQ_EXTRA(fp, atimes_ndx)->unum = (uint32)atime;
220 +#endif
221 +}
222 +
223  static inline int
224  to_wire_mode(mode_t mode)
225  {
226 diff --git a/log.c b/log.c
227 --- a/log.c
228 +++ b/log.c
229 @@ -721,7 +721,8 @@ static void log_formatted(enum logcode code, const char *format, const char *op,
230                         c[5] = !(iflags & ITEM_REPORT_PERMS) ? '.' : 'p';
231                         c[6] = !(iflags & ITEM_REPORT_OWNER) ? '.' : 'o';
232                         c[7] = !(iflags & ITEM_REPORT_GROUP) ? '.' : 'g';
233 -                       c[8] = !(iflags & ITEM_REPORT_ATIME) ? '.' : 'u';
234 +                       c[8] = !(iflags & ITEM_REPORT_ATIME) ? '.'
235 +                            : S_ISLNK(file->mode) ? 'U' : 'u';
236                         c[9] = !(iflags & ITEM_REPORT_ACL) ? '.' : 'a';
237                         c[10] = !(iflags & ITEM_REPORT_XATTR) ? '.' : 'x';
238                         c[11] = '\0';
239 diff --git a/options.c b/options.c
240 --- a/options.c
241 +++ b/options.c
242 @@ -59,6 +59,7 @@ int preserve_specials = 0;
243  int preserve_uid = 0;
244  int preserve_gid = 0;
245  int preserve_times = 0;
246 +int preserve_atimes = 0;
247  int update_only = 0;
248  int cvs_exclude = 0;
249  int dry_run = 0;
250 @@ -707,6 +708,7 @@ void usage(enum logcode F)
251    rprintf(F,"     --specials              preserve special files\n");
252    rprintf(F," -D                          same as --devices --specials\n");
253    rprintf(F," -t, --times                 preserve modification times\n");
254 +  rprintf(F," -U, --atimes                preserve access (last-used) times\n");
255    rprintf(F," -O, --omit-dir-times        omit directories from --times\n");
256    rprintf(F," -J, --omit-link-times       omit symlinks from --times\n");
257    rprintf(F,"     --super                 receiver attempts super-user activities\n");
258 @@ -865,6 +867,9 @@ static struct poptOption long_options[] = {
259    {"times",           't', POPT_ARG_VAL,    &preserve_times, 1, 0, 0 },
260    {"no-times",         0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
261    {"no-t",             0,  POPT_ARG_VAL,    &preserve_times, 0, 0, 0 },
262 +  {"atimes",          'U', POPT_ARG_VAL,    &preserve_atimes, 1, 0, 0 },
263 +  {"no-atimes",        0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
264 +  {"no-U",             0,  POPT_ARG_VAL,    &preserve_atimes, 0, 0, 0 },
265    {"omit-dir-times",  'O', POPT_ARG_VAL,    &omit_dir_times, 1, 0, 0 },
266    {"no-omit-dir-times",0,  POPT_ARG_VAL,    &omit_dir_times, 0, 0, 0 },
267    {"no-O",             0,  POPT_ARG_VAL,    &omit_dir_times, 0, 0, 0 },
268 @@ -2419,6 +2424,8 @@ void server_options(char **args, int *argc_p)
269                 argstr[x++] = 'D';
270         if (preserve_times)
271                 argstr[x++] = 't';
272 +       if (preserve_atimes)
273 +               argstr[x++] = 'U';
274         if (preserve_perms)
275                 argstr[x++] = 'p';
276         else if (preserve_executability && am_sender)
277 diff --git a/rsync.c b/rsync.c
278 --- a/rsync.c
279 +++ b/rsync.c
280 @@ -458,6 +458,7 @@ int set_file_attrs(const char *fname, struct file_struct *file, stat_x *sxp,
281         int updated = 0;
282         stat_x sx2;
283         int change_uid, change_gid;
284 +       time_t atime, mtime;
285         mode_t new_mode = file->mode;
286         int inherit;
287  
288 @@ -496,22 +497,40 @@ int set_file_attrs(const char *fname, struct file_struct *file, stat_x *sxp,
289                 set_xattr(fname, file, fnamecmp, sxp);
290  #endif
291  
292 +       /* This code must be the first update in the function due to
293 +        * how it uses the "updated" variable. */
294         if (!preserve_times
295          || (!(preserve_times & PRESERVE_DIR_TIMES) && S_ISDIR(sxp->st.st_mode))
296          || (!(preserve_times & PRESERVE_LINK_TIMES) && S_ISLNK(sxp->st.st_mode)))
297                 flags |= ATTRS_SKIP_MTIME;
298 +       if (!atimes_ndx || S_ISDIR(sxp->st.st_mode))
299 +               flags |= ATTRS_SKIP_ATIME;
300         if (!(flags & ATTRS_SKIP_MTIME)
301             && cmp_time(sxp->st.st_mtime, file->modtime) != 0) {
302 -               int ret = set_modtime(fname, file->modtime, F_MOD_NSEC(file), sxp->st.st_mode);
303 +               mtime = file->modtime;
304 +               updated = 1;
305 +       } else
306 +               mtime = sxp->st.st_mtime;
307 +       if (!(flags & ATTRS_SKIP_ATIME)) {
308 +               time_t file_atime = f_atime(file);
309 +               if (cmp_time(sxp->st.st_atime, file_atime) != 0) {
310 +                       atime = file_atime;
311 +                       updated = 1;
312 +               } else
313 +                       atime = sxp->st.st_atime;
314 +       } else
315 +               atime = sxp->st.st_atime;
316 +       if (updated) {
317 +               int ret = set_times(fname, mtime, F_MOD_NSEC(file), atime, sxp->st.st_mode);
318                 if (ret < 0) {
319                         rsyserr(FERROR_XFER, errno, "failed to set times on %s",
320                                 full_fname(fname));
321                         goto cleanup;
322                 }
323 -               if (ret == 0) /* ret == 1 if symlink could not be set */
324 -                       updated = 1;
325 -               else
326 +               if (ret > 0) { /* ret == 1 if symlink could not be set */
327 +                       updated = 0;
328                         file->flags |= FLAG_TIME_FAILED;
329 +               }
330         }
331  
332         change_uid = am_root && uid_ndx && sxp->st.st_uid != (uid_t)F_OWNER(file);
333 @@ -662,7 +681,7 @@ int finish_transfer(const char *fname, const char *fnametmp,
334  
335         /* Change permissions before putting the file into place. */
336         set_file_attrs(fnametmp, file, NULL, fnamecmp,
337 -                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
338 +                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME | ATTRS_SKIP_ATIME);
339  
340         /* move tmp file over real file */
341         if (DEBUG_GTE(RECV, 1))
342 @@ -687,7 +706,7 @@ int finish_transfer(const char *fname, const char *fnametmp,
343  
344    do_set_file_attrs:
345         set_file_attrs(fnametmp, file, NULL, fnamecmp,
346 -                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME);
347 +                      ok_to_set_time ? 0 : ATTRS_SKIP_MTIME | ATTRS_SKIP_ATIME);
348  
349         if (temp_copy_name) {
350                 if (do_rename(fnametmp, fname) < 0) {
351 diff --git a/rsync.h b/rsync.h
352 --- a/rsync.h
353 +++ b/rsync.h
354 @@ -62,6 +62,7 @@
355  #define XMIT_HLINK_FIRST (1<<12)       /* protocols 30 - now (HLINKED files only) */
356  #define XMIT_IO_ERROR_ENDLIST (1<<12)  /* protocols 31*- now (w/XMIT_EXTENDED_FLAGS) (also protocol 30 w/'f' compat flag) */
357  #define XMIT_MOD_NSEC (1<<13)          /* protocols 31 - now */
358 +#define XMIT_SAME_ATIME (1<<14)                /* protocols ?? - now */
359  
360  /* These flags are used in the live flist data. */
361  
362 @@ -165,6 +166,7 @@
363  
364  #define ATTRS_REPORT           (1<<0)
365  #define ATTRS_SKIP_MTIME       (1<<1)
366 +#define ATTRS_SKIP_ATIME       (1<<2)
367  
368  #define FULL_FLUSH     1
369  #define NORMAL_FLUSH   0
370 @@ -708,12 +710,14 @@ extern int file_extra_cnt;
371  extern int inc_recurse;
372  extern int uid_ndx;
373  extern int gid_ndx;
374 +extern int atimes_ndx;
375  extern int acls_ndx;
376  extern int xattrs_ndx;
377  
378  #define FILE_STRUCT_LEN (offsetof(struct file_struct, basename))
379  #define EXTRA_LEN (sizeof (union file_extras))
380  #define PTR_EXTRA_CNT ((sizeof (char *) + EXTRA_LEN - 1) / EXTRA_LEN)
381 +#define TIME_EXTRA_CNT ((SIZEOF_TIME_T + EXTRA_LEN - 1) / EXTRA_LEN)
382  #define DEV_EXTRA_CNT 2
383  #define DIRNODE_EXTRA_CNT 3
384  #define SUM_EXTRA_CNT ((MAX_DIGEST_LEN + EXTRA_LEN - 1) / EXTRA_LEN)
385 diff --git a/rsync.yo b/rsync.yo
386 --- a/rsync.yo
387 +++ b/rsync.yo
388 @@ -369,6 +369,7 @@ to the detailed description below for a complete description.  verb(
389       --specials              preserve special files
390   -D                          same as --devices --specials
391   -t, --times                 preserve modification times
392 + -U, --atimes                preserve access (use) times
393   -O, --omit-dir-times        omit directories from --times
394   -J, --omit-link-times       omit symlinks from --times
395       --super                 receiver attempts super-user activities
396 @@ -1162,6 +1163,12 @@ cause the next transfer to behave as if it used bf(-I), causing all files to be
397  updated (though rsync's delta-transfer algorithm will make the update fairly efficient
398  if the files haven't actually changed, you're much better off using bf(-t)).
399  
400 +dit(bf(-U, --atimes)) This tells rsync to set the access (use) times of the
401 +destination files to the same value as the source files.  Note that the
402 +reading of the source file may update the atime of the source files, so
403 +repeated rsync runs with --atimes may be needed if you want to force the
404 +access-time values to be 100% identical on the two systems.
405 +
406  dit(bf(-O, --omit-dir-times)) This tells rsync to omit directories when
407  it is preserving modification times (see bf(--times)).  If NFS is sharing
408  the directories on the receiving side, it is a good idea to use bf(-O).
409 @@ -2092,7 +2099,10 @@ quote(itemization(
410    sender's value (requires bf(--owner) and super-user privileges).
411    it() A bf(g) means the group is different and is being updated to the
412    sender's value (requires bf(--group) and the authority to set the group).
413 -  it() The bf(u) slot is reserved for future use.
414 +  it() A bf(u) means the access (use) time is different and is being updated to
415 +  the sender's value (requires bf(--atimes)).  An alternate value of bf(U)
416 +  means that the access time will be set to the transfer time, which happens
417 +  when a symlink or directory is updated.
418    it() The bf(a) means that the ACL information changed.
419    it() The bf(x) means that the extended attribute information changed.
420  ))
421 diff --git a/syscall.c b/syscall.c
422 --- a/syscall.c
423 +++ b/syscall.c
424 @@ -349,15 +349,15 @@ OFF_T do_lseek(int fd, OFF_T offset, int whence)
425  }
426  
427  #ifdef HAVE_UTIMENSAT
428 -int do_utimensat(const char *fname, time_t modtime, uint32 mod_nsec)
429 +int do_utimensat(const char *fname, time_t modtime, uint32 mod_nsec, time_t atime, uint32 a_nsec)
430  {
431         struct timespec t[2];
432  
433         if (dry_run) return 0;
434         RETURN_ERROR_IF_RO_OR_LO;
435  
436 -       t[0].tv_sec = 0;
437 -       t[0].tv_nsec = UTIME_NOW;
438 +       t[0].tv_sec = atime;
439 +       t[0].tv_nsec = a_nsec;
440         t[1].tv_sec = modtime;
441         t[1].tv_nsec = mod_nsec;
442         return utimensat(AT_FDCWD, fname, t, AT_SYMLINK_NOFOLLOW);
443 @@ -365,15 +365,15 @@ int do_utimensat(const char *fname, time_t modtime, uint32 mod_nsec)
444  #endif
445  
446  #ifdef HAVE_LUTIMES
447 -int do_lutimes(const char *fname, time_t modtime, uint32 mod_nsec)
448 +int do_lutimes(const char *fname, time_t modtime, uint32 mod_nsec, time_t atime, uint32 a_nsec)
449  {
450         struct timeval t[2];
451  
452         if (dry_run) return 0;
453         RETURN_ERROR_IF_RO_OR_LO;
454  
455 -       t[0].tv_sec = time(NULL);
456 -       t[0].tv_usec = 0;
457 +       t[0].tv_sec = atime;
458 +       t[0].tv_usec = a_nsec;
459         t[1].tv_sec = modtime;
460         t[1].tv_usec = mod_nsec / 1000;
461         return lutimes(fname, t);
462 @@ -381,22 +381,22 @@ int do_lutimes(const char *fname, time_t modtime, uint32 mod_nsec)
463  #endif
464  
465  #ifdef HAVE_UTIMES
466 -int do_utimes(const char *fname, time_t modtime, uint32 mod_nsec)
467 +int do_utimes(const char *fname, time_t modtime, uint32 mod_nsec, time_t atime, uint32 a_nsec)
468  {
469         struct timeval t[2];
470  
471         if (dry_run) return 0;
472         RETURN_ERROR_IF_RO_OR_LO;
473  
474 -       t[0].tv_sec = time(NULL);
475 -       t[0].tv_usec = 0;
476 +       t[0].tv_sec = atime;
477 +       t[0].tv_usec = a_nsec;
478         t[1].tv_sec = modtime;
479         t[1].tv_usec = mod_nsec / 1000;
480         return utimes(fname, t);
481  }
482  
483  #elif defined HAVE_UTIME
484 -int do_utime(const char *fname, time_t modtime, UNUSED(uint32 mod_nsec))
485 +int do_utime(const char *fname, time_t modtime, UNUSED(uint32 mod_nsec), time_t atime, UNUSED(uint32 a_nsec))
486  {
487  #ifdef HAVE_STRUCT_UTIMBUF
488         struct utimbuf tbuf;
489 @@ -408,11 +408,11 @@ int do_utime(const char *fname, time_t modtime, UNUSED(uint32 mod_nsec))
490         RETURN_ERROR_IF_RO_OR_LO;
491  
492  # ifdef HAVE_STRUCT_UTIMBUF
493 -       tbuf.actime = time(NULL);
494 +       tbuf.actime = atime;
495         tbuf.modtime = modtime;
496         return utime(fname, &tbuf);
497  # else
498 -       t[0] = time(NULL);
499 +       t[0] = atime;
500         t[1] = modtime;
501         return utime(fname, t);
502  # endif
503 diff --git a/testsuite/atimes.test b/testsuite/atimes.test
504 new file mode 100644
505 --- /dev/null
506 +++ b/testsuite/atimes.test
507 @@ -0,0 +1,17 @@
508 +#! /bin/sh
509 +
510 +# Test rsync copying atimes
511 +
512 +. "$suitedir/rsync.fns"
513 +
514 +mkdir "$fromdir"
515 +
516 +touch "$fromdir/foo"
517 +touch -a -t 200102031717.42 "$fromdir/foo"
518 +
519 +TLS_ARGS=--atimes
520 +
521 +checkit "$RSYNC -rtUgvvv \"$fromdir/\" \"$todir/\"" "$fromdir" "$todir"
522 +
523 +# The script would have aborted on error, so getting here means we've won.
524 +exit 0
525 diff --git a/testsuite/daemon.test b/testsuite/daemon.test
526 --- a/testsuite/daemon.test
527 +++ b/testsuite/daemon.test
528 @@ -27,7 +27,7 @@ outfile="$scratchdir/rsync.out"
529  SSH="src/support/lsh.sh --no-cd"
530  FILE_REPL='s/^\([^d][^ ]*\) *\(..........[0-9]\) /\1 \2 /'
531  DIR_REPL='s/^\(d[^ ]*\)  *[0-9][.,0-9]* /\1         DIR /'
532 -LS_REPL='s;[0-9][0-9][0-9][0-9]/[0-9][0-9]/[0-9][0-9] [0-9][0-9]:[0-9][0-9]:[0-9][0-9];####/##/## ##:##:##;'
533 +LS_REPL='s;[0-9][0-9][0-9][0-9]/[0-9][0-9]/[0-9][0-9] [0-9][0-9]:[0-9][0-9]:[0-9][0-9] ;####/##/## ##:##:## ;'
534  
535  build_rsyncd_conf
536  
537 diff --git a/testsuite/rsync.fns b/testsuite/rsync.fns
538 --- a/testsuite/rsync.fns
539 +++ b/testsuite/rsync.fns
540 @@ -219,6 +219,14 @@ checkit() {
541      # We can just write everything to stdout/stderr, because the
542      # wrapper hides it unless there is a problem.
543  
544 +    case "x$TLS_ARGS" in
545 +    *--atimes*)
546 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
547 +       ;;
548 +    *)
549 +       ;;
550 +    esac
551 +
552      echo "Running: \"$1\""  
553      eval "$1" 
554      status=$?
555 @@ -226,10 +234,17 @@ checkit() {
556         failed="$failed status=$status"
557      fi
558  
559 +    case "x$TLS_ARGS" in
560 +    *--atimes*)
561 +       ;;
562 +    *)
563 +       ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
564 +       ;;
565 +    esac
566 +
567      echo "-------------"
568      echo "check how the directory listings compare with diff:"
569      echo ""
570 -    ( cd "$2" && rsync_ls_lR . ) > "$tmpdir/ls-from"
571      ( cd "$3" && rsync_ls_lR . ) > "$tmpdir/ls-to"
572      diff $diffopt "$tmpdir/ls-from" "$tmpdir/ls-to" || failed="$failed dir-diff"
573  
574 diff --git a/tls.c b/tls.c
575 --- a/tls.c
576 +++ b/tls.c
577 @@ -109,6 +109,8 @@ static int stat_xattr(const char *fname, STRUCT_STAT *fst)
578  
579  #endif
580  
581 +static int display_atimes = 0;
582 +
583  static void failed(char const *what, char const *where)
584  {
585         fprintf(stderr, PROGRAM ": %s %s: %s\n",
586 @@ -116,13 +118,38 @@ static void failed(char const *what, char const *where)
587         exit(1);
588  }
589  
590 +static void storetime(char *dest, size_t destsize, time_t t, int nsecs)
591 +{
592 +       if (t) {
593 +               int len;
594 +               struct tm *mt = gmtime(&t);
595 +
596 +               len = snprintf(dest, destsize,
597 +                       "%04d-%02d-%02d %02d:%02d:%02d",
598 +                       (int)mt->tm_year + 1900,
599 +                       (int)mt->tm_mon + 1,
600 +                       (int)mt->tm_mday,
601 +                       (int)mt->tm_hour,
602 +                       (int)mt->tm_min,
603 +                       (int)mt->tm_sec);
604 +               if (nsecs >= 0 && len >= 0)
605 +                       snprintf(dest + len, destsize - len, ".%09d", nsecs);
606 +       } else {
607 +               int has_nsecs = nsecs >= 0 ? 1 : 0;
608 +               int len = MIN(19 + 9*has_nsecs, (int)destsize - 1);
609 +               memset(dest, ' ', len);
610 +               dest[len] = '\0';
611 +       }
612 +}
613 +
614  static void list_file(const char *fname)
615  {
616         STRUCT_STAT buf;
617         char permbuf[PERMSTRING_SIZE];
618 -       struct tm *mt;
619 -       char datebuf[50];
620 +       char mtimebuf[50];
621 +       char atimebuf[50];
622         char linkbuf[4096];
623 +       int nsecs;
624  
625         if (do_lstat(fname, &buf) < 0)
626                 failed("stat", fname);
627 @@ -159,30 +186,17 @@ static void list_file(const char *fname)
628         }
629  
630         permstring(permbuf, buf.st_mode);
631 -
632 -       if (buf.st_mtime) {
633 -               int len;
634 -               mt = gmtime(&buf.st_mtime);
635 -
636 -               len = snprintf(datebuf, sizeof datebuf,
637 -                       "%04d-%02d-%02d %02d:%02d:%02d",
638 -                       (int)mt->tm_year + 1900,
639 -                       (int)mt->tm_mon + 1,
640 -                       (int)mt->tm_mday,
641 -                       (int)mt->tm_hour,
642 -                       (int)mt->tm_min,
643 -                       (int)mt->tm_sec);
644  #ifdef ST_MTIME_NSEC
645 -               if (nsec_times) {
646 -                       snprintf(datebuf + len, sizeof datebuf - len,
647 -                               ".%09d", (int)buf.ST_MTIME_NSEC);
648 -               }
649 +       if (nsec_times)
650 +               nsecs = (int)buf.ST_MTIME_NSEC;
651 +       else
652  #endif
653 -       } else {
654 -               int len = MIN(19 + 9*nsec_times, (int)sizeof datebuf - 1);
655 -               memset(datebuf, ' ', len);
656 -               datebuf[len] = '\0';
657 -       }
658 +               nsecs = -1;
659 +       storetime(mtimebuf, sizeof mtimebuf, buf.st_mtime, nsecs);
660 +       if (display_atimes)
661 +               storetime(atimebuf, sizeof atimebuf, S_ISDIR(buf.st_mode) ? 0 : buf.st_atime, -1);
662 +       else
663 +               atimebuf[0] = '\0';
664  
665         /* TODO: Perhaps escape special characters in fname? */
666  
667 @@ -193,13 +207,14 @@ static void list_file(const char *fname)
668                     (long)minor(buf.st_rdev));
669         } else
670                 printf("%15s", do_big_num(buf.st_size, 1, NULL));
671 -       printf(" %6ld.%-6ld %6ld %s %s%s\n",
672 +       printf(" %6ld.%-6ld %6ld %s%s%s%s\n",
673                (long)buf.st_uid, (long)buf.st_gid, (long)buf.st_nlink,
674 -              datebuf, fname, linkbuf);
675 +              mtimebuf, atimebuf, fname, linkbuf);
676  }
677  
678  static struct poptOption long_options[] = {
679    /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
680 +  {"atimes",          'U', POPT_ARG_NONE,   &display_atimes, 0, 0, 0},
681    {"link-times",      'l', POPT_ARG_NONE,   &link_times, 0, 0, 0 },
682    {"link-owner",      'L', POPT_ARG_NONE,   &link_owner, 0, 0, 0 },
683  #ifdef SUPPORT_XATTRS
684 @@ -218,6 +233,7 @@ static void tls_usage(int ret)
685    fprintf(F,"usage: " PROGRAM " [OPTIONS] FILE ...\n");
686    fprintf(F,"Trivial file listing program for portably checking rsync\n");
687    fprintf(F,"\nOptions:\n");
688 +  fprintf(F," -U, --atimes                display access (last-used) times\n");
689    fprintf(F," -l, --link-times            display the time on a symlink\n");
690    fprintf(F," -L, --link-owner            display the owner+group on a symlink\n");
691  #ifdef SUPPORT_XATTRS
692 diff --git a/util.c b/util.c
693 --- a/util.c
694 +++ b/util.c
695 @@ -116,20 +116,24 @@ void print_child_argv(const char *prefix, char **cmd)
696  
697  /* This returns 0 for success, 1 for a symlink if symlink time-setting
698   * is not possible, or -1 for any other error. */
699 -int set_modtime(const char *fname, time_t modtime, uint32 mod_nsec, mode_t mode)
700 +int set_times(const char *fname, time_t modtime, uint32 mod_nsec, time_t atime, mode_t mode)
701  {
702         static int switch_step = 0;
703  
704         if (DEBUG_GTE(TIME, 1)) {
705 -               rprintf(FINFO, "set modtime of %s to (%ld) %s",
706 +               char mtimebuf[200];
707 +
708 +               strlcpy(mtimebuf, timestring(modtime), sizeof mtimebuf);
709 +               rprintf(FINFO,
710 +                       "set modtime, atime of %s to (%ld) %s, (%ld) %s\n",
711                         fname, (long)modtime,
712 -                       asctime(localtime(&modtime)));
713 +                       mtimebuf, (long)atime, timestring(atime));
714         }
715  
716         switch (switch_step) {
717  #ifdef HAVE_UTIMENSAT
718  #include "case_N.h"
719 -               if (do_utimensat(fname, modtime, mod_nsec) == 0)
720 +               if (do_utimensat(fname, modtime, mod_nsec, atime, 0) == 0)
721                         break;
722                 if (errno != ENOSYS)
723                         return -1;
724 @@ -139,7 +143,7 @@ int set_modtime(const char *fname, time_t modtime, uint32 mod_nsec, mode_t mode)
725  
726  #ifdef HAVE_LUTIMES
727  #include "case_N.h"
728 -               if (do_lutimes(fname, modtime, mod_nsec) == 0)
729 +               if (do_lutimes(fname, modtime, mod_nsec, atime, 0) == 0)
730                         break;
731                 if (errno != ENOSYS)
732                         return -1;
733 @@ -158,10 +162,10 @@ int set_modtime(const char *fname, time_t modtime, uint32 mod_nsec, mode_t mode)
734  
735  #include "case_N.h"
736  #ifdef HAVE_UTIMES
737 -               if (do_utimes(fname, modtime, mod_nsec) == 0)
738 +               if (do_utimes(fname, modtime, mod_nsec, atime, 0) == 0)
739                         break;
740  #else
741 -               if (do_utime(fname, modtime, mod_nsec) == 0)
742 +               if (do_utime(fname, modtime, mod_nsec, atime, 0) == 0)
743                         break;
744  #endif
745