Re: [libvirt] [v7 00/10] Support cache tune in libvirt
On Tue, Feb 21, 2017 at 10:25:13AM +0800, Eli Qiao wrote:
--
Best regards
Eli
天涯无处不重逢
a leaf duckweed belongs to the sea, where not to meet in life
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
On Tuesday, 21 February 2017 at 4:54 AM, Marcelo Tosatti wrote:
> Can you explain how the resource leak can happen?
>
1. libvirtd creates a qemu process and create a resctrl directory for it, add it’s pids
to tasks
2. stop libvirtd or it’s down.
3. qemu process, and tasks file is empty now.
4. libvirt restart, well, the resctrl directory should be deleted as qemu process gone.
So there are two problems:
P1) If libvirtd is not restarted and guest VM is poweredoff, then
CAT reservation leaks.
Ideally QEMU should free the resources.
How does libvirt deal with this for other resources?
P2) Libvirt restart.
Please remove resctrlfs directory only for
libvirt owned VMs (say if the directory name matches
the VM UUID), and not for all directories in resctrlfs
(should not change resctrlfs directories which libvirt
does not own).