Skip to content

rapido-vm: minor changes - #282

Merged
ddiss merged 4 commits into
rapido-linux:masterfrom
ddiss:vm-minor-changes
Oct 23, 2026
Merged

ddiss merged 4 commits into
rapido-linux:masterfrom
ddiss:vm-minor-changes

Conversation

@ddiss

@ddiss ddiss commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator
The following changes since commit 6f97b30b42c810cd3b147428b929476629d0ffe5:

  kmod-parser: ignore modules.builtin duplicates (2026-10-22 20:52:16 +1100)

are available in the Git repository at:

  https://github.com/ddiss/rapido.git vm-minor-changes

for you to fetch changes up to d18f56009811c2dea49babab44050fa0488cec29:

  rapido-vm: add test_vm_distro_kconfig_kernel (2026-10-23 16:54:13 +1100)

----------------------------------------------------------------
David Disseldorp (4):
      rapido-vm: improve ks_kernel_img() error output
      rapido-vm: avoid double get of KERNEL_SRC setting
      rapido-vm: rework vm_qemu_args.params initialization
      rapido-vm: add test_vm_distro_kconfig_kernel

 src/bin/rapido-vm.rs | 61 ++++++++++++++++++++++++++++++++--------------------
 1 file changed, 38 insertions(+), 23 deletions(-)

ddiss added 4 commits October 23, 2026 17:55
The error path is currently common for both distro and KERNEL_SRC based
kernels, which is confusing given that multiple search paths are used
for distro kernels.
Make the error message a bit more user friendly by showing search paths
based on KERNEL_SRC setting.

Signed-off-by: David Disseldorp <[email protected]>
This test checks that rapido's logic for finding the host's kconfig and
kernel image is working. Both tend to move around a bit between distros,
architectures and releases, so this will hopefully help us catch
regressions.

Signed-off-by: David Disseldorp <[email protected]>
@ddiss

ddiss commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator Author

v2: run rustfmt

@ddiss
ddiss merged commit ea26b83 into rapido-linux:master Oct 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant