
10 Dec
2015
10 Dec
'15
1:34 p.m.
On Thu, 2015-12-10 at 12:44 -0500, John Ferlan wrote:
Calling qemuDomainAdjustMaxMemLock() more than necessary shouldn't really cause any harm, but adding a check for
hostdev->source.subsys.type == VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI
around the call is even better.
Would that address your concerns?
You mean of course before the pesky "virDomainHostdevDefFree(hostdev)"?
You could perhaps do it after the qemuDomainRemovePCIHostDevice in the switch or within that qemuDomainRemovePCIHostDevice code.
I moved it into the switch. Tomorrow I'll do a round of tests and send out a v2. Cheers. -- Andrea Bolognani Software Engineer - Virtualization Team