From ba40899e84abdd1743d3f45b2200db641ae8acb8 Mon Sep 17 00:00:00 2001 From: Zach Brown Date: Mon, 11 Sep 2017 14:09:16 -0700 Subject: [PATCH] scoutfs: remove scoutfs_trans_wake_holders() This was used by compaction to wake local holders who were waiting for compaction to free up level 0 segments for them to enter the transaction. Throttling level 0 segment writes works differently now and doesn't involve blocking trans holders. Signed-off-by: Zach Brown --- kmod/src/trans.c | 12 ------------ kmod/src/trans.h | 1 - 2 files changed, 13 deletions(-) diff --git a/kmod/src/trans.c b/kmod/src/trans.c index ca05240f..d00d36c0 100644 --- a/kmod/src/trans.c +++ b/kmod/src/trans.c @@ -451,18 +451,6 @@ void scoutfs_release_trans(struct super_block *sb) wake_up(&sbi->trans_hold_wq); } -/* - * This is called to wake people waiting on holders when the conditions - * that they're waiting on change: levels being full, dirty count falling - * under a segment, or holders falling to 0. - */ -void scoutfs_trans_wake_holders(struct super_block *sb) -{ - struct scoutfs_sb_info *sbi = SCOUTFS_SB(sb); - - wake_up(&sbi->trans_hold_wq); -} - int scoutfs_setup_trans(struct super_block *sb) { struct scoutfs_sb_info *sbi = SCOUTFS_SB(sb); diff --git a/kmod/src/trans.h b/kmod/src/trans.h index 49db305d..d3c6f326 100644 --- a/kmod/src/trans.h +++ b/kmod/src/trans.h @@ -12,7 +12,6 @@ void scoutfs_trans_restart_sync_deadline(struct super_block *sb); int scoutfs_hold_trans(struct super_block *sb, struct scoutfs_item_count *cnt); bool scoutfs_trans_held(void); void scoutfs_release_trans(struct super_block *sb); -void scoutfs_trans_wake_holders(struct super_block *sb); void scoutfs_trans_track_item(struct super_block *sb, signed items, signed keys, signed vals);