diff options
author | Alex Auvolat <alex@adnab.me> | 2022-06-21 16:27:12 +0200 |
---|---|---|
committer | Alex Auvolat <alex@adnab.me> | 2022-06-21 16:27:12 +0200 |
commit | 14337d2a561887b94ea165727dd49263717cb478 (patch) | |
tree | 57331f5139799b96fb970b162711402808a06103 | |
parent | ba1ace6cf6edcea58aa904ccf6190155a6ac7c5e (diff) | |
download | garage-14337d2a561887b94ea165727dd49263717cb478.tar.gz garage-14337d2a561887b94ea165727dd49263717cb478.zip |
rename things
-rw-r--r-- | src/block/manager.rs | 24 | ||||
-rw-r--r-- | src/util/background/worker.rs | 1 |
2 files changed, 13 insertions, 12 deletions
diff --git a/src/block/manager.rs b/src/block/manager.rs index 54368faf..30d6e792 100644 --- a/src/block/manager.rs +++ b/src/block/manager.rs @@ -109,7 +109,7 @@ pub struct BlockManager { // it INSIDE a Mutex. struct BlockManagerLocked(); -enum BlockIterResult { +enum ResyncIterResult { BusyDidSomething, BusyDidNothing, IdleFor(Duration), @@ -478,7 +478,7 @@ impl BlockManager { fn spawn_background_worker(self: Arc<Self>) { // Launch a background workers for background resync loop processing let background = self.system.background.clone(); - let worker = BlockResyncWorker { + let worker = ResyncWorker { manager: self, tranquilizer: Tranquilizer::new(30), next_delay: Duration::from_secs(10), @@ -503,7 +503,7 @@ impl BlockManager { Ok(()) } - async fn resync_iter(&self) -> Result<BlockIterResult, db::Error> { + async fn resync_iter(&self) -> Result<ResyncIterResult, db::Error> { if let Some((time_bytes, hash_bytes)) = self.resync_queue.first()? { let time_msec = u64::from_be_bytes(time_bytes[0..8].try_into().unwrap()); let now = now_msec(); @@ -523,7 +523,7 @@ impl BlockManager { // (we want to do the remove after the insert to ensure // that the item is not lost if we crash in-between) self.resync_queue.remove(time_bytes)?; - return Ok(BlockIterResult::BusyDidNothing); + return Ok(ResyncIterResult::BusyDidNothing); } } @@ -570,9 +570,9 @@ impl BlockManager { self.resync_queue.remove(time_bytes)?; } - Ok(BlockIterResult::BusyDidSomething) + Ok(ResyncIterResult::BusyDidSomething) } else { - Ok(BlockIterResult::IdleFor(Duration::from_millis( + Ok(ResyncIterResult::IdleFor(Duration::from_millis( time_msec - now, ))) } @@ -583,7 +583,7 @@ impl BlockManager { // between the time we checked the queue and the first poll // to resync_notify.notified(): if that happens, we'll just loop // back 10 seconds later, which is fine. - Ok(BlockIterResult::IdleFor(Duration::from_secs(10))) + Ok(ResyncIterResult::IdleFor(Duration::from_secs(10))) } } @@ -716,14 +716,14 @@ impl EndpointHandler<BlockRpc> for BlockManager { } } -struct BlockResyncWorker { +struct ResyncWorker { manager: Arc<BlockManager>, tranquilizer: Tranquilizer, next_delay: Duration, } #[async_trait] -impl Worker for BlockResyncWorker { +impl Worker for ResyncWorker { fn name(&self) -> String { "Block resync worker".into() } @@ -734,14 +734,14 @@ impl Worker for BlockResyncWorker { ) -> Result<WorkerStatus, Error> { self.tranquilizer.reset(); match self.manager.resync_iter().await { - Ok(BlockIterResult::BusyDidSomething) => { + Ok(ResyncIterResult::BusyDidSomething) => { self.tranquilizer .tranquilize(self.manager.background_tranquility) .await; Ok(WorkerStatus::Busy) } - Ok(BlockIterResult::BusyDidNothing) => Ok(WorkerStatus::Busy), - Ok(BlockIterResult::IdleFor(delay)) => { + Ok(ResyncIterResult::BusyDidNothing) => Ok(WorkerStatus::Busy), + Ok(ResyncIterResult::IdleFor(delay)) => { self.next_delay = delay; Ok(WorkerStatus::Idle) } diff --git a/src/util/background/worker.rs b/src/util/background/worker.rs index e30fecd7..f916692d 100644 --- a/src/util/background/worker.rs +++ b/src/util/background/worker.rs @@ -86,6 +86,7 @@ impl WorkerProcessor { } worker = await_next_worker => { if let Some(mut worker) = worker { + trace!("{} (TID {}): {:?}", worker.worker.name(), worker.task_id, worker.status); // TODO save new worker status somewhere if worker.status == WorkerStatus::Done { info!("Worker {} (TID {}) exited", worker.worker.name(), worker.task_id); |