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

SWDEV-294351 Fixed zero bandwith for hsa-api and hip-api traces #59

Open
wants to merge 1 commit into
base: rocm-4.3.x
Choose a base branch
from

Conversation

kikimych
Copy link

Hsa ops traces correlation id is number of hsa async-copy called.

Hsa api does not log any correlation id, but can be calculated
implicitly as number of current hsa_amd_memory_async_copy called.

Hip ops correlation id index in rec_vals was 7 instead of 8.

Hip api and hip ops correlation id were consistent.

String manipulation in copy_line calculation in register_copy
and register_activity is increasing copy_line length. Replaced
string manipulation with class instance and fields modification.

[email protected]

  Hsa ops traces correlation id is number of hsa async-copy called.

  Hsa api does not log any correlation id, but can be calculated
  implicitly as number of current hsa_amd_memory_async_copy called.

  Hip ops correlation id index in rec_vals was 7 instead of 8.

  Hip api and hip ops correlation id were consistent.

  String manipulation in copy_line calculation in register_copy
  and register_activity is increasing copy_line length. Replaced
  string manipulation with class instance and fields modification.

  [email protected]
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.

1 participant