From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40812) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c1T83-0002XP-2l for qemu-devel@nongnu.org; Tue, 01 Nov 2016 03:05:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c1T7z-0006Np-34 for qemu-devel@nongnu.org; Tue, 01 Nov 2016 03:05:03 -0400 Received: from [59.151.112.132] (port=37700 helo=heian.cn.fujitsu.com) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c1T7y-0006Mf-6K for qemu-devel@nongnu.org; Tue, 01 Nov 2016 03:04:59 -0400 References: <1477971492-10766-1-git-send-email-zhangchen.fnst@cn.fujitsu.com> <58183554.8070101@huawei.com> From: Zhang Chen Message-ID: Date: Tue, 1 Nov 2016 15:06:22 +0800 MIME-Version: 1.0 In-Reply-To: <58183554.8070101@huawei.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] docs: fix COLO architecture diagram List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Hailiang Zhang , qemu devel , Amit Shah Cc: Jason Wang , Li Zhijian , "eddie . dong" On 11/01/2016 02:25 PM, Hailiang Zhang wrote: > Hmm, there are other contents in this file need to be updated, > for example, we support blockdev-add command for nbd now, > so we can convert the related hmp command to qmp command way. > But since we didn't integrate COLO frame with block replication > and proxy, It is OK to fix them later. > > For COLO, the basic capability is still incomplete, > but now we got into 'Soft feature freeze' stage, > I'm wondering if it is possible at this point to combine > COLO frame with proxy and block replication which only needs > three or four patches that only touch colo related files ... > Or uses still can't test COLO feature in qemu 2.8. > > Any ideas ? Thanks. I will send some patch about COLO-Proxy combine with COLO-frame in the future. > > On 2016/11/1 11:38, Zhang Chen wrote: >> Fix COLO-Proxy part of COLO architecture diagram >> >> Signed-off-by: Zhang Chen > > Reviewed-by: zhanghailiang > > All such patches can go through trivial branch. > > Cc: qemu-trivial@nongnu.org I think this patch about COLO architecture, So, I didn't cc qemu-trivial. Thanks Zhang Chen > >> --- >> docs/COLO-FT.txt | 72 >> +++++++++++++++++++++++++++++--------------------------- >> 1 file changed, 37 insertions(+), 35 deletions(-) >> >> diff --git a/docs/COLO-FT.txt b/docs/COLO-FT.txt >> index 6282938..e289be2 100644 >> --- a/docs/COLO-FT.txt >> +++ b/docs/COLO-FT.txt >> @@ -41,41 +41,43 @@ identical responses to all client requests. Once >> the differences in the outputs >> are detected between the PVM and SVM, COLO withholds transmission >> of the >> outbound packets until it has successfully synchronized the PVM >> state to the SVM. >> >> - Primary Node Secondary Node >> - +------------+ +-----------------------+ >> +------------------------+ +------------+ >> - | | | HeartBeat |<----->| HeartBeat >> | | | >> - | Primary VM | +-----------|-----------+ >> +-----------|------------+ |Secondary VM| >> - | | | | | | >> - | | +-----------|-----------+ >> +-----------|------------+ | | >> - | | |QEMU +---v----+ | |QEMU >> +----v---+ | | | >> - | | | |Failover| | | |Failover| >> | | | >> - | | | +--------+ | | +--------+ >> | | | >> - | | | +---------------+ | | >> +---------------+ | | | >> - | | | | VM Checkpoint |-------------->| VM Checkpoint >> | | | | >> - | | | +---------------+ | | >> +---------------+ | | | >> - | | | | | >> | | | >> - >> |Requests<---------------------------^------------------------------------------>Requests| >> - |Responses----------------------\ /--|--------------\ >> /------------------------Responses| >> - | | | | | | | | | >> | | | | >> - | | | +-----------+ | | | | | | | >> +------------+ | | | >> - | | | | COLO disk | | | | | | | | | COLO >> disk | | | | >> - | | | | Manager |-|-|--|--------------|--|->| >> Manager | | | | >> - | | | +|----------+ | | | | | | | >> +-----------|+ | | | >> - | | | | | | | | | | >> | | | | | >> - +------------+ +--|------------|-|--|--+ >> +---|--|--------------|--+ +------------+ >> - | | | | | | | >> - +-------------+ | +----------v-v--|--+ +---|--v-----------+ >> | +-------------+ >> - | VM Monitor | | | COLO Proxy | | COLO Proxy >> | | | VM Monitor | >> - | | | |(compare packet) | | (adjust >> sequence)| | | | >> - +-------------+ | +----------|----^--+ +------------------+ >> | +-------------+ >> - | | | | >> - +------------------|------------|----|--+ >> +---------------------|------------------+ >> - | Kernel | | | | | Kernel >> | | >> - +------------------|------------|----|--+ >> +---------------------|------------------+ >> - | | | | >> - +--------------v+ +--------v----|--+ +------------------+ >> +v-------------+ >> - | Storage | |External Network| | External Network >> | | Storage | >> - +---------------+ +----------------+ +------------------+ >> +--------------+ >> + Primary Node Secondary Node >> ++------------+ +-----------------------+ >> +------------------------+ +------------+ >> +| | | HeartBeat +<----->+ HeartBeat >> | | | >> +| Primary VM | +-----------+-----------+ >> +-----------+------------+ |Secondary VM| >> +| | | | | | >> +| | +-----------|-----------+ >> +-----------|------------+ | | >> +| | |QEMU +---v----+ | |QEMU >> +----v---+ | | | >> +| | | |Failover| | | |Failover| >> | | | >> +| | | +--------+ | | +--------+ >> | | | >> +| | | +---------------+ | | >> +---------------+ | | | >> +| | | | VM Checkpoint +-------------->+ VM Checkpoint >> | | | | >> +| | | +---------------+ | | >> +---------------+ | | | >> +|Requests<--------------------------\ /-----------------\ >> /--------------------->Requests| >> +| | | ^ ^ | | | >> | | | | >> +|Responses+---------------------\ /-|-|------------\ >> /-------------------------+Responses| >> +| | | | | | | | | | | | >> | | | | >> +| | | +-----------+ | | | | | | | | | | >> +----------+ | | | >> +| | | | COLO disk | | | | | | | | | | | | COLO >> disk| | | | >> +| | | | Manager +---------------------------->| >> Manager | | | | >> +| | | ++----------+ v v | | | | | v v | >> +---------++ | | | >> +| | | |+-----------+-+-+-++| | >> ++-+--+-+---------+ | | | | >> +| | | || COLO Proxy || | | COLO Proxy >> | | | | | >> +| | | || (compare packet || | |(adjust sequence >> | | | | | >> +| | | ||and mirror packet)|| | | and ACK) >> | | | | | >> +| | | |+------------+---+-+| | >> +-----------------+ | | | | >> ++------------+ +-----------------------+ >> +------------------------+ +------------+ >> ++------------+ | | | >> | +------------+ >> +| VM Monitor | | | | >> | | VM Monitor | >> ++------------+ | | | >> | +------------+ >> ++---------------------------------------+ >> +----------------------------------------+ >> +| Kernel | | | | | Kernel >> | | >> ++---------------------------------------+ >> +----------------------------------------+ >> + | | | | >> + +--------------v+ +---------v---+--+ +------------------+ >> +v-------------+ >> + | Storage | |External Network| | External Network | >> | Storage | >> + +---------------+ +----------------+ +------------------+ >> +--------------+ >> + >> >> == Components introduction == >> >> > > > > . > -- Thanks zhangchen