Add ProgramCacheUs to execute timings (#473)

jito-foundation · Apr 2, 2024 · 65f54be · 65f54be
1 parent e815925
commit 65f54be
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/program-runtime/src/timings.rs b/program-runtime/src/timings.rs
@@ -51,6 +51,7 @@ pub enum ExecuteTimingType {
     CollectLogsUs,
     TotalBatchesLen,
     UpdateTransactionStatuses,
+    ProgramCacheUs,
 }
 
 pub struct Metrics([u64; ExecuteTimingType::CARDINALITY]);
@@ -94,6 +95,13 @@ eager_macro_rules! { $eager_1
                     .index(ExecuteTimingType::CheckUs),
                 i64
             ),
+            (
+                "program_cache_us",
+                *$self
+                    .metrics
+                    .index(ExecuteTimingType::ProgramCacheUs),
+                i64
+            ),
             (
                 "load_us",
                 *$self

diff --git a/svm/src/transaction_processor.rs b/svm/src/transaction_processor.rs
@@ -211,6 +211,7 @@ impl<FG: ForkGraph> TransactionBatchProcessor<FG> {
         log_messages_bytes_limit: Option<usize>,
         limit_to_load_programs: bool,
     ) -> LoadAndExecuteSanitizedTransactionsOutput {
+        let mut program_cache_time = Measure::start("program_cache");
         let mut program_accounts_map = Self::filter_executable_program_accounts(
             callbacks,
             sanitized_txs,
@@ -234,6 +235,7 @@ impl<FG: ForkGraph> TransactionBatchProcessor<FG> {
                 execution_results: vec![],
             };
         }
+        program_cache_time.stop();
 
         let mut load_time = Measure::start("accounts_load");
         let mut loaded_transactions = load_accounts(
@@ -328,6 +330,10 @@ impl<FG: ForkGraph> TransactionBatchProcessor<FG> {
             sanitized_txs.len(),
         );
 
+        timings.saturating_add_in_place(
+            ExecuteTimingType::ProgramCacheUs,
+            program_cache_time.as_us(),
+        );
         timings.saturating_add_in_place(ExecuteTimingType::LoadUs, load_time.as_us());
         timings.saturating_add_in_place(ExecuteTimingType::ExecuteUs, execution_time.as_us());