[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v4 03/10] vl: clean up global property registeration
From: |
Peter Xu |
Subject: |
[Qemu-devel] [PATCH v4 03/10] vl: clean up global property registerations |
Date: |
Wed, 21 Jun 2017 15:52:01 +0800 |
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);
+}
+
+/*
+ * Note: we should see that these compat properties are actually
+ * having a priority: accel < machine < user. This means e.g. when
+ * user specifies something in "-global", it'll always be used with
+ * highest priority.
+ */
+static void register_global_properties(MachineState *ms)
+{
+ accel_register_compat_props(ms->accelerator);
+ machine_register_compat_props(ms);
+ user_register_compat_props();
+}
+
int main(int argc, char **argv, char **envp)
{
int i;
@@ -4571,11 +4590,11 @@ int main(int argc, char **argv, char **envp)
exit (i == 1 ? 1 : 0);
}
- accel_register_compat_props(current_machine->accelerator);
- machine_register_compat_props(current_machine);
-
- qemu_opts_foreach(qemu_find_opts("global"),
- global_init_func, NULL, NULL);
+ /*
+ * Register all the global properties, including accel properties,
+ * machine properties, and user-specified ones.
+ */
+ register_global_properties(current_machine);
/* This checkpoint is required by replay to separate prior clock
reading from the other reads, because timer polling functions query
--
2.7.4
- [Qemu-devel] [PATCH v4 00/10] migration: objectify MigrationState, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 02/10] accel: introduce AccelClass.global_props, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 01/10] machine: export register_compat_prop(), Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 03/10] vl: clean up global property registerations,
Peter Xu <=
- [Qemu-devel] [PATCH v4 04/10] migration: let MigrationState be a qdev, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 05/10] migration: move global_state.optional out, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 06/10] migration: move only_migratable to MigrationState, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 07/10] migration: move skip_configuration out, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 08/10] migration: move skip_section_footers, Peter Xu, 2017/06/21
- [Qemu-devel] [PATCH v4 10/10] migration: hmp: dump globals, Peter Xu, 2017/06/21