From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58101) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2EIr-0003DI-Ib for qemu-devel@nongnu.org; Thu, 03 Nov 2016 05:27:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c2EIo-0001Nn-DT for qemu-devel@nongnu.org; Thu, 03 Nov 2016 05:27:21 -0400 Received: from mx1.redhat.com ([209.132.183.28]:35690) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c2EIo-0001NA-6Q for qemu-devel@nongnu.org; Thu, 03 Nov 2016 05:27:18 -0400 Date: Thu, 3 Nov 2016 10:27:13 +0100 From: Igor Mammedov Message-ID: <20161103102713.6268b9c5@nial.brq.redhat.com> In-Reply-To: <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> <20161103065256-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/4] nvdimm: hotplug support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" 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 On Thu, 3 Nov 2016 06:53:33 +0200 "Michael S. Tsirkin" 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 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(-) > > >