Michael Haggerty <mhag...@alum.mit.edu> writes: > This patch series has the side effect that all of the directories > listed in GIT_CEILING_DIRECTORIES are accessed *unconditionally* to > resolve any symlinks that are present in their paths. It is > admittedly odd that a feature intended to avoid accessing expensive > directories would now *intentionally* access directories near the > expensive ones. In the above scenario this shouldn't be a problem, > because /home would be the directory listed in > GIT_CEILING_DIRECTORIES, and accessing /home itself shouldn't be > expensive.
Interesting observation. In the last sentence, "accessing /home" does not exactly mean accessing /home, but accessing / to learn about "home" in it, no? > But there might be other scenarios for which this patch > series causes a performance regression. Yeah, after merging this to 'next', we should ask people who care about CEILING to test it sufficiently. Thanks for rerolling. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
