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@xxxxxxxxxx>
