Skip to content

Commit aa32939

Browse files
vmggitster
authored andcommitted
rev-list: add bitmap mode to speed up object lists
The bitmap reachability index used to speed up the counting objects phase during `pack-objects` can also be used to optimize a normal rev-list if the only thing required are the SHA1s of the objects during the list (i.e., not the path names at which trees and blobs were found). Calling `git rev-list --objects --use-bitmap-index [committish]` will perform an object iteration based on a bitmap result instead of actually walking the object graph. These are some example timings for `torvalds/linux` (warm cache, best-of-five): $ time git rev-list --objects master > /dev/null real 0m34.191s user 0m33.904s sys 0m0.268s $ time git rev-list --objects --use-bitmap-index master > /dev/null real 0m1.041s user 0m0.976s sys 0m0.064s Likewise, using `git rev-list --count --use-bitmap-index` will speed up the counting operation by building the resulting bitmap and performing a fast popcount (number of bits set on the bitmap) on the result. Here are some sample timings of different ways to count commits in `torvalds/linux`: $ time git rev-list master | wc -l 399882 real 0m6.524s user 0m6.060s sys 0m3.284s $ time git rev-list --count master 399882 real 0m4.318s user 0m4.236s sys 0m0.076s $ time git rev-list --use-bitmap-index --count master 399882 real 0m0.217s user 0m0.176s sys 0m0.040s This also respects negative refs, so you can use it to count a slice of history: $ time git rev-list --count v3.0..master 144843 real 0m1.971s user 0m1.932s sys 0m0.036s $ time git rev-list --use-bitmap-index --count v3.0..master real 0m0.280s user 0m0.220s sys 0m0.056s Though note that the closer the endpoints, the less it helps. In the traversal case, we have fewer commits to cross, so we take less time. But the bitmap time is dominated by generating the pack revindex, which is constant with respect to the refs given. Note that you cannot yet get a fast --left-right count of a symmetric difference (e.g., "--count --left-right master...topic"). The slow part of that walk actually happens during the merge-base determination when we parse "master...topic". Even though a count does not actually need to know the real merge base (it only needs to take the symmetric difference of the bitmaps), the revision code would require some refactoring to handle this case. Additionally, a `--test-bitmap` flag has been added that will perform the same rev-list manually (i.e. using a normal revwalk) and using bitmaps, and verify that the results are the same. This can be used to exercise the bitmap code, and also to verify that the contents of the .bitmap file are sane. Signed-off-by: Vicent Marti <[email protected]> Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 6b8fda2 commit aa32939

File tree

3 files changed

+48
-0
lines changed

3 files changed

+48
-0
lines changed

Documentation/git-rev-list.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ SYNOPSIS
5555
[ \--reverse ]
5656
[ \--walk-reflogs ]
5757
[ \--no-walk ] [ \--do-walk ]
58+
[ \--use-bitmap-index ]
5859
<commit>... [ \-- <paths>... ]
5960

6061
DESCRIPTION

Documentation/rev-list-options.txt

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,14 @@ See also linkgit:git-reflog[1].
274274
Output excluded boundary commits. Boundary commits are
275275
prefixed with `-`.
276276

277+
ifdef::git-rev-list[]
278+
--use-bitmap-index::
279+
280+
Try to speed up the traversal using the pack bitmap index (if
281+
one is available). Note that when traversing with `--objects`,
282+
trees and blobs will not have their associated path printed.
283+
endif::git-rev-list[]
284+
277285
--
278286

279287
History Simplification

builtin/rev-list.c

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@
33
#include "diff.h"
44
#include "revision.h"
55
#include "list-objects.h"
6+
#include "pack.h"
7+
#include "pack-bitmap.h"
68
#include "builtin.h"
79
#include "log-tree.h"
810
#include "graph.h"
@@ -257,6 +259,18 @@ static int show_bisect_vars(struct rev_list_info *info, int reaches, int all)
257259
return 0;
258260
}
259261

262+
static int show_object_fast(
263+
const unsigned char *sha1,
264+
enum object_type type,
265+
int exclude,
266+
uint32_t name_hash,
267+
struct packed_git *found_pack,
268+
off_t found_offset)
269+
{
270+
fprintf(stdout, "%s\n", sha1_to_hex(sha1));
271+
return 1;
272+
}
273+
260274
int cmd_rev_list(int argc, const char **argv, const char *prefix)
261275
{
262276
struct rev_info revs;
@@ -265,6 +279,7 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix)
265279
int bisect_list = 0;
266280
int bisect_show_vars = 0;
267281
int bisect_find_all = 0;
282+
int use_bitmap_index = 0;
268283

269284
git_config(git_default_config, NULL);
270285
init_revisions(&revs, prefix);
@@ -306,6 +321,14 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix)
306321
bisect_show_vars = 1;
307322
continue;
308323
}
324+
if (!strcmp(arg, "--use-bitmap-index")) {
325+
use_bitmap_index = 1;
326+
continue;
327+
}
328+
if (!strcmp(arg, "--test-bitmap")) {
329+
test_bitmap_walk(&revs);
330+
return 0;
331+
}
309332
usage(rev_list_usage);
310333

311334
}
@@ -333,6 +356,22 @@ int cmd_rev_list(int argc, const char **argv, const char *prefix)
333356
if (bisect_list)
334357
revs.limited = 1;
335358

359+
if (use_bitmap_index) {
360+
if (revs.count && !revs.left_right && !revs.cherry_mark) {
361+
uint32_t commit_count;
362+
if (!prepare_bitmap_walk(&revs)) {
363+
count_bitmap_commit_list(&commit_count, NULL, NULL, NULL);
364+
printf("%d\n", commit_count);
365+
return 0;
366+
}
367+
} else if (revs.tag_objects && revs.tree_objects && revs.blob_objects) {
368+
if (!prepare_bitmap_walk(&revs)) {
369+
traverse_bitmap_commit_list(&show_object_fast);
370+
return 0;
371+
}
372+
}
373+
}
374+
336375
if (prepare_revision_walk(&revs))
337376
die("revision walk setup failed");
338377
if (revs.tree_objects)

0 commit comments

Comments
 (0)