From e85bb92450da5f1491c16a3e4dec02ffb7ee66a6 Mon Sep 17 00:00:00 2001
From: HaoranYi <haoran.yi@anza.xyz>
Date: Wed, 6 Nov 2024 21:55:46 +0000
Subject: [PATCH] remove cond_var

---
 .../src/verify_accounts_hash_in_background.rs | 35 +++++++------------
 ledger-tool/src/main.rs                       |  2 +-
 runtime/src/bank.rs                           |  6 ++--
 3 files changed, 16 insertions(+), 27 deletions(-)

diff --git a/accounts-db/src/verify_accounts_hash_in_background.rs b/accounts-db/src/verify_accounts_hash_in_background.rs
index f03e4e0482ce8e..5bb4e08fb06a63 100644
--- a/accounts-db/src/verify_accounts_hash_in_background.rs
+++ b/accounts-db/src/verify_accounts_hash_in_background.rs
@@ -1,22 +1,16 @@
 //! at startup, verify accounts hash in the background
-use {
-    crate::waitable_condvar::WaitableCondvar,
-    std::{
-        sync::{
-            atomic::{AtomicBool, Ordering},
-            Arc, Mutex,
-        },
-        thread::JoinHandle,
-        time::Duration,
+use std::{
+    sync::{
+        atomic::{AtomicBool, Ordering},
+        Arc, Mutex,
     },
+    thread::JoinHandle,
 };
 
 #[derive(Debug)]
 pub struct VerifyAccountsHashInBackground {
     /// true when verification has completed or never had to run in background
     pub verified: Arc<AtomicBool>,
-    /// enable waiting for verification to become complete
-    complete: Arc<WaitableCondvar>,
     /// thread doing verification
     thread: Mutex<Option<JoinHandle<bool>>>,
     /// set when background thread has completed
@@ -27,7 +21,6 @@ impl Default for VerifyAccountsHashInBackground {
     fn default() -> Self {
         // initialize, expecting possible background verification to be started
         Self {
-            complete: Arc::default(),
             // with default initialization, 'verified' is false
             verified: Arc::new(AtomicBool::new(false)),
             // no thread to start with
@@ -47,7 +40,6 @@ impl VerifyAccountsHashInBackground {
 
     /// notify that the bg process has completed
     pub fn background_finished(&self) {
-        self.complete.notify_all();
         self.background_completed.store(true, Ordering::Release);
     }
 
@@ -59,7 +51,7 @@ impl VerifyAccountsHashInBackground {
     }
 
     /// block until bg process is complete
-    pub fn wait_for_complete(&self) {
+    pub fn join_background_thread(&self) {
         // just now completing
         let mut lock = self.thread.lock().unwrap();
         if lock.is_none() {
@@ -81,14 +73,11 @@ impl VerifyAccountsHashInBackground {
             // already completed
             return true;
         }
-        if self.complete.wait_timeout(Duration::default())
-            && !self.background_completed.load(Ordering::Acquire)
-        {
-            // timed out, so not complete
+        if !self.background_completed.load(Ordering::Acquire) {
             false
         } else {
-            // Did not time out, so thread finished. Join it.
-            self.wait_for_complete();
+            // background thread has completed, so join the thread and panic if verify fails.
+            self.join_background_thread();
             true
         }
     }
@@ -134,7 +123,7 @@ pub mod tests {
         solana_logger::setup();
         let verify = Arc::new(VerifyAccountsHashInBackground::default());
         start_thread_and_return(&verify, true, || {});
-        verify.wait_for_complete();
+        verify.join_background_thread();
         assert!(verify.check_complete());
     }
 
@@ -143,7 +132,7 @@ pub mod tests {
     fn test_panic() {
         let verify = Arc::new(VerifyAccountsHashInBackground::default());
         start_thread_and_return(&verify, false, || {});
-        verify.wait_for_complete();
+        verify.join_background_thread();
         assert!(!verify.check_complete());
     }
 
@@ -159,7 +148,7 @@ pub mod tests {
         });
         assert!(!verify.check_complete());
         finish.store(true, Ordering::Relaxed);
-        verify.wait_for_complete();
+        verify.join_background_thread();
         assert!(verify.check_complete());
     }
 }
diff --git a/ledger-tool/src/main.rs b/ledger-tool/src/main.rs
index 1c5b1e28acdf01..ea8694a84fa05e 100644
--- a/ledger-tool/src/main.rs
+++ b/ledger-tool/src/main.rs
@@ -2101,7 +2101,7 @@ fn main() {
                         .accounts
                         .accounts_db
                         .verify_accounts_hash_in_bg
-                        .wait_for_complete();
+                        .join_background_thread();
 
                     let child_bank_required = rent_burn_percentage.is_ok()
                         || hashes_per_tick.is_some()
diff --git a/runtime/src/bank.rs b/runtime/src/bank.rs
index 805a22fc3db19a..35df4b4afea29f 100644
--- a/runtime/src/bank.rs
+++ b/runtime/src/bank.rs
@@ -5641,7 +5641,7 @@ impl Bank {
         accounts
             .accounts_db
             .verify_accounts_hash_in_bg
-            .wait_for_complete();
+            .join_background_thread();
 
         let slot = self.slot();
 
@@ -5968,7 +5968,7 @@ impl Bank {
             .accounts
             .accounts_db
             .verify_accounts_hash_in_bg
-            .wait_for_complete();
+            .join_background_thread();
         self.rc
             .accounts
             .accounts_db
@@ -7199,7 +7199,7 @@ impl Bank {
             .accounts
             .accounts_db
             .verify_accounts_hash_in_bg
-            .wait_for_complete()
+            .join_background_thread()
     }
 
     pub fn get_sysvar_cache_for_tests(&self) -> SysvarCache {