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

Add experimental extension for metric tracer #291

Open
wants to merge 6 commits into
base: staging/v1.10
Choose a base branch
from

Conversation

matcabral
Copy link
Contributor

No description provided.

@matcabral
Copy link
Contributor Author

implements #287 and #285

@wdamon-intel
Copy link
Contributor

@joshuaranjan please review

Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Show resolved Hide resolved
scripts/tools/metricTracer.yml Show resolved Hide resolved
matcabral and others added 2 commits March 15, 2024 16:27
* Fix scripts to support two digits minor version
* Extend `all_versions` array to include 1.8-1.10.
* Add `ignore_git_revision` flag to run script.

Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
Co-authored-by: Will Damon <will.damon@intel.com>
Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
@matcabral matcabral changed the base branch from master to staging/v1.10 March 15, 2024 20:33
@matcabral
Copy link
Contributor Author

Force push after rebasing for scripts updates

Copy link
Contributor

@joshuaranjan joshuaranjan left a comment

Choose a reason for hiding this comment

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

lgtm

@matcabral
Copy link
Contributor Author

@wdamon-intel

Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
scripts/tools/EXT_Exp_MetricTracer.rst Show resolved Hide resolved
scripts/tools/metric.yml Show resolved Hide resolved
scripts/tools/EXT_Exp_MetricTracer.rst Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
scripts/tools/metricTracer.yml Outdated Show resolved Hide resolved
Signed-off-by: Matias Cabral <matias.a.cabral@intel.com>
@matcabral
Copy link
Contributor Author

@aravindksg, @joshuaranjan , @joshuaranjan Added a new patch to extend the metric event entry to support subdeviceId. This follows the same approach sysman already has in place.

Copy link
Contributor

@aravindksg aravindksg left a comment

Choose a reason for hiding this comment

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

subdevice changes lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API: Tools enhancement New feature or request
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

Support decoding L0 Tracer Metrics Support for L0 Tracer Metrics
4 participants