Merge branch 'cc/shared-index-permbits'
authorJunio C Hamano <gitster@pobox.com>
Wed, 21 Nov 2018 11:39:03 +0000 (20:39 +0900)
committerJunio C Hamano <gitster@pobox.com>
Wed, 21 Nov 2018 11:39:03 +0000 (20:39 +0900)
The way .git/index and .git/sharedindex* files were initially
created gave these files different perm bits until they were
adjusted for shared repository settings. This was made consistent.

* cc/shared-index-permbits:
read-cache: make the split index obey umask settings

1  2 
read-cache.c
t/t1700-split-index.sh
diff --combined read-cache.c
index 42de59a163c1f88580b88281dfc1572ca0dcd5f0,2fc2acb5bcd95bf3bd5f8f2da8dbd0772193d4d0..bd45dc3e24d7dc28820d26cc6e6d377f92fda46d
@@@ -23,8 -23,6 +23,8 @@@
  #include "split-index.h"
  #include "utf8.h"
  #include "fsmonitor.h"
 +#include "thread-utils.h"
 +#include "progress.h"
  
  /* Mask for the name length in ce_flags in the on-disk index */
  
@@@ -45,8 -43,6 +45,8 @@@
  #define CACHE_EXT_LINK 0x6c696e6b       /* "link" */
  #define CACHE_EXT_UNTRACKED 0x554E5452          /* "UNTR" */
  #define CACHE_EXT_FSMONITOR 0x46534D4E          /* "FSMN" */
 +#define CACHE_EXT_ENDOFINDEXENTRIES 0x454F4945        /* "EOIE" */
 +#define CACHE_EXT_INDEXENTRYOFFSETTABLE 0x49454F54 /* "IEOT" */
  
  /* changes that can be kept in $GIT_DIR/index (basically all extensions) */
  #define EXTMASK (RESOLVE_UNDO_CHANGED | CACHE_TREE_CHANGED | \
