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

[intel-npu] Support new internal cached_model_buffer config for memory mapped cached blobs #27822

Conversation

MirceaDan99
Copy link
Contributor

@MirceaDan99 MirceaDan99 commented Nov 29, 2024

Details:

  • Based on new import_model API from PR New caching api #27644 new plugins property to pass mmap buffer from PR New plugins property to pass mmap buffer #27981
  • Added BlobContainer class for IGraph objects that may derive with BlobContainerAlignedBuffer for the new import_model API and BlobContainerVector for the old one
  • Refactored getGraphHandle function to allow passing const uint8_t and size_t params instead of `std::vector<uint8_t>

Tickets:

  • 157192

@github-actions github-actions bot added category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin category: NPU OpenVINO NPU plugin labels Nov 29, 2024
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from ed220a4 to 94e33c4 Compare November 29, 2024 14:36
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch 3 times, most recently from 539c459 to 8cfb954 Compare December 11, 2024 09:34
@github-actions github-actions bot removed category: inference OpenVINO Runtime library - Inference category: Core OpenVINO Core (aka ngraph) category: CPU OpenVINO CPU plugin labels Dec 11, 2024
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch 2 times, most recently from 1f163fa to 5a29e3c Compare December 11, 2024 10:11
@MirceaDan99 MirceaDan99 changed the title [intel-npu] Support new internal import_model API for memory mapped cached blobs [intel-npu] Support new internal cached_model_buffer config for memory mapped cached blobs Dec 11, 2024
@MirceaDan99 MirceaDan99 marked this pull request as ready for review December 11, 2024 10:30
@MirceaDan99 MirceaDan99 requested review from a team as code owners December 11, 2024 10:30
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from 5a29e3c to 60a36c4 Compare December 11, 2024 12:12
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch 6 times, most recently from 9859b15 to 99b823b Compare December 12, 2024 15:47
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from 99b823b to 122788e Compare January 13, 2025 13:24
@MirceaDan99 MirceaDan99 requested review from a team as code owners January 13, 2025 13:24
@github-actions github-actions bot added the category: CPU OpenVINO CPU plugin label Jan 13, 2025
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from 0ffe5d6 to d6beace Compare January 20, 2025 16:39
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from d6beace to 1801088 Compare January 21, 2025 08:56
@MirceaDan99 MirceaDan99 force-pushed the POC/add_caching_with_mmap_property branch from bcda564 to 02683cf Compare January 22, 2025 10:57
@github-actions github-actions bot removed the category: build OpenVINO cmake script / infra label Jan 22, 2025
@PatrikStepan PatrikStepan added this pull request to the merge queue Jan 27, 2025
Merged via the queue into openvinotoolkit:master with commit 685527a Jan 27, 2025
157 checks passed
aobolensk pushed a commit to aobolensk/openvino that referenced this pull request Jan 28, 2025
…ory mapped cached blobs (openvinotoolkit#27822)

### Details:
- *Based on <s>new `import_model` API from PR openvinotoolkit#27644</s>&emsp;new
plugins property to pass mmap buffer from PR openvinotoolkit#27981*
- *Added `BlobContainer` class for `IGraph` objects that may derive with
`BlobContainerAlignedBuffer` for the new `import_model` API and
`BlobContainerVector` for the old one*
- *Refactored `getGraphHandle` function to allow passing `const uint8_t`
and `size_t` params instead of `std::vector<uint8_t>*

### Tickets:
 - *157192*

---------

Signed-off-by: Alexandru Enache <[email protected]>
Co-authored-by: Alexandru Enache <[email protected]>
Co-authored-by: Oleg Pipikin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: NPU OpenVINO NPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants