On Thu, 3 Nov 2016 06:53:33 +0200 "Michael S. Tsirkin" <mst@xxxxxxxxxx> wrote: > On Wed, Nov 02, 2016 at 03:01:33PM +0100, Igor Mammedov wrote: > > On Sat, 29 Oct 2016 00:35:36 +0800 > > Xiao Guangrong <guangrong.xiao@xxxxxxxxxxxxxxx> wrote: > > > > > It is based on my previous patchset, > > > "[PATCH 0/8] nvdimm acpi: bug fix and cleanup", these two patchset are > > > against commit dea651a95af6dad099 (intel-iommu: Check IOAPIC's Trigger Mode > > > against the one in IRTE) on pci branch of Michael's git tree and can be > > > found at: > > > https://github.com/xiaogr/qemu.git nvdimm-hotplug-v3 > > > > > > Changelog in v3: > > > 1) use a dedicated interrupt for nvdimm device hotplug > > > 2) stop nvdimm device hot unplug > > > 3) reserve UUID and handle for QEMU internally used QEMU > > > 5) redesign fit buffer to avoid OSPM reading incomplete fit info > > > 6) bug fixes and cleanups > > > > > > Changelog in v2: > > > Fixed signed integer overflow pointed out by Stefan Hajnoczi > > > > > > This patchset enables nvdimm hotplug support, it is used as pc-dimm hotplug, > > > for example, a new nvdimm device can be plugged as follows: > > > object_add memory-backend-file,id=mem3,size=10G,mem-path=/home/eric/nvdimm3 > > > device_add nvdimm,id=nvdimm3,memdev=mem3 > > > > > > and unplug it as follows: > > > device_del nvdimm3 > > > object_del mem3 > > there is no unplug support > > > > > > instead of incremental fixups on top merged patches in followup series, > > I'd prefer it to make a clean revert for patches 2-4/4 first and > > them amended versions of them to follow. > > Let's get the fixes reviewed first, how to apply them is > a minor issue in my eyes. Incremental fixes would be a pain to review, clean revert + rewritten patches on top should make review much easier as one won't have to compare 3 variants of code. > > > > > > > Xiao Guangrong (4): > > > nvdimm acpi: prebuild nvdimm devices for available slots > > > nvdimm acpi: introduce fit buffer > > > nvdimm acpi: introduce _FIT > > > pc: memhp: enable nvdimm device hotplug > > > > > > docs/specs/acpi_mem_hotplug.txt | 3 + > > > docs/specs/acpi_nvdimm.txt | 58 ++++++- > > > hw/acpi/memory_hotplug.c | 31 +++- > > > hw/acpi/nvdimm.c | 286 +++++++++++++++++++++++++++++++---- > > > hw/core/hotplug.c | 11 ++ > > > hw/core/qdev.c | 20 ++- > > > hw/i386/acpi-build.c | 9 +- > > > hw/i386/pc.c | 31 ++++ > > > hw/mem/nvdimm.c | 4 - > > > include/hw/acpi/acpi_dev_interface.h | 1 + > > > include/hw/hotplug.h | 10 ++ > > > include/hw/mem/nvdimm.h | 27 +++- > > > 12 files changed, 443 insertions(+), 48 deletions(-) > > > -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
