Skip to content

build: improve hvisor-tool cross-build compatibility and ivc kernel A… - #85

Merged
agicy merged 1 commit into
syswonder:mainfrom
Stone749990226:main
Jul 27, 2026
Merged

build: improve hvisor-tool cross-build compatibility and ivc kernel A…#85
agicy merged 1 commit into
syswonder:mainfrom
Stone749990226:main

Conversation

@Stone749990226

@Stone749990226 Stone749990226 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

OpenHarmony use LLVM instead of CC. And its sysroot lacks virtio_mmio definitions. And its Linux Kernel version is 6.6. So this pr:

  1. Keep tools compiler selection flexible: only fall back to CROSS_COMPILE gcc when CC is the default from make, so explicit CC overrides still work.
  2. Add a local virtio mmio UAPI compatibility header for environments where the cross sysroot lacks linux virtio_mmio definitions.
  3. Make demo binaries optional via BUILD_DEMOS (default off), so the main userspace hvisor binary is not blocked by optional rpmsg demo headers.
  4. Update ivc driver to include linux version checks and use the newer class_create API on Linux 6.4+, while keeping backward-compatible behavior for older kernels.

…PI support

Keep tools compiler selection flexible: only fall back to CROSS_COMPILE gcc when CC is the default from make, so explicit CC overrides still work.
Add a local virtio mmio UAPI compatibility header for environments where the cross sysroot lacks linux virtio_mmio definitions.
Make demo binaries optional via BUILD_DEMOS (default off), so the main userspace hvisor binary is not blocked by optional rpmsg demo headers.
Update ivc driver to include linux version checks and use the newer class_create API on Linux 6.4+, while keeping backward-compatible behavior for older kernels.

@agicy agicy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@agicy
agicy merged commit fd65826 into syswonder:main Jul 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants