Re: [PATCH v3 00/14] FSMonitor edge cases on case-insensitive file systems
"Jeff Hostetler via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 37 quoted lines
> Here is version 3. I think I have addressed the remaining comments.
>
> I cleaned up the test code to use the test_expect_failure at the beginning
> and squashed in the test_expect_success version of tests into the final
> commit in the series.
>
> I moved the invalidate_ce_fsm() commit earlier in the series, so that the
> final commit actually uses all of the up-to-this-point changes to fix the
> problem.
>
> I converted a few "should not happens" to BUG()s.
>
> Thanks to everyone for their time and attention reviewing this. Jeff
>
> Jeff Hostetler (14):
> name-hash: add index_dir_find()
> t7527: add case-insensitve test for FSMonitor
> fsmonitor: refactor refresh callback on directory events
> fsmonitor: clarify handling of directory events in callback helper
> fsmonitor: refactor refresh callback for non-directory events
> dir: create untracked_cache_invalidate_trimmed_path()
> fsmonitor: refactor untracked-cache invalidation
> fsmonitor: move untracked-cache invalidation into helper functions
> fsmonitor: return invalidated cache-entry count on directory event
> fsmonitor: remove custom loop from non-directory path handler
> fsmonitor: return invalided cache-entry count on non-directory event
> fsmonitor: trace the new invalidated cache-entry count
> fsmonitor: refactor bit invalidation in refresh callback
> fsmonitor: support case-insensitive events
>
> dir.c | 20 +++
> dir.h | 7 +
> fsmonitor.c | 312 +++++++++++++++++++++++++++++------
> name-hash.c | 9 +-
> name-hash.h | 7 +-
> t/t7527-builtin-fsmonitor.sh | 223 +++++++++++++++++++++++++
> 6 files changed, 522 insertions(+), 56 deletions(-)
Much nicer. Will queue. Thanks.