On 3/8/19 5:05 PM, Jim Fehlig wrote:
xenbus is virtual controller (akin to virtio controllers) for Xen paravirtual devices. Although all Xen VMs have a xenbus, it has never been modeled in libvirt, or in Xen native VM config format for that matter.
Recently there have been requests to support Xen's max_grant_frames setting in libvirt. max_grant_frames is best modeled as an attribute of xenbus. It describes the maximum IO buffer space (or DMA space) available in xenbus for use by connected paravirtual devices. This patch introduces a new xenbus controller type that includes a maxGrantFrames attribute.
Signed-off-by: Jim Fehlig <jfehlig@suse.com> ---
With this patch applied, I'm now getting compilation failures on Fedora 29 with gcc 8.3.1: vbox/vbox_common.c: In function 'vboxSetStorageController': vbox/vbox_common.c:355:5: error: enumeration value 'VIR_DOMAIN_CONTROLLER_TYPE_XENBUS' not handled in switch [-Werror=switch] switch ((virDomainControllerType) controller->type) { ^~~~~~ CC vbox/libvirt_driver_vbox_impl_la-vbox_storage.lo I'm pushing this as the obvious fix under the build-breaker rule: diff --git i/src/vbox/vbox_common.c w/src/vbox/vbox_common.c index c410514d37..b8dfb55ef4 100644 --- i/src/vbox/vbox_common.c +++ w/src/vbox/vbox_common.c @@ -377,6 +377,7 @@ vboxSetStorageController(virDomainControllerDefPtr controller, case VIR_DOMAIN_CONTROLLER_TYPE_CCID: case VIR_DOMAIN_CONTROLLER_TYPE_USB: case VIR_DOMAIN_CONTROLLER_TYPE_PCI: + case VIR_DOMAIN_CONTROLLER_TYPE_XENBUS: case VIR_DOMAIN_CONTROLLER_TYPE_LAST: virReportError(VIR_ERR_CONFIG_UNSUPPORTED, _("The vbox driver does not support %s controller type"), -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org