Code

Revert bogus optimization that avoids index file writes
authorLinus Torvalds <torvalds@ppc970.osdl.org>
Fri, 6 May 2005 23:48:43 +0000 (16:48 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Fri, 6 May 2005 23:48:43 +0000 (16:48 -0700)
It didn't properly mark all cache updates as being dirty, and
causes merge errors due to that. In particular, it didn't notice
when a file was force-removed.

Besides, it was ugly as hell. I've put in place a slightly cleaner
version, but I've not enabled the optimization because I don't
want to be burned again.

cache.h
read-cache.c
update-cache.c

diff --git a/cache.h b/cache.h
index 9de7709f9ba6435e82e69aad6bbc51794ad2a1bd..a727cb4efe462d2f4dff7356d3976393f5fb55a1 100644 (file)
--- a/cache.h
+++ b/cache.h
@@ -98,7 +98,7 @@ static inline unsigned int create_ce_mode(unsigned int mode)
 
 const char *sha1_file_directory;
 struct cache_entry **active_cache;
-unsigned int active_nr, active_alloc;
+unsigned int active_nr, active_alloc, active_cache_changed;
 
 #define DB_ENVIRONMENT "SHA1_FILE_DIRECTORY"
 #define DEFAULT_DB_ENVIRONMENT ".git/objects"
index 2a88d18b16b5bc6b9cbbef16a125aa4705b5cb90..6a04cf194c98d1d04bee8f6748262d6e56bcbb46 100644 (file)
@@ -7,7 +7,7 @@
 #include "cache.h"
 
 struct cache_entry **active_cache = NULL;
-unsigned int active_nr = 0, active_alloc = 0;
+unsigned int active_nr = 0, active_alloc = 0, active_cache_changed = 0;
 
 int cache_match_stat(struct cache_entry *ce, struct stat *st)
 {
@@ -99,6 +99,7 @@ int cache_name_pos(const char *name, int namelen)
 /* Remove entry, return true if there are more entries to go.. */
 int remove_entry_at(int pos)
 {
+       active_cache_changed = 1;
        active_nr--;
        if (pos >= active_nr)
                return 0;
@@ -130,6 +131,7 @@ int add_cache_entry(struct cache_entry *ce, int ok_to_add)
 
        /* existing match? Just replace it */
        if (pos >= 0) {
+               active_cache_changed = 1;
                active_cache[pos] = ce;
                return 0;
        }
@@ -161,6 +163,7 @@ int add_cache_entry(struct cache_entry *ce, int ok_to_add)
        if (active_nr > pos)
                memmove(active_cache + pos + 1, active_cache + pos, (active_nr - pos - 1) * sizeof(ce));
        active_cache[pos] = ce;
+       active_cache_changed = 1;
        return 0;
 }
 
index 97d5e8b6b0695b82a235026430050a4140a1586a..735d19920d998a3d53dbfd9a68336cc13780ef23 100644 (file)
  */
 static int allow_add = 0, allow_remove = 0, not_new = 0;
 
-/*
- * update-cache --refresh may not touch anything at all, in which case
- * writing 1.6MB of the same thing is a waste.
- */
-static int cache_changed = 0;
-
 /* Three functions to allow overloaded pointer return; see linux/err.h */
 static inline void *ERR_PTR(long error)
 {
@@ -57,7 +51,7 @@ static void fill_stat_cache_info(struct cache_entry *ce, struct stat *st)
        ce->ce_size = htonl(st->st_size);
 }
 
-static int add_file_to_cache_1(char *path)
+static int add_file_to_cache(char *path)
 {
        int size, namelen;
        struct cache_entry *ce;
@@ -101,35 +95,9 @@ static int add_file_to_cache_1(char *path)
        default:
                return -1;
        }
-       if (!cache_changed) {
-               /* If we have not smudged the cache, be careful
-                * to keep it clean.  Find out if we have a matching
-                * cache entry that add_cache_entry would replace with,
-                * and if it matches then do not bother calling it.
-                */
-               int pos = cache_name_pos(ce->name, namelen);
-               if ((0 <= pos) &&
-                   !memcmp(active_cache[pos], ce, sizeof(*ce))) {
-                       free(ce);
-                       /* magic to tell add_file_to_cache that
-                        * we have not updated anything.
-                        */
-                       return 999;
-               }
-       }
        return add_cache_entry(ce, allow_add);
 }
 
-static int add_file_to_cache(char *path)
-{
-       int ret = add_file_to_cache_1(path);
-       if (ret == 0)
-               cache_changed = 1;
-       else if (ret == 999)
-               ret = 0;
-       return ret;
-}
-
 static int match_data(int fd, void *buffer, unsigned long size)
 {
        while (size) {
@@ -236,7 +204,6 @@ static struct cache_entry *refresh_entry(struct cache_entry *ce)
                return ERR_PTR(-EINVAL);
        }
 
-       cache_changed = 1;
        size = ce_size(ce);
        updated = xmalloc(size);
        memcpy(updated, ce, size);
@@ -270,6 +237,7 @@ static int refresh_cache(void)
                        }
                        continue;
                }
+               active_cache_changed = 1;
                active_cache[i] = new;
        }
        return has_errors;
@@ -317,7 +285,6 @@ static int add_cacheinfo(char *arg1, char *arg2, char *arg3)
        if (!verify_path(arg3))
                return -1;
 
-       cache_changed = 1;
        len = strlen(arg3);
        size = cache_entry_size(len);
        ce = xmalloc(size);
@@ -412,11 +379,7 @@ int main(int argc, char **argv)
                if (add_file_to_cache(path))
                        die("Unable to add %s to database", path);
        }
-
-       if (!cache_changed)
-               unlink(lockfile);
-       else if (write_cache(newfd, active_cache, active_nr) ||
-                rename(lockfile, indexfile))
+       if (write_cache(newfd, active_cache, active_nr) || rename(lockfile, indexfile))
                die("Unable to write new cachefile");
 
        lockfile_name = NULL;