[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 03/10] vl: clean up global property registera
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH v5 03/10] vl: clean up global property registerations |
Date: |
Mon, 26 Jun 2017 10:40:41 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Jun 23, 2017 at 06:35:29PM -0300, Eduardo Habkost wrote:
> On Fri, Jun 23, 2017 at 12:46:39PM +0800, Peter Xu wrote:
> > It's not that clear on how the global properties are registered to
> > global_props (and also its priority relationship). Let's provide a
> > single function to be called in main() for that, with comment to explain
> > it a bit.
> >
> > Signed-off-by: Peter Xu <address@hidden>
> > ---
> > vl.c | 29 ++++++++++++++++++++++++-----
> > 1 file changed, 24 insertions(+), 5 deletions(-)
> >
> > diff --git a/vl.c b/vl.c
> > index 4452d7a..cdd2ec8 100644
> > --- a/vl.c
> > +++ b/vl.c
> > @@ -2969,6 +2969,25 @@ static int qemu_read_default_config_file(void)
> > return 0;
> > }
> >
> > +static void user_register_compat_props(void)
> > +{
> > + qemu_opts_foreach(qemu_find_opts("global"),
> > + global_init_func, NULL, NULL);
>
> I suggest using "compat props" only when referring to global properties
> registered for internal use (because they help us ensure command-line
> compatibility). User-provided global properties are just "global
> properties", not "compat props".
Agree. Fixing up. Thanks,
--
Peter Xu
[Qemu-devel] [PATCH v5 05/10] migration: move global_state.optional out, Peter Xu, 2017/06/23
[Qemu-devel] [PATCH v5 06/10] migration: move only_migratable to MigrationState, Peter Xu, 2017/06/23
[Qemu-devel] [PATCH v5 07/10] migration: move skip_configuration out, Peter Xu, 2017/06/23
[Qemu-devel] [PATCH v5 08/10] migration: move skip_section_footers, Peter Xu, 2017/06/23