Re: [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare

[Dr. David Alan Gilbert]

* Zhang Chen (address@hidden) wrote:
> From: Zhang Chen <address@hidden>
> 
> We missed the iothread related args in this file.
> This patch is used to fix this issue.
> 
> Signed-off-by: Zhang Chen <address@hidden>

OK.


Reviewed-by: Dr. David Alan Gilbert <address@hidden>

> ---
>  qemu-options.hx | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/qemu-options.hx b/qemu-options.hx
> index 08749a3391..a4500c99ef 100644
> --- a/qemu-options.hx
> +++ b/qemu-options.hx
> @@ -4420,13 +4420,15 @@ Dump the network traffic on netdev @var{dev} to the 
> file specified by
>  The file format is libpcap, so it can be analyzed with tools such as tcpdump
>  or Wireshark.
>  
> address@hidden -object 
> colo-compare,address@hidden,address@hidden,address@hidden,address@hidden,vnet_hdr_support]
> address@hidden -object 
> colo-compare,address@hidden,address@hidden,address@hidden,address@hidden,address@hidden,vnet_hdr_support]
>  
>  Colo-compare gets packet from address@hidden and address@hidden, than 
> compare primary packet with
>  secondary packet. If the packets are same, we will output primary
>  packet to address@hidden, else we will notify colo-frame
>  do checkpoint and send primary packet to address@hidden
> -if it has the vnet_hdr_support flag, colo compare will send/recv packet with 
> vnet_hdr_len.
> +In order to improve efficiency, we need to put the task of comparison
> +in another thread. If it has the vnet_hdr_support flag, colo compare
> +will send/recv packet with vnet_hdr_len.
>  
>  we must use it with the help of filter-mirror and filter-redirector.
>  
> @@ -4441,10 +4443,11 @@ primary:
>  -chardev socket,id=compare0-0,host=3.3.3.3,port=9001
>  -chardev socket,id=compare_out,host=3.3.3.3,port=9005,server,nowait
>  -chardev socket,id=compare_out0,host=3.3.3.3,port=9005
> +-object iothread,id=iothread1
>  -object filter-mirror,id=m0,netdev=hn0,queue=tx,outdev=mirror0
>  -object filter-redirector,netdev=hn0,id=redire0,queue=rx,indev=compare_out
>  -object filter-redirector,netdev=hn0,id=redire1,queue=rx,outdev=compare0
> --object 
> colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,outdev=compare_out0
> +-object 
> colo-compare,id=comp0,primary_in=compare0-0,secondary_in=compare1,outdev=compare_out0,iothread=iothread1
>  
>  secondary:
>  -netdev tap,id=hn0,vhost=off,script=/etc/qemu-ifup,down 
> script=/etc/qemu-ifdown
> -- 
> 2.17.GIT
> 
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK