Bug 249362

Summary: sys.fs.tmpfs.times_test.{empty,non_empty} failed after (r365768, r365787]
Product: Base System Reporter: Li-Wen Hsu <lwhsu>
Component: testsAssignee: Konstantin Belousov <kib>
Status: Closed FIXED    
Severity: Affects Only Me CC: kib
Priority: ---    
Version: CURRENT   
Hardware: Any   
OS: Any   
See Also: https://reviews.freebsd.org/D26451

Comment 1 commit-hook freebsd_committer 2020-09-16 09:58:45 UTC
A commit references this bug:

Author: lwhsu
Date: Wed Sep 16 09:58:16 UTC 2020
New revision: 365793
URL: https://svnweb.freebsd.org/changeset/base/365793

Log:
  Temporarily skip sys.fs.tmpfs.times_test.{empty,non_empty} in CI

  PR:		249362
  Sponsored by:	The FreeBSD Foundation

Changes:
  head/contrib/netbsd-tests/fs/tmpfs/t_times.sh
Comment 2 Li-Wen Hsu freebsd_committer 2020-09-16 10:00:08 UTC
kib, can you help to check if this is related to r365783 to r365787? Should the codes or the tests be fixed? Thanks!
Comment 3 commit-hook freebsd_committer 2020-09-16 21:29:20 UTC
A commit references this bug:

Author: kib
Date: Wed Sep 16 21:28:19 UTC 2020
New revision: 365810
URL: https://svnweb.freebsd.org/changeset/base/365810

Log:
  tmpfs: restore atime updates for reads from page cache.

  Split TMPFS_NODE_ACCCESSED bit into dedicated byte that can be updated
  atomically without locks or (locked) atomics.

  tn_update_getattr() change also contains unrelated bug fix.

  Reported by:	lwhsu
  PR:	249362
  Reviewed by:	markj (previous version)
  Discussed with:	mjg
  Sponsored by:	The FreeBSD Foundation
  Differential revision:	https://reviews.freebsd.org/D26451

Changes:
  head/sys/fs/tmpfs/tmpfs.h
  head/sys/fs/tmpfs/tmpfs_fifoops.c
  head/sys/fs/tmpfs/tmpfs_subr.c
  head/sys/fs/tmpfs/tmpfs_vnops.c
Comment 4 commit-hook freebsd_committer 2020-09-17 07:14:43 UTC
A commit references this bug:

Author: lwhsu
Date: Thu Sep 17 07:13:42 UTC 2020
New revision: 365831
URL: https://svnweb.freebsd.org/changeset/base/365831

Log:
  Revert r249362, atime update in tmpfs is fixed in r365810

  PR:		249362
  Sponsored by:	The FreeBSD Foundation

Changes:
  head/contrib/netbsd-tests/fs/tmpfs/t_times.sh