[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: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] vl: Print warning when a default config file is loaded |
Date: |
Wed, 18 Jan 2017 14:24:09 -0200 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Wed, Jan 18, 2017 at 05:04:45PM +0100, Markus Armbruster wrote:
> 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?
I didnt't want to show the warning to users that won't be
affected by the change. I assume that users with empty qemu.conf
files won't care when we stop loading it.
>
> > 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;
> > }
--
Eduardo