@@@ -209,17 -205,15 +209,17 @@@ void fill_stat_cache_info(struct cache_
        }
  }
  
 -static int ce_compare_data(const struct cache_entry *ce, struct stat *st)
 +static int ce_compare_data(struct index_state *istate,
 +                         const struct cache_entry *ce,
 +                         struct stat *st)
  {
        int match = -1;
        int fd = git_open_cloexec(ce->name, O_RDONLY);
  
        if (fd >= 0) {
                struct object_id oid;
 -              if (!index_fd(&oid, fd, st, OBJ_BLOB, ce->name, 0))
 -                      match = oidcmp(&oid, &ce->oid);
 +              if (!index_fd(istate, &oid, fd, st, OBJ_BLOB, ce->name, 0))
 +                      match = !oideq(&oid, &ce->oid);
                /* index_fd() closed the file descriptor already */
        }
        return match;
@@@ -260,16 -254,14 +260,16 @@@ static int ce_compare_gitlink(const str
         */
        if (resolve_gitlink_ref(ce->name, "HEAD", &oid) < 0)
                return 0;
 -      return oidcmp(&oid, &ce->oid);
 +      return !oideq(&oid, &ce->oid);
  }
  
 -static int ce_modified_check_fs(const struct cache_entry *ce, struct stat *st)
 +static int ce_modified_check_fs(struct index_state *istate,
 +                              const struct cache_entry *ce,
 +                              struct stat *st)
  {
        switch (st->st_mode & S_IFMT) {
        case S_IFREG:
 -              if (ce_compare_data(ce, st))
 +              if (ce_compare_data(istate, ce, st))
                        return DATA_CHANGED;
                break;
        case S_IFLNK:
@@@ -345,7 -337,7 +345,7 @@@ static int is_racy_stat(const struct in
                );
  }
  
 -static int is_racy_timestamp(const struct index_state *istate,
 +int is_racy_timestamp(const struct index_state *istate,
                             const struct cache_entry *ce)
  {
        return (!S_ISGITLINK(ce->ce_mode) &&
@@@ -415,7 -407,7 +415,7 @@@ int ie_match_stat(struct index_state *i
                if (assume_racy_is_modified)
                        changed |= DATA_CHANGED;
                else
 -                      changed |= ce_modified_check_fs(ce, st);
 +                      changed |= ce_modified_check_fs(istate, ce, st);
        }
  
        return changed;
@@@ -455,7 -447,7 +455,7 @@@ int ie_modified(struct index_state *ist
            (S_ISGITLINK(ce->ce_mode) || ce->ce_stat_data.sd_size != 0))
                return changed;
  
 -      changed_fs = ce_modified_check_fs(ce, st);
 +      changed_fs = ce_modified_check_fs(istate, ce, st);
        if (changed_fs)
                return changed | changed_fs;
        return 0;
@@@ -761,7 -753,7 +761,7 @@@ int add_to_index(struct index_state *is
                }
        }
        if (!intent_only) {
 -              if (index_path(&ce->oid, path, st, newflags)) {
 +              if (index_path(istate, &ce->oid, path, st, newflags)) {
                        discard_cache_entry(ce);
                        return error("unable to index file %s", path);
                }
        /* It was suspected to be racily clean, but it turns out to be Ok */
        was_same = (alias &&
                    !ce_stage(alias) &&
 -                  !oidcmp(&alias->oid, &ce->oid) &&
 +                  oideq(&alias->oid, &ce->oid) &&
                    ce->ce_mode == alias->ce_mode);
  
        if (pretend)
@@@ -831,7 -823,7 +831,7 @@@ struct cache_entry *make_cache_entry(st
        ce->ce_namelen = len;
        ce->ce_mode = create_ce_mode(mode);
  
 -      ret = refresh_cache_entry(&the_index, ce, refresh_options);
 +      ret = refresh_cache_entry(istate, ce, refresh_options);
        if (ret != ce)
                discard_cache_entry(ce);
        return ret;
@@@ -1484,24 -1476,13 +1484,24 @@@ int refresh_index(struct index_state *i
        const char *typechange_fmt;
        const char *added_fmt;
        const char *unmerged_fmt;
 -      uint64_t start = getnanotime();
 +      struct progress *progress = NULL;
 +
 +      if (flags & REFRESH_PROGRESS && isatty(2))
 +              progress = start_delayed_progress(_("Refresh index"),
 +                                                istate->cache_nr);
  
 +      trace_performance_enter();
        modified_fmt = (in_porcelain ? "M\t%s\n" : "%s: needs update\n");
        deleted_fmt = (in_porcelain ? "D\t%s\n" : "%s: needs update\n");
        typechange_fmt = (in_porcelain ? "T\t%s\n" : "%s needs update\n");
        added_fmt = (in_porcelain ? "A\t%s\n" : "%s needs update\n");
        unmerged_fmt = (in_porcelain ? "U\t%s\n" : "%s: needs merge\n");
 +      /*
 +       * Use the multi-threaded preload_index() to refresh most of the
 +       * cache entries quickly then in the single threaded loop below,
 +       * we only have to do the special cases that are left.
 +       */
 +      preload_index(istate, pathspec, 0);
        for (i = 0; i < istate->cache_nr; i++) {
                struct cache_entry *ce, *new_entry;
                int cache_errno = 0;
                if (ignore_submodules && S_ISGITLINK(ce->ce_mode))
                        continue;
  
 -              if (pathspec && !ce_path_match(&the_index, ce, pathspec, seen))
 +              if (pathspec && !ce_path_match(istate, ce, pathspec, seen))
                        filtered = 1;
  
                if (ce_stage(ce)) {
                new_entry = refresh_cache_ent(istate, ce, options, &cache_errno, &changed);
                if (new_entry == ce)
                        continue;
 +              if (progress)
 +                      display_progress(progress, i);
                if (!new_entry) {
                        const char *fmt;
  
  
                replace_index_entry(istate, i, new_entry);
        }
 -      trace_performance_since(start, "refresh index");
 +      if (progress) {
 +              display_progress(progress, istate->cache_nr);
 +              stop_progress(&progress);
 +      }
 +      trace_performance_leave("refresh index");
        return has_errors;
  }
  
@@@ -1675,7 -1650,7 +1675,7 @@@ int verify_index_checksum
  /* Allow fsck to force verification of the cache entry order. */
  int verify_ce_order;
  
 -static int verify_hdr(struct cache_header *hdr, unsigned long size)
 +static int verify_hdr(const struct cache_header *hdr, unsigned long size)
  {
        git_hash_ctx c;
        unsigned char hash[GIT_MAX_RAWSZ];
        the_hash_algo->init_fn(&c);
        the_hash_algo->update_fn(&c, hdr, size - the_hash_algo->rawsz);
        the_hash_algo->final_fn(hash, &c);
 -      if (hashcmp(hash, (unsigned char *)hdr + size - the_hash_algo->rawsz))
 +      if (!hasheq(hash, (unsigned char *)hdr + size - the_hash_algo->rawsz))
                return error("bad index file sha1 signature");
        return 0;
  }
  
  static int read_index_extension(struct index_state *istate,
 -                              const char *ext, void *data, unsigned long sz)
 +                              const char *ext, const char *data, unsigned long sz)
  {
        switch (CACHE_EXT(ext)) {
        case CACHE_EXT_TREE:
        case CACHE_EXT_FSMONITOR:
                read_fsmonitor_extension(istate, data, sz);
                break;
 +      case CACHE_EXT_ENDOFINDEXENTRIES:
 +      case CACHE_EXT_INDEXENTRYOFFSETTABLE:
 +              /* already handled in do_read_index() */
 +              break;
        default:
                if (*ext < 'A' || 'Z' < *ext)
                        return error("index uses %.4s extension, which we do not understand",
@@@ -1742,25 -1713,63 +1742,25 @@@ int read_index(struct index_state *ista
        return read_index_from(istate, get_index_file(), get_git_dir());
  }
  
 -static struct cache_entry *cache_entry_from_ondisk(struct mem_pool *mem_pool,
 -                                                 struct ondisk_cache_entry *ondisk,
 -                                                 unsigned int flags,
 -                                                 const char *name,
 -                                                 size_t len)
 -{
 -      struct cache_entry *ce = mem_pool__ce_alloc(mem_pool, len);
 -
 -      ce->ce_stat_data.sd_ctime.sec = get_be32(&ondisk->ctime.sec);
 -      ce->ce_stat_data.sd_mtime.sec = get_be32(&ondisk->mtime.sec);
 -      ce->ce_stat_data.sd_ctime.nsec = get_be32(&ondisk->ctime.nsec);
 -      ce->ce_stat_data.sd_mtime.nsec = get_be32(&ondisk->mtime.nsec);
 -      ce->ce_stat_data.sd_dev   = get_be32(&ondisk->dev);
 -      ce->ce_stat_data.sd_ino   = get_be32(&ondisk->ino);
 -      ce->ce_mode  = get_be32(&ondisk->mode);
 -      ce->ce_stat_data.sd_uid   = get_be32(&ondisk->uid);
 -      ce->ce_stat_data.sd_gid   = get_be32(&ondisk->gid);
 -      ce->ce_stat_data.sd_size  = get_be32(&ondisk->size);
 -      ce->ce_flags = flags & ~CE_NAMEMASK;
 -      ce->ce_namelen = len;
 -      ce->index = 0;
 -      hashcpy(ce->oid.hash, ondisk->sha1);
 -      memcpy(ce->name, name, len);
 -      ce->name[len] = '\0';
 -      return ce;
 -}
 -
 -/*
 - * Adjacent cache entries tend to share the leading paths, so it makes
 - * sense to only store the differences in later entries.  In the v4
 - * on-disk format of the index, each on-disk cache entry stores the
 - * number of bytes to be stripped from the end of the previous name,
 - * and the bytes to append to the result, to come up with its name.
 - */
 -static unsigned long expand_name_field(struct strbuf *name, const char *cp_)
 -{
 -      const unsigned char *ep, *cp = (const unsigned char *)cp_;
 -      size_t len = decode_varint(&cp);
 -
 -      if (name->len < len)
 -              die("malformed name field in the index");
 -      strbuf_remove(name, name->len - len, len);
 -      for (ep = cp; *ep; ep++)
 -              ; /* find the end */
 -      strbuf_add(name, cp, ep - cp);
 -      return (const char *)ep + 1 - cp_;
 -}
 -
 -static struct cache_entry *create_from_disk(struct mem_pool *mem_pool,
 +static struct cache_entry *create_from_disk(struct mem_pool *ce_mem_pool,
 +                                          unsigned int version,
                                            struct ondisk_cache_entry *ondisk,
                                            unsigned long *ent_size,
 -                                          struct strbuf *previous_name)
 +                                          const struct cache_entry *previous_ce)
  {
        struct cache_entry *ce;
        size_t len;
        const char *name;
        unsigned int flags;
 +      size_t copy_len = 0;
 +      /*
 +       * Adjacent cache entries tend to share the leading paths, so it makes
 +       * sense to only store the differences in later entries.  In the v4
 +       * on-disk format of the index, each on-disk cache entry stores the
 +       * number of bytes to be stripped from the end of the previous name,
 +       * and the bytes to append to the result, to come up with its name.
 +       */
 +      int expand_name_field = version == 4;
  
        /* On-disk flags are just 16 bits */
        flags = get_be16(&ondisk->flags);
        else
                name = ondisk->name;
  
 -      if (!previous_name) {
 -              /* v3 and earlier */
 -              if (len == CE_NAMEMASK)
 -                      len = strlen(name);
 -              ce = cache_entry_from_ondisk(mem_pool, ondisk, flags, name, len);
 +      if (expand_name_field) {
 +              const unsigned char *cp = (const unsigned char *)name;
 +              size_t strip_len, previous_len;
 +
 +              /* If we're at the begining of a block, ignore the previous name */
 +              strip_len = decode_varint(&cp);
 +              if (previous_ce) {
 +                      previous_len = previous_ce->ce_namelen;
 +                      if (previous_len < strip_len)
 +                              die(_("malformed name field in the index, near path '%s'"),
 +                                      previous_ce->name);
 +                      copy_len = previous_len - strip_len;
 +              }
 +              name = (const char *)cp;
 +      }
  
 -              *ent_size = ondisk_ce_size(ce);
 -      } else {
 -              unsigned long consumed;
 -              consumed = expand_name_field(previous_name, name);
 -              ce = cache_entry_from_ondisk(mem_pool, ondisk, flags,
 -                                           previous_name->buf,
 -                                           previous_name->len);
 +      if (len == CE_NAMEMASK) {
 +              len = strlen(name);
 +              if (expand_name_field)
 +                      len += copy_len;
 +      }
 +
 +      ce = mem_pool__ce_alloc(ce_mem_pool, len);
 +
 +      ce->ce_stat_data.sd_ctime.sec = get_be32(&ondisk->ctime.sec);
 +      ce->ce_stat_data.sd_mtime.sec = get_be32(&ondisk->mtime.sec);
 +      ce->ce_stat_data.sd_ctime.nsec = get_be32(&ondisk->ctime.nsec);
 +      ce->ce_stat_data.sd_mtime.nsec = get_be32(&ondisk->mtime.nsec);
 +      ce->ce_stat_data.sd_dev   = get_be32(&ondisk->dev);
 +      ce->ce_stat_data.sd_ino   = get_be32(&ondisk->ino);
 +      ce->ce_mode  = get_be32(&ondisk->mode);
 +      ce->ce_stat_data.sd_uid   = get_be32(&ondisk->uid);
 +      ce->ce_stat_data.sd_gid   = get_be32(&ondisk->gid);
 +      ce->ce_stat_data.sd_size  = get_be32(&ondisk->size);
 +      ce->ce_flags = flags & ~CE_NAMEMASK;
 +      ce->ce_namelen = len;
 +      ce->index = 0;
 +      hashcpy(ce->oid.hash, ondisk->sha1);
  
 -              *ent_size = (name - ((char *)ondisk)) + consumed;
 +      if (expand_name_field) {
 +              if (copy_len)
 +                      memcpy(ce->name, previous_ce->name, copy_len);
 +              memcpy(ce->name + copy_len, name, len + 1 - copy_len);
 +              *ent_size = (name - ((char *)ondisk)) + len + 1 - copy_len;
 +      } else {
 +              memcpy(ce->name, name, len + 1);
 +              *ent_size = ondisk_ce_size(ce);
        }
        return ce;
  }
@@@ -1912,228 -1889,16 +1912,228 @@@ static size_t estimate_cache_size(size_
        return ondisk_size + entries * per_entry;
  }
  
 +struct index_entry_offset
 +{
 +      /* starting byte offset into index file, count of index entries in this block */
 +      int offset, nr;
 +};
 +
 +struct index_entry_offset_table
 +{
 +      int nr;
 +      struct index_entry_offset entries[FLEX_ARRAY];
 +};
 +
 +static struct index_entry_offset_table *read_ieot_extension(const char *mmap, size_t mmap_size, size_t offset);
 +static void write_ieot_extension(struct strbuf *sb, struct index_entry_offset_table *ieot);
 +
 +static size_t read_eoie_extension(const char *mmap, size_t mmap_size);
 +static void write_eoie_extension(struct strbuf *sb, git_hash_ctx *eoie_context, size_t offset);
 +
 +struct load_index_extensions
 +{
 +      pthread_t pthread;
 +      struct index_state *istate;
 +      const char *mmap;
 +      size_t mmap_size;
 +      unsigned long src_offset;
 +};
 +
 +static void *load_index_extensions(void *_data)
 +{
 +      struct load_index_extensions *p = _data;
 +      unsigned long src_offset = p->src_offset;
 +
 +      while (src_offset <= p->mmap_size - the_hash_algo->rawsz - 8) {
 +              /* After an array of active_nr index entries,
 +               * there can be arbitrary number of extended
 +               * sections, each of which is prefixed with
 +               * extension name (4-byte) and section length
 +               * in 4-byte network byte order.
 +               */
 +              uint32_t extsize = get_be32(p->mmap + src_offset + 4);
 +              if (read_index_extension(p->istate,
 +                                       p->mmap + src_offset,
 +                                       p->mmap + src_offset + 8,
 +                                       extsize) < 0) {
 +                      munmap((void *)p->mmap, p->mmap_size);
 +                      die(_("index file corrupt"));
 +              }
 +              src_offset += 8;
 +              src_offset += extsize;
 +      }
 +
 +      return NULL;
 +}
 +
 +/*
 + * A helper function that will load the specified range of cache entries
 + * from the memory mapped file and add them to the given index.
 + */
 +static unsigned long load_cache_entry_block(struct index_state *istate,
 +                      struct mem_pool *ce_mem_pool, int offset, int nr, const char *mmap,
 +                      unsigned long start_offset, const struct cache_entry *previous_ce)
 +{
 +      int i;
 +      unsigned long src_offset = start_offset;
 +
 +      for (i = offset; i < offset + nr; i++) {
 +              struct ondisk_cache_entry *disk_ce;
 +              struct cache_entry *ce;
 +              unsigned long consumed;
 +
 +              disk_ce = (struct ondisk_cache_entry *)(mmap + src_offset);
 +              ce = create_from_disk(ce_mem_pool, istate->version, disk_ce, &consumed, previous_ce);
 +              set_index_entry(istate, i, ce);
 +
 +              src_offset += consumed;
 +              previous_ce = ce;
 +      }
 +      return src_offset - start_offset;
 +}
 +
 +static unsigned long load_all_cache_entries(struct index_state *istate,
 +                      const char *mmap, size_t mmap_size, unsigned long src_offset)
 +{
 +      unsigned long consumed;
 +
 +      if (istate->version == 4) {
 +              mem_pool_init(&istate->ce_mem_pool,
 +                              estimate_cache_size_from_compressed(istate->cache_nr));
 +      } else {
 +              mem_pool_init(&istate->ce_mem_pool,
 +                              estimate_cache_size(mmap_size, istate->cache_nr));
 +      }
 +
 +      consumed = load_cache_entry_block(istate, istate->ce_mem_pool,
 +                                      0, istate->cache_nr, mmap, src_offset, NULL);
 +      return consumed;
 +}
 +
 +/*
 + * Mostly randomly chosen maximum thread counts: we
 + * cap the parallelism to online_cpus() threads, and we want
 + * to have at least 10000 cache entries per thread for it to
 + * be worth starting a thread.
 + */
 +
 +#define THREAD_COST           (10000)
 +
 +struct load_cache_entries_thread_data
 +{
 +      pthread_t pthread;
 +      struct index_state *istate;
 +      struct mem_pool *ce_mem_pool;
 +      int offset;
 +      const char *mmap;
 +      struct index_entry_offset_table *ieot;
 +      int ieot_start;         /* starting index into the ieot array */
 +      int ieot_blocks;        /* count of ieot entries to process */
 +      unsigned long consumed; /* return # of bytes in index file processed */
 +};
 +
 +/*
 + * A thread proc to run the load_cache_entries() computation
 + * across multiple background threads.
 + */
 +static void *load_cache_entries_thread(void *_data)
 +{
 +      struct load_cache_entries_thread_data *p = _data;
 +      int i;
 +
 +      /* iterate across all ieot blocks assigned to this thread */
 +      for (i = p->ieot_start; i < p->ieot_start + p->ieot_blocks; i++) {
 +              p->consumed += load_cache_entry_block(p->istate, p->ce_mem_pool,
 +                      p->offset, p->ieot->entries[i].nr, p->mmap, p->ieot->entries[i].offset, NULL);
 +              p->offset += p->ieot->entries[i].nr;
 +      }
 +      return NULL;
 +}
 +
 +static unsigned long load_cache_entries_threaded(struct index_state *istate, const char *mmap, size_t mmap_size,
 +                      unsigned long src_offset, int nr_threads, struct index_entry_offset_table *ieot)
 +{
 +      int i, offset, ieot_blocks, ieot_start, err;
 +      struct load_cache_entries_thread_data *data;
 +      unsigned long consumed = 0;
 +
 +      /* a little sanity checking */
 +      if (istate->name_hash_initialized)
 +              BUG("the name hash isn't thread safe");
 +
 +      mem_pool_init(&istate->ce_mem_pool, 0);
 +
 +      /* ensure we have no more threads than we have blocks to process */
 +      if (nr_threads > ieot->nr)
 +              nr_threads = ieot->nr;
 +      data = xcalloc(nr_threads, sizeof(*data));
 +
 +      offset = ieot_start = 0;
 +      ieot_blocks = DIV_ROUND_UP(ieot->nr, nr_threads);
 +      for (i = 0; i < nr_threads; i++) {
 +              struct load_cache_entries_thread_data *p = &data[i];
 +              int nr, j;
 +
 +              if (ieot_start + ieot_blocks > ieot->nr)
 +                      ieot_blocks = ieot->nr - ieot_start;
 +
 +              p->istate = istate;
 +              p->offset = offset;
 +              p->mmap = mmap;
 +              p->ieot = ieot;
 +              p->ieot_start = ieot_start;
 +              p->ieot_blocks = ieot_blocks;
 +
 +              /* create a mem_pool for each thread */
 +              nr = 0;
 +              for (j = p->ieot_start; j < p->ieot_start + p->ieot_blocks; j++)
 +                      nr += p->ieot->entries[j].nr;
 +              if (istate->version == 4) {
 +                      mem_pool_init(&p->ce_mem_pool,
 +                              estimate_cache_size_from_compressed(nr));
 +              } else {
 +                      mem_pool_init(&p->ce_mem_pool,
 +                              estimate_cache_size(mmap_size, nr));
 +              }
 +
 +              err = pthread_create(&p->pthread, NULL, load_cache_entries_thread, p);
 +              if (err)
 +                      die(_("unable to create load_cache_entries thread: %s"), strerror(err));
 +
 +              /* increment by the number of cache entries in the ieot block being processed */
 +              for (j = 0; j < ieot_blocks; j++)
 +                      offset += ieot->entries[ieot_start + j].nr;
 +              ieot_start += ieot_blocks;
 +      }
 +
 +      for (i = 0; i < nr_threads; i++) {
 +              struct load_cache_entries_thread_data *p = &data[i];
 +
 +              err = pthread_join(p->pthread, NULL);
 +              if (err)
 +                      die(_("unable to join load_cache_entries thread: %s"), strerror(err));
 +              mem_pool_combine(istate->ce_mem_pool, p->ce_mem_pool);
 +              consumed += p->consumed;
 +      }
 +
 +      free(data);
 +
 +      return consumed;
 +}
 +
  /* remember to discard_cache() before reading a different cache! */
  int do_read_index(struct index_state *istate, const char *path, int must_exist)
  {
 -      int fd, i;
 +      int fd;
        struct stat st;
        unsigned long src_offset;
 -      struct cache_header *hdr;
 -      void *mmap;
 +      const struct cache_header *hdr;
 +      const char *mmap;
        size_t mmap_size;
 -      struct strbuf previous_name_buf = STRBUF_INIT, *previous_name;
 +      struct load_index_extensions p;
 +      size_t extension_offset = 0;
 +      int nr_threads, cpus;
 +      struct index_entry_offset_table *ieot = NULL;
  
        if (istate->initialized)
                return istate->cache_nr;
                die_errno("unable to map index file");
        close(fd);
  
 -      hdr = mmap;
 +      hdr = (const struct cache_header *)mmap;
        if (verify_hdr(hdr, mmap_size) < 0)
                goto unmap;
  
        istate->cache = xcalloc(istate->cache_alloc, sizeof(*istate->cache));
        istate->initialized = 1;
  
 -      if (istate->version == 4) {
 -              previous_name = &previous_name_buf;
 -              mem_pool_init(&istate->ce_mem_pool,
 -                            estimate_cache_size_from_compressed(istate->cache_nr));
 -      } else {
 -              previous_name = NULL;
 -              mem_pool_init(&istate->ce_mem_pool,
 -                            estimate_cache_size(mmap_size, istate->cache_nr));
 -      }
 +      p.istate = istate;
 +      p.mmap = mmap;
 +      p.mmap_size = mmap_size;
  
        src_offset = sizeof(*hdr);
 -      for (i = 0; i < istate->cache_nr; i++) {
 -              struct ondisk_cache_entry *disk_ce;
 -              struct cache_entry *ce;
 -              unsigned long consumed;
  
 -              disk_ce = (struct ondisk_cache_entry *)((char *)mmap + src_offset);
 -              ce = create_from_disk(istate->ce_mem_pool, disk_ce, &consumed, previous_name);
 -              set_index_entry(istate, i, ce);
 +      if (git_config_get_index_threads(&nr_threads))
 +              nr_threads = 1;
  
 -              src_offset += consumed;
 +      /* TODO: does creating more threads than cores help? */
 +      if (!nr_threads) {
 +              nr_threads = istate->cache_nr / THREAD_COST;
 +              cpus = online_cpus();
 +              if (nr_threads > cpus)
 +                      nr_threads = cpus;
        }
 -      strbuf_release(&previous_name_buf);
 +
 +      if (!HAVE_THREADS)
 +              nr_threads = 1;
 +
 +      if (nr_threads > 1) {
 +              extension_offset = read_eoie_extension(mmap, mmap_size);
 +              if (extension_offset) {
 +                      int err;
 +
 +                      p.src_offset = extension_offset;
 +                      err = pthread_create(&p.pthread, NULL, load_index_extensions, &p);
 +                      if (err)
 +                              die(_("unable to create load_index_extensions thread: %s"), strerror(err));
 +
 +                      nr_threads--;
 +              }
 +      }
 +
 +      /*
 +       * Locate and read the index entry offset table so that we can use it
 +       * to multi-thread the reading of the cache entries.
 +       */
 +      if (extension_offset && nr_threads > 1)
 +              ieot = read_ieot_extension(mmap, mmap_size, extension_offset);
 +
 +      if (ieot) {
 +              src_offset += load_cache_entries_threaded(istate, mmap, mmap_size, src_offset, nr_threads, ieot);
 +              free(ieot);
 +      } else {
 +              src_offset += load_all_cache_entries(istate, mmap, mmap_size, src_offset);
 +      }
 +
        istate->timestamp.sec = st.st_mtime;
        istate->timestamp.nsec = ST_MTIME_NSEC(st);
  
 -      while (src_offset <= mmap_size - the_hash_algo->rawsz - 8) {
 -              /* After an array of active_nr index entries,
 -               * there can be arbitrary number of extended
 -               * sections, each of which is prefixed with
 -               * extension name (4-byte) and section length
 -               * in 4-byte network byte order.
 -               */
 -              uint32_t extsize;
 -              memcpy(&extsize, (char *)mmap + src_offset + 4, 4);
 -              extsize = ntohl(extsize);
 -              if (read_index_extension(istate,
 -                                       (const char *) mmap + src_offset,
 -                                       (char *) mmap + src_offset + 8,
 -                                       extsize) < 0)
 -                      goto unmap;
 -              src_offset += 8;
 -              src_offset += extsize;
 +      /* if we created a thread, join it otherwise load the extensions on the primary thread */
 +      if (extension_offset) {
 +              int ret = pthread_join(p.pthread, NULL);
 +              if (ret)
 +                      die(_("unable to join load_index_extensions thread: %s"), strerror(ret));
 +      } else {
 +              p.src_offset = src_offset;
 +              load_index_extensions(&p);
        }
 -      munmap(mmap, mmap_size);
 +      munmap((void *)mmap, mmap_size);
        return istate->cache_nr;
  
  unmap:
 -      munmap(mmap, mmap_size);
 +      munmap((void *)mmap, mmap_size);
        die("index file corrupt");
  }
  
@@@ -2253,6 -2002,7 +2253,6 @@@ static void freshen_shared_index(const 
  int read_index_from(struct index_state *istate, const char *path,
                    const char *gitdir)
  {
 -      uint64_t start = getnanotime();
        struct split_index *split_index;
        int ret;
        char *base_oid_hex;
        if (istate->initialized)
                return istate->cache_nr;
  
 +      trace_performance_enter();
        ret = do_read_index(istate, path, 0);
 -      trace_performance_since(start, "read cache %s", path);
 +      trace_performance_leave("read cache %s", path);
  
        split_index = istate->split_index;
        if (!split_index || is_null_oid(&split_index->base_oid)) {
                return ret;
        }
  
 +      trace_performance_enter();
        if (split_index->base)
                discard_index(split_index->base);
        else
        base_oid_hex = oid_to_hex(&split_index->base_oid);
        base_path = xstrfmt("%s/sharedindex.%s", gitdir, base_oid_hex);
        ret = do_read_index(split_index->base, base_path, 1);
 -      if (oidcmp(&split_index->base_oid, &split_index->base->oid))
 +      if (!oideq(&split_index->base_oid, &split_index->base->oid))
                die("broken index, expect %s in %s, got %s",
                    base_oid_hex, base_path,
                    oid_to_hex(&split_index->base->oid));
        freshen_shared_index(base_path, 0);
        merge_base_index(istate);
        post_read_index_from(istate);
 -      trace_performance_since(start, "read cache %s", base_path);
 +      trace_performance_leave("read cache %s", base_path);
        free(base_path);
        return ret;
  }
@@@ -2374,7 -2122,7 +2374,7 @@@ int unmerged_index(const struct index_s
        return 0;
  }
  
 -int index_has_changes(const struct index_state *istate,
 +int index_has_changes(struct index_state *istate,
                      struct tree *tree,
                      struct strbuf *sb)
  {
        if (tree || !get_oid_tree("HEAD", &cmp)) {
                struct diff_options opt;
  
 -              diff_setup(&opt);
 +              repo_diff_setup(the_repository, &opt);
                opt.flags.exit_with_status = 1;
                if (!sb)
                        opt.flags.quick = 1;
@@@ -2450,15 -2198,11 +2450,15 @@@ static int ce_write(git_hash_ctx *conte
        return 0;
  }
  
 -static int write_index_ext_header(git_hash_ctx *context, int fd,
 -                                unsigned int ext, unsigned int sz)
 +static int write_index_ext_header(git_hash_ctx *context, git_hash_ctx *eoie_context,
 +                                int fd, unsigned int ext, unsigned int sz)
  {
        ext = htonl(ext);
        sz = htonl(sz);
 +      if (eoie_context) {
 +              the_hash_algo->update_fn(eoie_context, &ext, 4);
 +              the_hash_algo->update_fn(eoie_context, &sz, 4);
 +      }
        return ((ce_write(context, fd, &ext, 4) < 0) ||
                (ce_write(context, fd, &sz, 4) < 0)) ? -1 : 0;
  }
@@@ -2486,8 -2230,7 +2486,8 @@@ static int ce_flush(git_hash_ctx *conte
        return (write_in_full(fd, write_buffer, left) < 0) ? -1 : 0;
  }
  
 -static void ce_smudge_racily_clean_entry(struct cache_entry *ce)
 +static void ce_smudge_racily_clean_entry(struct index_state *istate,
 +                                       struct cache_entry *ce)
  {
        /*
         * The only thing we care about in this function is to smudge the
                return;
        if (ce_match_stat_basic(ce, &st))
                return;
 -      if (ce_modified_check_fs(ce, &st)) {
 +      if (ce_modified_check_fs(istate, ce, &st)) {
                /* This is "racily clean"; smudge it.  Note that this
                 * is a tricky code.  At first glance, it may appear
                 * that it can break with this sequence:
@@@ -2652,7 -2395,7 +2652,7 @@@ static int verify_index_from(const stru
        if (n != the_hash_algo->rawsz)
                goto out;
  
 -      if (hashcmp(istate->oid.hash, hash))
 +      if (!hasheq(istate->oid.hash, hash))
                goto out;
  
        close(fd);
@@@ -2690,36 -2433,6 +2690,36 @@@ void update_index_if_able(struct index_
                rollback_lock_file(lockfile);
  }
  
 +static int record_eoie(void)
 +{
 +      int val;
 +
 +      if (!git_config_get_bool("index.recordendofindexentries", &val))
 +              return val;
 +
 +      /*
 +       * As a convenience, the end of index entries extension
 +       * used for threading is written by default if the user
 +       * explicitly requested threaded index reads.
 +       */
 +      return !git_config_get_index_threads(&val) && val != 1;
 +}
 +
 +static int record_ieot(void)
 +{
 +      int val;
 +
 +      if (!git_config_get_bool("index.recordoffsettable", &val))
 +              return val;
 +
 +      /*
 +       * As a convenience, the offset table used for threading is
 +       * written by default if the user explicitly requested
 +       * threaded index reads.
 +       */
 +      return !git_config_get_index_threads(&val) && val != 1;
 +}
 +
  /*
   * On success, `tempfile` is closed. If it is the temporary file
   * of a `struct lock_file`, we will therefore effectively perform
@@@ -2732,7 -2445,7 +2732,7 @@@ static int do_write_index(struct index_
  {
        uint64_t start = getnanotime();
        int newfd = tempfile->fd;
 -      git_hash_ctx c;
 +      git_hash_ctx c, eoie_c;
        struct cache_header hdr;
        int i, err = 0, removed, extended, hdr_version;
        struct cache_entry **cache = istate->cache;
        struct ondisk_cache_entry_extended ondisk;
        struct strbuf previous_name_buf = STRBUF_INIT, *previous_name;
        int drop_cache_tree = istate->drop_cache_tree;
 +      off_t offset;
 +      int ieot_entries = 1;
 +      struct index_entry_offset_table *ieot = NULL;
 +      int nr, nr_threads;
  
        for (i = removed = extended = 0; i < entries; i++) {
                if (cache[i]->ce_flags & CE_REMOVE)
        if (ce_write(&c, newfd, &hdr, sizeof(hdr)) < 0)
                return -1;
  
 +      if (!HAVE_THREADS || git_config_get_index_threads(&nr_threads))
 +              nr_threads = 1;
 +
 +      if (nr_threads != 1 && record_ieot()) {
 +              int ieot_blocks, cpus;
 +
 +              /*
 +               * ensure default number of ieot blocks maps evenly to the
 +               * default number of threads that will process them leaving
 +               * room for the thread to load the index extensions.
 +               */
 +              if (!nr_threads) {
 +                      ieot_blocks = istate->cache_nr / THREAD_COST;
 +                      cpus = online_cpus();
 +                      if (ieot_blocks > cpus - 1)
 +                              ieot_blocks = cpus - 1;
 +              } else {
 +                      ieot_blocks = nr_threads;
 +                      if (ieot_blocks > istate->cache_nr)
 +                              ieot_blocks = istate->cache_nr;
 +              }
 +
 +              /*
 +               * no reason to write out the IEOT extension if we don't
 +               * have enough blocks to utilize multi-threading
 +               */
 +              if (ieot_blocks > 1) {
 +                      ieot = xcalloc(1, sizeof(struct index_entry_offset_table)
 +                              + (ieot_blocks * sizeof(struct index_entry_offset)));
 +                      ieot_entries = DIV_ROUND_UP(entries, ieot_blocks);
 +              }
 +      }
 +
 +      offset = lseek(newfd, 0, SEEK_CUR);
 +      if (offset < 0) {
 +              free(ieot);
 +              return -1;
 +      }
 +      offset += write_buffer_len;
 +      nr = 0;
        previous_name = (hdr_version == 4) ? &previous_name_buf : NULL;
  
        for (i = 0; i < entries; i++) {
                if (ce->ce_flags & CE_REMOVE)
                        continue;
                if (!ce_uptodate(ce) && is_racy_timestamp(istate, ce))
 -                      ce_smudge_racily_clean_entry(ce);
 +                      ce_smudge_racily_clean_entry(istate, ce);
                if (is_null_oid(&ce->oid)) {
                        static const char msg[] = "cache entry has null sha1: %s";
                        static int allow = -1;
  
                        drop_cache_tree = 1;
                }
 +              if (ieot && i && (i % ieot_entries == 0)) {
 +                      ieot->entries[ieot->nr].nr = nr;
 +                      ieot->entries[ieot->nr].offset = offset;
 +                      ieot->nr++;
 +                      /*
 +                       * If we have a V4 index, set the first byte to an invalid
 +                       * character to ensure there is nothing common with the previous
 +                       * entry
 +                       */
 +                      if (previous_name)
 +                              previous_name->buf[0] = 0;
 +                      nr = 0;
 +                      offset = lseek(newfd, 0, SEEK_CUR);
 +                      if (offset < 0) {
 +                              free(ieot);
 +                              return -1;
 +                      }
 +                      offset += write_buffer_len;
 +              }
                if (ce_write_entry(&c, newfd, ce, previous_name, (struct ondisk_cache_entry *)&ondisk) < 0)
                        err = -1;
  
                if (err)
                        break;
 +              nr++;
 +      }
 +      if (ieot && nr) {
 +              ieot->entries[ieot->nr].nr = nr;
 +              ieot->entries[ieot->nr].offset = offset;
 +              ieot->nr++;
        }
        strbuf_release(&previous_name_buf);
  
 -      if (err)
 +      if (err) {
 +              free(ieot);
                return err;
 +      }
  
        /* Write extension data here */
 +      offset = lseek(newfd, 0, SEEK_CUR);
 +      if (offset < 0) {
 +              free(ieot);
 +              return -1;
 +      }
 +      offset += write_buffer_len;
 +      the_hash_algo->init_fn(&eoie_c);
 +
 +      /*
 +       * Lets write out CACHE_EXT_INDEXENTRYOFFSETTABLE first so that we
 +       * can minimize the number of extensions we have to scan through to
 +       * find it during load.  Write it out regardless of the
 +       * strip_extensions parameter as we need it when loading the shared
 +       * index.
 +       */
 +      if (ieot) {
 +              struct strbuf sb = STRBUF_INIT;
 +
 +              write_ieot_extension(&sb, ieot);
 +              err = write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_INDEXENTRYOFFSETTABLE, sb.len) < 0
 +                      || ce_write(&c, newfd, sb.buf, sb.len) < 0;
 +              strbuf_release(&sb);
 +              free(ieot);
 +              if (err)
 +                      return -1;
 +      }
 +
        if (!strip_extensions && istate->split_index) {
                struct strbuf sb = STRBUF_INIT;
  
                err = write_link_extension(&sb, istate) < 0 ||
 -                      write_index_ext_header(&c, newfd, CACHE_EXT_LINK,
 +                      write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_LINK,
                                               sb.len) < 0 ||
                        ce_write(&c, newfd, sb.buf, sb.len) < 0;
                strbuf_release(&sb);
                struct strbuf sb = STRBUF_INIT;
  
                cache_tree_write(&sb, istate->cache_tree);
 -              err = write_index_ext_header(&c, newfd, CACHE_EXT_TREE, sb.len) < 0
 +              err = write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_TREE, sb.len) < 0
                        || ce_write(&c, newfd, sb.buf, sb.len) < 0;
                strbuf_release(&sb);
                if (err)
                struct strbuf sb = STRBUF_INIT;
  
                resolve_undo_write(&sb, istate->resolve_undo);
 -              err = write_index_ext_header(&c, newfd, CACHE_EXT_RESOLVE_UNDO,
 +              err = write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_RESOLVE_UNDO,
                                             sb.len) < 0
                        || ce_write(&c, newfd, sb.buf, sb.len) < 0;
                strbuf_release(&sb);
                struct strbuf sb = STRBUF_INIT;
  
                write_untracked_extension(&sb, istate->untracked);
 -              err = write_index_ext_header(&c, newfd, CACHE_EXT_UNTRACKED,
 +              err = write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_UNTRACKED,
                                             sb.len) < 0 ||
                        ce_write(&c, newfd, sb.buf, sb.len) < 0;
                strbuf_release(&sb);
                struct strbuf sb = STRBUF_INIT;
  
                write_fsmonitor_extension(&sb, istate);
 -              err = write_index_ext_header(&c, newfd, CACHE_EXT_FSMONITOR, sb.len) < 0
 +              err = write_index_ext_header(&c, &eoie_c, newfd, CACHE_EXT_FSMONITOR, sb.len) < 0
 +                      || ce_write(&c, newfd, sb.buf, sb.len) < 0;
 +              strbuf_release(&sb);
 +              if (err)
 +                      return -1;
 +      }
 +
 +      /*
 +       * CACHE_EXT_ENDOFINDEXENTRIES must be written as the last entry before the SHA1
 +       * so that it can be found and processed before all the index entries are
 +       * read.  Write it out regardless of the strip_extensions parameter as we need it
 +       * when loading the shared index.
 +       */
 +      if (offset && record_eoie()) {
 +              struct strbuf sb = STRBUF_INIT;
 +
 +              write_eoie_extension(&sb, &eoie_c, offset);
 +              err = write_index_ext_header(&c, NULL, newfd, CACHE_EXT_ENDOFINDEXENTRIES, sb.len) < 0
                        || ce_write(&c, newfd, sb.buf, sb.len) < 0;
                strbuf_release(&sb);
                if (err)
@@@ -3145,9 -2743,6 +3145,9 @@@ int write_locked_index(struct index_sta
        int new_shared_index, ret;
        struct split_index *si = istate->split_index;
  
 +      if (git_env_bool("GIT_TEST_CHECK_CACHE_TREE", 0))
 +              cache_tree_verify(istate);
 +
        if ((flags & SKIP_IF_UNCHANGED) && !istate->cache_changed) {
                if (flags & COMMIT_LOCK)
                        rollback_lock_file(lock);
                struct tempfile *temp;
                int saved_errno;
  
-               temp = mks_tempfile(git_path("sharedindex_XXXXXX"));
+               /* Same initial permissions as the main .git/index file */
+               temp = mks_tempfile_sm(git_path("sharedindex_XXXXXX"), 0, 0666);
                if (!temp) {
                        oidclr(&si->base_oid);
                        ret = do_write_locked_index(istate, lock, flags);
@@@ -3344,8 -2940,6 +3345,8 @@@ void move_index_extensions(struct index
  {
        dst->untracked = src->untracked;
        src->untracked = NULL;
 +      dst->cache_tree = src->cache_tree;
 +      src->cache_tree = NULL;
  }
  
  struct cache_entry *dup_cache_entry(const struct cache_entry *ce,
@@@ -3385,179 -2979,3 +3386,179 @@@ int should_validate_cache_entries(void
  
        return validate_index_cache_entries;
  }
 +
 +#define EOIE_SIZE (4 + GIT_SHA1_RAWSZ) /* <4-byte offset> + <20-byte hash> */
 +#define EOIE_SIZE_WITH_HEADER (4 + 4 + EOIE_SIZE) /* <4-byte signature> + <4-byte length> + EOIE_SIZE */
 +
 +static size_t read_eoie_extension(const char *mmap, size_t mmap_size)
 +{
 +      /*
 +       * The end of index entries (EOIE) extension is guaranteed to be last
 +       * so that it can be found by scanning backwards from the EOF.
 +       *
 +       * "EOIE"
 +       * <4-byte length>
 +       * <4-byte offset>
 +       * <20-byte hash>
 +       */
 +      const char *index, *eoie;
 +      uint32_t extsize;
 +      size_t offset, src_offset;
 +      unsigned char hash[GIT_MAX_RAWSZ];
 +      git_hash_ctx c;
 +
 +      /* ensure we have an index big enough to contain an EOIE extension */
 +      if (mmap_size < sizeof(struct cache_header) + EOIE_SIZE_WITH_HEADER + the_hash_algo->rawsz)
 +              return 0;
 +
 +      /* validate the extension signature */
 +      index = eoie = mmap + mmap_size - EOIE_SIZE_WITH_HEADER - the_hash_algo->rawsz;
 +      if (CACHE_EXT(index) != CACHE_EXT_ENDOFINDEXENTRIES)
 +              return 0;
 +      index += sizeof(uint32_t);
 +
 +      /* validate the extension size */
 +      extsize = get_be32(index);
 +      if (extsize != EOIE_SIZE)
 +              return 0;
 +      index += sizeof(uint32_t);
 +
 +      /*
 +       * Validate the offset we're going to look for the first extension
 +       * signature is after the index header and before the eoie extension.
 +       */
 +      offset = get_be32(index);
 +      if (mmap + offset < mmap + sizeof(struct cache_header))
 +              return 0;
 +      if (mmap + offset >= eoie)
 +              return 0;
 +      index += sizeof(uint32_t);
 +
 +      /*
 +       * The hash is computed over extension types and their sizes (but not
 +       * their contents).  E.g. if we have "TREE" extension that is N-bytes
 +       * long, "REUC" extension that is M-bytes long, followed by "EOIE",
 +       * then the hash would be:
 +       *
 +       * SHA-1("TREE" + <binary representation of N> +
 +       *       "REUC" + <binary representation of M>)
 +       */
 +      src_offset = offset;
 +      the_hash_algo->init_fn(&c);
 +      while (src_offset < mmap_size - the_hash_algo->rawsz - EOIE_SIZE_WITH_HEADER) {
 +              /* After an array of active_nr index entries,
 +               * there can be arbitrary number of extended
 +               * sections, each of which is prefixed with
 +               * extension name (4-byte) and section length
 +               * in 4-byte network byte order.
 +               */
 +              uint32_t extsize;
 +              memcpy(&extsize, mmap + src_offset + 4, 4);
 +              extsize = ntohl(extsize);
 +
 +              /* verify the extension size isn't so large it will wrap around */
 +              if (src_offset + 8 + extsize < src_offset)
 +                      return 0;
 +
 +              the_hash_algo->update_fn(&c, mmap + src_offset, 8);
 +
 +              src_offset += 8;
 +              src_offset += extsize;
 +      }
 +      the_hash_algo->final_fn(hash, &c);
 +      if (!hasheq(hash, (const unsigned char *)index))
 +              return 0;
 +
 +      /* Validate that the extension offsets returned us back to the eoie extension. */
 +      if (src_offset != mmap_size - the_hash_algo->rawsz - EOIE_SIZE_WITH_HEADER)
 +              return 0;
 +
 +      return offset;
 +}
 +
 +static void write_eoie_extension(struct strbuf *sb, git_hash_ctx *eoie_context, size_t offset)
 +{
 +      uint32_t buffer;
 +      unsigned char hash[GIT_MAX_RAWSZ];
 +
 +      /* offset */
 +      put_be32(&buffer, offset);
 +      strbuf_add(sb, &buffer, sizeof(uint32_t));
 +
 +      /* hash */
 +      the_hash_algo->final_fn(hash, eoie_context);
 +      strbuf_add(sb, hash, the_hash_algo->rawsz);
 +}
 +
 +#define IEOT_VERSION  (1)
 +
 +static struct index_entry_offset_table *read_ieot_extension(const char *mmap, size_t mmap_size, size_t offset)
 +{
 +       const char *index = NULL;
 +       uint32_t extsize, ext_version;
 +       struct index_entry_offset_table *ieot;
 +       int i, nr;
 +
 +       /* find the IEOT extension */
 +       if (!offset)
 +             return NULL;
 +       while (offset <= mmap_size - the_hash_algo->rawsz - 8) {
 +             extsize = get_be32(mmap + offset + 4);
 +             if (CACHE_EXT((mmap + offset)) == CACHE_EXT_INDEXENTRYOFFSETTABLE) {
 +                     index = mmap + offset + 4 + 4;
 +                     break;
 +             }
 +             offset += 8;
 +             offset += extsize;
 +       }
 +       if (!index)
 +             return NULL;
 +
 +       /* validate the version is IEOT_VERSION */
 +       ext_version = get_be32(index);
 +       if (ext_version != IEOT_VERSION) {
 +             error("invalid IEOT version %d", ext_version);
 +             return NULL;
 +       }
 +       index += sizeof(uint32_t);
 +
 +       /* extension size - version bytes / bytes per entry */
 +       nr = (extsize - sizeof(uint32_t)) / (sizeof(uint32_t) + sizeof(uint32_t));
 +       if (!nr) {
 +             error("invalid number of IEOT entries %d", nr);
 +             return NULL;
 +       }
 +       ieot = xmalloc(sizeof(struct index_entry_offset_table)
 +             + (nr * sizeof(struct index_entry_offset)));
 +       ieot->nr = nr;
 +       for (i = 0; i < nr; i++) {
 +             ieot->entries[i].offset = get_be32(index);
 +             index += sizeof(uint32_t);
 +             ieot->entries[i].nr = get_be32(index);
 +             index += sizeof(uint32_t);
 +       }
 +
 +       return ieot;
 +}
 +
 +static void write_ieot_extension(struct strbuf *sb, struct index_entry_offset_table *ieot)
 +{
 +       uint32_t buffer;
 +       int i;
 +
 +       /* version */
 +       put_be32(&buffer, IEOT_VERSION);
 +       strbuf_add(sb, &buffer, sizeof(uint32_t));
 +
 +       /* ieot */
 +       for (i = 0; i < ieot->nr; i++) {
 +
 +             /* offset */
 +             put_be32(&buffer, ieot->entries[i].offset);
 +             strbuf_add(sb, &buffer, sizeof(uint32_t));
 +
 +             /* count */
 +             put_be32(&buffer, ieot->entries[i].nr);
 +             strbuf_add(sb, &buffer, sizeof(uint32_t));
 +       }
 +}
diff --combined t/t1700-split-index.sh
index 0cbac64e28b26eebe2805c51748be99d647e039d,91f9cc5ab4acb120e928e8753a8f9b2c3b09f063..4667e1a1909a01e638b13b7264b9041805a438b6
@@@ -6,27 -6,13 +6,27 @@@ test_description='split index mode test
  
  # We need total control of index splitting here
  sane_unset GIT_TEST_SPLIT_INDEX
 -sane_unset GIT_FSMONITOR_TEST
 +
 +# Testing a hard coded SHA against an index with an extension
 +# that can vary from run to run is problematic so we disable
 +# those extensions.
 +sane_unset GIT_TEST_FSMONITOR
 +sane_unset GIT_TEST_INDEX_THREADS
 +
 +# Create a file named as $1 with content read from stdin.
 +# Set the file's mtime to a few seconds in the past to avoid racy situations.
 +create_non_racy_file () {
 +      cat >"$1" &&
 +      test-tool chmtime =-5 "$1"
 +}
  
  test_expect_success 'enable split index' '
        git config splitIndex.maxPercentChange 100 &&
        git update-index --split-index &&
        test-tool dump-split-index .git/index >actual &&
        indexversion=$(test-tool index-version <.git/index) &&
 +
 +      # NEEDSWORK: Stop hard-coding checksums.
        if test "$indexversion" = "4"
        then
                own=432ef4b63f32193984f339431fd50ca796493569
@@@ -35,7 -21,6 +35,7 @@@
                own=8299b0bcd1ac364e5f1d7768efb62fa2da79a339
                base=39d890139ee5356c7ef572216cebcd27aa41f9df
        fi &&
 +
        cat >expect <<-EOF &&
        own $own
        base $base
@@@ -46,7 -31,7 +46,7 @@@
  '
  
  test_expect_success 'add one file' '
 -      : >one &&
 +      create_non_racy_file one &&
        git update-index --add one &&
        git ls-files --stage >ls-files.actual &&
        cat >ls-files.expect <<-EOF &&
@@@ -72,7 -57,7 +72,7 @@@ test_expect_success 'disable split inde
        EOF
        test_cmp ls-files.expect ls-files.actual &&
  
 -      BASE=$(test-tool dump-split-index .git/index | grep "^own" | sed "s/own/base/") &&
 +      BASE=$(test-tool dump-split-index .git/index | sed -n "s/^own/base/p") &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        cat >expect <<-EOF &&
        not a split index
@@@ -98,7 -83,7 +98,7 @@@ test_expect_success 'enable split inde
  '
  
  test_expect_success 'modify original file, base index untouched' '
 -      echo modified >one &&
 +      echo modified | create_non_racy_file one &&
        git update-index one &&
        git ls-files --stage >ls-files.actual &&
        cat >ls-files.expect <<-EOF &&
  '
  
  test_expect_success 'add another file, which stays index' '
 -      : >two &&
 +      create_non_racy_file two &&
        git update-index --add two &&
        git ls-files --stage >ls-files.actual &&
        cat >ls-files.expect <<-EOF &&
@@@ -170,7 -155,7 +170,7 @@@ test_expect_success 'remove file in bas
  '
  
  test_expect_success 'add original file back' '
 -      : >one &&
 +      create_non_racy_file one &&
        git update-index --add one &&
        git ls-files --stage >ls-files.actual &&
        cat >ls-files.expect <<-EOF &&
  '
  
  test_expect_success 'add new file' '
 -      : >two &&
 +      create_non_racy_file two &&
        git update-index --add two &&
        git ls-files --stage >actual &&
        cat >expect <<-EOF &&
@@@ -233,7 -218,7 +233,7 @@@ test_expect_success 'rev-parse --shared
  
  test_expect_success 'set core.splitIndex config variable to true' '
        git config core.splitIndex true &&
 -      : >three &&
 +      create_non_racy_file three &&
        git update-index --add three &&
        git ls-files --stage >ls-files.actual &&
        cat >ls-files.expect <<-EOF &&
@@@ -268,9 -253,9 +268,9 @@@ test_expect_success 'set core.splitInde
        test_cmp expect actual
  '
  
 -test_expect_success 'set core.splitIndex config variable to true' '
 +test_expect_success 'set core.splitIndex config variable back to true' '
        git config core.splitIndex true &&
 -      : >three &&
 +      create_non_racy_file three &&
        git update-index --add three &&
        BASE=$(test-tool dump-split-index .git/index | grep "^base") &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        deletions:
        EOF
        test_cmp expect actual &&
 -      : >four &&
 +      create_non_racy_file four &&
        git update-index --add four &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        cat >expect <<-EOF &&
  
  test_expect_success 'check behavior with splitIndex.maxPercentChange unset' '
        git config --unset splitIndex.maxPercentChange &&
 -      : >five &&
 +      create_non_racy_file five &&
        git update-index --add five &&
        BASE=$(test-tool dump-split-index .git/index | grep "^base") &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        deletions:
        EOF
        test_cmp expect actual &&
 -      : >six &&
 +      create_non_racy_file six &&
        git update-index --add six &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        cat >expect <<-EOF &&
  
  test_expect_success 'check splitIndex.maxPercentChange set to 0' '
        git config splitIndex.maxPercentChange 0 &&
 -      : >seven &&
 +      create_non_racy_file seven &&
        git update-index --add seven &&
        BASE=$(test-tool dump-split-index .git/index | grep "^base") &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
        deletions:
        EOF
        test_cmp expect actual &&
 -      : >eight &&
 +      create_non_racy_file eight &&
        git update-index --add eight &&
        BASE=$(test-tool dump-split-index .git/index | grep "^base") &&
        test-tool dump-split-index .git/index | sed "/^own/d" >actual &&
  '
  
  test_expect_success 'shared index files expire after 2 weeks by default' '
 -      : >ten &&
 +      create_non_racy_file ten &&
        git update-index --add ten &&
        test $(ls .git/sharedindex.* | wc -l) -gt 2 &&
        just_under_2_weeks_ago=$((5-14*86400)) &&
        test-tool chmtime =$just_under_2_weeks_ago .git/sharedindex.* &&
 -      : >eleven &&
 +      create_non_racy_file eleven &&
        git update-index --add eleven &&
        test $(ls .git/sharedindex.* | wc -l) -gt 2 &&
        just_over_2_weeks_ago=$((-1-14*86400)) &&
        test-tool chmtime =$just_over_2_weeks_ago .git/sharedindex.* &&
 -      : >twelve &&
 +      create_non_racy_file twelve &&
        git update-index --add twelve &&
        test $(ls .git/sharedindex.* | wc -l) -le 2
  '
  test_expect_success 'check splitIndex.sharedIndexExpire set to 16 days' '
        git config splitIndex.sharedIndexExpire "16.days.ago" &&
        test-tool chmtime =$just_over_2_weeks_ago .git/sharedindex.* &&
 -      : >thirteen &&
 +      create_non_racy_file thirteen &&
        git update-index --add thirteen &&
        test $(ls .git/sharedindex.* | wc -l) -gt 2 &&
        just_over_16_days_ago=$((-1-16*86400)) &&
        test-tool chmtime =$just_over_16_days_ago .git/sharedindex.* &&
 -      : >fourteen &&
 +      create_non_racy_file fourteen &&
        git update-index --add fourteen &&
        test $(ls .git/sharedindex.* | wc -l) -le 2
  '
@@@ -373,17 -358,37 +373,37 @@@ test_expect_success 'check splitIndex.s
        git config splitIndex.sharedIndexExpire never &&
        just_10_years_ago=$((-365*10*86400)) &&
        test-tool chmtime =$just_10_years_ago .git/sharedindex.* &&
 -      : >fifteen &&
 +      create_non_racy_file fifteen &&
        git update-index --add fifteen &&
        test $(ls .git/sharedindex.* | wc -l) -gt 2 &&
        git config splitIndex.sharedIndexExpire now &&
        just_1_second_ago=-1 &&
        test-tool chmtime =$just_1_second_ago .git/sharedindex.* &&
 -      : >sixteen &&
 +      create_non_racy_file sixteen &&
        git update-index --add sixteen &&
        test $(ls .git/sharedindex.* | wc -l) -le 2
  '
  
+ test_expect_success POSIXPERM 'same mode for index & split index' '
+       git init same-mode &&
+       (
+               cd same-mode &&
+               test_commit A &&
+               test_modebits .git/index >index_mode &&
+               test_must_fail git config core.sharedRepository &&
+               git -c core.splitIndex=true status &&
+               shared=$(ls .git/sharedindex.*) &&
+               case "$shared" in
+               *" "*)
+                       # we have more than one???
+                       false ;;
+               *)
+                       test_modebits "$shared" >split_index_mode &&
+                       test_cmp index_mode split_index_mode ;;
+               esac
+       )
+ '
  while read -r mode modebits
  do
        test_expect_success POSIXPERM "split index respects core.sharedrepository $mode" '
                # Create one new shared index file
                git config core.sharedrepository "$mode" &&
                git config core.splitIndex true &&
 -              : >one &&
 +              create_non_racy_file one &&
                git update-index --add one &&
                echo "$modebits" >expect &&
                test_modebits .git/index >actual &&