From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([209.51.188.92]:41021) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hBHjj-0006XX-Q0 for qemu-devel@nongnu.org; Tue, 02 Apr 2019 07:37:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hBHji-0006yu-QC for qemu-devel@nongnu.org; Tue, 02 Apr 2019 07:37:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44606) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hBHji-0006yV-Ik for qemu-devel@nongnu.org; Tue, 02 Apr 2019 07:37:50 -0400 From: Markus Armbruster References: <20190401090827.20793-1-armbru@redhat.com> <20190401090827.20793-2-armbru@redhat.com> <20190402125034.1a052992@redhat.com> Date: Tue, 02 Apr 2019 13:37:45 +0200 In-Reply-To: <20190402125034.1a052992@redhat.com> (Igor Mammedov's message of "Tue, 2 Apr 2019 12:50:34 +0200") Message-ID: <87r2akd4dy.fsf@dusky.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH v2 1/5] Revert "vl: Fix to create migration object before block backends again" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: kwolf@redhat.com, quintela@redhat.com, dgilbert@redhat.com, peterx@redhat.com, qemu-devel@nongnu.org, anthony.perard@citrix.com Igor Mammedov writes: > On Mon, 1 Apr 2019 11:08:23 +0200 > Markus Armbruster wrote: > >> This reverts commit e60483f2f8498ae08ae79ca4c6fb03a3317f5e1e. >> >> Recent commit cda4aa9a5a0 moved block backend creation before machine >> property evaluation. > > Side note: > I've also looked at > "[PATCH v3 11/12] pc: Support firmware configuration with -blockdev" > that caused movement and have a similar issue with hostmem backends > where I wanted to set machine.inital-ram-memdev property pointing to > backend. But hostmem backends are created, since they have to be created > after machine_set_properties/configure_accelerator due to dependencies. > > After some playing around with link property on machine and trying > to untangle dependency chain I gave up (doesn't look feasible). > Idea with aliasing like with blockdev won't work either due to > dependencies (also it looks to me to be quite complicated one). > So I've resorted to "usual" machine practice 'machine.set_string_property' > and deal with 'id' referred by it later at machine_run_board_init() time to > instantiate initial RAM, when we already created all backends. That's... horrible. How many of these monstrosities have crept in? Let's discuss once the RC circus has moved to the next town. >> This broke block backends registering migration >> blockers. Commit e60483f2f84 fixed it by moving migration object >> creation before block backend creation. This broke migration with >> Xen. Turns out we need to configure the accelerator before we create >> the migration object so that Xen's accelerator compat properties get >> applied. Revert the flawed commit. This fixes the Xen regression, >> but brings back the block backend regression. The next commits will >> fix it again. >> >> Signed-off-by: Markus Armbruster > > Reviewed-by: Igor Mammedov Thanks!