In D43433#991937, @jhibbits wrote:

Does psim even still work?

Looks good.

Nice work, thanks.

Looks good.

Removing guest pages from the host domain has the appearance of offering some additional security, but VM control structures are still accessible (EPT, VMCS etc). Also, it only protects guests that have ppt devices configured, and doesn't do anything for other guests.

The use of num_vcpus_allowed() could probably be collapsed at some point into calc_topology()

Maybe a note to fix the spelling later in the commit :)

Been lucky that most if not all 64-bit BARs have been prefetchable.

Works fine.

This may have been leftover from some early soft-partitioning work where the host was running on CPU 0 (and low CPUs) with bhyve and a single-guest running on the remainder with halt-exits disabled.

I'd say until snapshots are enabled by default, there are no guarantees.

Looks fine.

In practice this hasn't been an issue since a) guests use ECAM for config access, or b) mask off the lower bits (e.g. Linux routines in arch/x86/pci/direct.c).

In D31578#712408, @c.koehne_beckhoff.com wrote:

Add option -l fwcfg,<name> to select either QEMU or bhyve fwcfg.

What are the features that you're looking to get from Qemu fwcfg ?

• An E820 table. It could help D26209.

...

• ACPI/SMBIOS tables. Don't know if it's benefitial, yet.

Double-accounting for memory is a never-ending issue in bhyve :(

There's zero reason to change for this alone, and it requires a flag day for both bhyve and UEFI firmware which has thus far been avoided.

As next step bhyve could implement more fwcfg items from QEMU to use QEMU's features without developing an own firmware implementation.

Bhyve's fwctl only passes the number of vcpus to the guest. That could be done easily by QEMU's fwcfg.

No POLA concerns: this is fine. Thanks bz.

• Invert sense of SEVL printing.

The register printing may be a bit verbose: feedback welcome.

AArch32 Instruction Set Attributes 5 = <CRC32,SHA2,SHA1,AES+VMULL>
AArch32 Media and VFP Features 0 = <FPRound,FPSqrt,FPDivide,DP VFPv3+v4,SP VFPv3+v4,AdvSIMD>
AArch32 Media and VFP Features 1 = <SIMDFMAC,FPHP DP Conv,SIMDHP SP Conv,SIMDSP,SIMDInt,SIMDLS,FPDNaN,FPFtZ>

• Remove HWCAP32_THUMBEE as this is unused in ARMv8
• Update COMPAT HWCAP comments to reflect ARMv8 usage.
• Print aarch32 regs, and compare across all CPUs.
• Remove _EL1 from register defines.
• Fix typo in ID_ISAR5_VCMA_IMPL constant.
• FPshVec -> FPShVec in register defines.
• Review feedback on register names

Thanks: I'll pull in changes for this with the fix for checking caps across CPUs.

• Remove HWCAP32_THUMBEE as this is unused in ARMv8
• Update COMPAT HWCAP comments to reflect ARMv8 usage.

• Merge e1000 commit 4b38eed76d for promisc filter handling.

• Use correct SPDX identifier and move to after copyrights.

I don't see any glaring problems with it that should block it from going upstream. Are you going to announce this addition on the mailing lists?

• "ethernet" -> "Ethernet"

• Fix up copyrights based on review feedback.
• Consistent capitalization of I225

Even absent my new draft, there's already a few files in the tree with this construct, so these should be fine going in prior to it being finalized.

Verified this works with a mouse on Win10.