Skip to content

Enable build with TBB on Android #31779

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 1 commit into from
Aug 20, 2025
Merged

Enable build with TBB on Android #31779

merged 1 commit into from
Aug 20, 2025

Conversation

aobolensk
Copy link
Contributor

Details:

Tickets:

  • N/A

@aobolensk aobolensk requested a review from a team as a code owner August 18, 2025 16:32
@github-actions github-actions bot added the category: build OpenVINO cmake script / infra label Aug 18, 2025
@aobolensk aobolensk added the platform: android OpenVINO on Android label Aug 18, 2025
@aobolensk aobolensk changed the title Enable TBB build on Android Enable build with TBB on Android Aug 18, 2025
@allnes allnes requested a review from alvoron August 19, 2025 10:44
@allnes allnes assigned alvoron and unassigned allnes Aug 19, 2025
alvoron
alvoron reviewed Aug 19, 2025
View reviewed changes
Copy link
Contributor

@alvoron alvoron left a comment

Choose a reason for hiding this comment

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

How do you test TBB on Android?

@aobolensk
Copy link
Contributor Author

How do you test TBB on Android?

The problem reported in the comment (reason on why it was switched to SEQ) was related with segfault during the compilation process on Android with TBB. Build is being tested on CI. Test steps are absent there as of now

@aobolensk aobolensk requested a review from alvoron August 19, 2025 11:30
@alvoron
Copy link
Contributor

alvoron commented Aug 19, 2025

How do you test TBB on Android?

The problem reported in the comment (reason on why it was switched to SEQ) was related with segfault during the compilation process on Android with TBB. Build is being tested on CI. Test steps are absent there as of now

What comment are you referring to?

@aobolensk
Copy link
Contributor Author

How do you test TBB on Android?

The problem reported in the comment (reason on why it was switched to SEQ) was related with segfault during the compilation process on Android with TBB. Build is being tested on CI. Test steps are absent there as of now

What comment are you referring to?

The one that I have removed in this patch:

# on Android we experience SEGFAULT during compilation

@alvoron
Copy link
Contributor

alvoron commented Aug 19, 2025

As discussed offline, I'd be great to run tests to verify inference.

@aobolensk
Copy link
Contributor Author

aobolensk commented Aug 20, 2025 •
edited
Loading

As discussed offline, I'd be great to run tests to verify inference.

Verified locally on the device, there is no functional regression of the tests after switch to TBB

@aobolensk aobolensk added this pull request to the merge queue Aug 20, 2025
Merged via the queue into openvinotoolkit:master with commit c355b8c Aug 20, 2025
214 of 216 checks passed
@aobolensk aobolensk deleted the android-tbb branch August 20, 2025 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alvoron alvoron alvoron approved these changes

@allnes allnes allnes approved these changes

Assignees

@alvoron alvoron

Labels
category: build OpenVINO cmake script / infra platform: android OpenVINO on Android
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aobolensk @alvoron @allnes