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

refactor cpu pinning #23099

Merged
merged 5 commits into from
Feb 29, 2024
Merged

Conversation

wangleis
Copy link
Contributor

@wangleis wangleis commented Feb 27, 2024

Details:

  • Porting PR 22489
    -- add ov::affinity deprecation in CPU plugin introduction
    -- refactoring get_cpu_pinning()
    -- Default CPU pinning on Windows is false on all platform
  • Enable CPU pinning in part of CPU functional test cases. Now Intel CPU plugin func tests on Windows finish at 0h:27m:32.21s in this PR and 0h:30m:50.99s in PR 23129

Tickets:

@wangleis wangleis requested review from a team as code owners February 27, 2024 03:25
@wangleis wangleis requested review from zKulesza and removed request for a team February 27, 2024 03:25
@github-actions github-actions bot added category: IE Tests OpenVINO Test: plugins and common category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation labels Feb 27, 2024
@@ -17,6 +17,7 @@ void core_configuration(ov::test::SubgraphBaseTest* test) {
// todo: issue: 123320
test->convert_precisions.insert({ov::element::bf16, ov::element::f32});
test->convert_precisions.insert({ov::element::f16, ov::element::f32});
test->configuration.insert({ov::hint::enable_cpu_pinning.name(), true});
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add the comment why we need this setting

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

@dmitry-gorokhov
Copy link
Contributor

@wangleis Please also update PR description with Win test pass time change (before/after).

@wangleis
Copy link
Contributor Author

@wangleis Please also update PR description with Win test pass time change (before/after).

Updated

@wangleis wangleis added this pull request to the merge queue Feb 29, 2024
Merged via the queue into openvinotoolkit:master with commit 9e759e4 Feb 29, 2024
101 checks passed
@wangleis wangleis deleted the refactor_cpu_pinning branch February 29, 2024 03:18
@wangleis wangleis mentioned this pull request Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants