The Wayback Machine - https://web.archive.org/web/20210910054054/https://github.com/pytorch/pytorch/pull/61934
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

Support system-provided Intel TBB #61934

Closed
wants to merge 1 commit into from
Closed

Support system-provided Intel TBB #61934

wants to merge 1 commit into from

Conversation

@cbalioglu
Copy link
Contributor

@cbalioglu cbalioglu commented Jul 20, 2021

This PR: (1) enables the use of a system-provided Intel TBB for building PyTorch, (2) removes tbb:task_scheduler_init references since it has been removed from TBB a while ago (3) marks the implementation of _internal_set_num_threads with a TODO as it requires a revision that fixes its thread allocation logic.

Tested with test/run_test; no new tests are introduced since there are no behavioral changes (removal of tbb::task_scheduler_init has no impact on the runtime behavior).

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented Jul 20, 2021

🔗 Helpful links

💊 CI failures summary and remediations

As of commit 9bbfe4a (more details on the Dr. CI page):


  • 3/3 failures possibly* introduced in this PR
    • 1/3 non-scanned failure(s)

2 failures not recognized by patterns:

Job Step Action
CircleCI pytorch_cpp_doc_build Doc Build and Push 🔁 rerun
CircleCI pytorch_xla_linux_bionic_py3_6_clang9_test Run tests 🔁 rerun

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented Jul 20, 2021

@cbalioglu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@cbalioglu cbalioglu requested a review from pytorch/pytorch-dev-infra Jul 21, 2021
@cbalioglu cbalioglu force-pushed the balioglu-sys-tbb branch from 60839ce to 9bbfe4a Jul 30, 2021
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented Jul 30, 2021

@cbalioglu has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@malfet
malfet approved these changes Jul 30, 2021
@facebook-github-bot
Copy link
Contributor

@facebook-github-bot facebook-github-bot commented Aug 2, 2021

@cbalioglu merged this pull request in 7565039.

@cbalioglu cbalioglu deleted the balioglu-sys-tbb branch Aug 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants