James Youngman wrote: > On Sun, Mar 8, 2009 at 4:35 PM, Jim Meyering <[email protected]> wrote: >> James Youngman wrote: >> ... >>> +#24342: -inum predicate shoud use dirent.d_ino instead of stat.st_ino >>> +(this is a performance bug). >> >> You might want to mention that dirent.d_ino values are unreliable on some >> systems (Linux, Solaris, *BSD get it wrong, but Cygwin gets it right). > > The text could have perhaps been more specific there, but in our case, > we just discard d_ino for directories, and stat them instead.
Ahh... good. Except for Cygwin. >> The problem is with mount points. There, stat.st_ino does not always >> match the corresponding dirent.d_ino. This is a problem even with the >> latest Linux kernels. > > Yes, I'm aware of the problem. Heh ;-) I should have looked at the code.
