Skip to content

Add profiling messages [AP-650]#1401

Merged
woodfell merged 5 commits intomasterfrom
woodfell/native_thread_profiling
Jun 17, 2024
Merged

Add profiling messages [AP-650]#1401
woodfell merged 5 commits intomasterfrom
woodfell/native_thread_profiling

Conversation

@woodfell
Copy link
Copy Markdown
Contributor

@woodfell woodfell commented Feb 1, 2024

Description

@swift-nav/devinfra

Add definitions and tests for 3 new messages

  • MSG_PROFILING_RESOURCE_COUNTER
  • MSG_PROFILING_SYSTEM_INFO
  • MSG_PROFILING_THREAD_INFO

These messages are not intended for wide public use, the definitions and significance are defined elsewhere

As always, only the changes under the spec directory are interesting, everything else is generated

API compatibility

Does this change introduce a API compatibility risk?

No

API compatibility plan

If the above is "Yes", please detail the compatibility (or migration) plan:

N/A

JIRA Reference

https://swift-nav.atlassian.net/browse/AP-650

@woodfell woodfell force-pushed the woodfell/native_thread_profiling branch from e95f2ce to 470c354 Compare February 2, 2024 01:20
@woodfell woodfell changed the title Woodfell/native thread profiling Add profiling messages [AP-650] Feb 2, 2024
@woodfell woodfell marked this pull request as ready for review February 2, 2024 01:21
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Feb 2, 2024

Quality Gate Passed Quality Gate passed for 'libsbp-c'

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
92.8% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Comment on lines +152 to +156
id: 0xCF03
short_desc: Information about resource buckets
desc: >
Information about resource buckets. Refer to product documentation to understand
the meaning and values in this message.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this description correct? I guess it contains buckets but not exactly what I was expecting give the message name

@woodfell woodfell force-pushed the woodfell/native_thread_profiling branch from 470c354 to 99701e0 Compare June 15, 2024 02:04
@sonarqubecloud
Copy link
Copy Markdown

@woodfell woodfell merged commit 7da3bf7 into master Jun 17, 2024
@woodfell woodfell deleted the woodfell/native_thread_profiling branch June 17, 2024 05:17
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.

3 participants