Skip to content

Commit a8a003d

Browse files
committed
Merge pull request #1468 from atetubou/fscache_checkout_flush
checkout.c: enable fscache for checkout again Signed-off-by: Johannes Schindelin <[email protected]>
2 parents 43751fe + b08310b commit a8a003d

File tree

6 files changed

+60
-0
lines changed

6 files changed

+60
-0
lines changed

builtin/checkout.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -345,6 +345,7 @@ static int checkout_worktree(const struct checkout_opts *opts)
345345
state.refresh_cache = 1;
346346
state.istate = &the_index;
347347

348+
enable_fscache(1);
348349
enable_delayed_checkout(&state);
349350
for (pos = 0; pos < active_nr; pos++) {
350351
struct cache_entry *ce = active_cache[pos];
@@ -368,6 +369,7 @@ static int checkout_worktree(const struct checkout_opts *opts)
368369
remove_marked_cache_entries(&the_index, 1);
369370
remove_scheduled_dirs();
370371
errs |= finish_delayed_checkout(&state, &nr_checkouts);
372+
enable_fscache(0);
371373

372374
if (opts->count_checkout_paths) {
373375
if (nr_unmerged)

compat/win32/fscache.c

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -433,6 +433,18 @@ int fscache_enable(int enable)
433433
return result;
434434
}
435435

436+
/*
437+
* Flush cached stats result when fscache is enabled.
438+
*/
439+
void fscache_flush(void)
440+
{
441+
if (enabled) {
442+
EnterCriticalSection(&mutex);
443+
fscache_clear();
444+
LeaveCriticalSection(&mutex);
445+
}
446+
}
447+
436448
/*
437449
* Lstat replacement, uses the cache if enabled, otherwise redirects to
438450
* mingw_lstat.

compat/win32/fscache.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ int fscache_enable(int enable);
77
int fscache_enabled(const char *path);
88
#define is_fscache_enabled(path) fscache_enabled(path)
99

10+
void fscache_flush(void);
11+
#define flush_fscache() fscache_flush()
12+
1013
DIR *fscache_opendir(const char *dir);
1114
int fscache_lstat(const char *file_name, struct stat *buf);
1215

entry.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -367,6 +367,9 @@ static int write_entry(struct cache_entry *ce,
367367
}
368368

369369
finish:
370+
/* Flush cached lstat in fscache after writing to disk. */
371+
flush_fscache();
372+
370373
if (state->refresh_cache) {
371374
assert(state->istate);
372375
if (!fstat_done)

git-compat-util.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1311,6 +1311,10 @@ static inline int is_missing_file_error(int errno_)
13111311
#define is_fscache_enabled(path) (0)
13121312
#endif
13131313

1314+
#ifndef flush_fscache
1315+
#define flush_fscache() /* noop */
1316+
#endif
1317+
13141318
int cmd_main(int, const char **);
13151319

13161320
/*

t/t7201-co.sh

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,42 @@ fill () {
3232
}
3333

3434

35+
test_expect_success MINGW 'fscache flush cache' '
36+
37+
git init fscache-test &&
38+
cd fscache-test &&
39+
git config core.fscache 1 &&
40+
echo A > test.txt &&
41+
git add test.txt &&
42+
git commit -m A &&
43+
echo B >> test.txt &&
44+
git checkout . &&
45+
test -z "$(git status -s)" &&
46+
echo A > expect.txt &&
47+
test_cmp expect.txt test.txt &&
48+
cd .. &&
49+
rm -rf fscache-test
50+
'
51+
52+
test_expect_success MINGW 'fscache flush cache dir' '
53+
54+
git init fscache-test &&
55+
cd fscache-test &&
56+
git config core.fscache 1 &&
57+
echo A > test.txt &&
58+
git add test.txt &&
59+
git commit -m A &&
60+
rm test.txt &&
61+
mkdir test.txt &&
62+
touch test.txt/test.txt &&
63+
git checkout . &&
64+
test -z "$(git status -s)" &&
65+
echo A > expect.txt &&
66+
test_cmp expect.txt test.txt &&
67+
cd .. &&
68+
rm -rf fscache-test
69+
'
70+
3571
test_expect_success setup '
3672
3773
fill x y z > same &&

0 commit comments

Comments
 (0)