Skip to content

[CPU] Use ITT regions API #31499

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 54 commits into from
Oct 30, 2025
Merged

[CPU] Use ITT regions API #31499

merged 54 commits into from
Oct 30, 2025

Conversation

@aobolensk
Copy link
Contributor

@aobolensk aobolensk commented Jul 28, 2025 •
edited
Loading

Details:

  • Enable basic ITT tracing by default (introduce 3 options -DENABLE_PROFILING_ITT=OFF/BASE/FULL)
  • [CPU] Introduce usage of region tracing ITT API
  • [CPU] Align naming of the events

Tickets:

@github-actions github-actions bot added category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin labels Jul 28, 2025
@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: samples OpenVINO Runtime Samples category: inference OpenVINO Runtime library - Inference category: dependency_changes Pull requests that update a dependency file labels Aug 25, 2025
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code and removed category: samples OpenVINO Runtime Samples labels Aug 27, 2025
praasz
praasz reviewed Oct 27, 2025
View reviewed changes
@aobolensk aobolensk requested review from EgorDuplensky, eparshut, praasz and t-jankowski and removed request for t-jankowski October 27, 2025 13:25
eparshut
eparshut reviewed Oct 27, 2025
View reviewed changes
src/inference/src/cpp/core.cpp Outdated
CompiledModel Core::compile_model(const std::shared_ptr<const ov::Model>& model,
const std::string& device_name,
const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated
}

CompiledModel Core::compile_model(const std::string& model_path, const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated

#ifdef OPENVINO_ENABLE_UNICODE_PATH_SUPPORT
CompiledModel Core::compile_model(const std::wstring& model_path, const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated
#endif

CompiledModel Core::compile_model(const std::string& model_path, const std::string& device_name, const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated
CompiledModel Core::compile_model(const std::wstring& model_path,
const std::string& device_name,
const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated
const ov::Tensor& weights,
const std::string& device_name,
const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

src/inference/src/cpp/core.cpp Outdated
CompiledModel Core::compile_model(const std::shared_ptr<const ov::Model>& model,
const RemoteContext& context,
const AnyMap& config) {
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");
Copy link

Choose a reason for hiding this comment

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

we can remove this extra "Compile model" task since we already have a region

Suggested change
OV_ITT_SCOPED_TASK_BASE(ov::itt::domains::OV, "Compile model");

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@aobolensk aobolensk requested a review from eparshut October 27, 2025 15:00
eparshut
eparshut reviewed Oct 28, 2025
View reviewed changes
Copy link

@eparshut eparshut left a comment

Choose a reason for hiding this comment

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

LGTM 🚀 (Approved)

@EgorDuplensky EgorDuplensky removed their assignment Oct 29, 2025
@aobolensk aobolensk added this pull request to the merge queue Oct 30, 2025
Merged via the queue into openvinotoolkit:master with commit c6d55b9 Oct 30, 2025
212 checks passed
@aobolensk aobolensk deleted the itt-regions branch October 30, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EgorDuplensky EgorDuplensky EgorDuplensky approved these changes

@praasz praasz praasz approved these changes

@zKulesza zKulesza Awaiting requested review from zKulesza zKulesza is a code owner automatically assigned from openvinotoolkit/openvino-docs-maintainers

@CuriousPanCake CuriousPanCake Awaiting requested review from CuriousPanCake CuriousPanCake is a code owner automatically assigned from openvinotoolkit/openvino-ie-transformations-maintainers

@olpipi olpipi Awaiting requested review from olpipi

@tsavina tsavina Awaiting requested review from tsavina

@t-jankowski t-jankowski Awaiting requested review from t-jankowski

+1 more reviewer

@eparshut eparshut eparshut left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@maxnick maxnick

Labels

category: build OpenVINO cmake script / infra category: CI OpenVINO public CI category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: dependency_changes Pull requests that update a dependency file category: docs OpenVINO documentation category: inference OpenVINO Runtime library - Inference Code Freeze github_actions Pull requests that update GitHub Actions code no-match-files

Projects

None yet

Milestone

2025.4

Development

Successfully merging this pull request may close these issues.

7 participants

@aobolensk @maxnick @EgorDuplensky @eparshut @t-jankowski @praasz @moslex