v1.17: Fix - Removes redundant TotalBatchesLen from `ThreadExecuteT…

…imings::accumulate()` (backport of #34179) (#34347) Fix - Removes redundant `TotalBatchesLen` from `ThreadExecuteTimings::accumulate()` (#34179) Removes redundant TotalBatchesLen from ThreadExecuteTimings::accumulate(). (cherry picked from commit c0a4fc8) Co-authored-by: Alexander Meißner <[email protected]>
solana-labs · Dec 7, 2023 · d17243f · d17243f
1 parent 311e986
commit d17243f
Showing 1 changed file with 0 additions and 7 deletions.
diff --git a/program-runtime/src/timings.rs b/program-runtime/src/timings.rs
@@ -300,13 +300,6 @@ impl ThreadExecuteTimings {
     }
 
     pub fn accumulate(&mut self, other: &ThreadExecuteTimings) {
-        self.execute_timings.saturating_add_in_place(
-            ExecuteTimingType::TotalBatchesLen,
-            *other
-                .execute_timings
-                .metrics
-                .index(ExecuteTimingType::TotalBatchesLen),
-        );
         self.execute_timings.accumulate(&other.execute_timings);
         saturating_add_assign!(self.total_thread_us, other.total_thread_us);
         saturating_add_assign!(