Skip to content

VMRun: allow running riscv64 guests with RVVM#17

Merged
markjdb merged 2 commits intomarkjdb:master-pythonfrom
svmhdvn:push-tvurywyqowpu
Apr 8, 2026
Merged

VMRun: allow running riscv64 guests with RVVM#17
markjdb merged 2 commits intomarkjdb:master-pythonfrom
svmhdvn:push-tvurywyqowpu

Conversation

@svmhdvn
Copy link
Copy Markdown
Contributor

@svmhdvn svmhdvn commented Feb 19, 2026

See individual commits.

NOTE: RVVM support requires upstream RVVM, not the current emulators/rvvm freebsd port. Currently pending the addition of emulators/rvvm-devel tracking upstream HEAD.

Copy link
Copy Markdown
Owner

@markjdb markjdb left a comment

Choose a reason for hiding this comment

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

I'm ok with this if we avoid silently disabling network in RVVM.

@svmhdvn svmhdvn force-pushed the push-tvurywyqowpu branch from 0a57191 to ce562ba Compare April 2, 2026 15:03
@svmhdvn
Copy link
Copy Markdown
Contributor Author

svmhdvn commented Apr 2, 2026

Split out the other unrelated commits, will post PRs for those separately.

I've added a NetworkDriver.NONE enum value, that should handle the feedback you mentioned. Let me know if this looks good.

@svmhdvn svmhdvn changed the title More improvements after riscv64 testing VMRun: allow running riscv64 guests with RVVM Apr 2, 2026
@markjdb
Copy link
Copy Markdown
Owner

markjdb commented Apr 3, 2026

Is the intent to use RVVM for running the regression test suite? The lack of networking means that we can't fetch the kyua report using scp.

@markjdb
Copy link
Copy Markdown
Owner

markjdb commented Apr 3, 2026

Looks ok to me in any case.

@markjdb
Copy link
Copy Markdown
Owner

markjdb commented Apr 3, 2026

(but there are some conflicts, sorry)

@svmhdvn
Copy link
Copy Markdown
Contributor Author

svmhdvn commented Apr 3, 2026

Is the intent to use RVVM for running the regression test suite? The lack of networking means that we can't fetch the kyua report using scp.

RVVM has networking enabled by default, The -nonet is only necessary to disable networking.

@svmhdvn svmhdvn force-pushed the push-tvurywyqowpu branch from ce562ba to bdffc9e Compare April 3, 2026 13:03
@markjdb
Copy link
Copy Markdown
Owner

markjdb commented Apr 3, 2026

Is the intent to use RVVM for running the regression test suite? The lack of networking means that we can't fetch the kyua report using scp.

RVVM has networking enabled by default, The -nonet is only necessary to disable networking.

But we want to configure a specific address/port for ssh access, see ssh_addr in the VMRun class.

With this, we can choose to disable networking in VMs.
@svmhdvn svmhdvn force-pushed the push-tvurywyqowpu branch from bdffc9e to 5e28506 Compare April 6, 2026 15:43
RVVM seems to have a significant performance advantage
over QEMU.
@svmhdvn svmhdvn force-pushed the push-tvurywyqowpu branch from 5e28506 to 7c70751 Compare April 6, 2026 15:45
@svmhdvn
Copy link
Copy Markdown
Contributor Author

svmhdvn commented Apr 6, 2026

Is the intent to use RVVM for running the regression test suite? The lack of networking means that we can't fetch the kyua report using scp.

RVVM has networking enabled by default, The -nonet is only necessary to disable networking.

But we want to configure a specific address/port for ssh access, see ssh_addr in the VMRun class.

Thanks for this info, I've updated it now to properly port forward the ssh addr and port. I've tested this with #22 and it works well.

@markjdb markjdb merged commit c210fc9 into markjdb:master-python Apr 8, 2026
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