]> git.ipfire.org Git - thirdparty/git.git/blobdiff - commit-graph.c
strmap: enable allocations to come from a mem_pool
[thirdparty/git.git] / commit-graph.c
index 0ed003e218fb9bb2c024fb913a2af4fca9441039..cb042bdba8c83288cfc1e42853447d5bc06f47fe 100644 (file)
@@ -172,7 +172,7 @@ static char *get_split_graph_filename(struct object_directory *odb,
                       oid_hex);
 }
 
-static char *get_chain_filename(struct object_directory *odb)
+char *get_commit_graph_chain_filename(struct object_directory *odb)
 {
        return xstrfmt("%s/info/commit-graphs/commit-graph-chain", odb->path);
 }
@@ -231,7 +231,8 @@ int open_commit_graph(const char *graph_file, int *fd, struct stat *st)
        return 1;
 }
 
-struct commit_graph *load_commit_graph_one_fd_st(int fd, struct stat *st,
+struct commit_graph *load_commit_graph_one_fd_st(struct repository *r,
+                                                int fd, struct stat *st,
                                                 struct object_directory *odb)
 {
        void *graph_map;
@@ -247,7 +248,7 @@ struct commit_graph *load_commit_graph_one_fd_st(int fd, struct stat *st,
        }
        graph_map = xmmap(NULL, graph_size, PROT_READ, MAP_PRIVATE, fd, 0);
        close(fd);
-       ret = parse_commit_graph(graph_map, graph_size);
+       ret = parse_commit_graph(r, graph_map, graph_size);
 
        if (ret)
                ret->odb = odb;
@@ -287,7 +288,8 @@ static int verify_commit_graph_lite(struct commit_graph *g)
        return 0;
 }
 
-struct commit_graph *parse_commit_graph(void *graph_map, size_t graph_size)
+struct commit_graph *parse_commit_graph(struct repository *r,
+                                       void *graph_map, size_t graph_size)
 {
        const unsigned char *data, *chunk_lookup;
        uint32_t i;
@@ -325,6 +327,8 @@ struct commit_graph *parse_commit_graph(void *graph_map, size_t graph_size)
                return NULL;
        }
 
+       prepare_repo_settings(r);
+
        graph = alloc_commit_graph();
 
        graph->hash_len = the_hash_algo->rawsz;
@@ -401,14 +405,14 @@ struct commit_graph *parse_commit_graph(void *graph_map, size_t graph_size)
                case GRAPH_CHUNKID_BLOOMINDEXES:
                        if (graph->chunk_bloom_indexes)
                                chunk_repeated = 1;
-                       else
+                       else if (r->settings.commit_graph_read_changed_paths)
                                graph->chunk_bloom_indexes = data + chunk_offset;
                        break;
 
                case GRAPH_CHUNKID_BLOOMDATA:
                        if (graph->chunk_bloom_data)
                                chunk_repeated = 1;
-                       else {
+                       else if (r->settings.commit_graph_read_changed_paths) {
                                uint32_t hash_version;
                                graph->chunk_bloom_data = data + chunk_offset;
                                hash_version = get_be32(data + chunk_offset);
@@ -420,6 +424,7 @@ struct commit_graph *parse_commit_graph(void *graph_map, size_t graph_size)
                                graph->bloom_filter_settings->hash_version = hash_version;
                                graph->bloom_filter_settings->num_hashes = get_be32(data + chunk_offset + 4);
                                graph->bloom_filter_settings->bits_per_entry = get_be32(data + chunk_offset + 8);
+                               graph->bloom_filter_settings->max_changed_paths = DEFAULT_BLOOM_MAX_CHANGES;
                        }
                        break;
                }
@@ -452,7 +457,8 @@ free_and_return:
        return NULL;
 }
 
-static struct commit_graph *load_commit_graph_one(const char *graph_file,
+static struct commit_graph *load_commit_graph_one(struct repository *r,
+                                                 const char *graph_file,
                                                  struct object_directory *odb)
 {
 
@@ -464,7 +470,7 @@ static struct commit_graph *load_commit_graph_one(const char *graph_file,
        if (!open_ok)
                return NULL;
 
-       g = load_commit_graph_one_fd_st(fd, &st, odb);
+       g = load_commit_graph_one_fd_st(r, fd, &st, odb);
 
        if (g)
                g->filename = xstrdup(graph_file);
@@ -476,7 +482,7 @@ static struct commit_graph *load_commit_graph_v1(struct repository *r,
                                                 struct object_directory *odb)
 {
        char *graph_name = get_commit_graph_filename(odb);
-       struct commit_graph *g = load_commit_graph_one(graph_name, odb);
+       struct commit_graph *g = load_commit_graph_one(r, graph_name, odb);
        free(graph_name);
 
        return g;
@@ -523,7 +529,7 @@ static struct commit_graph *load_commit_graph_chain(struct repository *r,
        struct stat st;
        struct object_id *oids;
        int i = 0, valid = 1, count;
-       char *chain_name = get_chain_filename(odb);
+       char *chain_name = get_commit_graph_chain_filename(odb);
        FILE *fp;
        int stat_res;
 
@@ -557,7 +563,7 @@ static struct commit_graph *load_commit_graph_chain(struct repository *r,
                valid = 0;
                for (odb = r->objects->odb; odb; odb = odb->next) {
                        char *graph_name = get_split_graph_filename(odb, line.buf);
-                       struct commit_graph *g = load_commit_graph_one(graph_name, odb);
+                       struct commit_graph *g = load_commit_graph_one(r, graph_name, odb);
 
                        free(graph_name);
 
@@ -667,6 +673,17 @@ int generation_numbers_enabled(struct repository *r)
        return !!first_generation;
 }
 
+struct bloom_filter_settings *get_bloom_filter_settings(struct repository *r)
+{
+       struct commit_graph *g = r->objects->commit_graph;
+       while (g) {
+               if (g->bloom_filter_settings)
+                       return g->bloom_filter_settings;
+               g = g->base_graph;
+       }
+       return NULL;
+}
+
 static void close_commit_graph_one(struct commit_graph *g)
 {
        if (!g)
@@ -952,9 +969,14 @@ struct write_commit_graph_context {
                 changed_paths:1,
                 order_by_pack:1;
 
-       const struct split_commit_graph_opts *split_opts;
+       const struct commit_graph_opts *opts;
        size_t total_bloom_filter_data_size;
        const struct bloom_filter_settings *bloom_settings;
+
+       int count_bloom_filter_computed;
+       int count_bloom_filter_not_computed;
+       int count_bloom_filter_trunc_empty;
+       int count_bloom_filter_trunc_large;
 };
 
 static int write_graph_chunk_fanout(struct hashfile *f,
@@ -1166,7 +1188,7 @@ static int write_graph_chunk_bloom_indexes(struct hashfile *f,
        uint32_t cur_pos = 0;
 
        while (list < last) {
-               struct bloom_filter *filter = get_bloom_filter(ctx->r, *list, 0);
+               struct bloom_filter *filter = get_bloom_filter(ctx->r, *list);
                size_t len = filter ? filter->len : 0;
                cur_pos += len;
                display_progress(ctx->progress, ++ctx->progress_cnt);
@@ -1185,6 +1207,7 @@ static void trace2_bloom_filter_settings(struct write_commit_graph_context *ctx)
        jw_object_intmax(&jw, "hash_version", ctx->bloom_settings->hash_version);
        jw_object_intmax(&jw, "num_hashes", ctx->bloom_settings->num_hashes);
        jw_object_intmax(&jw, "bits_per_entry", ctx->bloom_settings->bits_per_entry);
+       jw_object_intmax(&jw, "max_changed_paths", ctx->bloom_settings->max_changed_paths);
        jw_end(&jw);
 
        trace2_data_json("bloom", ctx->r, "settings", &jw);
@@ -1205,7 +1228,7 @@ static int write_graph_chunk_bloom_data(struct hashfile *f,
        hashwrite_be32(f, ctx->bloom_settings->bits_per_entry);
 
        while (list < last) {
-               struct bloom_filter *filter = get_bloom_filter(ctx->r, *list, 0);
+               struct bloom_filter *filter = get_bloom_filter(ctx->r, *list);
                size_t len = filter ? filter->len : 0;
 
                display_progress(ctx->progress, ++ctx->progress_cnt);
@@ -1270,8 +1293,8 @@ static void close_reachable(struct write_commit_graph_context *ctx)
 {
        int i;
        struct commit *commit;
-       enum commit_graph_split_flags flags = ctx->split_opts ?
-               ctx->split_opts->flags : COMMIT_GRAPH_SPLIT_UNSPECIFIED;
+       enum commit_graph_split_flags flags = ctx->opts ?
+               ctx->opts->split_flags : COMMIT_GRAPH_SPLIT_UNSPECIFIED;
 
        if (ctx->report_progress)
                ctx->progress = start_delayed_progress(
@@ -1375,11 +1398,24 @@ static void compute_generation_numbers(struct write_commit_graph_context *ctx)
        stop_progress(&ctx->progress);
 }
 
+static void trace2_bloom_filter_write_statistics(struct write_commit_graph_context *ctx)
+{
+       trace2_data_intmax("commit-graph", ctx->r, "filter-computed",
+                          ctx->count_bloom_filter_computed);
+       trace2_data_intmax("commit-graph", ctx->r, "filter-not-computed",
+                          ctx->count_bloom_filter_not_computed);
+       trace2_data_intmax("commit-graph", ctx->r, "filter-trunc-empty",
+                          ctx->count_bloom_filter_trunc_empty);
+       trace2_data_intmax("commit-graph", ctx->r, "filter-trunc-large",
+                          ctx->count_bloom_filter_trunc_large);
+}
+
 static void compute_bloom_filters(struct write_commit_graph_context *ctx)
 {
        int i;
        struct progress *progress = NULL;
        struct commit **sorted_commits;
+       int max_new_filters;
 
        init_bloom_filters();
 
@@ -1396,13 +1432,34 @@ static void compute_bloom_filters(struct write_commit_graph_context *ctx)
        else
                QSORT(sorted_commits, ctx->commits.nr, commit_gen_cmp);
 
+       max_new_filters = ctx->opts && ctx->opts->max_new_filters >= 0 ?
+               ctx->opts->max_new_filters : ctx->commits.nr;
+
        for (i = 0; i < ctx->commits.nr; i++) {
+               enum bloom_filter_computed computed = 0;
                struct commit *c = sorted_commits[i];
-               struct bloom_filter *filter = get_bloom_filter(ctx->r, c, 1);
-               ctx->total_bloom_filter_data_size += sizeof(unsigned char) * filter->len;
+               struct bloom_filter *filter = get_or_compute_bloom_filter(
+                       ctx->r,
+                       c,
+                       ctx->count_bloom_filter_computed < max_new_filters,
+                       ctx->bloom_settings,
+                       &computed);
+               if (computed & BLOOM_COMPUTED) {
+                       ctx->count_bloom_filter_computed++;
+                       if (computed & BLOOM_TRUNC_EMPTY)
+                               ctx->count_bloom_filter_trunc_empty++;
+                       if (computed & BLOOM_TRUNC_LARGE)
+                               ctx->count_bloom_filter_trunc_large++;
+               } else if (computed & BLOOM_NOT_COMPUTED)
+                       ctx->count_bloom_filter_not_computed++;
+               ctx->total_bloom_filter_data_size += filter
+                       ? sizeof(unsigned char) * filter->len : 0;
                display_progress(progress, i + 1);
        }
 
+       if (trace2_is_enabled())
+               trace2_bloom_filter_write_statistics(ctx);
+
        free(sorted_commits);
        stop_progress(&progress);
 }
@@ -1431,7 +1488,7 @@ static int add_ref_to_set(const char *refname,
 
 int write_commit_graph_reachable(struct object_directory *odb,
                                 enum commit_graph_write_flags flags,
-                                const struct split_commit_graph_opts *split_opts)
+                                const struct commit_graph_opts *opts)
 {
        struct oidset commits = OIDSET_INIT;
        struct refs_cb_data data;
@@ -1448,7 +1505,7 @@ int write_commit_graph_reachable(struct object_directory *odb,
        stop_progress(&data.progress);
 
        result = write_commit_graph(odb, NULL, &commits,
-                                   flags, split_opts);
+                                   flags, opts);
 
        oidset_clear(&commits);
        return result;
@@ -1563,8 +1620,8 @@ static uint32_t count_distinct_commits(struct write_commit_graph_context *ctx)
 static void copy_oids_to_commits(struct write_commit_graph_context *ctx)
 {
        uint32_t i;
-       enum commit_graph_split_flags flags = ctx->split_opts ?
-               ctx->split_opts->flags : COMMIT_GRAPH_SPLIT_UNSPECIFIED;
+       enum commit_graph_split_flags flags = ctx->opts ?
+               ctx->opts->split_flags : COMMIT_GRAPH_SPLIT_UNSPECIFIED;
 
        ctx->num_extra_edges = 0;
        if (ctx->report_progress)
@@ -1646,15 +1703,6 @@ static int write_commit_graph_file(struct write_commit_graph_context *ctx)
        int num_chunks = 3;
        uint64_t chunk_offset;
        struct object_id file_hash;
-       struct bloom_filter_settings bloom_settings = DEFAULT_BLOOM_FILTER_SETTINGS;
-
-       if (!ctx->bloom_settings) {
-               bloom_settings.bits_per_entry = git_env_ulong("GIT_TEST_BLOOM_SETTINGS_BITS_PER_ENTRY",
-                                                             bloom_settings.bits_per_entry);
-               bloom_settings.num_hashes = git_env_ulong("GIT_TEST_BLOOM_SETTINGS_NUM_HASHES",
-                                                         bloom_settings.num_hashes);
-               ctx->bloom_settings = &bloom_settings;
-       }
 
        if (ctx->split) {
                struct strbuf tmp_file = STRBUF_INIT;
@@ -1675,7 +1723,7 @@ static int write_commit_graph_file(struct write_commit_graph_context *ctx)
        }
 
        if (ctx->split) {
-               char *lock_name = get_chain_filename(ctx->odb);
+               char *lock_name = get_commit_graph_chain_filename(ctx->odb);
 
                hold_lock_file_for_update_mode(&lk, lock_name,
                                               LOCK_DIE_ON_ERROR, 0444);
@@ -1858,13 +1906,13 @@ static void split_graph_merge_strategy(struct write_commit_graph_context *ctx)
        int max_commits = 0;
        int size_mult = 2;
 
-       if (ctx->split_opts) {
-               max_commits = ctx->split_opts->max_commits;
+       if (ctx->opts) {
+               max_commits = ctx->opts->max_commits;
 
-               if (ctx->split_opts->size_multiple)
-                       size_mult = ctx->split_opts->size_multiple;
+               if (ctx->opts->size_multiple)
+                       size_mult = ctx->opts->size_multiple;
 
-               flags = ctx->split_opts->flags;
+               flags = ctx->opts->split_flags;
        }
 
        g = ctx->r->objects->commit_graph;
@@ -2042,10 +2090,10 @@ static void expire_commit_graphs(struct write_commit_graph_context *ctx)
        size_t dirnamelen;
        timestamp_t expire_time = time(NULL);
 
-       if (ctx->split_opts && ctx->split_opts->expire_time)
-               expire_time = ctx->split_opts->expire_time;
+       if (ctx->opts && ctx->opts->expire_time)
+               expire_time = ctx->opts->expire_time;
        if (!ctx->split) {
-               char *chain_file_name = get_chain_filename(ctx->odb);
+               char *chain_file_name = get_commit_graph_chain_filename(ctx->odb);
                unlink(chain_file_name);
                free(chain_file_name);
                ctx->num_commit_graphs_after = 0;
@@ -2094,12 +2142,13 @@ int write_commit_graph(struct object_directory *odb,
                       struct string_list *pack_indexes,
                       struct oidset *commits,
                       enum commit_graph_write_flags flags,
-                      const struct split_commit_graph_opts *split_opts)
+                      const struct commit_graph_opts *opts)
 {
        struct write_commit_graph_context *ctx;
        uint32_t i, count_distinct = 0;
        int res = 0;
        int replace = 0;
+       struct bloom_filter_settings bloom_settings = DEFAULT_BLOOM_FILTER_SETTINGS;
 
        if (!commit_graph_compatible(the_repository))
                return 0;
@@ -2110,9 +2159,17 @@ int write_commit_graph(struct object_directory *odb,
        ctx->append = flags & COMMIT_GRAPH_WRITE_APPEND ? 1 : 0;
        ctx->report_progress = flags & COMMIT_GRAPH_WRITE_PROGRESS ? 1 : 0;
        ctx->split = flags & COMMIT_GRAPH_WRITE_SPLIT ? 1 : 0;
-       ctx->split_opts = split_opts;
+       ctx->opts = opts;
        ctx->total_bloom_filter_data_size = 0;
 
+       bloom_settings.bits_per_entry = git_env_ulong("GIT_TEST_BLOOM_SETTINGS_BITS_PER_ENTRY",
+                                                     bloom_settings.bits_per_entry);
+       bloom_settings.num_hashes = git_env_ulong("GIT_TEST_BLOOM_SETTINGS_NUM_HASHES",
+                                                 bloom_settings.num_hashes);
+       bloom_settings.max_changed_paths = git_env_ulong("GIT_TEST_BLOOM_SETTINGS_MAX_CHANGED_PATHS",
+                                                        bloom_settings.max_changed_paths);
+       ctx->bloom_settings = &bloom_settings;
+
        if (flags & COMMIT_GRAPH_WRITE_BLOOM_FILTERS)
                ctx->changed_paths = 1;
        if (!(flags & COMMIT_GRAPH_NO_WRITE_BLOOM_FILTERS)) {
@@ -2150,15 +2207,15 @@ int write_commit_graph(struct object_directory *odb,
                        }
                }
 
-               if (ctx->split_opts)
-                       replace = ctx->split_opts->flags & COMMIT_GRAPH_SPLIT_REPLACE;
+               if (ctx->opts)
+                       replace = ctx->opts->split_flags & COMMIT_GRAPH_SPLIT_REPLACE;
        }
 
        ctx->approx_nr_objects = approximate_object_count();
        ctx->oids.alloc = ctx->approx_nr_objects / 32;
 
-       if (ctx->split && split_opts && ctx->oids.alloc > split_opts->max_commits)
-               ctx->oids.alloc = split_opts->max_commits;
+       if (ctx->split && opts && ctx->oids.alloc > opts->max_commits)
+               ctx->oids.alloc = opts->max_commits;
 
        if (ctx->append) {
                prepare_commit_graph_one(ctx->r, ctx->odb);