Thread (8 messages) flat view 8 messages, 2 authors, 2016-06-15

Re: Alternate Patch: [PATCH] Don't include device number in cache invalidation when running on NFS

From: Thomas Glanzmann <hidden>
Date: 2016-06-15 22:41:58

Hello,
This is _really_ Linux-specific afaik. Which is ok for git, but at the
same time it really makes me go "Ewww". It's testing that the major number 
is 0, and it would be a lot more cleaner to use 
	if (!major(st.st_dev))
but even that is very Linux-specific.
I see.
I'll have to think about it. Maybe I should just remove the st_dev check. 
I guess inode/size/mtime/ctime should be plenty safe enough in practice.
I think so. At least I kick this one out because it is just getting on
my nerves. :-)

	Thomas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help