diff options
| author | Eric Wong | 2019-01-01 13:03:08 +0000 |
|---|---|---|
| committer | June McEnroe | 2022-02-13 11:55:52 -0500 |
| commit | a8e11db6570261d2a3d7d9396b796b72998b7ec5 (patch) | |
| tree | f8f37dcb98ef6f5eb6976223c06c9b59aeed329f /cache.c | |
| parent | Remove redundant title on repo anchors (diff) | |
| download | cgit-a8e11db6570261d2a3d7d9396b796b72998b7ec5.tar.gz cgit-a8e11db6570261d2a3d7d9396b796b72998b7ec5.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 'cache.c')
| -rw-r--r-- | cache.c | 7 |
1 files changed, 7 insertions, 0 deletions
| diff --git a/cache.c b/cache.c index 2c70be7..580c0e8 100644 --- a/cache.c +++ b/cache.c | |||
| @@ -265,6 +265,13 @@ static int process_slot(struct cache_slot *slot) | |||
| 265 | { | 265 | { |
| 266 | int err; | 266 | int err; |
| 267 | 267 | ||
| 268 | /* | ||
| 269 | * Make sure any buffered data is flushed before we redirect, | ||
| 270 | * do sendfile(2) or write(2) | ||
| 271 | */ | ||
| 272 | if (fflush(stdout)) | ||
| 273 | return errno; | ||
| 274 | |||
| 268 | err = open_slot(slot); | 275 | err = open_slot(slot); |
| 269 | if (!err && slot->match) { | 276 | if (!err && slot->match) { |
| 270 | if (is_expired(slot)) { | 277 | if (is_expired(slot)) { |