Project

General

Profile

Actions

Feature #13674

closed

bhyve upstream sync 2021 March

Added by Andy Fiddaman about 1 year ago. Updated about 1 year 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
Actions #1

Updated by Electric Monk about 1 year ago

  • Gerrit CR set to 1383
Actions #2

Updated by Andy Fiddaman about 1 year ago

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

Updated by Andy Fiddaman about 1 year 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
Actions #4

Updated by Electric Monk about 1 year 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>

Actions

Also available in: Atom PDF