[v1] CH: Add support for a configuration file and log level configuration

[PATCH 0/2] CH: Add support for a configuration file and log level configuration

Stefan Kober posted 2 patches 3 months, 2 weeks ago

Diff against v2 v3 v4
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20250522062122.227648-1-stefan.kober@cyberus-technology.de

There is a newer version of this series

src/ch/ch_conf.c    | 32 ++++++++++++++++++++++++++++++++
src/ch/ch_conf.h    | 18 ++++++++++++++++++
src/ch/ch_driver.c  |  6 ++++++
src/ch/ch_monitor.c |  6 ++++++
4 files changed, 62 insertions(+)

Expand all Fold all

[PATCH 0/2] CH: Add support for a configuration file and log level configuration

Posted by Stefan Kober 3 months, 2 weeks ago

Similar to the qemu.conf file that allows QEMU specific configurations we add
the possibility to specify a ch.conf file for Cloud Hypervisor configuration.

The initial single config option is the log level, which sets the verbosity of
the Cloud Hypervisor process. This allows for simpler debugging when using
Cloud Hypervisor via libvirt.

Stefan Kober (2):
  ch: Add config file support
  ch: add log level configuration option

 src/ch/ch_conf.c    | 32 ++++++++++++++++++++++++++++++++
 src/ch/ch_conf.h    | 18 ++++++++++++++++++
 src/ch/ch_driver.c  |  6 ++++++
 src/ch/ch_monitor.c |  6 ++++++
 4 files changed, 62 insertions(+)

-- 
2.49.0

Re: [PATCH 0/2] CH: Add support for a configuration file and log level configuration

Posted by Michal Prívozník via Devel 3 months, 2 weeks ago

On 5/22/25 08:21, Stefan Kober wrote:
> Similar to the qemu.conf file that allows QEMU specific configurations we add
> the possibility to specify a ch.conf file for Cloud Hypervisor configuration.
> 
> The initial single config option is the log level, which sets the verbosity of
> the Cloud Hypervisor process. This allows for simpler debugging when using
> Cloud Hypervisor via libvirt.
> 
> Stefan Kober (2):
>   ch: Add config file support
>   ch: add log level configuration option
> 
>  src/ch/ch_conf.c    | 32 ++++++++++++++++++++++++++++++++
>  src/ch/ch_conf.h    | 18 ++++++++++++++++++
>  src/ch/ch_driver.c  |  6 ++++++
>  src/ch/ch_monitor.c |  6 ++++++
>  4 files changed, 62 insertions(+)
> 

The idea is sound, but I think we need a couple of more files.
Also, this deserves to be mentioned in NEWS.rst ;-)

Michal