From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [Qemu-devel] [PATCH v3 0/4] nvdimm: hotplug support Date: Thu, 3 Nov 2016 06:53:33 +0200 Message-ID: <20161103065256-mutt-send-email-mst@kernel.org> References: <1477672540-27952-1-git-send-email-guangrong.xiao@linux.intel.com> <20161102150133.043b4923@nial.brq.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Xiao Guangrong , pbonzini@redhat.com, ehabkost@redhat.com, kvm@vger.kernel.org, gleb@kernel.org, mtosatti@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com, dan.j.williams@intel.com, rth@twiddle.net To: Igor Mammedov Return-path: Received: from mx1.redhat.com ([209.132.183.28]:41448 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753184AbcKCExg (ORCPT ); Thu, 3 Nov 2016 00:53:36 -0400 Content-Disposition: inline In-Reply-To: <20161102150133.043b4923@nial.brq.redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Wed, Nov 02, 2016 at 03:01:33PM +0100, Igor Mammedov wrote: > On Sat, 29 Oct 2016 00:35:36 +0800 > Xiao Guangrong 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. > > > > 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(-) > >