diff options
| author |  Linus Torvalds <torvalds@linux-foundation.org> | 2008-06-14 11:32:37 -0700 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2008-06-14 14:39:22 -0700 |
| commit | 44d1c19ee8ab405108b90ab9c02cd86a014639e8 (patch) | |
| tree | b01368f217f11c3963f23f3a0265b5afa92de756 /builtin-add.c | |
| parent | Avoid cross-directory renames and linking on object creation (diff) | |
| download | tgif-44d1c19ee8ab405108b90ab9c02cd86a014639e8.tar.xz | |
Make loose object file reading more careful
We used to do 'stat()+open()+mmap()+close()' to read the loose object
file data, which does work fine, but has a couple of problems:
- it unnecessarily walks the filename twice (at 'stat()' time and then
again to open it)
- NFS generally has open-close consistency guarantees, which means that
the initial 'stat()' was technically done outside of the normal
consistency rules.
So change it to do 'open()+fstat()+mmap()+close()' instead, which avoids
both these issues.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-add.c')
0 files changed, 0 insertions, 0 deletions