Re: [Qemu-devel] [PATCH 3/4] hostmem: Register TYPE_MEMORY_BACKEND properties as class properties

[Igor Mammedov]

On Thu, 13 Oct 2016 18:26:41 -0300
Eduardo Habkost  wrote:

> The NULL errp arguments on the property registration calls were
> changed to _abort.
> 
> Signed-off-by: Eduardo Habkost 
Reviewed-by: Igor Mammedov 

> ---
>  backends/hostmem.c | 42 ++
>  1 file changed, 22 insertions(+), 20 deletions(-)
> 
> diff --git a/backends/hostmem.c b/backends/hostmem.c
> index b7a208d..4256d24 100644
> --- a/backends/hostmem.c
> +++ b/backends/hostmem.c
> @@ -241,26 +241,6 @@ static void host_memory_backend_init(Object *obj)
>  backend->merge = machine_mem_merge(machine);
>  backend->dump = machine_dump_guest_core(machine);
>  backend->prealloc = mem_prealloc;
> -
> -object_property_add_bool(obj, "merge",
> -host_memory_backend_get_merge,
> -host_memory_backend_set_merge, NULL);
> -object_property_add_bool(obj, "dump",
> -host_memory_backend_get_dump,
> -host_memory_backend_set_dump, NULL);
> -object_property_add_bool(obj, "prealloc",
> -host_memory_backend_get_prealloc,
> -host_memory_backend_set_prealloc, NULL);
> -object_property_add(obj, "size", "int",
> -host_memory_backend_get_size,
> -host_memory_backend_set_size, NULL, NULL, NULL);
> -object_property_add(obj, "host-nodes", "int",
> -host_memory_backend_get_host_nodes,
> -host_memory_backend_set_host_nodes, NULL, NULL, 
> NULL);
> -object_property_add_enum(obj, "policy", "HostMemPolicy",
> - HostMemPolicy_lookup,
> - host_memory_backend_get_policy,
> - host_memory_backend_set_policy, NULL);
>  }
>  
>  MemoryRegion *
> @@ -375,6 +355,28 @@ host_memory_backend_class_init(ObjectClass *oc, void 
> *data)
>  
>  ucc->complete = host_memory_backend_memory_complete;
>  ucc->can_be_deleted = host_memory_backend_can_be_deleted;
> +
> +object_class_property_add_bool(oc, "merge",
> +host_memory_backend_get_merge,
> +host_memory_backend_set_merge, _abort);
> +object_class_property_add_bool(oc, "dump",
> +host_memory_backend_get_dump,
> +host_memory_backend_set_dump, _abort);
> +object_class_property_add_bool(oc, "prealloc",
> +host_memory_backend_get_prealloc,
> +host_memory_backend_set_prealloc, _abort);
> +object_class_property_add(oc, "size", "int",
> +host_memory_backend_get_size,
> +host_memory_backend_set_size,
> +NULL, NULL, _abort);
> +object_class_property_add(oc, "host-nodes", "int",
> +host_memory_backend_get_host_nodes,
> +host_memory_backend_set_host_nodes,
> +NULL, NULL, _abort);
> +object_class_property_add_enum(oc, "policy", "HostMemPolicy",
> +HostMemPolicy_lookup,
> +host_memory_backend_get_policy,
> +host_memory_backend_set_policy, _abort);
>  }
>  
>  static const TypeInfo host_memory_backend_info = {

[Qemu-devel] [PATCH 3/4] hostmem: Register TYPE_MEMORY_BACKEND properties as class properties

[Eduardo Habkost]

The NULL errp arguments on the property registration calls were
changed to _abort.

Signed-off-by: Eduardo Habkost 
---
 backends/hostmem.c | 42 ++
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/backends/hostmem.c b/backends/hostmem.c
index b7a208d..4256d24 100644
--- a/backends/hostmem.c
+++ b/backends/hostmem.c
@@ -241,26 +241,6 @@ static void host_memory_backend_init(Object *obj)
 backend->merge = machine_mem_merge(machine);
 backend->dump = machine_dump_guest_core(machine);
 backend->prealloc = mem_prealloc;
-
-object_property_add_bool(obj, "merge",
-host_memory_backend_get_merge,
-host_memory_backend_set_merge, NULL);
-object_property_add_bool(obj, "dump",
-host_memory_backend_get_dump,
-host_memory_backend_set_dump, NULL);
-object_property_add_bool(obj, "prealloc",
-host_memory_backend_get_prealloc,
-host_memory_backend_set_prealloc, NULL);
-object_property_add(obj, "size", "int",
-host_memory_backend_get_size,
-host_memory_backend_set_size, NULL, NULL, NULL);
-object_property_add(obj, "host-nodes", "int",
-host_memory_backend_get_host_nodes,
-host_memory_backend_set_host_nodes, NULL, NULL, NULL);
-object_property_add_enum(obj, "policy", "HostMemPolicy",
- HostMemPolicy_lookup,
- host_memory_backend_get_policy,
- host_memory_backend_set_policy, NULL);
 }
 
 MemoryRegion *
@@ -375,6 +355,28 @@ host_memory_backend_class_init(ObjectClass *oc, void *data)
 
 ucc->complete = host_memory_backend_memory_complete;
 ucc->can_be_deleted = host_memory_backend_can_be_deleted;
+
+object_class_property_add_bool(oc, "merge",
+host_memory_backend_get_merge,
+host_memory_backend_set_merge, _abort);
+object_class_property_add_bool(oc, "dump",
+host_memory_backend_get_dump,
+host_memory_backend_set_dump, _abort);
+object_class_property_add_bool(oc, "prealloc",
+host_memory_backend_get_prealloc,
+host_memory_backend_set_prealloc, _abort);
+object_class_property_add(oc, "size", "int",
+host_memory_backend_get_size,
+host_memory_backend_set_size,
+NULL, NULL, _abort);
+object_class_property_add(oc, "host-nodes", "int",
+host_memory_backend_get_host_nodes,
+host_memory_backend_set_host_nodes,
+NULL, NULL, _abort);
+object_class_property_add_enum(oc, "policy", "HostMemPolicy",
+HostMemPolicy_lookup,
+host_memory_backend_get_policy,
+host_memory_backend_set_policy, _abort);
 }
 
 static const TypeInfo host_memory_backend_info = {
-- 
2.7.4