[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 04/22] memory-device: introduce separate conf
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH v3 04/22] memory-device: introduce separate config option |
Date: |
Fri, 21 Sep 2018 14:59:20 +1000 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Thu, Sep 20, 2018 at 12:32:25PM +0200, David Hildenbrand wrote:
> Some architectures might support memory devices, while they don't
> support DIMM/NVDIMM. So let's
> - Rename CONFIG_MEM_HOTPLUG to CONFIG_MEM_DEVICE
> - Introduce CONFIG_DIMM and use it similarly to CONFIG NVDIMM
>
> CONFIG_DIMM and CONFIG_NVDIMM require CONFIG_MEM_DEVICE.
>
> Reviewed-by: Igor Mammedov <address@hidden>
> Signed-off-by: David Hildenbrand <address@hidden>
Reviewed-by: David Gibson <address@hidden>
and ppc part
Acked-by: David Gibson <address@hidden>
> ---
> default-configs/i386-softmmu.mak | 3 ++-
> default-configs/ppc64-softmmu.mak | 3 ++-
> hw/Makefile.objs | 2 +-
> hw/mem/Makefile.objs | 4 ++--
> qapi/misc.json | 2 +-
> 5 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/default-configs/i386-softmmu.mak
> b/default-configs/i386-softmmu.mak
> index 8c7d4a0fa0..4c1637338b 100644
> --- a/default-configs/i386-softmmu.mak
> +++ b/default-configs/i386-softmmu.mak
> @@ -50,7 +50,8 @@ CONFIG_PCI_Q35=y
> CONFIG_APIC=y
> CONFIG_IOAPIC=y
> CONFIG_PVPANIC=y
> -CONFIG_MEM_HOTPLUG=y
> +CONFIG_MEM_DEVICE=y
> +CONFIG_DIMM=y
> CONFIG_NVDIMM=y
> CONFIG_ACPI_NVDIMM=y
> CONFIG_PCIE_PORT=y
> diff --git a/default-configs/ppc64-softmmu.mak
> b/default-configs/ppc64-softmmu.mak
> index b94af6c7c6..f550573782 100644
> --- a/default-configs/ppc64-softmmu.mak
> +++ b/default-configs/ppc64-softmmu.mak
> @@ -16,4 +16,5 @@ CONFIG_VIRTIO_VGA=y
> CONFIG_XICS=$(CONFIG_PSERIES)
> CONFIG_XICS_SPAPR=$(CONFIG_PSERIES)
> CONFIG_XICS_KVM=$(call land,$(CONFIG_PSERIES),$(CONFIG_KVM))
> -CONFIG_MEM_HOTPLUG=y
> +CONFIG_MEM_DEVICE=y
> +CONFIG_DIMM=y
> diff --git a/hw/Makefile.objs b/hw/Makefile.objs
> index a19c1417ed..58872e27e0 100644
> --- a/hw/Makefile.objs
> +++ b/hw/Makefile.objs
> @@ -33,7 +33,7 @@ devices-dirs-$(CONFIG_SOFTMMU) += vfio/
> devices-dirs-$(CONFIG_SOFTMMU) += virtio/
> devices-dirs-$(CONFIG_SOFTMMU) += watchdog/
> devices-dirs-$(CONFIG_SOFTMMU) += xen/
> -devices-dirs-$(CONFIG_MEM_HOTPLUG) += mem/
> +devices-dirs-$(CONFIG_MEM_DEVICE) += mem/
> devices-dirs-$(CONFIG_SOFTMMU) += smbios/
> devices-dirs-y += core/
> common-obj-y += $(devices-dirs-y)
> diff --git a/hw/mem/Makefile.objs b/hw/mem/Makefile.objs
> index 10be4df2a2..3e2f7c5ca2 100644
> --- a/hw/mem/Makefile.objs
> +++ b/hw/mem/Makefile.objs
> @@ -1,3 +1,3 @@
> -common-obj-$(CONFIG_MEM_HOTPLUG) += pc-dimm.o
> -common-obj-$(CONFIG_MEM_HOTPLUG) += memory-device.o
> +common-obj-$(CONFIG_DIMM) += pc-dimm.o
> +common-obj-$(CONFIG_MEM_DEVICE) += memory-device.o
> common-obj-$(CONFIG_NVDIMM) += nvdimm.o
> diff --git a/qapi/misc.json b/qapi/misc.json
> index d450cfef21..7c36de0464 100644
> --- a/qapi/misc.json
> +++ b/qapi/misc.json
> @@ -2061,7 +2061,7 @@
> #
> # @plugged-memory: size of memory that can be hot-unplugged. This field
> # is omitted if target doesn't support memory hotplug
> -# (i.e. CONFIG_MEM_HOTPLUG not defined on build time).
> +# (i.e. CONFIG_MEM_DEVICE not defined at build time).
> #
> # Since: 2.11.0
> ##
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH v3 00/22] memory-device: complete refactoring + virtio-pmem, David Hildenbrand, 2018/09/20
- [Qemu-devel] [PATCH v3 01/22] memory-device: fix error message when hinted address is too small, David Hildenbrand, 2018/09/20
- [Qemu-devel] [PATCH v3 02/22] memory-device: handle integer overflows properly, David Hildenbrand, 2018/09/20
- [Qemu-devel] [PATCH v3 03/22] memory-device: use memory device terminology in error messages, David Hildenbrand, 2018/09/20
- [Qemu-devel] [PATCH v3 04/22] memory-device: introduce separate config option, David Hildenbrand, 2018/09/20
- Re: [Qemu-devel] [PATCH v3 04/22] memory-device: introduce separate config option,
David Gibson <=
- [Qemu-devel] [PATCH v3 05/22] memory-device: forward errors in get_region_size()/get_plugged_size(), David Hildenbrand, 2018/09/20
- [Qemu-devel] [PATCH v3 06/22] memory-device: document MemoryDeviceClass, David Hildenbrand, 2018/09/20