GGLinnk/bees
1
0
Fork 0
mirror of https://github.com/Zygo/bees.git synced 2025-05-18 05:45:45 +02:00
Code Packages

context: drop dead code in dedup wrapper

Browse Source 
This code has been #if 0 for a long time, and it seems unlikely it
will ever be useful in the future.

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
This commit is contained in:
Zygo Blaxell 2017-09-16 16:37:04 -04:00
parent 59fe9f4617
commit 917fc8c412
1 changed files with 0 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/bees-context.cc
View File

@ -172,24 +172,6 @@ BeesContext::dedup(const BeesRangePair &brp)
brp.first.fd(shared_from_this());
brp.second.fd(shared_from_this());
#if 0
// This avoids some sort of kernel race condition;
// however, it also doubles our dedup times.
// Is avoiding a crash every few weeks worth it?
bees_sync(brp.first.fd());
#endif
// dedup isn't very long-running compared to LOGICAL_INO.
// Also we are approaching saturation on systems with many cores.
// Hopefully the lock on the extent bytenr will avoid hitting
// kernel deadlocks too often!
#if 0
// To avoid hammering all the cores with long-running ioctls,
// only do one dedup at any given time.
BEESNOTE("Waiting to dedup " << brp);
auto dedup_lock = bees_ioctl_lock_set.make_lock(gettid());
#endif
BEESNOTE("dedup " << brp);
BEESTOOLONG("dedup " << brp);