Page MenuHomeFreeBSD
Differential D39315

bhyve/fwcfg: add QEMU_FWCFG_INDEX_MAX_CPUS item
ClosedPublic
Actions

Authored by corvink on Mar 29 2023, 12:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jan 27, 4:50 PM
Unknown Object (File)
Thu, Jan 23, 6:26 PM
Unknown Object (File)
Thu, Jan 23, 12:35 PM
Unknown Object (File)
Sun, Jan 19, 11:16 PM
Unknown Object (File)
Sun, Jan 19, 11:15 PM
Unknown Object (File)
Sun, Jan 19, 11:15 PM
Unknown Object (File)
Sun, Jan 19, 7:15 PM
Unknown Object (File)
Sat, Jan 18, 9:31 PM
View All 68 Files
Subscribers
bcran
imp
rew
rgrimes

Details

Reviewers
jhb
markj
Group Reviewers
bhyve
Commits
rG82622189fe02: bhyve: add QEMU_FWCFG_INDEX_MAX_CPUS item
rG305edaa47918: bhyve: add QEMU_FWCFG_INDEX_MAX_CPUS item
Summary

Requested-by: coreboot

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

corvink created this revision.Mar 29 2023, 12:44 PM
Herald added subscribers: bcran, rgrimes, imp. · View Herald TranscriptMar 29 2023, 12:44 PM
corvink requested review of this revision.Mar 29 2023, 12:44 PM
Harbormaster completed remote builds in B50646: Diff 119604.Mar 29 2023, 12:44 PM
rew added a subscriber: rew.Mar 29 2023, 3:17 PM
rew added inline comments.
usr.sbin/bhyve/qemu_fwcfg.c
184

not suggesting for this review

but this could probably return EEXIST?

236

inverted error?

241–243

()'s on return

also, why the uint8_t cast on fwcfg_max_cpus?

rew mentioned this in D39316: bhyve/fwcfg: add QEMU_FWCFG_INDEX_NB_CPUS item.Mar 29 2023, 5:20 PM
corvink updated this revision to Diff 119655.Mar 30 2023, 5:56 AM
  • fix suggested style issues
Harbormaster completed remote builds in B50673: Diff 119655.Mar 30 2023, 5:56 AM
corvink marked 3 inline comments as done.Mar 30 2023, 5:57 AM
corvink added inline comments.
usr.sbin/bhyve/qemu_fwcfg.c
184

D39338

markj added inline comments.Apr 3 2023, 6:17 PM
usr.sbin/bhyve/qemu_fwcfg.c
239

Since this looks a bit odd, I'd add a comment noting that we don't support CPU hotplug (assuming that that's the reason for setting it equal to guest_ncpus and not maxcpus.

corvink updated this revision to Diff 120134.Apr 11 2023, 7:14 AM
corvink marked an inline comment as done.
  • add comment to explain that we don't support cpu hotplug
corvink marked an inline comment as done.Apr 11 2023, 7:14 AM
Harbormaster completed remote builds in B50831: Diff 120134.Apr 11 2023, 7:15 AM
markj accepted this revision as: markj.Apr 11 2023, 2:16 PM
This revision is now accepted and ready to land.Apr 11 2023, 2:16 PM
Closed by commit rG305edaa47918: bhyve: add QEMU_FWCFG_INDEX_MAX_CPUS item (authored by corvink). · Explain WhyApr 12 2023, 7:35 AM
This revision was automatically updated to reflect the committed changes.
corvink added a commit: rG305edaa47918: bhyve: add QEMU_FWCFG_INDEX_MAX_CPUS item.
corvink added a commit: rG82622189fe02: bhyve: add QEMU_FWCFG_INDEX_MAX_CPUS item.Apr 28 2023, 10:41 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
bhyve/
23 lines

Diff 120184

View Options

usr.sbin/bhyve/qemu_fwcfg.c

Loading...