Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

add execute detail timings #15638

Merged
merged 1 commit into from
Mar 3, 2021
Merged

add execute detail timings #15638

merged 1 commit into from
Mar 3, 2021

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Mar 2, 2021 •
edited
Loading

Problem

performance

Summary of Changes

add metrics for serialize, create vm, execute, and deserialize

Fixes #

@jeffwashington jeffwashington added the CI Pull Request is ready to enter CI label Mar 2, 2021
@jeffwashington jeffwashington force-pushed the execute_timings branch 2 times, most recently from afb4fa7 to 7f1cce5 Compare March 2, 2021 21:14
@jeffwashington
Copy link
Contributor Author

jeffwashington commented Mar 2, 2021 •
edited
Loading

example output:
confirm_full_slot ExecuteTimings load_us 110642 execute_us 300904 store_us 77064 details ExecuteDetailsTimings serialize_us 42568 create_vm_us 7660 execute_us 56902 deserialize_us 29593 changed_account_count 336 total_account_count 5348 total_data_size 347398402 data_size_changed 13661928
ledger processed in 1 minute and 4 seconds. 3955 MB allocated. root slot is 67266403 6 forks at 67266419 67266448 67266456 67266459 67266471 67266487 with 81 frozen banks

@jeffwashington jeffwashington added CI Pull Request is ready to enter CI and removed CI Pull Request is ready to enter CI labels Mar 2, 2021
@jeffwashington jeffwashington force-pushed the execute_timings branch 2 times, most recently from 8a977d9 to 07c728a Compare March 2, 2021 21:54
@jeffwashington jeffwashington added CI Pull Request is ready to enter CI and removed CI Pull Request is ready to enter CI labels Mar 2, 2021
@solana-grimes solana-grimes removed the CI Pull Request is ready to enter CI label Mar 2, 2021
@jeffwashington jeffwashington force-pushed the execute_timings branch 5 times, most recently from 40d2728 to e522b2e Compare March 3, 2021 15:51
@jeffwashington jeffwashington marked this pull request as ready for review March 3, 2021 18:42
@jeffwashington jeffwashington requested a review from jackcmay March 3, 2021 20:44
@jeffwashington jeffwashington merged commit be35c1c into solana-labs:master Mar 3, 2021
@jeffwashington jeffwashington deleted the execute_timings branch March 3, 2021 23:07
@jstarry jstarry mentioned this pull request Aug 10, 2022
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jackcmay jackcmay jackcmay approved these changes

@sakridge sakridge sakridge approved these changes

@Lichtso Lichtso Lichtso approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jeffwashington @Lichtso @sakridge @jackcmay @solana-grimes