Merge branch 'ym/fix-opportunistic-index-update-race'
commit9af098c29b2faa89225556258ae0e3676741c981
authorJunio C Hamano <gitster@pobox.com>
Tue, 3 Jun 2014 19:06:41 +0000 (3 12:06 -0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 3 Jun 2014 19:06:41 +0000 (3 12:06 -0700)
treeba6752e3109e5206465c08fdf2538deb9ee4704b
parent2cc70cefdd4a249fab895943890d21071e03f8c7
parent426ddeead6112955dfb50ccf9bb4af05d1ca9082
Merge branch 'ym/fix-opportunistic-index-update-race'

Read-only operations such as "git status" that internally refreshes
the index write out the refreshed index to the disk to optimize
future accesses to the working tree, but this could race with a
"read-write" operation that modify the index while it is running.
Detect such a race and avoid overwriting the index.

Duy raised a good point that we may need to do the same for the
normal writeout codepath, not just the "opportunistic" update
codepath.  While that is true, nobody sane would be running two
simultaneous operations that are clearly write-oriented competing
with each other against the same index file.  So in that sense that
can be done as a less urgent follow-up for this topic.

* ym/fix-opportunistic-index-update-race:
  read-cache.c: verify index file before we opportunistically update it
  wrapper.c: add xpread() similar to xread()
builtin/index-pack.c
cache.h
git-compat-util.h
read-cache.c