summary refs log tree commit diff
path: root/www/git.causal.agency/cgit/ui-snapshot.c
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2019-01-01 13:03:08 +0000
committerJune McEnroe <june@causal.agency>2021-01-17 20:03:45 +0000
commite146ca6bb88dc8a70a7897dcfd07940e046f7e33 (patch)
treeb0cf8366f3fe12b4bb65e4e246845fb22778aac1 /www/git.causal.agency/cgit/ui-snapshot.c
parentRemove redundant title on repo anchors (diff)
downloadsrc-e146ca6bb88dc8a70a7897dcfd07940e046f7e33.tar.gz
src-e146ca6bb88dc8a70a7897dcfd07940e046f7e33.zip
Use buffered stdio
Our generation of HTML triggers many small write(2) syscalls
which is inefficient.

Time output on a horrible query against my git.git mirror
shows significant performance improvement:

QUERY_STRING='id=2b93bfac0f5bcabbf60f174f4e7bfa9e318e64d5&id2=d6da71a9d16b8cf27f9d8f90692d3625c849cbc8'
PATH_INFO=/mirrors/git.git/diff
export QUERY_STRING PATH_INFO
time ./cgit >/dev/null

Before:
real    0m1.585s
user    0m0.904s
sys     0m0.658s

After:
real    0m0.750s
user    0m0.666s
sys     0m0.076s
Diffstat (limited to '')
-rw-r--r--www/git.causal.agency/cgit/ui-snapshot.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/www/git.causal.agency/cgit/ui-snapshot.c b/www/git.causal.agency/cgit/ui-snapshot.c
index 556d3ed4..8b81e374 100644
--- a/www/git.causal.agency/cgit/ui-snapshot.c
+++ b/www/git.causal.agency/cgit/ui-snapshot.c
@@ -37,6 +37,9 @@ static int write_archive_type(const char *format, const char *hex, const char *p
 	/* argv_array guarantees a trailing NULL entry. */
 	memcpy(nargv, argv.argv, sizeof(char *) * (argv.argc + 1));
 
+	if (fflush(stdout))
+		return errno;
+
 	result = write_archive(argv.argc, nargv, NULL, the_repository, NULL, 0);
 	argv_array_clear(&argv);
 	free(nargv);