Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Fri, Dec 07, 2018 at 04:46:29PM +, Daniel P. Berrangé wrote: > I'm not convinced it is useful enough to justify playing games in qdev > with dynamically hiding devices. This adds complexity to the code which > will make it harder to maintain and debug at runtime. I actually think a hidden device is a useful concept to model. E.g. you can have a powered off slot and a PCI device in such a slot isn't visible but isn't gone either. Right now we force-eject such devices. But it sounds reasonable that e.g. a bunch of guests cooperate and share an assigned device and then whoever wants to use it, powers it up. These patches do not implement this of course but it's a step in that direction. -- MST
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Dec 06, 2018 at 10:06:18AM +, Daniel P. Berrangé wrote: > On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > > From: Sameeh Jubran > > > > > > > > Hi all, > > > > > > > > Background: > > > > > > > > There has been a few attempts to implement the standby feature for vfio > > > > assigned devices which aims to enable the migration of such devices. > > > > This > > > > is another attempt. > > > > > > > > The series implements an infrastructure for hiding devices from the bus > > > > upon boot. What it does is the following: > > > > > > > > * In the first patch the infrastructure for hiding the device is added > > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > > state and it is set based on a callback to the standby device which > > > > registers itself for handling the assessment: "should the primary > > > > device > > > > be hidden?" by cross validating the ids of the devices. > > > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > > device and unhides it when the feature is acked. > > > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > > devices to the guest, one emulated, one physical PCI device. The guest > > > would > > > put them in a bonded pair. Before migration the PCI device is unplugged & > > > a > > > new PCI device plugged on target after migration. The guest traffic > > > continues > > > without interuption due to the emulate device. > > > > > > This kind of conceptual approach can already be implemented today by > > > management > > > apps. The only hard problem that exists today is how the guest OS can > > > figure > > > out that a particular pair of devices it has are intended to be used > > > together. > > > > > > With this series, IIUC, the virtio-net device is getting a given property > > > which > > > defines the qdev ID of the associated VFIO device. When the guest OS > > > activates > > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > > unhides the associated VFIO device. > > > > > > AFAICT the guest has to infer that the device which suddenly appears is > > > the one > > > associated with the virtio-net device it just initialized, for purposes of > > > setting up the NIC bonding. There doesn't appear to be any explicit > > > assocation > > > between the devices exposed to the guest. > > > > > > This feels pretty fragile for a guest needing to match up devices when > > > there > > > are many pairs of devices exposed to a single guest. > > > > The impression I get from > > linux.git:Documentation/networking/net_failover.rst > > is that the matching is done based on the primary/standby NICs having > > the same MAC address. In theory you pass both to a guest and based on > > MAC it essentially does automatic, and if you additionally add STANDBY > > it'll know to use a virtio-net device specifically for failover. > > > > None of this requires any sort of hiding/plugging of devices from > > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live > > migration > > and the VFIO hotplug on the other end to switch back over). > > > > That simplifies things greatly, but also introduces the problem of how > > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > > is why this series is looking at hotplugging the VFIO device only after > > we confirm STANDBY is supported by the virtio-net device, and why it's > > being done transparent to management. > > > > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always > > > has > > > to be available at the time QEMU is started. There's no way to boot a > > > guest > > > and then later hotplug a VFIO device to accelerate the existing > > > virtio-net NIC. > > > Or similarly after migration there might not be any VFIO device available > > > initially when QEMU is started to accept the incoming migration. So it > > > might > > > need to run in degraded mode for an extended period of time until one > > > becomes > > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > > the > > > qdev ID of the future VFIO device would need to be decided upfront before > > > it > > > even exists. > > > > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of > > > devices. I > > > would much prefer to see some way to expose an explicit relationship > > > between > > > the devices to the guest. > > > > If we place the burden of determining whether the guest supports STANDBY > > on the part of users/management, a lot of this complexity goes away. For > > instance, one possible implementation is to simply fail migration and say > > "sorry your VFIO device is still there" if the VFIO device is still around > > at the start of migration (whether
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Dec 06, 2018 at 10:06:18AM +, Daniel P. Berrangé wrote: > On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > > From: Sameeh Jubran > > > > > > > > Hi all, > > > > > > > > Background: > > > > > > > > There has been a few attempts to implement the standby feature for vfio > > > > assigned devices which aims to enable the migration of such devices. > > > > This > > > > is another attempt. > > > > > > > > The series implements an infrastructure for hiding devices from the bus > > > > upon boot. What it does is the following: > > > > > > > > * In the first patch the infrastructure for hiding the device is added > > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > > state and it is set based on a callback to the standby device which > > > > registers itself for handling the assessment: "should the primary > > > > device > > > > be hidden?" by cross validating the ids of the devices. > > > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > > device and unhides it when the feature is acked. > > > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > > devices to the guest, one emulated, one physical PCI device. The guest > > > would > > > put them in a bonded pair. Before migration the PCI device is unplugged & > > > a > > > new PCI device plugged on target after migration. The guest traffic > > > continues > > > without interuption due to the emulate device. > > > > > > This kind of conceptual approach can already be implemented today by > > > management > > > apps. The only hard problem that exists today is how the guest OS can > > > figure > > > out that a particular pair of devices it has are intended to be used > > > together. > > > > > > With this series, IIUC, the virtio-net device is getting a given property > > > which > > > defines the qdev ID of the associated VFIO device. When the guest OS > > > activates > > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > > unhides the associated VFIO device. > > > > > > AFAICT the guest has to infer that the device which suddenly appears is > > > the one > > > associated with the virtio-net device it just initialized, for purposes of > > > setting up the NIC bonding. There doesn't appear to be any explicit > > > assocation > > > between the devices exposed to the guest. > > > > > > This feels pretty fragile for a guest needing to match up devices when > > > there > > > are many pairs of devices exposed to a single guest. > > > > The impression I get from > > linux.git:Documentation/networking/net_failover.rst > > is that the matching is done based on the primary/standby NICs having > > the same MAC address. In theory you pass both to a guest and based on > > MAC it essentially does automatic, and if you additionally add STANDBY > > it'll know to use a virtio-net device specifically for failover. > > > > None of this requires any sort of hiding/plugging of devices from > > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live > > migration > > and the VFIO hotplug on the other end to switch back over). > > > > That simplifies things greatly, but also introduces the problem of how > > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > > is why this series is looking at hotplugging the VFIO device only after > > we confirm STANDBY is supported by the virtio-net device, and why it's > > being done transparent to management. > > > > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always > > > has > > > to be available at the time QEMU is started. There's no way to boot a > > > guest > > > and then later hotplug a VFIO device to accelerate the existing > > > virtio-net NIC. > > > Or similarly after migration there might not be any VFIO device available > > > initially when QEMU is started to accept the incoming migration. So it > > > might > > > need to run in degraded mode for an extended period of time until one > > > becomes > > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > > the > > > qdev ID of the future VFIO device would need to be decided upfront before > > > it > > > even exists. > > > > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of > > > devices. I > > > would much prefer to see some way to expose an explicit relationship > > > between > > > the devices to the guest. > > > > If we place the burden of determining whether the guest supports STANDBY > > on the part of users/management, a lot of this complexity goes away. For > > instance, one possible implementation is to simply fail migration and say > > "sorry your VFIO device is still there" if the VFIO device is still around > > at the start of migration (whether
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Fri, Dec 07, 2018 at 02:36:07PM -0200, Eduardo Habkost wrote: > On Thu, Dec 06, 2018 at 10:06:18AM +, Daniel P. Berrangé wrote: > > On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > > > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > > > From: Sameeh Jubran > > > > > > > > > > Hi all, > > > > > > > > > > Background: > > > > > > > > > > There has been a few attempts to implement the standby feature for > > > > > vfio > > > > > assigned devices which aims to enable the migration of such devices. > > > > > This > > > > > is another attempt. > > > > > > > > > > The series implements an infrastructure for hiding devices from the > > > > > bus > > > > > upon boot. What it does is the following: > > > > > > > > > > * In the first patch the infrastructure for hiding the device is added > > > > > for the qbus and qdev APIs. A "hidden" boolean is added to the > > > > > device > > > > > state and it is set based on a callback to the standby device which > > > > > registers itself for handling the assessment: "should the primary > > > > > device > > > > > be hidden?" by cross validating the ids of the devices. > > > > > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > > > device and unhides it when the feature is acked. > > > > > > > > IIUC, the general idea is that we want to provide a pair of associated > > > > NIC > > > > devices to the guest, one emulated, one physical PCI device. The guest > > > > would > > > > put them in a bonded pair. Before migration the PCI device is unplugged > > > > & a > > > > new PCI device plugged on target after migration. The guest traffic > > > > continues > > > > without interuption due to the emulate device. > > > > > > > > This kind of conceptual approach can already be implemented today by > > > > management > > > > apps. The only hard problem that exists today is how the guest OS can > > > > figure > > > > out that a particular pair of devices it has are intended to be used > > > > together. > > > > > > > > With this series, IIUC, the virtio-net device is getting a given > > > > property which > > > > defines the qdev ID of the associated VFIO device. When the guest OS > > > > activates > > > > the virtio-net device and acknowledges the STANDBY feature bit, qdev > > > > then > > > > unhides the associated VFIO device. > > > > > > > > AFAICT the guest has to infer that the device which suddenly appears is > > > > the one > > > > associated with the virtio-net device it just initialized, for purposes > > > > of > > > > setting up the NIC bonding. There doesn't appear to be any explicit > > > > assocation > > > > between the devices exposed to the guest. > > > > > > > > This feels pretty fragile for a guest needing to match up devices when > > > > there > > > > are many pairs of devices exposed to a single guest. > > > > > > The impression I get from > > > linux.git:Documentation/networking/net_failover.rst > > > is that the matching is done based on the primary/standby NICs having > > > the same MAC address. In theory you pass both to a guest and based on > > > MAC it essentially does automatic, and if you additionally add STANDBY > > > it'll know to use a virtio-net device specifically for failover. > > > > > > None of this requires any sort of hiding/plugging of devices from > > > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live > > > migration > > > and the VFIO hotplug on the other end to switch back over). > > > > > > That simplifies things greatly, but also introduces the problem of how > > > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > > > is why this series is looking at hotplugging the VFIO device only after > > > we confirm STANDBY is supported by the virtio-net device, and why it's > > > being done transparent to management. > > > > > > > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device > > > > always has > > > > to be available at the time QEMU is started. There's no way to boot a > > > > guest > > > > and then later hotplug a VFIO device to accelerate the existing > > > > virtio-net NIC. > > > > Or similarly after migration there might not be any VFIO device > > > > available > > > > initially when QEMU is started to accept the incoming migration. So it > > > > might > > > > need to run in degraded mode for an extended period of time until one > > > > becomes > > > > available for hotplugging. The use of qdev IDs makes this troublesome, > > > > as the > > > > qdev ID of the future VFIO device would need to be decided upfront > > > > before it > > > > even exists. > > > > > > > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of > > > > devices. I > > > > would much prefer to see some way to expose an explicit relationship > > > > between > > > > the devices to the guest. > > > > > >
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Dec 06, 2018 at 10:06:18AM +, Daniel P. Berrangé wrote: > On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > > From: Sameeh Jubran > > > > > > > > Hi all, > > > > > > > > Background: > > > > > > > > There has been a few attempts to implement the standby feature for vfio > > > > assigned devices which aims to enable the migration of such devices. > > > > This > > > > is another attempt. > > > > > > > > The series implements an infrastructure for hiding devices from the bus > > > > upon boot. What it does is the following: > > > > > > > > * In the first patch the infrastructure for hiding the device is added > > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > > state and it is set based on a callback to the standby device which > > > > registers itself for handling the assessment: "should the primary > > > > device > > > > be hidden?" by cross validating the ids of the devices. > > > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > > device and unhides it when the feature is acked. > > > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > > devices to the guest, one emulated, one physical PCI device. The guest > > > would > > > put them in a bonded pair. Before migration the PCI device is unplugged & > > > a > > > new PCI device plugged on target after migration. The guest traffic > > > continues > > > without interuption due to the emulate device. > > > > > > This kind of conceptual approach can already be implemented today by > > > management > > > apps. The only hard problem that exists today is how the guest OS can > > > figure > > > out that a particular pair of devices it has are intended to be used > > > together. > > > > > > With this series, IIUC, the virtio-net device is getting a given property > > > which > > > defines the qdev ID of the associated VFIO device. When the guest OS > > > activates > > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > > unhides the associated VFIO device. > > > > > > AFAICT the guest has to infer that the device which suddenly appears is > > > the one > > > associated with the virtio-net device it just initialized, for purposes of > > > setting up the NIC bonding. There doesn't appear to be any explicit > > > assocation > > > between the devices exposed to the guest. > > > > > > This feels pretty fragile for a guest needing to match up devices when > > > there > > > are many pairs of devices exposed to a single guest. > > > > The impression I get from > > linux.git:Documentation/networking/net_failover.rst > > is that the matching is done based on the primary/standby NICs having > > the same MAC address. In theory you pass both to a guest and based on > > MAC it essentially does automatic, and if you additionally add STANDBY > > it'll know to use a virtio-net device specifically for failover. > > > > None of this requires any sort of hiding/plugging of devices from > > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live > > migration > > and the VFIO hotplug on the other end to switch back over). > > > > That simplifies things greatly, but also introduces the problem of how > > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > > is why this series is looking at hotplugging the VFIO device only after > > we confirm STANDBY is supported by the virtio-net device, and why it's > > being done transparent to management. > > > > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always > > > has > > > to be available at the time QEMU is started. There's no way to boot a > > > guest > > > and then later hotplug a VFIO device to accelerate the existing > > > virtio-net NIC. > > > Or similarly after migration there might not be any VFIO device available > > > initially when QEMU is started to accept the incoming migration. So it > > > might > > > need to run in degraded mode for an extended period of time until one > > > becomes > > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > > the > > > qdev ID of the future VFIO device would need to be decided upfront before > > > it > > > even exists. > > > > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of > > > devices. I > > > would much prefer to see some way to expose an explicit relationship > > > between > > > the devices to the guest. > > > > If we place the burden of determining whether the guest supports STANDBY > > on the part of users/management, a lot of this complexity goes away. For > > instance, one possible implementation is to simply fail migration and say > > "sorry your VFIO device is still there" if the VFIO device is still around > > at the start of migration (whether
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > From: Sameeh Jubran > > > > > > Hi all, > > > > > > Background: > > > > > > There has been a few attempts to implement the standby feature for vfio > > > assigned devices which aims to enable the migration of such devices. This > > > is another attempt. > > > > > > The series implements an infrastructure for hiding devices from the bus > > > upon boot. What it does is the following: > > > > > > * In the first patch the infrastructure for hiding the device is added > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > state and it is set based on a callback to the standby device which > > > registers itself for handling the assessment: "should the primary device > > > be hidden?" by cross validating the ids of the devices. > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > device and unhides it when the feature is acked. > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > devices to the guest, one emulated, one physical PCI device. The guest would > > put them in a bonded pair. Before migration the PCI device is unplugged & a > > new PCI device plugged on target after migration. The guest traffic > > continues > > without interuption due to the emulate device. > > > > This kind of conceptual approach can already be implemented today by > > management > > apps. The only hard problem that exists today is how the guest OS can figure > > out that a particular pair of devices it has are intended to be used > > together. > > > > With this series, IIUC, the virtio-net device is getting a given property > > which > > defines the qdev ID of the associated VFIO device. When the guest OS > > activates > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > unhides the associated VFIO device. > > > > AFAICT the guest has to infer that the device which suddenly appears is the > > one > > associated with the virtio-net device it just initialized, for purposes of > > setting up the NIC bonding. There doesn't appear to be any explicit > > assocation > > between the devices exposed to the guest. > > > > This feels pretty fragile for a guest needing to match up devices when there > > are many pairs of devices exposed to a single guest. > > The impression I get from linux.git:Documentation/networking/net_failover.rst > is that the matching is done based on the primary/standby NICs having > the same MAC address. In theory you pass both to a guest and based on > MAC it essentially does automatic, and if you additionally add STANDBY > it'll know to use a virtio-net device specifically for failover. > > None of this requires any sort of hiding/plugging of devices from > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live migration > and the VFIO hotplug on the other end to switch back over). > > That simplifies things greatly, but also introduces the problem of how > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > is why this series is looking at hotplugging the VFIO device only after > we confirm STANDBY is supported by the virtio-net device, and why it's > being done transparent to management. > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always has > > to be available at the time QEMU is started. There's no way to boot a guest > > and then later hotplug a VFIO device to accelerate the existing virtio-net > > NIC. > > Or similarly after migration there might not be any VFIO device available > > initially when QEMU is started to accept the incoming migration. So it might > > need to run in degraded mode for an extended period of time until one > > becomes > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > the > > qdev ID of the future VFIO device would need to be decided upfront before it > > even exists. > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of devices. I > > would much prefer to see some way to expose an explicit relationship between > > the devices to the guest. > > If we place the burden of determining whether the guest supports STANDBY > on the part of users/management, a lot of this complexity goes away. For > instance, one possible implementation is to simply fail migration and say > "sorry your VFIO device is still there" if the VFIO device is still around > at the start of migration (whether due to unplug failure or a > user/management forgetting to do it manually beforehand). > > So how important is it that setting F_STANDBY cap doesn't break older > guests? If the idea is to support live migration with VFs then aren't > we still dead in the water if the guest boots okay but doesn't have > the requisite functionality to be migrated later?
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Wed, Dec 05, 2018 at 03:57:14PM -0500, Michael S. Tsirkin wrote: > On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always > > > has > > > to be available at the time QEMU is started. There's no way to boot a > > > guest > > > and then later hotplug a VFIO device to accelerate the existing > > > virtio-net NIC. > > > Or similarly after migration there might not be any VFIO device available > > > initially when QEMU is started to accept the incoming migration. So it > > > might > > > need to run in degraded mode for an extended period of time until one > > > becomes > > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > > the > > > qdev ID of the future VFIO device would need to be decided upfront before > > > it > > > even exists. > > > > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of > > > devices. I > > > would much prefer to see some way to expose an explicit relationship > > > between > > > the devices to the guest. > > > > If we place the burden of determining whether the guest supports STANDBY > > on the part of users/management, a lot of this complexity goes away. For > > instance, one possible implementation is to simply fail migration and say > > "sorry your VFIO device is still there" if the VFIO device is still around > > at the start of migration (whether due to unplug failure or a > > user/management forgetting to do it manually beforehand). > > It's a bit different. What happens is that migration just doesn't > finish. Same as it sometimes doesn't when guest dirties too much memory. > Upper layers usually handle that in a way similar to what you describe. > If it's desirable that the reason for migration not finishing is > reported to user, we can add that information for sure. Though most > users likely won't care. Users absolutely *do* care why migration is not finishing. A migration that does not finish is a major problem for mgmt apps in many case of the use cases for migration. Especially important when evacuating VMs from a host in order to do a software upgrade or replace faulty hardware. As mentioned previously, they will also often serialize migrations to prevent eh network being overutilized, so a migration that runs indefinitely will stall evacuation of additional VMs too. Predictable execution of migration and clear error reporting/handling are critical features. IMHO this is the key reason VFIO unplug/plug needs to be done explicitly by the mgmt app, so it can be in control over when each part of the process takes place. > > So how important is it that setting F_STANDBY cap doesn't break older > > guests? If the idea is to support live migration with VFs then aren't > > we still dead in the water if the guest boots okay but doesn't have > > the requisite functionality to be migrated later? > > No because such legacy guest will never see the PT device at all. So it > can migrate. PCI devices are a precious finite resource. If a guest is not going to use it, we must never add the VFIO device to QEMU in the first place. Adding a PCI device that is never activated wastes precious resources, preventing other guests that need PCI devices from being launched on the host. Regards, Daniel -- |: https://berrange.com -o-https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o-https://fstop138.berrange.com :| |: https://entangle-photo.org-o-https://www.instagram.com/dberrange :|
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Wed, Dec 05, 2018 at 02:44:38PM -0600, Michael Roth wrote: > > So how important is it that setting F_STANDBY cap doesn't break older > > guests? If the idea is to support live migration with VFs then aren't > > we still dead in the water if the guest boots okay but doesn't have > > the requisite functionality to be migrated later? Shouldn't that all > > Well, I guess that's not really the scenario with this approach. Instead > they'd run with degraded network performance but could still at least be > migrated. Thanks, that's a good summary. And instead of degraded we call it un-accelerated. -- MST
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Wed, Dec 05, 2018 at 02:24:32PM -0600, Michael Roth wrote: > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > From: Sameeh Jubran > > > > > > Hi all, > > > > > > Background: > > > > > > There has been a few attempts to implement the standby feature for vfio > > > assigned devices which aims to enable the migration of such devices. This > > > is another attempt. > > > > > > The series implements an infrastructure for hiding devices from the bus > > > upon boot. What it does is the following: > > > > > > * In the first patch the infrastructure for hiding the device is added > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > state and it is set based on a callback to the standby device which > > > registers itself for handling the assessment: "should the primary device > > > be hidden?" by cross validating the ids of the devices. > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > device and unhides it when the feature is acked. > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > devices to the guest, one emulated, one physical PCI device. The guest would > > put them in a bonded pair. Before migration the PCI device is unplugged & a > > new PCI device plugged on target after migration. The guest traffic > > continues > > without interuption due to the emulate device. > > > > This kind of conceptual approach can already be implemented today by > > management > > apps. The only hard problem that exists today is how the guest OS can figure > > out that a particular pair of devices it has are intended to be used > > together. > > > > With this series, IIUC, the virtio-net device is getting a given property > > which > > defines the qdev ID of the associated VFIO device. When the guest OS > > activates > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > unhides the associated VFIO device. > > > > AFAICT the guest has to infer that the device which suddenly appears is the > > one > > associated with the virtio-net device it just initialized, for purposes of > > setting up the NIC bonding. There doesn't appear to be any explicit > > assocation > > between the devices exposed to the guest. > > > > This feels pretty fragile for a guest needing to match up devices when there > > are many pairs of devices exposed to a single guest. > > The impression I get from linux.git:Documentation/networking/net_failover.rst > is that the matching is done based on the primary/standby NICs having > the same MAC address. In theory you pass both to a guest and based on > MAC it essentially does automatic, and if you additionally add STANDBY > it'll know to use a virtio-net device specifically for failover. > > None of this requires any sort of hiding/plugging of devices from > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live migration > and the VFIO hotplug on the other end to switch back over). > > That simplifies things greatly, but also introduces the problem of how > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > is why this series is looking at hotplugging the VFIO device only after > we confirm STANDBY is supported by the virtio-net device, and why it's > being done transparent to management. Exactly, thanks for the summary. > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always has > > to be available at the time QEMU is started. There's no way to boot a guest > > and then later hotplug a VFIO device to accelerate the existing virtio-net > > NIC. > > Or similarly after migration there might not be any VFIO device available > > initially when QEMU is started to accept the incoming migration. So it might > > need to run in degraded mode for an extended period of time until one > > becomes > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > the > > qdev ID of the future VFIO device would need to be decided upfront before it > > even exists. > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of devices. I > > would much prefer to see some way to expose an explicit relationship between > > the devices to the guest. > > If we place the burden of determining whether the guest supports STANDBY > on the part of users/management, a lot of this complexity goes away. For > instance, one possible implementation is to simply fail migration and say > "sorry your VFIO device is still there" if the VFIO device is still around > at the start of migration (whether due to unplug failure or a > user/management forgetting to do it manually beforehand). It's a bit different. What happens is that migration just doesn't finish. Same as it sometimes doesn't when guest dirties too much memory. Upper layers usually handle that in a way similar to what you describe. If it's desirable that
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
Quoting Michael Roth (2018-12-05 14:24:32) > Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > > From: Sameeh Jubran > > > > > > Hi all, > > > > > > Background: > > > > > > There has been a few attempts to implement the standby feature for vfio > > > assigned devices which aims to enable the migration of such devices. This > > > is another attempt. > > > > > > The series implements an infrastructure for hiding devices from the bus > > > upon boot. What it does is the following: > > > > > > * In the first patch the infrastructure for hiding the device is added > > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > > state and it is set based on a callback to the standby device which > > > registers itself for handling the assessment: "should the primary device > > > be hidden?" by cross validating the ids of the devices. > > > > > > * In the second patch the virtio-net uses the API to hide the vfio > > > device and unhides it when the feature is acked. > > > > IIUC, the general idea is that we want to provide a pair of associated NIC > > devices to the guest, one emulated, one physical PCI device. The guest would > > put them in a bonded pair. Before migration the PCI device is unplugged & a > > new PCI device plugged on target after migration. The guest traffic > > continues > > without interuption due to the emulate device. > > > > This kind of conceptual approach can already be implemented today by > > management > > apps. The only hard problem that exists today is how the guest OS can figure > > out that a particular pair of devices it has are intended to be used > > together. > > > > With this series, IIUC, the virtio-net device is getting a given property > > which > > defines the qdev ID of the associated VFIO device. When the guest OS > > activates > > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > > unhides the associated VFIO device. > > > > AFAICT the guest has to infer that the device which suddenly appears is the > > one > > associated with the virtio-net device it just initialized, for purposes of > > setting up the NIC bonding. There doesn't appear to be any explicit > > assocation > > between the devices exposed to the guest. > > > > This feels pretty fragile for a guest needing to match up devices when there > > are many pairs of devices exposed to a single guest. > > The impression I get from linux.git:Documentation/networking/net_failover.rst > is that the matching is done based on the primary/standby NICs having > the same MAC address. In theory you pass both to a guest and based on > MAC it essentially does automatic, and if you additionally add STANDBY > it'll know to use a virtio-net device specifically for failover. > > None of this requires any sort of hiding/plugging of devices from > QEMU/libvirt (except for the VFIO unplug we'd need to initiate live migration > and the VFIO hotplug on the other end to switch back over). > > That simplifies things greatly, but also introduces the problem of how > an older guest will handle seeing 2 NICs with the same MAC, which IIUC > is why this series is looking at hotplugging the VFIO device only after > we confirm STANDBY is supported by the virtio-net device, and why it's > being done transparent to management. > > > > > Unless I'm mis-reading the patches, it looks like the VFIO device always has > > to be available at the time QEMU is started. There's no way to boot a guest > > and then later hotplug a VFIO device to accelerate the existing virtio-net > > NIC. > > Or similarly after migration there might not be any VFIO device available > > initially when QEMU is started to accept the incoming migration. So it might > > need to run in degraded mode for an extended period of time until one > > becomes > > available for hotplugging. The use of qdev IDs makes this troublesome, as > > the > > qdev ID of the future VFIO device would need to be decided upfront before it > > even exists. > > > > > So overall I'm not really a fan of the dynamic hiding/unhiding of devices. I > > would much prefer to see some way to expose an explicit relationship between > > the devices to the guest. > > If we place the burden of determining whether the guest supports STANDBY > on the part of users/management, a lot of this complexity goes away. For > instance, one possible implementation is to simply fail migration and say > "sorry your VFIO device is still there" if the VFIO device is still around > at the start of migration (whether due to unplug failure or a > user/management forgetting to do it manually beforehand). > > So how important is it that setting F_STANDBY cap doesn't break older > guests? If the idea is to support live migration with VFs then aren't > we still dead in the water if the guest boots okay but doesn't have > the requisite functionality to be migrated later? Shouldn't that all
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
Quoting Daniel P. Berrangé (2018-12-05 11:18:18) > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > From: Sameeh Jubran > > > > Hi all, > > > > Background: > > > > There has been a few attempts to implement the standby feature for vfio > > assigned devices which aims to enable the migration of such devices. This > > is another attempt. > > > > The series implements an infrastructure for hiding devices from the bus > > upon boot. What it does is the following: > > > > * In the first patch the infrastructure for hiding the device is added > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > state and it is set based on a callback to the standby device which > > registers itself for handling the assessment: "should the primary device > > be hidden?" by cross validating the ids of the devices. > > > > * In the second patch the virtio-net uses the API to hide the vfio > > device and unhides it when the feature is acked. > > IIUC, the general idea is that we want to provide a pair of associated NIC > devices to the guest, one emulated, one physical PCI device. The guest would > put them in a bonded pair. Before migration the PCI device is unplugged & a > new PCI device plugged on target after migration. The guest traffic continues > without interuption due to the emulate device. > > This kind of conceptual approach can already be implemented today by > management > apps. The only hard problem that exists today is how the guest OS can figure > out that a particular pair of devices it has are intended to be used > together. > > With this series, IIUC, the virtio-net device is getting a given property > which > defines the qdev ID of the associated VFIO device. When the guest OS activates > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > unhides the associated VFIO device. > > AFAICT the guest has to infer that the device which suddenly appears is the > one > associated with the virtio-net device it just initialized, for purposes of > setting up the NIC bonding. There doesn't appear to be any explicit assocation > between the devices exposed to the guest. > > This feels pretty fragile for a guest needing to match up devices when there > are many pairs of devices exposed to a single guest. The impression I get from linux.git:Documentation/networking/net_failover.rst is that the matching is done based on the primary/standby NICs having the same MAC address. In theory you pass both to a guest and based on MAC it essentially does automatic, and if you additionally add STANDBY it'll know to use a virtio-net device specifically for failover. None of this requires any sort of hiding/plugging of devices from QEMU/libvirt (except for the VFIO unplug we'd need to initiate live migration and the VFIO hotplug on the other end to switch back over). That simplifies things greatly, but also introduces the problem of how an older guest will handle seeing 2 NICs with the same MAC, which IIUC is why this series is looking at hotplugging the VFIO device only after we confirm STANDBY is supported by the virtio-net device, and why it's being done transparent to management. > > Unless I'm mis-reading the patches, it looks like the VFIO device always has > to be available at the time QEMU is started. There's no way to boot a guest > and then later hotplug a VFIO device to accelerate the existing virtio-net > NIC. > Or similarly after migration there might not be any VFIO device available > initially when QEMU is started to accept the incoming migration. So it might > need to run in degraded mode for an extended period of time until one becomes > available for hotplugging. The use of qdev IDs makes this troublesome, as the > qdev ID of the future VFIO device would need to be decided upfront before it > even exists. > > So overall I'm not really a fan of the dynamic hiding/unhiding of devices. I > would much prefer to see some way to expose an explicit relationship between > the devices to the guest. If we place the burden of determining whether the guest supports STANDBY on the part of users/management, a lot of this complexity goes away. For instance, one possible implementation is to simply fail migration and say "sorry your VFIO device is still there" if the VFIO device is still around at the start of migration (whether due to unplug failure or a user/management forgetting to do it manually beforehand). So how important is it that setting F_STANDBY cap doesn't break older guests? If the idea is to support live migration with VFs then aren't we still dead in the water if the guest boots okay but doesn't have the requisite functionality to be migrated later? Shouldn't that all be sorted out as early as possible? Is a very clear QEMU error message in this case insufficient? And if backward compatibility is important, are there alternative approaches? Like maybe starting off with a dummy MAC and switching over to the duplicate MAC
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Wed, Dec 05, 2018 at 05:18:18PM +, Daniel P. Berrangé wrote: > On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > > From: Sameeh Jubran > > > > Hi all, > > > > Background: > > > > There has been a few attempts to implement the standby feature for vfio > > assigned devices which aims to enable the migration of such devices. This > > is another attempt. > > > > The series implements an infrastructure for hiding devices from the bus > > upon boot. What it does is the following: > > > > * In the first patch the infrastructure for hiding the device is added > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > state and it is set based on a callback to the standby device which > > registers itself for handling the assessment: "should the primary device > > be hidden?" by cross validating the ids of the devices. > > > > * In the second patch the virtio-net uses the API to hide the vfio > > device and unhides it when the feature is acked. > > IIUC, the general idea is that we want to provide a pair of associated NIC > devices to the guest, one emulated, one physical PCI device. The guest would > put them in a bonded pair. Before migration the PCI device is unplugged & a > new PCI device plugged on target after migration. The guest traffic continues > without interuption due to the emulate device. > > This kind of conceptual approach can already be implemented today by > management > apps. The only hard problem that exists today is how the guest OS can figure > out that a particular pair of devices it has are intended to be used > together. > > With this series, IIUC, the virtio-net device is getting a given property > which > defines the qdev ID of the associated VFIO device. When the guest OS activates > the virtio-net device and acknowledges the STANDBY feature bit, qdev then > unhides the associated VFIO device. > > AFAICT the guest has to infer that the device which suddenly appears is the > one > associated with the virtio-net device it just initialized, for purposes of > setting up the NIC bonding. There doesn't appear to be any explicit assocation > between the devices exposed to the guest. > > This feels pretty fragile for a guest needing to match up devices when there > are many pairs of devices exposed to a single guest. > > Unless I'm mis-reading the patches, it looks like the VFIO device always has > to be available at the time QEMU is started. There's no way to boot a guest > and then later hotplug a VFIO device to accelerate the existing virtio-net > NIC. That should be supported. > Or similarly after migration there might not be any VFIO device available > initially when QEMU is started to accept the incoming migration. So it might > need to run in degraded mode for an extended period of time until one becomes > available for hotplugging. That should work too. > The use of qdev IDs makes this troublesome, as the > qdev ID of the future VFIO device would need to be decided upfront before it > even exists. I agree this sounds problematic. > > So overall I'm not really a fan of the dynamic hiding/unhiding of devices. Dynamic hiding is an orthogonal issue though. It's needed for error handling in case of migration failure: we do not want to close the VFIO device but we do need to hide it from guest. libvirt should not be involved in this aspect though. > I > would much prefer to see some way to expose an explicit relationship between > the devices to the guest. > > > Disclaimers: > > > > * I have only scratch tested this and from qemu side, it seems to be > > working. > > * This is an RFC so it lacks some proper error handling in few cases > > and proper resource freeing. I wanted to get some feedback first > > before it is finalized. > > > > Command line example: > > > > /home/sameeh/Builds/failover/qemu/x86_64-softmmu/qemu-system-x86_64 \ > > -netdev > > tap,id=hostnet0,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_71 > > \ > > -netdev > > tap,vhost=on,id=hostnet1,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_72,queues=4 > > \ > > -device > > virtio-net,host_mtu=1500,netdev=hostnet1,id=cc1_72,vectors=10,mq=on,primary=cc1_71 > > \ > > -device e1000,netdev=hostnet0,id=cc1_71,standby=cc1_72 \ > > > > Migration support: > > > > Pre migration or during setup phase of the migration we should send an > > unplug request to the guest to unplug the primary device. I haven't had > > the chance to implement that part yet but should do soon. Do you know > > what's the best approach to do so? I wanted to have a callback to the > > virtio-net device which tries to send an unplug request to the guest and > > if succeeds then the migration continues. It needs to handle the case where > > the migration fails and then it has to replug the primary device back. > > Having QEMU do this internally gets into a world of pain when you have > multiple devices in the guest. > > Consider if we have 2
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > From: Sameeh Jubran > > Hi all, > > Background: > > There has been a few attempts to implement the standby feature for vfio > assigned devices which aims to enable the migration of such devices. This > is another attempt. > > The series implements an infrastructure for hiding devices from the bus > upon boot. What it does is the following: > > * In the first patch the infrastructure for hiding the device is added > for the qbus and qdev APIs. A "hidden" boolean is added to the device > state and it is set based on a callback to the standby device which > registers itself for handling the assessment: "should the primary device > be hidden?" by cross validating the ids of the devices. > > * In the second patch the virtio-net uses the API to hide the vfio > device and unhides it when the feature is acked. IIUC, the general idea is that we want to provide a pair of associated NIC devices to the guest, one emulated, one physical PCI device. The guest would put them in a bonded pair. Before migration the PCI device is unplugged & a new PCI device plugged on target after migration. The guest traffic continues without interuption due to the emulate device. This kind of conceptual approach can already be implemented today by management apps. The only hard problem that exists today is how the guest OS can figure out that a particular pair of devices it has are intended to be used together. With this series, IIUC, the virtio-net device is getting a given property which defines the qdev ID of the associated VFIO device. When the guest OS activates the virtio-net device and acknowledges the STANDBY feature bit, qdev then unhides the associated VFIO device. AFAICT the guest has to infer that the device which suddenly appears is the one associated with the virtio-net device it just initialized, for purposes of setting up the NIC bonding. There doesn't appear to be any explicit assocation between the devices exposed to the guest. This feels pretty fragile for a guest needing to match up devices when there are many pairs of devices exposed to a single guest. Unless I'm mis-reading the patches, it looks like the VFIO device always has to be available at the time QEMU is started. There's no way to boot a guest and then later hotplug a VFIO device to accelerate the existing virtio-net NIC. Or similarly after migration there might not be any VFIO device available initially when QEMU is started to accept the incoming migration. So it might need to run in degraded mode for an extended period of time until one becomes available for hotplugging. The use of qdev IDs makes this troublesome, as the qdev ID of the future VFIO device would need to be decided upfront before it even exists. So overall I'm not really a fan of the dynamic hiding/unhiding of devices. I would much prefer to see some way to expose an explicit relationship between the devices to the guest. > Disclaimers: > > * I have only scratch tested this and from qemu side, it seems to be > working. > * This is an RFC so it lacks some proper error handling in few cases > and proper resource freeing. I wanted to get some feedback first > before it is finalized. > > Command line example: > > /home/sameeh/Builds/failover/qemu/x86_64-softmmu/qemu-system-x86_64 \ > -netdev > tap,id=hostnet0,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_71 > \ > -netdev > tap,vhost=on,id=hostnet1,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_72,queues=4 > \ > -device > virtio-net,host_mtu=1500,netdev=hostnet1,id=cc1_72,vectors=10,mq=on,primary=cc1_71 > \ > -device e1000,netdev=hostnet0,id=cc1_71,standby=cc1_72 \ > > Migration support: > > Pre migration or during setup phase of the migration we should send an > unplug request to the guest to unplug the primary device. I haven't had > the chance to implement that part yet but should do soon. Do you know > what's the best approach to do so? I wanted to have a callback to the > virtio-net device which tries to send an unplug request to the guest and > if succeeds then the migration continues. It needs to handle the case where > the migration fails and then it has to replug the primary device back. Having QEMU do this internally gets into a world of pain when you have multiple devices in the guest. Consider if we have 2 pairs of devices. We unplug one VFIO device, but unplugging the second VFIO device fails, thus we try to replug the first VFIO device but this now fails too. We don't even get as far as starting the migration before we have to return an error. The mgmt app will just see that the migration failed, but it will not be sure which devices are now actually exposed to the guest OS correctly. The similar problem hits if we started the migration data stream, but then had to abort and so need to tear try to replug in the source but failed for some reasons. Doing the VFIO device plugging/unplugging
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
Quoting Sameeh Jubran (2018-10-25 13:01:10) > On Thu, Oct 25, 2018 at 5:06 PM Sameeh Jubran wrote: > > > > From: Sameeh Jubran > > > > Hi all, > > > > Background: > > > > There has been a few attempts to implement the standby feature for vfio > > assigned devices which aims to enable the migration of such devices. This > > is another attempt. > > > > The series implements an infrastructure for hiding devices from the bus > > upon boot. What it does is the following: > > > > * In the first patch the infrastructure for hiding the device is added > > for the qbus and qdev APIs. A "hidden" boolean is added to the device > > state and it is set based on a callback to the standby device which > > registers itself for handling the assessment: "should the primary device > > be hidden?" by cross validating the ids of the devices. > > > > * In the second patch the virtio-net uses the API to hide the vfio > > device and unhides it when the feature is acked. > > > > Disclaimers: > > > > * I have only scratch tested this and from qemu side, it seems to be > > working. > > * This is an RFC so it lacks some proper error handling in few cases > > and proper resource freeing. I wanted to get some feedback first > > before it is finalized. > > > > Command line example: > > > > /home/sameeh/Builds/failover/qemu/x86_64-softmmu/qemu-system-x86_64 \ > > -netdev > > tap,id=hostnet0,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_71 > > \ > > -netdev > > tap,vhost=on,id=hostnet1,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_72,queues=4 > > \ > > -device > > virtio-net,host_mtu=1500,netdev=hostnet1,id=cc1_72,vectors=10,mq=on,primary=cc1_71 > > \ > > -device e1000,netdev=hostnet0,id=cc1_71,standby=cc1_72 \ > > > > Migration support: > > > > Pre migration or during setup phase of the migration we should send an > > unplug request to the guest to unplug the primary device. I haven't had > > the chance to implement that part yet but should do soon. Do you know > > what's the best approach to do so? I wanted to have a callback to the > > virtio-net device which tries to send an unplug request to the guest and > > if succeeds then the migration continues. It needs to handle the case where > > the migration fails and then it has to replug the primary device back. > I think that the "add_migration_state_change_notifier" API call can be used > from within the virtio-net device to achieve this, what do you think? I think it would be good to hear from the libvirt folks (on Cc:) on this as having QEMU unplug a device without libvirt's involvement seems like it could cause issues. Personally I think it seems cleaner to just have QEMU handle the 'hidden' aspects of the device and leave it to QMP/libvirt to do the unplug beforehand. On the libvirt side I could imagine adding an option like virsh migrate --switch-to-standby-networking or something along that line to do it automatically (if we decide doing it automatically is even needed on that end). > > > > The following terms are used as interchangeable: > > standby - virtio-net > > primary - vfio-device - physical device - assigned device > > > > Please share your thoughts and suggestions, > > Thanks! > > > > Sameeh Jubran (2): > > qdev/qbus: Add hidden device support > > virtio-net: Implement VIRTIO_NET_F_STANDBY feature > > > > hw/core/qdev.c | 48 +--- > > hw/net/virtio-net.c| 25 +++ > > hw/pci/pci.c | 1 + > > include/hw/pci/pci.h | 2 ++ > > include/hw/qdev-core.h | 11 ++- > > include/hw/virtio/virtio-net.h | 5 +++ > > qdev-monitor.c | 58 -- > > 7 files changed, 142 insertions(+), 8 deletions(-) > > > > -- > > 2.17.0 > > > > > -- > Respectfully, > Sameeh Jubran > Linkedin > Software Engineer @ Daynix. >
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Oct 25, 2018 at 05:06:29PM +0300, Sameeh Jubran wrote: > From: Sameeh Jubran > > Hi all, > > Background: > > There has been a few attempts to implement the standby feature for vfio > assigned devices which aims to enable the migration of such devices. This > is another attempt. > > The series implements an infrastructure for hiding devices from the bus > upon boot. What it does is the following: > > * In the first patch the infrastructure for hiding the device is added > for the qbus and qdev APIs. A "hidden" boolean is added to the device > state and it is set based on a callback to the standby device which > registers itself for handling the assessment: "should the primary device > be hidden?" by cross validating the ids of the devices. > > * In the second patch the virtio-net uses the API to hide the vfio > device and unhides it when the feature is acked. > > Disclaimers: > > * I have only scratch tested this and from qemu side, it seems to be > working. > * This is an RFC so it lacks some proper error handling in few cases > and proper resource freeing. I wanted to get some feedback first > before it is finalized. > > Command line example: > > /home/sameeh/Builds/failover/qemu/x86_64-softmmu/qemu-system-x86_64 \ > -netdev > tap,id=hostnet0,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_71 > \ > -netdev > tap,vhost=on,id=hostnet1,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_72,queues=4 > \ > -device > virtio-net,host_mtu=1500,netdev=hostnet1,id=cc1_72,vectors=10,mq=on,primary=cc1_71 > \ > -device e1000,netdev=hostnet0,id=cc1_71,standby=cc1_72 \ > > Migration support: > > Pre migration or during setup phase of the migration we should send an > unplug request to the guest to unplug the primary device. I haven't had > the chance to implement that part yet but should do soon. Do you know > what's the best approach to do so? I wanted to have a callback to the > virtio-net device which tries to send an unplug request to the guest and > if succeeds then the migration continues. It needs to handle the case where > the migration fails and then it has to replug the primary device back. > > The following terms are used as interchangeable: > standby - virtio-net > primary - vfio-device - physical device - assigned device > > Please share your thoughts and suggestions, > Thanks! Didn't have time to look at code yet. Could you test with a VF please? That's the real test, isn't it? > Sameeh Jubran (2): > qdev/qbus: Add hidden device support > virtio-net: Implement VIRTIO_NET_F_STANDBY feature > > hw/core/qdev.c | 48 +--- > hw/net/virtio-net.c| 25 +++ > hw/pci/pci.c | 1 + > include/hw/pci/pci.h | 2 ++ > include/hw/qdev-core.h | 11 ++- > include/hw/virtio/virtio-net.h | 5 +++ > qdev-monitor.c | 58 -- > 7 files changed, 142 insertions(+), 8 deletions(-) > > -- > 2.17.0
Re: [Qemu-devel] [RFC 0/2] Attempt to implement the standby feature for assigned network devices
On Thu, Oct 25, 2018 at 5:06 PM Sameeh Jubran wrote: > > From: Sameeh Jubran > > Hi all, > > Background: > > There has been a few attempts to implement the standby feature for vfio > assigned devices which aims to enable the migration of such devices. This > is another attempt. > > The series implements an infrastructure for hiding devices from the bus > upon boot. What it does is the following: > > * In the first patch the infrastructure for hiding the device is added > for the qbus and qdev APIs. A "hidden" boolean is added to the device > state and it is set based on a callback to the standby device which > registers itself for handling the assessment: "should the primary device > be hidden?" by cross validating the ids of the devices. > > * In the second patch the virtio-net uses the API to hide the vfio > device and unhides it when the feature is acked. > > Disclaimers: > > * I have only scratch tested this and from qemu side, it seems to be > working. > * This is an RFC so it lacks some proper error handling in few cases > and proper resource freeing. I wanted to get some feedback first > before it is finalized. > > Command line example: > > /home/sameeh/Builds/failover/qemu/x86_64-softmmu/qemu-system-x86_64 \ > -netdev > tap,id=hostnet0,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_71 > \ > -netdev > tap,vhost=on,id=hostnet1,script=world_bridge_standalone.sh,downscript=no,ifname=cc1_72,queues=4 > \ > -device > virtio-net,host_mtu=1500,netdev=hostnet1,id=cc1_72,vectors=10,mq=on,primary=cc1_71 > \ > -device e1000,netdev=hostnet0,id=cc1_71,standby=cc1_72 \ > > Migration support: > > Pre migration or during setup phase of the migration we should send an > unplug request to the guest to unplug the primary device. I haven't had > the chance to implement that part yet but should do soon. Do you know > what's the best approach to do so? I wanted to have a callback to the > virtio-net device which tries to send an unplug request to the guest and > if succeeds then the migration continues. It needs to handle the case where > the migration fails and then it has to replug the primary device back. I think that the "add_migration_state_change_notifier" API call can be used from within the virtio-net device to achieve this, what do you think? > > The following terms are used as interchangeable: > standby - virtio-net > primary - vfio-device - physical device - assigned device > > Please share your thoughts and suggestions, > Thanks! > > Sameeh Jubran (2): > qdev/qbus: Add hidden device support > virtio-net: Implement VIRTIO_NET_F_STANDBY feature > > hw/core/qdev.c | 48 +--- > hw/net/virtio-net.c| 25 +++ > hw/pci/pci.c | 1 + > include/hw/pci/pci.h | 2 ++ > include/hw/qdev-core.h | 11 ++- > include/hw/virtio/virtio-net.h | 5 +++ > qdev-monitor.c | 58 -- > 7 files changed, 142 insertions(+), 8 deletions(-) > > -- > 2.17.0 > -- Respectfully, Sameeh Jubran Linkedin Software Engineer @ Daynix.