Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751446AbXBVSJc (ORCPT ); Thu, 22 Feb 2007 13:09:32 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751440AbXBVSJc (ORCPT ); Thu, 22 Feb 2007 13:09:32 -0500 Received: from mail-gw1.sa.eol.hu ([212.108.200.67]:44759 "EHLO mail-gw1.sa.eol.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751104AbXBVSJb (ORCPT ); Thu, 22 Feb 2007 13:09:31 -0500 To: staubach@redhat.com CC: miklos@szeredi.hu, akpm@linux-foundation.org, hugh@veritas.com, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org In-reply-to: <45DDD55F.4060106@redhat.com> (message from Peter Staubach on Thu, 22 Feb 2007 12:39:43 -0500) Subject: Re: [PATCH] update ctime and mtime for mmaped write References: <20070221202615.a0a167f4.akpm@linux-foundation.org> <45DDD55F.4060106@redhat.com> Message-Id: From: Miklos Szeredi Date: Thu, 22 Feb 2007 19:08:43 +0100 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1333 Lines: 35 > >> Why is the flag checked in __fput()? > >> > > > > It's because of this bit in the standard: > > > > If there is no such call and if the underlying file is modified > > as a result of a write reference, then these fields shall be > > marked for update at some time after the write reference. > > > > It could be done in munmap/mremap, but it seemed more difficult to > > track down all the places where the vma is removed. But yes, that may > > be a nicer solution. > > It seems to me that, with this support, a file, which is mmap'd, > modified, but never msync'd or munmap'd, will never get its mtime > updated. Or did I miss that? > > I also don't see how an mmap'd block device will get its mtime > updated either. __fput() will be called when there are no more references to 'file', then it will update the time if the flag is set. This applies to regular files as well as devices. But I've moved the check from __fput to remove_vma() in the next revision of the patch, which would give slightly nicer semantics, and be equally conforming. Thanks, Miklos - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/