From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43580) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c2bA9-0003ai-2N for qemu-devel@nongnu.org; Fri, 04 Nov 2016 05:51:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c2bA4-0004TQ-3X for qemu-devel@nongnu.org; Fri, 04 Nov 2016 05:51:53 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51322) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c2bA3-0004Sm-TS for qemu-devel@nongnu.org; Fri, 04 Nov 2016 05:51:48 -0400 Date: Fri, 4 Nov 2016 10:51:43 +0100 From: Igor Mammedov Message-ID: <20161104105143.27dd2a70@nial.brq.redhat.com> In-Reply-To: <20161104091548.GA3403@stefanha-x1.localdomain> References: <1478198186-45204-1-git-send-email-guangrong.xiao@linux.intel.com> <20161104060029-mutt-send-email-mst@kernel.org> <20161104062143-mutt-send-email-mst@kernel.org> <20161104091548.GA3403@stefanha-x1.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v5 0/5] nvdimm: hotplug support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: "Michael S. Tsirkin" , Xiao Guangrong , pbonzini@redhat.com, gleb@kernel.org, mtosatti@redhat.com, rth@twiddle.net, ehabkost@redhat.com, dan.j.williams@intel.com, kvm@vger.kernel.org, qemu-devel@nongnu.org On Fri, 4 Nov 2016 09:15:48 +0000 Stefan Hajnoczi wrote: > On Fri, Nov 04, 2016 at 06:22:26AM +0200, Michael S. Tsirkin wrote: > > On Fri, Nov 04, 2016 at 06:01:54AM +0200, Michael S. Tsirkin wrote: > > > On Fri, Nov 04, 2016 at 11:50:19AM +0800, Xiao Guangrong wrote: > > > > On 11/04/2016 02:36 AM, Xiao Guangrong wrote: > > > > > Hi Michael, > > > > > > > > > > This patchset can replace the patches from [PULL 36/47] to [PULL 39/47] > > > > > in your pull request: > > > > > [PULL 36/47] nvdimm acpi: prebuild nvdimm devices for available slots > > > > > [PULL 37/47] nvdimm acpi: introduce fit buffer > > > > > [PULL 38/47] nvdimm acpi: introduce _FIT > > > > > [PULL 39/47] pc: memhp: enable nvdimm device hotplug > > > > > > > > > > Thanks for your patience also thank Igor and Stefan for their review. > > > > > > > > Hi, > > > > > > > > As the pull request has been upstream (Cool! :)), i will post > > > > diff changes based on that. > > > > > > > > Thanks! > > > > > > Igor prefers seeing revert+patches, I prefer seeing a diff. > > > Can you send both? A global diff would be ok for me > > > as it's small and easy enough to generate. > > > > Stefan, I wonder what's easier for you to review? > > Since it has been merged into qemu.git/master I'd now like to see > follow-up patches. Not a global diff but real individual changes on top > of qemu.git/master. These fixes can be merged during softfreeze. Incremental followup patches will make review a bit harder as they should remove code that's shouldn't have been there is the first place and would be fixing existing mess. But since majority prefers incremental followup patches lets do it this way. > > Stefan