From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41118) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YlYKc-0000yd-9F for qemu-devel@nongnu.org; Fri, 24 Apr 2015 03:47:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YlYKb-0000ua-5f for qemu-devel@nongnu.org; Fri, 24 Apr 2015 03:47:26 -0400 Message-ID: <5539F4F6.10507@redhat.com> Date: Fri, 24 Apr 2015 09:47:02 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <20150423101716.GF5289@noname.redhat.com> <5538CA77.4030708@redhat.com> <20150423104045.GG5289@noname.redhat.com> <5538CD0F.1060100@redhat.com> <20150423113631.GH5289@noname.redhat.com> <5538DD52.3020101@redhat.com> <20150423120533.GF2177@work-vm> <5538E174.9020201@redhat.com> <20150423121953.GG2177@work-vm> <5538E459.5030801@redhat.com> <20150424020149.GL2723@ad.nay.redhat.com> <5539A78D.1020206@cn.fujitsu.com> In-Reply-To: <5539A78D.1020206@cn.fujitsu.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH COLO v3 01/14] docs: block replication's description List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Wen Congyang , Fam Zheng Cc: Kevin Wolf , Yang Hongyang , Lai Jiangshan , qemu block , armbru@redhat.com, jcody@redhat.com, Jiang Yunhong , Dong Eddie , "Dr. David Alan Gilbert" , qemu devel , Gonglei , Stefan Hajnoczi , Max Reitz , zhanghailiang On 24/04/2015 04:16, Wen Congyang wrote: >>> >> I think the primary shouldn't do any I/O after failover (and the >>> >> secondary should close the NBD server) so it is probably okay to ignore >>> >> the removal for now. Inserting the filter dynamically is probably >>> >> needed though. >> > >> > Or maybe just enabling/disabling? > Hmm, after failover, the secondary qemu should become primary qemu, but we don't > know the nbd server's IP/port when we execute the secondary qemu. So we need > to inserting nbd client dynamically after failover. True, but secondary->primary switch is already not supported in v3. Kevin/Stefan, is there a design document somewhere that covers at least static filters? Paolo