Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

RUN-636: Add call context instruction counter #185

Merged
merged 9 commits into from
Oct 12, 2023

Conversation

mraszyk
Copy link
Contributor

@mraszyk mraszyk commented Jun 28, 2023

The current execution instruction counter (performance counter type 0) gets reset at the beginning of each execution, i.e. at the beginning or each call, callback, or a system task. This makes the async code profiling difficult, as the profiling data must be stored across executions and then consolidated.

This PR introduces a new per-call instruction counter (performance counter type 1) which is monotonically increasing across nested calls and await points, greatly facilitating the async code profiling and optimization.

@mraszyk mraszyk requested a review from a team as a code owner June 28, 2023 05:33
@netlify
Copy link

netlify bot commented Jun 28, 2023

Deploy Preview for ic-interface-spec ready!

Name Link
🔨 Latest commit 96c3a93
🔍 Latest deploy log https://app.netlify.com/sites/ic-interface-spec/deploys/649bc639cf54e4000825ed33
😎 Deploy Preview https://deploy-preview-185--ic-interface-spec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@Dfinity-Bjoern Dfinity-Bjoern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good from a spec perspective.

@dfinity-berestovskyy dfinity-berestovskyy changed the title RUN-636: Add per-call instruction counter RUN-636: Add call context instruction counter Sep 12, 2023
@dfinity-berestovskyy
Copy link
Contributor

LGTM, thanks!

@mraszyk mraszyk merged commit 34de08e into master Oct 12, 2023
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants