зеркало из https://github.com/microsoft/git.git
dir.c: make add_excludes aware of fscache during status
Teach read_directory_recursive() and add_excludes() to be aware of optional fscache and avoid trying to open() and fstat() non-existant ".gitignore" files in every directory in the worktree. The current code in add_excludes() calls open() and then fstat() for a ".gitignore" file in each directory present in the worktree. Change that when fscache is enabled to call lstat() first and if present, call open(). This seems backwards because both lstat needs to do more work than fstat. But when fscache is enabled, fscache will already know if the .gitignore file exists and can completely avoid the IO calls. This works because of the lstat diversion to mingw_lstat when fscache is enabled. This reduced status times on a 350K file enlistment of the Windows repo on a NVMe SSD by 0.25 seconds. Signed-off-by: Jeff Hostetler <jeffhost@microsoft.com>
This commit is contained in:
Родитель
abdac1fa92
Коммит
50a4d7c57b
|
@ -9,6 +9,11 @@ static struct hashmap map;
|
|||
static CRITICAL_SECTION mutex;
|
||||
static struct trace_key trace_fscache = TRACE_KEY_INIT(FSCACHE);
|
||||
|
||||
int fscache_is_enabled(void)
|
||||
{
|
||||
return enabled;
|
||||
}
|
||||
|
||||
/*
|
||||
* An entry in the file system cache. Used for both entire directory listings
|
||||
* and file entries.
|
||||
|
|
|
@ -4,6 +4,9 @@
|
|||
int fscache_enable(int enable);
|
||||
#define enable_fscache(x) fscache_enable(x)
|
||||
|
||||
int fscache_is_enabled(void);
|
||||
#define is_fscache_enabled() (fscache_is_enabled())
|
||||
|
||||
DIR *fscache_opendir(const char *dir);
|
||||
int fscache_lstat(const char *file_name, struct stat *buf);
|
||||
|
||||
|
|
17
dir.c
17
dir.c
|
@ -785,12 +785,27 @@ static int add_excludes(const char *fname, const char *base, int baselen,
|
|||
size_t size = 0;
|
||||
char *buf;
|
||||
|
||||
if (is_fscache_enabled()) {
|
||||
if (lstat(fname, &st) < 0) {
|
||||
fd = -1;
|
||||
} else {
|
||||
fd = open(fname, O_RDONLY);
|
||||
if (fd < 0)
|
||||
warn_on_fopen_errors(fname);
|
||||
}
|
||||
} else {
|
||||
fd = open(fname, O_RDONLY);
|
||||
if (fd < 0 || fstat(fd, &st) < 0) {
|
||||
if (fd < 0)
|
||||
warn_on_fopen_errors(fname);
|
||||
else
|
||||
else {
|
||||
close(fd);
|
||||
fd = -1;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (fd < 0) {
|
||||
if (!istate)
|
||||
return -1;
|
||||
r = read_skip_worktree_file_from_index(istate, fname,
|
||||
|
|
|
@ -1298,6 +1298,10 @@ static inline int is_missing_file_error(int errno_)
|
|||
#define enable_fscache(x) /* noop */
|
||||
#endif
|
||||
|
||||
#ifndef is_fscache_enabled
|
||||
#define is_fscache_enabled() (0)
|
||||
#endif
|
||||
|
||||
int cmd_main(int, const char **);
|
||||
|
||||
/*
|
||||
|
|
Загрузка…
Ссылка в новой задаче