* [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
In this series we optimize codes and fix some tiny issues.
Zhang Chen (3):
migration/colo.c: Remove redundant input parameter
migration/colo.h: Remove obsolete codes
qemu-option.hx: Update missed parameter for colo-compare
include/migration/colo.h | 4 +---
migration/colo-failover.c | 2 +-
migration/colo.c | 2 +-
qemu-options.hx | 9 ++++++---
4 files changed, 9 insertions(+), 8 deletions(-)
--
2.17.GIT
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
In this series we optimize codes and fix some tiny issues.
Zhang Chen (3):
migration/colo.c: Remove redundant input parameter
migration/colo.h: Remove obsolete codes
qemu-option.hx: Update missed parameter for colo-compare
include/migration/colo.h | 4 +---
migration/colo-failover.c | 2 +-
migration/colo.c | 2 +-
qemu-options.hx | 9 ++++++---
4 files changed, 9 insertions(+), 8 deletions(-)
--
2.17.GIT
^ permalink raw reply [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 1/3] migration/colo.c: Remove redundant input parameter
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
The colo_do_failover no need the input parameter.
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
include/migration/colo.h | 2 +-
migration/colo-failover.c | 2 +-
migration/colo.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/migration/colo.h b/include/migration/colo.h
index 99ce17aca7..ddebe0ad27 100644
--- a/include/migration/colo.h
+++ b/include/migration/colo.h
@@ -37,7 +37,7 @@ bool migration_incoming_in_colo_state(void);
COLOMode get_colo_mode(void);
/* failover */
-void colo_do_failover(MigrationState *s);
+void colo_do_failover(void);
void colo_checkpoint_notify(void *opaque);
#endif
diff --git a/migration/colo-failover.c b/migration/colo-failover.c
index 4854a96c92..e9ca0b4774 100644
--- a/migration/colo-failover.c
+++ b/migration/colo-failover.c
@@ -39,7 +39,7 @@ static void colo_failover_bh(void *opaque)
return;
}
- colo_do_failover(NULL);
+ colo_do_failover();
}
void failover_request_active(Error **errp)
diff --git a/migration/colo.c b/migration/colo.c
index 238a6d62c7..8c1644091f 100644
--- a/migration/colo.c
+++ b/migration/colo.c
@@ -193,7 +193,7 @@ COLOMode get_colo_mode(void)
}
}
-void colo_do_failover(MigrationState *s)
+void colo_do_failover(void)
{
/* Make sure VM stopped while failover happened. */
if (!colo_runstate_is_stopped()) {
--
2.17.GIT
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 1/3] migration/colo.c: Remove redundant input parameter
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
The colo_do_failover no need the input parameter.
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
include/migration/colo.h | 2 +-
migration/colo-failover.c | 2 +-
migration/colo.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/include/migration/colo.h b/include/migration/colo.h
index 99ce17aca7..ddebe0ad27 100644
--- a/include/migration/colo.h
+++ b/include/migration/colo.h
@@ -37,7 +37,7 @@ bool migration_incoming_in_colo_state(void);
COLOMode get_colo_mode(void);
/* failover */
-void colo_do_failover(MigrationState *s);
+void colo_do_failover(void);
void colo_checkpoint_notify(void *opaque);
#endif
diff --git a/migration/colo-failover.c b/migration/colo-failover.c
index 4854a96c92..e9ca0b4774 100644
--- a/migration/colo-failover.c
+++ b/migration/colo-failover.c
@@ -39,7 +39,7 @@ static void colo_failover_bh(void *opaque)
return;
}
- colo_do_failover(NULL);
+ colo_do_failover();
}
void failover_request_active(Error **errp)
diff --git a/migration/colo.c b/migration/colo.c
index 238a6d62c7..8c1644091f 100644
--- a/migration/colo.c
+++ b/migration/colo.c
@@ -193,7 +193,7 @@ COLOMode get_colo_mode(void)
}
}
-void colo_do_failover(MigrationState *s)
+void colo_do_failover(void)
{
/* Make sure VM stopped while failover happened. */
if (!colo_runstate_is_stopped()) {
--
2.17.GIT
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 2/3] migration/colo.h: Remove obsolete codes
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
include/migration/colo.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/include/migration/colo.h b/include/migration/colo.h
index ddebe0ad27..f6fbe23ec9 100644
--- a/include/migration/colo.h
+++ b/include/migration/colo.h
@@ -22,8 +22,6 @@ enum colo_event {
COLO_EVENT_FAILOVER,
};
-void colo_info_init(void);
-
void migrate_start_colo_process(MigrationState *s);
bool migration_in_colo_state(void);
--
2.17.GIT
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 2/3] migration/colo.h: Remove obsolete codes
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
include/migration/colo.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/include/migration/colo.h b/include/migration/colo.h
index ddebe0ad27..f6fbe23ec9 100644
--- a/include/migration/colo.h
+++ b/include/migration/colo.h
@@ -22,8 +22,6 @@ enum colo_event {
COLO_EVENT_FAILOVER,
};
-void colo_info_init(void);
-
void migrate_start_colo_process(MigrationState *s);
bool migration_in_colo_state(void);
--
2.17.GIT
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
We missed the iothread related args in this file.
This patch is used to fix this issue.
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
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.
-@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid}[,vnet_hdr_support]
+@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid},iothread=@var{id}[,vnet_hdr_support]
Colo-compare gets packet from primary_in@var{chardevid} and secondary_in@var{chardevid}, than compare primary packet with
secondary packet. If the packets are same, we will output primary
packet to outdev@var{chardevid}, else we will notify colo-frame
do checkpoint and send primary packet to outdev@var{chardevid}.
-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
^ permalink raw reply related [flat|nested] 17+ messages in thread
* [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare
@ 2019-04-26 9:07 ` Zhang Chen
0 siblings, 0 replies; 17+ messages in thread
From: Zhang Chen @ 2019-04-26 9:07 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen, Zhang Chen
From: Zhang Chen <chen.zhang@intel.com>
We missed the iothread related args in this file.
This patch is used to fix this issue.
Signed-off-by: Zhang Chen <chen.zhang@intel.com>
---
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.
-@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid}[,vnet_hdr_support]
+@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid},iothread=@var{id}[,vnet_hdr_support]
Colo-compare gets packet from primary_in@var{chardevid} and secondary_in@var{chardevid}, than compare primary packet with
secondary packet. If the packets are same, we will output primary
packet to outdev@var{chardevid}, else we will notify colo-frame
do checkpoint and send primary packet to outdev@var{chardevid}.
-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
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 1/3] migration/colo.c: Remove redundant input parameter
@ 2019-04-29 12:21 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:21 UTC (permalink / raw)
To: Zhang Chen
Cc: Laurent Vivier, Juan Quintela, zhanghailiang, Markus Armbruster,
qemu-dev, Zhang Chen
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> The colo_do_failover no need the input parameter.
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> include/migration/colo.h | 2 +-
> migration/colo-failover.c | 2 +-
> migration/colo.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/migration/colo.h b/include/migration/colo.h
> index 99ce17aca7..ddebe0ad27 100644
> --- a/include/migration/colo.h
> +++ b/include/migration/colo.h
> @@ -37,7 +37,7 @@ bool migration_incoming_in_colo_state(void);
> COLOMode get_colo_mode(void);
>
> /* failover */
> -void colo_do_failover(MigrationState *s);
> +void colo_do_failover(void);
>
> void colo_checkpoint_notify(void *opaque);
> #endif
> diff --git a/migration/colo-failover.c b/migration/colo-failover.c
> index 4854a96c92..e9ca0b4774 100644
> --- a/migration/colo-failover.c
> +++ b/migration/colo-failover.c
> @@ -39,7 +39,7 @@ static void colo_failover_bh(void *opaque)
> return;
> }
>
> - colo_do_failover(NULL);
> + colo_do_failover();
> }
>
> void failover_request_active(Error **errp)
> diff --git a/migration/colo.c b/migration/colo.c
> index 238a6d62c7..8c1644091f 100644
> --- a/migration/colo.c
> +++ b/migration/colo.c
> @@ -193,7 +193,7 @@ COLOMode get_colo_mode(void)
> }
> }
>
> -void colo_do_failover(MigrationState *s)
> +void colo_do_failover(void)
> {
> /* Make sure VM stopped while failover happened. */
> if (!colo_runstate_is_stopped()) {
> --
> 2.17.GIT
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 1/3] migration/colo.c: Remove redundant input parameter
@ 2019-04-29 12:21 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:21 UTC (permalink / raw)
To: Zhang Chen
Cc: zhanghailiang, Juan Quintela, qemu-dev, Markus Armbruster,
Zhang Chen, Laurent Vivier
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> The colo_do_failover no need the input parameter.
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> include/migration/colo.h | 2 +-
> migration/colo-failover.c | 2 +-
> migration/colo.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/include/migration/colo.h b/include/migration/colo.h
> index 99ce17aca7..ddebe0ad27 100644
> --- a/include/migration/colo.h
> +++ b/include/migration/colo.h
> @@ -37,7 +37,7 @@ bool migration_incoming_in_colo_state(void);
> COLOMode get_colo_mode(void);
>
> /* failover */
> -void colo_do_failover(MigrationState *s);
> +void colo_do_failover(void);
>
> void colo_checkpoint_notify(void *opaque);
> #endif
> diff --git a/migration/colo-failover.c b/migration/colo-failover.c
> index 4854a96c92..e9ca0b4774 100644
> --- a/migration/colo-failover.c
> +++ b/migration/colo-failover.c
> @@ -39,7 +39,7 @@ static void colo_failover_bh(void *opaque)
> return;
> }
>
> - colo_do_failover(NULL);
> + colo_do_failover();
> }
>
> void failover_request_active(Error **errp)
> diff --git a/migration/colo.c b/migration/colo.c
> index 238a6d62c7..8c1644091f 100644
> --- a/migration/colo.c
> +++ b/migration/colo.c
> @@ -193,7 +193,7 @@ COLOMode get_colo_mode(void)
> }
> }
>
> -void colo_do_failover(MigrationState *s)
> +void colo_do_failover(void)
> {
> /* Make sure VM stopped while failover happened. */
> if (!colo_runstate_is_stopped()) {
> --
> 2.17.GIT
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 2/3] migration/colo.h: Remove obsolete codes
@ 2019-04-29 12:21 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:21 UTC (permalink / raw)
To: Zhang Chen
Cc: Laurent Vivier, Juan Quintela, zhanghailiang, Markus Armbruster,
qemu-dev, Zhang Chen
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> include/migration/colo.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/include/migration/colo.h b/include/migration/colo.h
> index ddebe0ad27..f6fbe23ec9 100644
> --- a/include/migration/colo.h
> +++ b/include/migration/colo.h
> @@ -22,8 +22,6 @@ enum colo_event {
> COLO_EVENT_FAILOVER,
> };
>
> -void colo_info_init(void);
> -
> void migrate_start_colo_process(MigrationState *s);
> bool migration_in_colo_state(void);
>
> --
> 2.17.GIT
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 2/3] migration/colo.h: Remove obsolete codes
@ 2019-04-29 12:21 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:21 UTC (permalink / raw)
To: Zhang Chen
Cc: zhanghailiang, Juan Quintela, qemu-dev, Markus Armbruster,
Zhang Chen, Laurent Vivier
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> include/migration/colo.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/include/migration/colo.h b/include/migration/colo.h
> index ddebe0ad27..f6fbe23ec9 100644
> --- a/include/migration/colo.h
> +++ b/include/migration/colo.h
> @@ -22,8 +22,6 @@ enum colo_event {
> COLO_EVENT_FAILOVER,
> };
>
> -void colo_info_init(void);
> -
> void migrate_start_colo_process(MigrationState *s);
> bool migration_in_colo_state(void);
>
> --
> 2.17.GIT
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare
@ 2019-04-29 12:25 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:25 UTC (permalink / raw)
To: Zhang Chen
Cc: Laurent Vivier, Juan Quintela, zhanghailiang, Markus Armbruster,
qemu-dev, Zhang Chen
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> We missed the iothread related args in this file.
> This patch is used to fix this issue.
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
OK.
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> 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.
>
> -@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid}[,vnet_hdr_support]
> +@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid},iothread=@var{id}[,vnet_hdr_support]
>
> Colo-compare gets packet from primary_in@var{chardevid} and secondary_in@var{chardevid}, than compare primary packet with
> secondary packet. If the packets are same, we will output primary
> packet to outdev@var{chardevid}, else we will notify colo-frame
> do checkpoint and send primary packet to outdev@var{chardevid}.
> -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 / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare
@ 2019-04-29 12:25 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-04-29 12:25 UTC (permalink / raw)
To: Zhang Chen
Cc: zhanghailiang, Juan Quintela, qemu-dev, Markus Armbruster,
Zhang Chen, Laurent Vivier
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> We missed the iothread related args in this file.
> This patch is used to fix this issue.
>
> Signed-off-by: Zhang Chen <chen.zhang@intel.com>
OK.
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> ---
> 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.
>
> -@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid}[,vnet_hdr_support]
> +@item -object colo-compare,id=@var{id},primary_in=@var{chardevid},secondary_in=@var{chardevid},outdev=@var{chardevid},iothread=@var{id}[,vnet_hdr_support]
>
> Colo-compare gets packet from primary_in@var{chardevid} and secondary_in@var{chardevid}, than compare primary packet with
> secondary packet. If the packets are same, we will output primary
> packet to outdev@var{chardevid}, else we will notify colo-frame
> do checkpoint and send primary packet to outdev@var{chardevid}.
> -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 / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description
2019-04-26 9:07 ` Zhang Chen
` (3 preceding siblings ...)
(?)
@ 2019-05-10 2:11 ` Zhang, Chen
2019-05-14 11:07 ` Dr. David Alan Gilbert
-1 siblings, 1 reply; 17+ messages in thread
From: Zhang, Chen @ 2019-05-10 2:11 UTC (permalink / raw)
To: Laurent Vivier, Dr. David Alan Gilbert, Juan Quintela,
zhanghailiang, Markus Armbruster, qemu-dev
Cc: Zhang Chen
Hi Dave,
I noticed that you have reviewed all the patches in this series, can you queue it?
Thanks
Zhang Chen
> -----Original Message-----
> From: Zhang, Chen
> Sent: Friday, April 26, 2019 5:07 PM
> To: Laurent Vivier <laurent@vivier.eu>; Dr. David Alan Gilbert
> <dgilbert@redhat.com>; Juan Quintela <quintela@redhat.com>; zhanghailiang
> <zhang.zhanghailiang@huawei.com>; Markus Armbruster
> <armbru@redhat.com>; qemu-dev <qemu-devel@nongnu.org>
> Cc: Zhang Chen <zhangckid@gmail.com>; Zhang, Chen <chen.zhang@intel.com>
> Subject: [PATCH 0/3] Optimize COLO related codes and description
>
> From: Zhang Chen <chen.zhang@intel.com>
>
> In this series we optimize codes and fix some tiny issues.
>
> Zhang Chen (3):
> migration/colo.c: Remove redundant input parameter
> migration/colo.h: Remove obsolete codes
> qemu-option.hx: Update missed parameter for colo-compare
>
> include/migration/colo.h | 4 +---
> migration/colo-failover.c | 2 +-
> migration/colo.c | 2 +-
> qemu-options.hx | 9 ++++++---
> 4 files changed, 9 insertions(+), 8 deletions(-)
>
> --
> 2.17.GIT
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description
2019-05-10 2:11 ` [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description Zhang, Chen
@ 2019-05-14 11:07 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-05-14 11:07 UTC (permalink / raw)
To: Zhang, Chen
Cc: zhanghailiang, Juan Quintela, qemu-dev, Markus Armbruster,
Zhang Chen, Laurent Vivier
* Zhang, Chen (chen.zhang@intel.com) wrote:
> Hi Dave,
>
> I noticed that you have reviewed all the patches in this series, can you queue it?
Yes, I'm about to start a migration pull now.
Dave
> Thanks
> Zhang Chen
>
>
> > -----Original Message-----
> > From: Zhang, Chen
> > Sent: Friday, April 26, 2019 5:07 PM
> > To: Laurent Vivier <laurent@vivier.eu>; Dr. David Alan Gilbert
> > <dgilbert@redhat.com>; Juan Quintela <quintela@redhat.com>; zhanghailiang
> > <zhang.zhanghailiang@huawei.com>; Markus Armbruster
> > <armbru@redhat.com>; qemu-dev <qemu-devel@nongnu.org>
> > Cc: Zhang Chen <zhangckid@gmail.com>; Zhang, Chen <chen.zhang@intel.com>
> > Subject: [PATCH 0/3] Optimize COLO related codes and description
> >
> > From: Zhang Chen <chen.zhang@intel.com>
> >
> > In this series we optimize codes and fix some tiny issues.
> >
> > Zhang Chen (3):
> > migration/colo.c: Remove redundant input parameter
> > migration/colo.h: Remove obsolete codes
> > qemu-option.hx: Update missed parameter for colo-compare
> >
> > include/migration/colo.h | 4 +---
> > migration/colo-failover.c | 2 +-
> > migration/colo.c | 2 +-
> > qemu-options.hx | 9 ++++++---
> > 4 files changed, 9 insertions(+), 8 deletions(-)
> >
> > --
> > 2.17.GIT
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description
2019-04-26 9:07 ` Zhang Chen
` (4 preceding siblings ...)
(?)
@ 2019-05-14 14:06 ` Dr. David Alan Gilbert
-1 siblings, 0 replies; 17+ messages in thread
From: Dr. David Alan Gilbert @ 2019-05-14 14:06 UTC (permalink / raw)
To: Zhang Chen
Cc: zhanghailiang, Juan Quintela, qemu-dev, Markus Armbruster,
Zhang Chen, Laurent Vivier
* Zhang Chen (chen.zhang@intel.com) wrote:
> From: Zhang Chen <chen.zhang@intel.com>
>
> In this series we optimize codes and fix some tiny issues.
Queued
>
> Zhang Chen (3):
> migration/colo.c: Remove redundant input parameter
> migration/colo.h: Remove obsolete codes
> qemu-option.hx: Update missed parameter for colo-compare
>
> include/migration/colo.h | 4 +---
> migration/colo-failover.c | 2 +-
> migration/colo.c | 2 +-
> qemu-options.hx | 9 ++++++---
> 4 files changed, 9 insertions(+), 8 deletions(-)
>
> --
> 2.17.GIT
>
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2019-05-14 14:19 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-04-26 9:07 [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description Zhang Chen
2019-04-26 9:07 ` Zhang Chen
2019-04-26 9:07 ` [Qemu-devel] [PATCH 1/3] migration/colo.c: Remove redundant input parameter Zhang Chen
2019-04-26 9:07 ` Zhang Chen
2019-04-29 12:21 ` Dr. David Alan Gilbert
2019-04-29 12:21 ` Dr. David Alan Gilbert
2019-04-26 9:07 ` [Qemu-devel] [PATCH 2/3] migration/colo.h: Remove obsolete codes Zhang Chen
2019-04-26 9:07 ` Zhang Chen
2019-04-29 12:21 ` Dr. David Alan Gilbert
2019-04-29 12:21 ` Dr. David Alan Gilbert
2019-04-26 9:07 ` [Qemu-devel] [PATCH 3/3] qemu-option.hx: Update missed parameter for colo-compare Zhang Chen
2019-04-26 9:07 ` Zhang Chen
2019-04-29 12:25 ` Dr. David Alan Gilbert
2019-04-29 12:25 ` Dr. David Alan Gilbert
2019-05-10 2:11 ` [Qemu-devel] [PATCH 0/3] Optimize COLO related codes and description Zhang, Chen
2019-05-14 11:07 ` Dr. David Alan Gilbert
2019-05-14 14:06 ` Dr. David Alan Gilbert
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.