On Fri, 2016-04-01 at 10:59 -0400, Laine Stump wrote: > On 04/01/2016 09:38 AM, Andrea Bolognani wrote: > > > > This reverts commit ee4cfb56436b50345b072c706b87aff82e06d760. > > > > Since we're still not persisting our bookkeeping lists across > > daemon restarts, we might have lost some information > > virPCIDeviceReattach() relies on, for example whether the > > device needs to be unbound from the stub driver. > > > > As a result, if the daemon has been restarted in the meantime, > > the device might end up remaining bound to the stub driver even > > after 'virsh nodedev-reattach' or similar has been called, with > > no way of giving it back to the host short of messing with > > sysfs behind libvirt's back. > > > > Revert back to the previous behavior of always trying to bind > > the device to the host driver, regardless of its status when it > > was detached, until persistent bookkeeping lists have been > > implemented. > > --- > > src/util/virhostdev.c | 10 ++++------ > > 1 file changed, 4 insertions(+), 6 deletions(-) > > I haven't been able to test yet if this has any effect on the problem I > encountered, but it definitely does fix a known regression that we > should get in before the release - ACK. Pushed, thanks :) -- Andrea Bolognani Software Engineer - Virtualization Team -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list