Skip to content

chore(submodule): bump libvnpu to latest hami-vnpu-core main#88

Merged
hami-robot[bot] merged 1 commit into
Project-HAMi:mainfrom
maverick123123:chore/bump-libvnpu-submodule
Jun 29, 2026
Merged

chore(submodule): bump libvnpu to latest hami-vnpu-core main#88
hami-robot[bot] merged 1 commit into
Project-HAMi:mainfrom
maverick123123:chore/bump-libvnpu-submodule

Conversation

@maverick123123

@maverick123123 maverick123123 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

What

Bump the libvnpu submodule from 32b3bd8 (Initial commit) to 7b3b4a1, the current tip of Project-HAMi/hami-vnpu-core main.

Why

The submodule pointer was pinned at the initial commit and had fallen behind upstream main, which now includes DCMI process-level memory tracking (hami-vnpu-core#8). This bump aligns the device plugin with the latest hami-vnpu-core so the soft-slicing (libvnpu.so interception) path builds against current code.

Changes

  • libvnpu: 32b3bd87b3b4a1

Submodule-pointer-only change; no source files in this repo are modified.

Summary by CodeRabbit

  • Chores
    • Updated the underlying libvnpu dependency to a newer revision to incorporate recent improvements and fixes.

@coderabbitai

coderabbitai Bot commented Jun 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c26ac84d-8738-4720-a2ea-4097dbaaba64

📥 Commits

Reviewing files that changed from the base of the PR and between e0bb2fd and a29fb70.

📒 Files selected for processing (1)
  • libvnpu

📝 Walkthrough

Walkthrough

The PR updates the libvnpu submodule pointer from one pinned commit to another. No other code logic or public interface changes are shown.

Changes

libvnpu submodule update

Layer / File(s) Summary
Pinned revision update
libvnpu
The pinned libvnpu revision changes from 32b3bd8... to f795831....

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I’m a rabbit with twitchy ears,
one tiny hop, and the pin appears.
libvnpu got a brand-new nest,
a little hash change, fluffy and blessed. 🐰

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: bumping the libvnpu submodule to the latest hami-vnpu-core main revision.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@hami-robot hami-robot Bot added the size/XS label Jun 26, 2026
Update the libvnpu submodule pointer from 32b3bd8 (Initial commit) to f795831, the current tip of Project-HAMi/hami-vnpu-core main — now includes per-container memory monitoring (file-backed local shmem) and DCMI process-level memory tracking.

Signed-off-by: maverick123123 <yuming.wu@dynamia.ai>
@maverick123123 maverick123123 force-pushed the chore/bump-libvnpu-submodule branch from e0bb2fd to a29fb70 Compare June 29, 2026 09:12

@archlitchi archlitchi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@hami-robot

hami-robot Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: archlitchi, maverick123123

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hami-robot hami-robot Bot added the approved label Jun 29, 2026
@hami-robot hami-robot Bot merged commit 92ef213 into Project-HAMi:main Jun 29, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants