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

latency_project_files #560

Closed
wants to merge 1 commit into from
Closed

Conversation

Creed2002
Copy link

Description

PEBS Sandy Bridge microarchitecture
Latency Measurement Program

Type of change

Program Added

How Has This Been Tested?

Tested within Linux Bash script using Intel documentation on LLNL Machines.

Checklist:

  • I have run ./scripts/check-code-format.sh and confirm my code code follows the style guidelines of variorum
  • I have added comments in my code
  • [x ] My changes generate no new warnings (build with -DENABLE_WARNINGS=ON)
  • [x ] New and existing unit tests pass with my changes

Thank you for taking the time to contribute to Variorum!

@rountree
Copy link
Collaborator

@Creed2002 I'd like to see this functionality included in the library---as in, an application that is linked to the Variorum library can call a C function and measure latency. Try compiling and examining some of the sample programs that come with Variorum to see how this works.

@Creed2002
Copy link
Author

Creed2002 commented Jun 27, 2024 via email

@tpatki
Copy link
Member

tpatki commented Jun 27, 2024

@rountree @Creed2002

  • Can you describe this PR a bit more clearly and open an associated issue of what the high-level problem you're trying to solve?
  • What is the PR trying to do with PEBS and how does this connect with Variorum?
    Where do we expect this to extend existing functionality: are we thinking of adding a new API to Variorum eventually?
  • Is this expected supported only Intel systems? Or Intel and AMD (x86)? If it is specific to one platform, we should consider creating an advanced API header within Variorum. As of now, I see two scripts with some wrmsr commands for PEBS and not sure how these are fitting in to Variorum.
  • Is it intended for production or research (I can tag accordingly in milestones)?

@tpatki tpatki added status-work-in-progress In progress, not ready to merge. area-feature-support labels Jun 27, 2024
@tpatki tpatki marked this pull request as draft June 28, 2024 19:32
@tpatki
Copy link
Member

tpatki commented Jul 11, 2024

Superseded by #566.

@tpatki tpatki closed this Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-feature-support status-work-in-progress In progress, not ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants