[PATCH 0/6] ch_driver: Misc cleanups

Tim Wiederhake twiederh at redhat.com
Fri Feb 11 08:32:12 UTC 2022


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 at redhat.com>




More information about the libvir-list mailing list