* Zhang Chen (zhangchen.f...@cn.fujitsu.com) wrote: > From: zhangchen <zhangchen.f...@cn.fujitsu.com> > > add colo-proxy to vl.c and qemu-options.hx > add trace-colo-proxy relation > > Signed-off-by: zhangchen <zhangchen.f...@cn.fujitsu.com> > Signed-off-by: zhanghailiang <zhang.zhanghaili...@huawei.com> > --- > qemu-options.hx | 6 ++++++ > trace-events | 8 ++++++++ > vl.c | 3 ++- > 3 files changed, 16 insertions(+), 1 deletion(-) > > diff --git a/qemu-options.hx b/qemu-options.hx > index 0eea4ee..6daa3f0 100644 > --- a/qemu-options.hx > +++ b/qemu-options.hx > @@ -3670,6 +3670,12 @@ queue @var{all|rx|tx} is an option that can be applied > to any netfilter. > @option{tx}: the filter is attached to the transmit queue of the netdev, > where it will receive packets sent by the netdev. > > +@item -object > colo-proxy,id=@var{id},netdev=@var{netdevid},addr=@var{host:port},mode=@var{primary|secondary}[,queue=@var{all}] > + > +Colo-proxy on netdev @var{netdevid},set colo mode @var{primary|secondary} > +connect other colo through addr@var{host:port},and colo needs queue all > +packet arriving in queue=@var{all} > + > @item -object > filter-dump,id=@var{id},netdev=@var{dev},file=@var{filename}][,maxlen=@var{len}] > > Dump the network traffic on netdev @var{dev} to the file specified by > diff --git a/trace-events b/trace-events > index 5f95b3c..a957fb3 100644 > --- a/trace-events > +++ b/trace-events > @@ -1586,6 +1586,14 @@ colo_failover_set_state(int new_state) "new state %d" > colo_start_block_replication(void) "Block replication is started" > colo_stop_block_replication(const char *reason) "Block replication is > stopped(reason: '%s')" > > +# net/colo-proxy.c > +colo_proxy(const char *sta) ": %s"
You use the 'colo_proxy' trace in a lot of different places; it would be better to use individual trace entries, so for example you could just trace miscompares. Dave > +colo_proxy_with_ret(const char *sta, ssize_t ret) ": %s ret = %zu" > +colo_proxy_packet_src(const char *src) ":ipsrc = %s" > +colo_proxy_packet_dst(const char *dst) ":ipdst = %s" > +colo_proxy_packet_size(int size) ": %d" > +colo_proxy_queue_size(int size) ": %d" > + > # kvm-all.c > kvm_ioctl(int type, void *arg) "type 0x%x, arg %p" > kvm_vm_ioctl(int type, void *arg) "type 0x%x, arg %p" > diff --git a/vl.c b/vl.c > index 8dc34ce..dcfb3a9 100644 > --- a/vl.c > +++ b/vl.c > @@ -2838,7 +2838,8 @@ static bool object_create_initial(const char *type) > * they depend on netdevs already existing > */ > if (g_str_equal(type, "filter-buffer") || > - g_str_equal(type, "filter-dump")) { > + g_str_equal(type, "filter-dump") || > + g_str_equal(type, "colo-proxy")) { > return false; > } > > -- > 1.9.1 > > > -- Dr. David Alan Gilbert / dgilb...@redhat.com / Manchester, UK