Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DrvX instrumentation for stat collection #90

Merged
merged 19 commits into from
May 9, 2024
Merged

Conversation

ywwu928
Copy link
Contributor

@ywwu928 ywwu928 commented May 9, 2024

Description

Important -- Read Before Creating a Pull Request

PR description

This PR write a high-level description of your pull request

Checklist

  • [v] The additions follow the code standards in the developer guide.
  • [v] New or existing tests cover these changes.
  • [v] The documentation is up to date with these changes.

@ywwu928
Copy link
Contributor Author

ywwu928 commented May 9, 2024

Requires rebuilding the container

@AdityaAtulTewari
Copy link
Contributor

I'd like some more context on what this is trying to do.

@ywwu928
Copy link
Contributor Author

ywwu928 commented May 9, 2024

I'd like some more context on what this is trying to do.

Collecting stat required for performance and system bottleneck investigation.

  1. adding stat for stall cycles when a thread is ready
  2. Let programmer specify which stage it is in and only collect stats got the actual execution stage but not the graph import stage
  3. Collect per phase stat. The support for dynamic stat counter registration is bad so preallocate certain amount of phase stat counters specified and flush it out to the output stat file once it is full
  4. add option to only collect and print out necessary stats

@ywwu928 ywwu928 merged commit 7ae5e38 into main May 9, 2024
20 checks passed
@ywwu928 ywwu928 deleted the ywwu/Drv_Instrumentation branch May 9, 2024 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants