git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/5] git-blame: further performance preview

From
David Kastrup <dak@gnu.org>
Date
Feb 3, 2014, 19:14 UTC
Message-ID
<1391454849-26558-1-git-send-email-dak@gnu.org>

Ok, I'm progressing rather like molasses with getting -M and -C options back to work. In the mean time, here is another performance preview without them. The main patch in the middle has basically gotten some formatting/style fixes as opposed to last time round and one small bug fix (concerning incremental output).

It still contains a significant amount of dead code: this series is not supposed to be merged, it's just supposed to be exciting to see how it performs.

There are two simple performance patches on top of the main patch, the first of which offers somewhat significant savings in I/O time (which was quite unaffected by the main rewrite so far). The gist of that patch makes convenient use of the changed data layout to avoid discarding blob data predictably required again right away.

It's likely that this is not the only opportunity to save performance by better data management.

The second "performance" patch is not likely to measurably affect overall performance. Avoiding irrelevant iterations might make debugging more pleasant, however.

David Kastrup (5):
  builtin/blame.c: struct blame_entry does not need a prev link
  Eliminate same_suspect function in builtin/blame.c
  builtin/blame.c: large-scale rewrite
  Performance improvement: don't drop origin blobs that are going to get
    tested next.
  Avoid queuing commits multiple times for the same origin
 builtin/blame.c | 595 +++++++++++++++++++++++++++++++++++---------------------
 1 file changed, 371 insertions(+), 224 deletions(-)
-- 
1.8.3.2
Next: David Kastrup
Message 1 of 6 in “git-blame: further performance preview”
  1. 0/5 git-blame: further performance previewDavid Kastrup, Feb 3, 2014
  2. 1/5 builtin/blame.c: struct blame_entry does not need a prev linkDavid Kastrup, Feb 3, 2014
  3. 2/5 Eliminate same_suspect function in builtin/blame.cDavid Kastrup, Feb 3, 2014
  4. 3/5 builtin/blame.c: large-scale rewriteDavid Kastrup, Feb 3, 2014
  5. 4/5 Performance improvement: don't drop origin blobs that are going to get tested next.David Kastrup, Feb 3, 2014
  6. 5/5 Avoid queuing commits multiple times for the same originDavid Kastrup, Feb 3, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.