On Fri, Jul 7, 2023 at 8:41 PM Alexandru Elisei
<alexandru.elisei@xxxxxxx> wrote:
>
> kvmtool can be unnecessarily verbose at times, and Will proposed in a chat
> we had a while ago to add a --loglevel command line argument to choose
> which type of messages to silence. This is me taking a stab at it.
>
> Build tested for all arches and run tested lightly on a rockpro64 and my
> x86 machine.
>
> Base commit is 3b1cdcf9e78f ("virtio/vhost: Clear VIRTIO_F_ACCESS_PLATFORM").
>
> Changelog in each patch.
>
> Alexandru Elisei (4):
> util: Make pr_err() return void
> Replace printf/fprintf with pr_* macros
> util: Use __pr_debug() instead of pr_info() to print debug messages
> Add --loglevel argument for the run command
Looks good for KVMTOOL RISC-V.
Reviewed-by: Anup Patel <anup@xxxxxxxxxxxxxx>
Regards,
Anup
>
> arm/gic.c | 5 ++--
> builtin-run.c | 69 ++++++++++++++++++++++++++++++--------------
> builtin-setup.c | 16 +++++-----
> guest_compat.c | 2 +-
> include/kvm/util.h | 14 ++++++---
> kvm-cpu.c | 12 ++++----
> mmio.c | 2 +-
> util/parse-options.c | 28 ++++++++++--------
> util/util.c | 27 +++++++++++++++--
> 9 files changed, 116 insertions(+), 59 deletions(-)
>
> --
> 2.41.0
>
