[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/27] dimm: implement dimm device abstraction
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 08/27] dimm: implement dimm device abstraction |
Date: |
Mon, 25 Nov 2013 13:57:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130923 Thunderbird/17.0.9 |
Il 21/11/2013 03:38, Igor Mammedov ha scritto:
> +static void dimm_initfn(Object *obj)
> +{
> + object_property_add(obj, "memdev", "string", dimm_get_memdev,
> + dimm_set_memdev, NULL, NULL, NULL);
Perhaps, instead of relying on memory_region_owner, you could have
separate pointers for the MemoryBackend* and MemoryRegion*. Then this
property could be a link<MemoryBackend> type (it looks like a link, and
quacks like a link).
Paolo
> + object_property_add(obj, "size", "int", dimm_get_size,
> + NULL, NULL, NULL, NULL);
> +}
> +
- Re: [Qemu-devel] [PATCH 04/27] vl: convert -m to qemu_opts_parse(), (continued)
- [Qemu-devel] [PATCH 02/27] rename pci_hotplug_fn to hotplug_fn and make it available for other devices, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 05/27] qapi: add SIZE type parser to string_input_visitor, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 06/27] get reference to /backend container via qemu_get_backend(), Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 07/27] add memdev backend infrastructure, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 08/27] dimm: implement dimm device abstraction, Igor Mammedov, 2013/11/20
- Re: [Qemu-devel] [PATCH 08/27] dimm: implement dimm device abstraction,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 09/27] dimm: map DimmDevice into DimBus provided address space, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 10/27] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 11/27] dimm: add busy address check and address auto-allocation, Igor Mammedov, 2013/11/20
- [Qemu-devel] [PATCH 13/27] acpi: memory hotplug ACPI hardware implementation, Igor Mammedov, 2013/11/20