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

[DOCS] [GSoC] add docs about how to build OV on RISC-V devices #26056

Merged

Conversation

BHbean
Copy link
Contributor

@BHbean BHbean commented Aug 14, 2024

Details:

  • add build_riscv64.md file giving instructions on how to build OV on RISC-V devices

Tickets:

  • N/A

@BHbean BHbean requested a review from a team as a code owner August 14, 2024 07:34
@BHbean BHbean requested review from tsavina and removed request for a team August 14, 2024 07:34
@github-actions github-actions bot added the category: docs OpenVINO documentation label Aug 14, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Aug 14, 2024
@a-sidorova a-sidorova added the platform: risc-v OpenVINO on RISC-V label Aug 14, 2024
@a-sidorova a-sidorova self-assigned this Aug 14, 2024
@a-sidorova a-sidorova added this to the 2024.4 milestone Aug 14, 2024
@dmitry-gorokhov dmitry-gorokhov added the category: CPU OpenVINO CPU plugin label Aug 14, 2024
docs/dev/build_riscv64.md Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
4. To cross compile OpenVINO Runtime for RISC-V devices, run `cmake` with specified `CMAKE_TOOLCHAIN_FILE` and `RISCV_TOOLCHAIN_ROOT`:
```sh
cmake .. \
-DCMAKE_BUILD_TYPE=Release \
Copy link
Contributor

Choose a reason for hiding this comment

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

The same: this is optional key. Could you please remove it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So you mean the key -DCMAKE_BUILD_TYPE is also optional?

Copy link
Contributor

Choose a reason for hiding this comment

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

In general, it is. Moreover, Release type is chosen by default.
But I don't insist

docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Show resolved Hide resolved

- [CMake](https://cmake.org/download/) 3.13 or higher
- GCC 7.5 or higher (for non-RVV) / [xuantie-gnu-toolchain](https://github.com/XUANTIE-RV/xuantie-gnu-toolchain) (for RVV)
- Python 3.10 for OpenVINO Runtime Python API
Copy link
Contributor

Choose a reason for hiding this comment

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

Python 3.8 or higher

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently only Python 3.10 is tested, and there is something wrong with Python 3.11. So I am not sure if it is suitable here to mention "Python 3.8 or higher" 🤔

docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
docs/dev/build_riscv64.md Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the category: CPU OpenVINO CPU plugin label Aug 19, 2024
docs/dev/build.md Outdated Show resolved Hide resolved
Copy link
Contributor

@a-sidorova a-sidorova left a comment

Choose a reason for hiding this comment

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

Great! Now we have tutorial how to cross-compile OpenVINO with optimized primitives for riscv64 devices! 🎉

@a-sidorova
Copy link
Contributor

build_jenkins

@dmitry-gorokhov dmitry-gorokhov added this pull request to the merge queue Aug 22, 2024
Merged via the queue into openvinotoolkit:master with commit 486fd0b Aug 22, 2024
104 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: docs OpenVINO documentation ExternalPR External contributor platform: risc-v OpenVINO on RISC-V
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants