From owner-freebsd-bugs@FreeBSD.ORG Wed Jan 11 06:40:10 2006 Return-Path: X-Original-To: freebsd-bugs@hub.freebsd.org Delivered-To: freebsd-bugs@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 7AEF816A41F for ; Wed, 11 Jan 2006 06:40:10 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id 43BA343D48 for ; Wed, 11 Jan 2006 06:40:10 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.4/8.13.4) with ESMTP id k0B6e9Ag060781 for ; Wed, 11 Jan 2006 06:40:09 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.4/8.13.4/Submit) id k0B6e9Iu060779; Wed, 11 Jan 2006 06:40:09 GMT (envelope-from gnats) Date: Wed, 11 Jan 2006 06:40:09 GMT Message-Id: <200601110640.k0B6e9Iu060779@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org From: Bruce Evans Cc: Subject: Re: bin/91622: /bin/cp does not update atime of the source file X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Bruce Evans List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 11 Jan 2006 06:40:10 -0000 The following reply was made to PR bin/91622; it has been noted by GNATS. From: Bruce Evans To: joe Cc: freebsd-gnats-submit@FreeBSD.org, freebsd-bugs@FreeBSD.org, dds@FreeBSD.org Subject: Re: bin/91622: /bin/cp does not update atime of the source file Date: Wed, 11 Jan 2006 17:34:58 +1100 (EST) On Wed, 11 Jan 2006, joe wrote: >> Environment: > RELENG_4 20051028 >> Description: > [cp fails to update atime] > > this occurs regardless of whether or not the user has > rwx permissions on the source file. does not apply of > course if the source filesystem is mounted read-only. > they are read-write here. > > since the source file is indeed being read, the atime should be updated. This is because cp(1) doesn't actually read(2) the file. It mmap(2)'s the file and reads memory. POSIX doesn't seem to require that cp sets atimes or otherwise acts as if it reads the file. Similar for cmp(1) and other utilities that use mmap() to read files (install(1) is another). The problem is mostly fixed in -current by "setting" (not actually setting, but marking for update) atimes in mmap(). I think to be strictly correct, setting of atimes should be nearer to munmap(2) than mmap(). No read has occurred from userland's viewpoint at mmap() time. However, since the OS can't tell if applications read mmapped memory unless their is a pagefault, the atime would have to be set on every pagefault to be as correct as possible, and I think mmap() prefaults some pages so it would have to set the atime whether or not the application reads the pages. So I think setting atimes in mmap() is required in practice. This leaves the problem that later pagefaults never set the atime, so the atime set by mmap() becomes out of date. I think the atime should be set by mmap() of there is at least one later pagefault. Setting it for every pagefault may be too inefficient. Bruce