[Btrfs-devel] btrfs and git-reflog
Junio C Hamano
gitster at pobox.com
Sun Jan 27 00:08:59 PST 2008
Linus Torvalds <torvalds at linux-foundation.org> writes:
> Yes, but the potential problem is actually very different:
> - directory contains 'a', 'b' and 'c'
> * opendir()
> * readdir() returns 'a'
> * unlink('a');
> * readdir() returns 'c', having skipped 'b'.
> This is something that could in theory happen if a directory is indexed
> using the *position* of a filename in a directory. 'a' was position 1, 'b'
> was position 2, and 'c' was position 3. After the first readdir(), the
> file position was 2 (pointing at 'b'), but when we removed 'a', the other
> entries positions moved down, and now 'b' is at position 1, and 'c' is at
> position '2'. When we call readdir() the next time, it skips 'b' (because
> it already returned position 1!), and returns 'c'.
Yes, I had that one in mind too, but that will cause us to miss
'b' and its only effect is to leave 'b' unpacked. If we somehow
dropped 'b' and lost information that is a different story, but
I do not think this failure mode would cause our current code to
do so. So I do not think is a big deal even if some filesystem
had a bug like that.
More information about the Btrfs-devel