From 4e1f37f8120105053ca91f1a8127a75c79061db7 Mon Sep 17 00:00:00 2001 From: Andrea Bolognani Date: Thu, 7 Jan 2016 17:50:15 +0100 Subject: [PATCH] pci: Log debug messages when manipulating the inactive list Most of the changes to the list of active and inactive PCI devices happen in virHostdev, where they are properly logged. virPCIDeviceDetach() and virPCIDeviceReattach(), however, change the inactive list as well, so they should be logging similar messages. --- src/util/virpci.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/util/virpci.c b/src/util/virpci.c index 5ebd408700..505c1f33a8 100644 --- a/src/util/virpci.c +++ b/src/util/virpci.c @@ -1370,9 +1370,10 @@ virPCIDeviceDetach(virPCIDevicePtr dev, /* Add *a copy of* the dev into list inactiveDevs, if * it's not already there. */ - if (inactiveDevs && !virPCIDeviceListFind(inactiveDevs, dev) && - virPCIDeviceListAddCopy(inactiveDevs, dev) < 0) { - return -1; + if (inactiveDevs && !virPCIDeviceListFind(inactiveDevs, dev)) { + VIR_DEBUG("Adding PCI device %s to inactive list", dev->name); + if (virPCIDeviceListAddCopy(inactiveDevs, dev) < 0) + return -1; } return 0; @@ -1393,8 +1394,10 @@ virPCIDeviceReattach(virPCIDevicePtr dev, return -1; /* Steal the dev from list inactiveDevs */ - if (inactiveDevs) + if (inactiveDevs) { + VIR_DEBUG("Removing PCI device %s from inactive list", dev->name); virPCIDeviceListDel(inactiveDevs, dev); + } return 0; } -- 2.47.2