From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41387) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c3jbi-0000sn-9M for qemu-devel@nongnu.org; Mon, 07 Nov 2016 08:05:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c3jbe-0003hP-BD for qemu-devel@nongnu.org; Mon, 07 Nov 2016 08:05:02 -0500 Received: from mail-wm0-x244.google.com ([2a00:1450:400c:c09::244]:36363) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1c3jbe-0003gF-39 for qemu-devel@nongnu.org; Mon, 07 Nov 2016 08:04:58 -0500 Received: by mail-wm0-x244.google.com with SMTP id c17so16335039wmc.3 for ; Mon, 07 Nov 2016 05:04:57 -0800 (PST) Date: Mon, 7 Nov 2016 13:04:54 +0000 From: Stefan Hajnoczi Message-ID: <20161107130454.GI5036@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> <20161104105143.27dd2a70@nial.brq.redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="FUFe+yI/t+r3nyH4" Content-Disposition: inline In-Reply-To: <20161104105143.27dd2a70@nial.brq.redhat.com> Subject: Re: [Qemu-devel] [PATCH v5 0/5] nvdimm: hotplug support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: Stefan Hajnoczi , Xiao Guangrong , ehabkost@redhat.com, kvm@vger.kernel.org, "Michael S. Tsirkin" , gleb@kernel.org, mtosatti@redhat.com, qemu-devel@nongnu.org, pbonzini@redhat.com, dan.j.williams@intel.com, rth@twiddle.net --FUFe+yI/t+r3nyH4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Fri, Nov 04, 2016 at 10:51:43AM +0100, Igor Mammedov wrote: > 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: = =20 > > > > On Fri, Nov 04, 2016 at 11:50:19AM +0800, Xiao Guangrong wrote: =20 > > > > > On 11/04/2016 02:36 AM, Xiao Guangrong wrote: =20 > > > > > > Hi Michael, > > > > > >=20 > > > > > > This patchset can replace the patches from [PULL 36/47] to [PUL= L 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 > > > > > >=20 > > > > > > Thanks for your patience also thank Igor and Stefan for their r= eview. =20 > > > > >=20 > > > > > Hi, > > > > >=20 > > > > > As the pull request has been upstream (Cool! :)), i will post > > > > > diff changes based on that. > > > > >=20 > > > > > Thanks! =20 > > > >=20 > > > > 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. =20 > > >=20 > > > Stefan, I wonder what's easier for you to review? =20 > >=20 > > 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. >=20 > 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. >=20 > But since majority prefers incremental followup patches lets do > it this way. For the record I would have preferred to get the patch series right before merging it. I guess that Michael favors development velocity and there are arguments why merging code before it's perfect can help it mature more quickly. If we merge things before they are ready then it's important that incomplete features are marked experimental/unstable. But in this case I guess the strategy will be to merge the fixes during soft-freeze (before Nov 15th) so that QEMU 2.8 ships the finalized code. Stefan --FUFe+yI/t+r3nyH4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEcBAEBAgAGBQJYIHv2AAoJEJykq7OBq3PIs64H/iUXB0RGv/drzw7V3VHIX5dQ glceI/Q1Om77wAxI5NElUI86zsDHCqYIkWbnY2OM6re/TyhcaBCNHtAXGgbJJ+Dt VCCEU2aGw32AnV1KRhyQx3tGRJUprvUhvWQ7mtV//hj+MHaoRGnDmD6WM/q9ebds liWgk40VWFEOSK8uwSVmzKJhZiPP+xABzAMBsckAJlZy+szFXWt/WN3WIoZ/5mUL nnvT6HnVB9ekc+Xv6PG7a5SueLCkNn5BHrWinrE/irxGa5vYIVbE6thn+ZKFGCJR ki0fes4fo14l02yxcGgLJxRvLld+HadozEsx4zFNRiCdnAD9kO7Nx0C+79hnMTk= =HTRw -----END PGP SIGNATURE----- --FUFe+yI/t+r3nyH4--