public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrew Pimlott <andrew@pimlott.net>
To: Pavel Machek <pavel@suse.cz>
Cc: Andi Kleen <ak@suse.de>, linux-kernel@vger.kernel.org
Subject: Re: The return of the return of crunch time (2.5 merge candidate list 1.6)
Date: Fri, 8 Nov 2002 17:02:13 -0500	[thread overview]
Message-ID: <20021108220213.GC16051@pimlott.net> (raw)
In-Reply-To: <20020115174416.GC2015@zaurus>

On Tue, Jan 15, 2002 at 12:44:28PM -0500, Pavel Machek wrote:
> > The point of my patchkit is to allow the file systems
> > who support better resolution to handle it properly. Other filesystems
> > are not worse than before when they flush inodes (and better off when
> > they keep everything in ram for your build because then they will enjoy 
> > full time resolution) 
> 
> What about always rounding down even when inode is
> in memory? That is both simple and consistent.

I assume you mean, for filesystems that don't support high
resolution.  That is what I think should be done as well.  People
have gotten along with second resolution all these years, so it's no
great tragedy to make them continue; plus, it seems like the common
filesystems will support high resolution soon anyway.

Paul Eggert listed lots of programs that could be broken if
timestamps jump around.  They could all implement heuristic
work-arounds, but that would be 1) a miracle and 2) a waste of
effort.

The only issue (as far as I can tell) is knowing when to round.
Hard-coding a list of filesystems seems reasonable (though even that
could be wrong if I load a newer filesystem module).  Ideally,
though, you would add a simple hook into the filesystem that asks it
to round the timestamp for you.  This would also accommodate
filesystems that don't store seconds or nanoseconds, but some other
resolution.

Andrew

  reply	other threads:[~2002-11-09  1:10 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200210251557.55202.landley@trommello.org.suse.lists.linux.kernel>
2002-10-26  7:53 ` The return of the return of crunch time (2.5 merge candidate list 1.6) Andi Kleen
2002-10-26  8:13   ` Andreas Dilger
     [not found]   ` <20021026190906.GA20571@pimlott.net>
2002-10-27  7:01     ` Andi Kleen
2002-10-27 15:20       ` Andrew Pimlott
2002-10-27 17:57         ` Rob Landley
2002-10-28  4:06           ` Andrew Pimlott
2002-10-28  4:32             ` Andi Kleen
2002-10-28  4:09           ` Andi Kleen
2002-10-28  4:30         ` Andi Kleen
2002-01-15 17:44           ` Pavel Machek
2002-11-08 22:02             ` Andrew Pimlott [this message]
2002-10-28  6:03           ` Andrew Pimlott
2002-10-25 20:57 Rob Landley
2002-10-26  2:42 ` Hans Reiser
2002-10-27 15:11   ` Rob Landley
2002-10-28  0:27     ` Hans Reiser
2002-10-26 15:09 ` Eric W. Biederman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20021108220213.GC16051@pimlott.net \
    --to=andrew@pimlott.net \
    --cc=ak@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox