Re: [libvirt PATCH v3 06/13] ch_monitor: Get nicindexes in prep for cgroup mgmt

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/10/21 21:34, Praveen K Paladugu wrote:
> From: Vineeth Pillai <viremana@xxxxxxxxxxxxxxxxxxx>
> 
> Signed-off-by: Vineeth Pillai <viremana@xxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Praveen K Paladugu <prapal@xxxxxxxxxxxxxxxxxxx>
> ---
>  src/ch/ch_conf.h    |   2 +
>  src/ch/ch_domain.c  |  27 ++++++-
>  src/ch/ch_domain.h  |   2 +
>  src/ch/ch_driver.c  |   1 +
>  src/ch/ch_monitor.c | 185 ++++++++++++++++++++++++++++++--------------
>  src/ch/ch_monitor.h |  14 +++-
>  src/ch/ch_process.c |   6 +-
>  src/ch/meson.build  |   1 +
>  8 files changed, 179 insertions(+), 59 deletions(-)
> 

> diff --git a/src/ch/ch_process.c b/src/ch/ch_process.c
> index 9f82e04485..65ca03cfaf 100644
> --- a/src/ch/ch_process.c
> +++ b/src/ch/ch_process.c

> @@ -169,6 +172,7 @@ int virCHProcessStart(virCHDriver *driver,
>          goto cleanup;
>      }
>  
> +    priv->machineName = virCHDomainGetMachineName(vm);

This needs to be coupled with corresponding free call in
virCHProcessStop() and virCHDomainObjPrivateFree() (the latter one I'm
not 100% sure, but it's better to be safe than sorry).

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux