]> git.hungrycats.org Git - bees/commitdiff
scan_one_extent: remove the unreadahead after benchmark results
authorZygo Blaxell <bees@furryterror.org>
Sat, 30 Nov 2024 00:22:53 +0000 (19:22 -0500)
committerZygo Blaxell <bees@furryterror.org>
Sun, 1 Dec 2024 04:30:33 +0000 (23:30 -0500)
That unreadahead used to result in a 10% hit on benchmarks.  Now it's
closer to 75%.

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
src/bees-context.cc

index e259b211693a76db1479f6aa26264e5b4ae5660a..194c124c1865ee4930da0ef505da971394efb8ed 100644 (file)
@@ -839,8 +839,8 @@ BeesContext::scan_one_extent(const BeesFileRange &bfr, const Extent &e)
        }
        lock_seen.unlock();
 
-       // Still hurts benchmarks...or does it?
-       bees_unreadahead(bfr.fd(), bfr.begin(), bfr.size());
+       // Now causes 75% loss of performance in benchmarks
+       // bees_unreadahead(bfr.fd(), bfr.begin(), bfr.size());
 }
 
 shared_ptr<Exclusion>