bhyve: Assert success in a couple of libvmmapi calls
ClosedPublic
Actions

Authored by markj on Jul 31 2024, 3:19 PM.

Details

Reviewers

jhb
andrew

Group Reviewers

bhyve

Commits

rG4c5188bbeb88: bhyve: Assert success in a couple of libvmmapi calls

Summary

In vmexit_smccc(), copy an assertion from amd64.
In fbsdrun_addcpu(), make sure that our vm_suspend_cpu() call is succesful.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 58901
Build 55788: arc lint + arc unit

Event Timeline

markj created this revision.Jul 31 2024, 3:19 PM

Herald added a reviewer: andrew. · View Herald TranscriptJul 31 2024, 3:19 PM

Herald added a reviewer: bhyve. · View Herald Transcript

Herald added subscribers: bcran, rgrimes, emaste, imp. · View Herald Transcript

markj requested review of this revision.Jul 31 2024, 3:19 PM

markj added a parent revision: D46196: bhyve: Optionally put vCPUs back in the debug state after resuming.

Harbormaster completed remote builds in B58901: Diff 141596.Jul 31 2024, 3:20 PM

jhb accepted this revision.Jul 31 2024, 9:21 PM

This revision is now accepted and ready to land.Jul 31 2024, 9:21 PM

Closed by commit rG4c5188bbeb88: bhyve: Assert success in a couple of libvmmapi calls (authored by markj). · Explain WhyJul 31 2024, 11:45 PM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG4c5188bbeb88: bhyve: Assert success in a couple of libvmmapi calls.

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

aarch64/

vmexit.c

3 lines

bhyverun.c

3 lines

Diff 141596

View Options

usr.sbin/bhyve/aarch64/vmexit.c