From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34867) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ecbHd-00061P-6M for qemu-devel@nongnu.org; Fri, 19 Jan 2018 13:20:58 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ecbHc-0007Du-ET for qemu-devel@nongnu.org; Fri, 19 Jan 2018 13:20:57 -0500 References: <20171220154945.88410-1-vsementsov@virtuozzo.com> <20171220154945.88410-14-vsementsov@virtuozzo.com> <20171228063330.GL9192@lemon> <17c25946-8cce-21e8-d70f-856fb1924b8c@redhat.com> <753ed688-ea9f-4070-b98f-ddbd35f49a32@virtuozzo.com> From: John Snow Message-ID: <45b5c74c-a863-362d-93c6-4573e7c65b25@redhat.com> Date: Fri, 19 Jan 2018 13:20:36 -0500 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v9 13/13] iotests: add dirty bitmap postcopy test List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Vladimir Sementsov-Ogievskiy , Fam Zheng Cc: qemu-block@nongnu.org, qemu-devel@nongnu.org, kwolf@redhat.com, peter.maydell@linaro.org, lirans@il.ibm.com, quintela@redhat.com, armbru@redhat.com, mreitz@redhat.com, stefanha@redhat.com, den@openvz.org, amit.shah@redhat.com, pbonzini@redhat.com, dgilbert@redhat.com On 01/19/2018 01:08 PM, Vladimir Sementsov-Ogievskiy wrote: > 18.01.2018 12:57, Vladimir Sementsov-Ogievskiy wrote: >> 17.01.2018 21:30, John Snow wrote: >>> >>> On 12/28/2017 06:49 AM, Vladimir Sementsov-Ogievskiy wrote: >>>> Thank you for reviewing my code! >>>> >>> Time for the re-spin? There's pretty good pressure to get this into 2= .12 >>> and say the non-nbd workflow model is feature complete. >> >> Yes, you are right. Hope to do it today or tomorrow. >> >=20 > I've rebased, applied comments from review, and even one some new fixes= , > but now I understand that it is too early for re-spin. >=20 > Now this series depends on > 1. [PATCH v2 0/3] fix bitmaps migration through shared storage > =C2=A0 - we need to decide, how to fix it. May be, we just do not need > bitmaps in inactive state, and should not load them in do_qcow2_open in > this case > =C2=A0 [I can ignore it, just dropping one test case from new iotest, a= nd fix > it later, but.. > 2. [PATCH v2 0/6] qmp dirty bitmap API > =C2=A0- here autoload is dropped, and migration should be rebased on it= . >=20 > so, I'll re-spin migration after these two questions are resolved. >=20 You got it, thanks.