From: Linus Torvalds <torvalds@linux-foundation.org>
To: Junio C Hamano <gitster@pobox.com>
Cc: Kjetil Barvik <barvik@broadpark.no>,
Robin Rosenberg <robin.rosenberg.lists@dewire.com>,
Git Mailing List <git@vger.kernel.org>,
spearce@spearce.org
Subject: Re: [PATCH] Add core.trustlowlevelstat for diffs in dev,ino,uid and gid
Date: Mon, 27 Apr 2009 08:00:38 -0700 (PDT) [thread overview]
Message-ID: <alpine.LFD.2.00.0904270757410.22156@localhost.localdomain> (raw)
In-Reply-To: <7vab628u23.fsf@gitster.siamese.dyndns.org>
On Sun, 26 Apr 2009, Junio C Hamano wrote:
>
> Was ignoring st_dev checking primarily for a work tree over NFS?
Yes, but I think there were other issues too (like git repositories on
removable media).
The inode number really has similar concerns - different operating systems
will use different inode numbers for both NFS and for things like FAT.
It's not nearly as noticeable, because people don't tend to switch OS's as
much as they might switch between two machines.
> I like the end result.
>
> But I am not sure about dropping the nanosecond resolution timestamps.
> The area was extended recently in preparation for ext4; we can take
> advantage of it to reduce the chance the racy-git avoidance codepath
> triggers if we keep it.
>
> fba2f38 (make USE_NSEC work as expected, 2009-02-19)
>
> c06ff49 (Record ns-timestamps if possible, but do not use it without
> USE_NSEC, 2009-03-04)
Hey, we can leave the NSEC support in. Admittedly removing that was about
half the patch, but even with it left in, it would be a cleanup.
Linus
next prev parent reply other threads:[~2009-04-27 15:04 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-26 10:55 [PATCH] Add core.trustlowlevelstat for diffs in dev,ino,uid and gid Robin Rosenberg
2009-04-26 11:06 ` Robin Rosenberg
2009-04-26 12:06 ` [PATCH 1/2] Silence diffs due to use by non-C code Robin Rosenberg
2009-04-26 18:38 ` [PATCH] Add core.trustlowlevelstat for diffs in dev,ino,uid and gid Junio C Hamano
2009-04-26 19:25 ` Linus Torvalds
2009-04-26 22:02 ` Robin Rosenberg
2009-04-27 6:55 ` Junio C Hamano
2009-04-27 15:00 ` Linus Torvalds [this message]
2009-04-27 15:58 ` Kjetil Barvik
2010-03-14 20:51 ` Robin Rosenberg
2010-03-15 6:50 ` Junio C Hamano
2010-03-15 7:41 ` Alex Riesen
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=alpine.LFD.2.00.0904270757410.22156@localhost.localdomain \
--to=torvalds@linux-foundation.org \
--cc=barvik@broadpark.no \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=robin.rosenberg.lists@dewire.com \
--cc=spearce@spearce.org \
/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;
as well as URLs for NNTP newsgroup(s).