On Thu, 2022-02-10 at 16:51 +0100, Michal Privoznik wrote:
While reviewing Tim's patches [1] I've found couple of places
in CH
driver that are downright bugs. Fix them.
1:
https://listman.redhat.com/archives/libvir-list/2022-February/msg00226.html
Michal Prívozník (6):
ch_driver: Don't lock driver when getting version
ch_driver: Don't lock driver when looking up domains
chDomainCreateXML: Drop spurious driver unlock
ch_driver: End job properly on failed chDomainCreateXML()
ch_driver: Introduce and use virCHDomainRemoveInactive()
ch_process: Check whether domain is already running before starting
it
src/ch/ch_domain.c | 9 +++++++++
src/ch/ch_domain.h | 4 ++++
src/ch/ch_driver.c | 32 +++++++++++---------------------
src/ch/ch_process.c | 6 ++++++
4 files changed, 30 insertions(+), 21 deletions(-)
Reviewed-by: Tim Wiederhake <twiederh(a)redhat.com>