From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 18 Jan 2008 13:28:50 -0500 From: Rik van Riel Subject: Re: [PATCH -v6 2/2] Updating ctime and mtime for memory-mapped files Message-ID: <20080118132850.044537e5@bree.surriel.com> In-Reply-To: References: <12006091182260-git-send-email-salikhmetov@gmail.com> <12006091211208-git-send-email-salikhmetov@gmail.com> <1200651337.5920.9.camel@twins> <1200651958.5920.12.camel@twins> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org Return-Path: To: Miklos Szeredi Cc: torvalds@linux-foundation.org, peterz@infradead.org, salikhmetov@gmail.com, linux-mm@kvack.org, jakob@unthought.net, linux-kernel@vger.kernel.org, valdis.kletnieks@vt.edu, ksm@42.dk, staubach@redhat.com, jesper.juhl@gmail.com, akpm@linux-foundation.org, protasnb@gmail.com, r.e.wolff@bitwizard.nl, hidave.darkstar@gmail.com, hch@infradead.org List-ID: On Fri, 18 Jan 2008 19:11:47 +0100 Miklos Szeredi wrote: > > And even in that four-liner, I suspect that the *last* two lines are > > actually incorrect: there's no point in updating the file time when the > > page *becomes* dirty, > > Actually all four lines do that. The first two for a write access on > a present, read-only pte, the other two for a write on a non-present > pte. > > > we should update the file time when it is marked > > clean, and "msync(MS_SYNC)" should update it as part of *that*. > > That would need a new page flag (PG_mmap_dirty?). Do we have one > available? I thought the page writing stuff looked at (and cleared) the pte dirty bit, too? -- All rights reversed. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org