Re: [libvirt] [PATCH 1/2] Introduce a lock for libxl long-running api

Bamvor Jian Zhang wrote: > This patch introduce a lock for protecting the long-running > api (save, dump, migration and so on) from the other api > which may update the status of the virtual machine. > Hi Bamvor, Thanks for the patches and sorry for the delayed response. I've been traveling quite a bit lately and just got around to reviewing and testing your work. Testing so far looks good. I can save and dump vm's while at the same time list and retrieve info. See my comments inline, but did want to raise a more general comment first. There is a quite a bit of code here borrowed from the qemu driver, which in general is fine since the libxl driver does not need the same locking features as the qemu one. I'd like to hear the opinion of other libvirt maintainers wrt the duplicated code.