GNOME Bugzilla – Bug 682792
Boxes should do vm removal async
Last modified: 2016-03-31 13:56:56 UTC
VM (storage really) removal is a pretty heavy operation ans should be done async. Here is the backtrace i get when this is slow:
+ Trace 230740
Created attachment 222546 [details] [review] Mark callback function as owned in run_in_thread Without this vala will not automatically ref/unref any data (such as locals blocks) during the runtime of the operation.
Created attachment 222547 [details] [review] Do vm removal async This can be a pretty expensive operation if it deletes files and all the calls are sync, so we do them in a thread.
Attachment 222546 [details] pushed as d3c7a34 - Mark callback function as owned in run_in_thread Attachment 222547 [details] pushed as 6c9995d - Do vm removal async
(In reply to comment #1) > Created an attachment (id=222546) [details] [review] > Mark callback function as owned in run_in_thread > > Without this vala will not automatically ref/unref any data > (such as locals blocks) during the runtime of the operation. that patch causes very annoying leaks, ultimately leading to libvirtd running out of fds: https://bugzilla.gnome.org/show_bug.cgi?id=683600