[libvirt] [PATCH 5/6] qemu: Reduce memlock limit after detaching hostdev

Andrea Bolognani abologna at redhat.com
Thu Dec 10 18:34:30 UTC 2015


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




More information about the libvir-list mailing list