[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v18 07/14] add memdev backend infrastructure
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v18 07/14] add memdev backend infrastructure |
Date: |
Wed, 19 Feb 2014 10:15:44 +0100 |
On Wed, 19 Feb 2014 15:53:58 +0800
Hu Tao <address@hidden> wrote:
> Provides framework for splitting host RAM allocation/
> policies into a separate backend that could be used
> by devices.
>
> Initially only legacy RAM backend is provided, which
> uses memory_region_init_ram() allocator and compatible
> with every CLI option that affects memory_region_init_ram().
>
> Signed-off-by: Igor Mammedov <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Hu Tao <address@hidden>
>
> Signed-off-by: Hu Tao <address@hidden>
> ---
this doesn't use complete() mathod of user_creatable
pls, pickup more correct patch:
https://github.com/imammedo/qemu/commit/19a4c596f4fab35706335e6dd01e94413195aa30
> backends/Makefile.objs | 2 +
> backends/hostmem-ram.c | 48 ++++++++++++++++++
> backends/hostmem.c | 125
> +++++++++++++++++++++++++++++++++++++++++++++++
> include/sysemu/hostmem.h | 63 ++++++++++++++++++++++++
> 4 files changed, 238 insertions(+)
> create mode 100644 backends/hostmem-ram.c
> create mode 100644 backends/hostmem.c
> create mode 100644 include/sysemu/hostmem.h
>
> diff --git a/backends/Makefile.objs b/backends/Makefile.objs
> index 42557d5..e6bdc11 100644
> --- a/backends/Makefile.objs
> +++ b/backends/Makefile.objs
> @@ -6,3 +6,5 @@ common-obj-$(CONFIG_BRLAPI) += baum.o
> $(obj)/baum.o: QEMU_CFLAGS += $(SDL_CFLAGS)
>
> common-obj-$(CONFIG_TPM) += tpm.o
> +
> +common-obj-y += hostmem.o hostmem-ram.o
> diff --git a/backends/hostmem-ram.c b/backends/hostmem-ram.c
> new file mode 100644
> index 0000000..a496dbd
> --- /dev/null
> +++ b/backends/hostmem-ram.c
> @@ -0,0 +1,48 @@
> +/*
> + * QEMU Host Memory Backend
> + *
> + * Copyright (C) 2013 Red Hat Inc
> + *
> + * Authors:
> + * Igor Mammedov <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#include "sysemu/hostmem.h"
> +
> +#define TYPE_MEMORY_BACKEND_RAM "memory-ram"
> +
> +
> +static int
> +ram_backend_memory_init(HostMemoryBackend *backend, Error **errp)
> +{
> + if (!memory_region_size(&backend->mr)) {
> + memory_region_init_ram(&backend->mr, OBJECT(backend),
> + object_get_canonical_path(OBJECT(backend)),
> + backend->size);
> + }
> +
> + return 0;
> +}
> +
> +static void
> +ram_backend_class_init(ObjectClass *oc, void *data)
> +{
> + HostMemoryBackendClass *bc = MEMORY_BACKEND_CLASS(oc);
> +
> + bc->memory_init = ram_backend_memory_init;
> +}
> +
> +static const TypeInfo ram_backend_info = {
> + .name = TYPE_MEMORY_BACKEND_RAM,
> + .parent = TYPE_MEMORY_BACKEND,
> + .class_init = ram_backend_class_init,
> +};
> +
> +static void register_types(void)
> +{
> + type_register_static(&ram_backend_info);
> +}
> +
> +type_init(register_types);
> diff --git a/backends/hostmem.c b/backends/hostmem.c
> new file mode 100644
> index 0000000..4b8fd8d
> --- /dev/null
> +++ b/backends/hostmem.c
> @@ -0,0 +1,125 @@
> +/*
> + * QEMU Host Memory Backend
> + *
> + * Copyright (C) 2013 Red Hat Inc
> + *
> + * Authors:
> + * Igor Mammedov <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#include "sysemu/hostmem.h"
> +#include "sysemu/sysemu.h"
> +#include "qapi/visitor.h"
> +#include "qapi/qmp/qerror.h"
> +#include "qemu/config-file.h"
> +#include "qom/object_interfaces.h"
> +
> +static void
> +hostmemory_backend_get_size(Object *obj, Visitor *v, void *opaque,
> + const char *name, Error **errp)
> +{
> + HostMemoryBackend *backend = MEMORY_BACKEND(obj);
> + uint64_t value = backend->size;
> +
> + visit_type_size(v, &value, name, errp);
> +}
> +
> +static void
> +hostmemory_backend_set_size(Object *obj, Visitor *v, void *opaque,
> + const char *name, Error **errp)
> +{
> + HostMemoryBackend *backend = MEMORY_BACKEND(obj);
> + uint64_t value;
> +
> + if (memory_region_size(&backend->mr)) {
> + error_setg(errp, "cannot change property value\n");
> + return;
> + }
> +
> + visit_type_size(v, &value, name, errp);
> + if (error_is_set(errp)) {
> + return;
> + }
> + if (!value) {
> + error_setg(errp, "Property '%s.%s' doesn't take value '%" PRIu64 "'",
> + object_get_typename(obj), name , value);
> + return;
> + }
> + backend->size = value;
> +}
> +
> +static void hostmemory_backend_initfn(Object *obj)
> +{
> + object_property_add(obj, "size", "int",
> + hostmemory_backend_get_size,
> + hostmemory_backend_set_size, NULL, NULL, NULL);
> +}
> +
> +static void hostmemory_backend_finalize(Object *obj)
> +{
> + HostMemoryBackend *backend = MEMORY_BACKEND(obj);
> +
> + if (memory_region_size(&backend->mr)) {
> + memory_region_destroy(&backend->mr);
> + }
> +}
> +
> +static int
> +hostmemory_backend_memory_init(HostMemoryBackend *backend, Error **errp)
> +{
> + error_setg(errp, "memory_init is not implemented for type [%s]",
> + object_get_typename(OBJECT(backend)));
> +
> + return -1;
> +}
> +
> +MemoryRegion *
> +host_memory_backend_get_memory(HostMemoryBackend *backend, Error **errp)
> +{
> + HostMemoryBackendClass *bc = MEMORY_BACKEND_GET_CLASS(backend);
> + Object *obj = OBJECT(backend);
> +
> + if (!backend->size) {
> + error_setg(errp, "Invalid property [%s.size] value: %" PRIu64,
> + object_get_typename(obj), backend->size);
> + return NULL;
> + }
> +
> + if (bc->memory_init(backend, errp) < 0) {
> + return NULL;
> + }
> +
> + return memory_region_size(&backend->mr) ? &backend->mr : NULL;
> +}
> +
> +static void
> +hostmemory_backend_class_init(ObjectClass *oc, void *data)
> +{
> + HostMemoryBackendClass *bc = MEMORY_BACKEND_CLASS(oc);
> +
> + bc->memory_init = hostmemory_backend_memory_init;
> +}
> +
> +static const TypeInfo hostmemory_backend_info = {
> + .name = TYPE_MEMORY_BACKEND,
> + .parent = TYPE_OBJECT,
> + .abstract = true,
> + .class_size = sizeof(HostMemoryBackendClass),
> + .class_init = hostmemory_backend_class_init,
> + .instance_size = sizeof(HostMemoryBackend),
> + .instance_init = hostmemory_backend_initfn,
> + .instance_finalize = hostmemory_backend_finalize,
> + .interfaces = (InterfaceInfo[]) {
> + { TYPE_USER_CREATABLE },
> + { }
> + }
> +};
> +
> +static void register_types(void)
> +{
> + type_register_static(&hostmemory_backend_info);
> +}
> +
> +type_init(register_types);
> diff --git a/include/sysemu/hostmem.h b/include/sysemu/hostmem.h
> new file mode 100644
> index 0000000..5847943
> --- /dev/null
> +++ b/include/sysemu/hostmem.h
> @@ -0,0 +1,63 @@
> +/*
> + * QEMU Host Memory Backend
> + *
> + * Copyright (C) 2013 Red Hat Inc
> + *
> + * Authors:
> + * Igor Mammedov <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL, version 2 or later.
> + * See the COPYING file in the top-level directory.
> + */
> +#ifndef QEMU_RAM_H
> +#define QEMU_RAM_H
> +
> +#include "qom/object.h"
> +#include "qapi/error.h"
> +#include "exec/memory.h"
> +#include "qemu/option.h"
> +
> +#define TYPE_MEMORY_BACKEND "memory"
> +#define MEMORY_BACKEND(obj) \
> + OBJECT_CHECK(HostMemoryBackend, (obj), TYPE_MEMORY_BACKEND)
> +#define MEMORY_BACKEND_GET_CLASS(obj) \
> + OBJECT_GET_CLASS(HostMemoryBackendClass, (obj), TYPE_MEMORY_BACKEND)
> +#define MEMORY_BACKEND_CLASS(klass) \
> + OBJECT_CLASS_CHECK(HostMemoryBackendClass, (klass), TYPE_MEMORY_BACKEND)
> +
> +typedef struct HostMemoryBackend HostMemoryBackend;
> +typedef struct HostMemoryBackendClass HostMemoryBackendClass;
> +
> +/**
> + * HostMemoryBackendClass:
> + * @parent_class: opaque parent class container
> + * @memory_init: hook for derived classes to perform memory allocation
> + */
> +struct HostMemoryBackendClass {
> + ObjectClass parent_class;
> +
> + int (*memory_init)(HostMemoryBackend *backend, Error **errp);
> +};
> +
> +/**
> + * @HostMemoryBackend
> + *
> + * @parent: opaque parent object container
> + * @size: amount of memory backend provides
> + * @id: unique identification string in memdev namespace
> + * @mr: MemoryRegion representing host memory belonging to backend
> + */
> +struct HostMemoryBackend {
> + /* private */
> + Object parent;
> +
> + /* protected */
> + uint64_t size;
> +
> + MemoryRegion mr;
> +};
> +
> +MemoryRegion *host_memory_backend_get_memory(HostMemoryBackend *backend,
> + Error **errp);
> +
> +#endif
- [Qemu-devel] [PATCH v18 00/14] Add support for binding guest numa nodes to host numa nodes, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 01/14] NUMA: move numa related code to new file numa.c, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 11/14] qapi: make string input visitor parse int list, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 03/14] NUMA: Add numa_info structure to contain numa nodes info, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 04/14] NUMA: convert -numa option to use OptsVisitor, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 07/14] add memdev backend infrastructure, Hu Tao, 2014/02/19
- Re: [Qemu-devel] [PATCH v18 07/14] add memdev backend infrastructure,
Igor Mammedov <=
- [Qemu-devel] [PATCH v18 06/14] qapi: add SIZE type parser to string_input_visitor, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 09/14] numa: introduce memory_region_allocate_system_memory, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 10/14] numa: add -numa node, memdev= option, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 08/14] pc: pass QEMUMachineInitArgs to pc_memory_init, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 02/14] NUMA: check if the total numa memory size is equal to ram_size, Hu Tao, 2014/02/19
- [Qemu-devel] [PATCH v18 14/14] amp: add query-memdev, Hu Tao, 2014/02/19