Project

General

Profile

Feature #13674

bhyve upstream sync 2021 March

Added by Andy Fiddaman 25 days ago. Updated about 17 hours ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
bhyve
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

Sync upstream activity in FreeBSD's bhyve into illumos.


Related issues

Blocked by illumos gate - Bug #13712: bhyve allows vmspace mapping inside existing rangeClosedAndy Fiddaman

Actions
#1

Updated by Electric Monk 19 days ago

  • Gerrit CR set to 1383
#2

Updated by Andy Fiddaman 8 days ago

  • Blocked by Bug #13712: bhyve allows vmspace mapping inside existing range added
#3

Updated by Andy Fiddaman 1 day ago

This change has been tested on a range of different host systems, with both Intel and AMD processors, and with a range of different guests. Combinations of options such as framebuffer(vnc) and PCI pass-through were used.

During testing, two pre-existing bugs in bhyve were exposed and fixed:

  • #13692 bhyve panic if vmm_drv_purge() fails
  • #13712 bhyve allows vmspace mapping inside existing range

The following testing notes are copied from Gerrit and email threads:

I tested on a patched version of OmniOS, running the following guests, with and without framebuffer, with the virtio RNG on the Linux guests and passing through an HDMI audio controller (the only spare PCI device on this test box!)

  • Windows 10
  • Solaris 11.4
  • FreeBSD 12
  • Ubuntu 20.04
  • OpenIndiana
  • OmniOS

Jorge Schrauwen tested with a patched SmartOS PI and reported:

Windows 10:

VNC: OK
passthru:
  - USB controller: OK
  - i40e NIC: OK
NVMe: OK
AHCI: OK
vioblk: OK
FreeBSD:

NVMe: OK
passthru:
  - USB Controller: OK
illumos:
viovlk: OK
passthru:
  - USB controller: OK
Mike Zeller tested the following guests on OmniOS, and also did some testing with the patch applied to SmartOS
  • alpine
  • centos
  • freebsd
  • haiku
  • SmartOS guest
#4

Updated by Electric Monk about 17 hours ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

git commit 2b9481465d6ee67ac62c160dbf79c3ec3348c611

commit  2b9481465d6ee67ac62c160dbf79c3ec3348c611
Author: Andy Fiddaman <omnios@citrus-it.co.uk>
Date:   2021-04-19T16:13:12.000Z

    13674 bhyve upstream sync 2021 March
    Reviewed by: Patrick Mooney <pmooney@pfmooney.com>
    Reviewed by: Jorge Schrauwen <sjorge@blackdot.be>
    Reviewed by: Mike Zeller <mike.zeller@joyent.com>
    Reviewed by: C Fraire <cfraire@me.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF