bhyve: Don't force an upper bound on vCPUs when parsing pinning.
ClosedPublic
Actions

Authored by jhb on Mar 8 2022, 10:39 PM.

Details

Reviewers

markj
grehan

Group Reviewers

bhyve

Commits

rG27537022c5a4: bhyve: Don't force an upper bound on vCPUs when parsing pinning.
rGfd6f92946f02: bhyve: Don't force an upper bound on vCPUs when parsing pinning.

Summary

Even today it is possible to specify pinning for a vCPU higher than
the configured number of CPUs but lower than VM_MAXCPU without raising
an error.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 44704
Build 41592: arc lint + arc unit

Event Timeline

jhb created this revision.Mar 8 2022, 10:39 PM

Herald added a reviewer: bhyve. · View Herald TranscriptMar 8 2022, 10:39 PM

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

jhb requested review of this revision.Mar 8 2022, 10:39 PM

jhb added a parent revision: D34491: bhyve: Allocate dynamic arrays to hold per-VCPU state..

jhb added a child revision: D34493: bhyve: Use vm_get_topology to query kernel's maximum vCPU count..

Harbormaster completed remote builds in B44704: Diff 103653.Mar 8 2022, 10:39 PM

grehan accepted this revision.Mar 8 2022, 10:54 PM

This revision is now accepted and ready to land.Mar 8 2022, 10:54 PM

Closed by commit rGfd6f92946f02: bhyve: Don't force an upper bound on vCPUs when parsing pinning. (authored by jhb). · Explain WhyMar 9 2022, 11:58 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGfd6f92946f02: bhyve: Don't force an upper bound on vCPUs when parsing pinning..

jhb added a commit: rG27537022c5a4: bhyve: Don't force an upper bound on vCPUs when parsing pinning..May 11 2022, 6:56 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

bhyverun.c

5 lines

Diff 103653

View Options

bhyve: Don't force an upper bound on vCPUs when parsing pinning.ClosedPublicActions