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

RUN-636: Add per-call instruction counter #172

Conversation

dfinity-berestovskyy
Copy link
Contributor

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 new per-call instruction counter (performance counter type 1) is monotonically increasing across nested calls and await points, greatly facilitating the async code profiling and optimization.

@dfinity-berestovskyy dfinity-berestovskyy requested a review from a team as a code owner June 7, 2023 23:17
@netlify
Copy link

netlify bot commented Jun 7, 2023

Deploy Preview for ic-interface-spec ready!

Name Link
🔨 Latest commit 84244b6
🔍 Latest deploy log https://app.netlify.com/sites/ic-interface-spec/deploys/64996c0e114b890008118c1a
😎 Deploy Preview https://deploy-preview-172--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 settings.

spec/index.adoc Outdated Show resolved Hide resolved
Copy link
Contributor

@mraszyk mraszyk left a comment

Choose a reason for hiding this comment

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

I'd link the type 1 counter to the notion of "call context" so that its specification is well-defined.

@mraszyk mraszyk marked this pull request as draft June 13, 2023 09:26
@dfinity-berestovskyy dfinity-berestovskyy force-pushed the andriy/run-636-per-call-counter branch 4 times, most recently from 86311b8 to 72502c0 Compare June 23, 2023 11:10
@dfinity-berestovskyy dfinity-berestovskyy force-pushed the andriy/run-636-per-call-counter branch from 72502c0 to f561ae0 Compare June 23, 2023 11:33
spec/index.adoc Outdated Show resolved Hide resolved
@mraszyk mraszyk marked this pull request as ready for review June 26, 2023 05:48
@mraszyk
Copy link
Contributor

mraszyk commented Jun 28, 2023

Closing in favor of #185 which is converted from AsciiDoc to MarkDown.

@mraszyk mraszyk closed this Jun 28, 2023
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