[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] vl: Print warning when a default config fil
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] vl: Print warning when a default config file is loaded |
Date: |
Wed, 18 Jan 2017 17:04:45 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Eduardo Habkost <address@hidden> writes:
> In case there were options set in the default config file, print
> a warning so users can update their scripts.
Can you explain why you don't warn on an empty qemu.conf?
> If somebody wants to keep the config file as-is, avoid the
> warning and use a command-line that will work in future QEMU
> versions, they can use:
>
> $QEMU -nodefconfig -readconfig /etc/qemu/qemu.conf
>
> I was going to add an additional message suggesting it as a
> solution, but I thought it could make it more confusing. The
> solution can be documented in the QEMU 2.9 ChangeLog.
>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> vl.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/vl.c b/vl.c
> index b563f9b924..e80b6da4bd 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2999,6 +2999,12 @@ static int qemu_read_default_config_file(void)
> return ret;
> }
>
> + if (ret > 0) {
> + loc_set_none();
> + error_report("Warning: Future QEMU versions won't load %s
> automatically",
> + CONFIG_QEMU_CONFDIR "/qemu.conf");
> + }
> +
> return 0;
> }