Memoize program cache write lock

anza-xyz · Apr 26, 2024 · cce3075 · cce3075
1 parent d50c11c
commit cce3075
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/runtime/src/bank.rs b/runtime/src/bank.rs
@@ -4156,18 +4156,23 @@ impl Bank {
 
         let mut store_executors_which_were_deployed_time =
             Measure::start("store_executors_which_were_deployed_time");
+        let mut cache = None;
         for execution_result in &execution_results {
             if let TransactionExecutionResult::Executed {
                 details,
                 programs_modified_by_tx,
             } = execution_result
             {
                 if details.status.is_ok() && !programs_modified_by_tx.is_empty() {
-                    let mut cache = self.transaction_processor.program_cache.write().unwrap();
-                    cache.merge(programs_modified_by_tx);
+                    cache
+                        .get_or_insert_with(|| {
+                            self.transaction_processor.program_cache.write().unwrap()
+                        })
+                        .merge(programs_modified_by_tx);
                 }
             }
         }
+        drop(cache);
         store_executors_which_were_deployed_time.stop();
         saturating_add_assign!(
             timings.execute_accessories.update_executors_us,