[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v3 4/6] net/colo-compare.c: Only hexdump packets if tracing i
From: |
Zhang, Chen |
Subject: |
RE: [PATCH v3 4/6] net/colo-compare.c: Only hexdump packets if tracing is enabled |
Date: |
Mon, 27 Apr 2020 03:37:52 +0000 |
> -----Original Message-----
> From: Lukas Straub <address@hidden>
> Sent: Monday, April 27, 2020 5:19 AM
> To: qemu-devel <address@hidden>
> Cc: Zhang, Chen <address@hidden>; Li Zhijian
> <address@hidden>; Jason Wang <address@hidden>; Marc-
> André Lureau <address@hidden>; Paolo Bonzini
> <address@hidden>
> Subject: [PATCH v3 4/6] net/colo-compare.c: Only hexdump packets if tracing
> is enabled
>
> Else the log will be flooded if there is a lot of network traffic.
>
> Signed-off-by: Lukas Straub <address@hidden>
>
Looks good for me.
Reviewed-by: Zhang Chen <address@hidden>
---
> net/colo-compare.c | 10 ++++++----
> 1 file changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/net/colo-compare.c b/net/colo-compare.c index
> ff6a740284..6634911770 100644
> --- a/net/colo-compare.c
> +++ b/net/colo-compare.c
> @@ -479,10 +479,12 @@ sec:
> g_queue_push_head(&conn->primary_list, ppkt);
> g_queue_push_head(&conn->secondary_list, spkt);
>
> - qemu_hexdump((char *)ppkt->data, stderr,
> - "colo-compare ppkt", ppkt->size);
> - qemu_hexdump((char *)spkt->data, stderr,
> - "colo-compare spkt", spkt->size);
> + if
> (trace_event_get_state_backends(TRACE_COLO_COMPARE_MISCOMPARE))
> {
> + qemu_hexdump((char *)ppkt->data, stderr,
> + "colo-compare ppkt", ppkt->size);
> + qemu_hexdump((char *)spkt->data, stderr,
> + "colo-compare spkt", spkt->size);
> + }
>
> colo_compare_inconsistency_notify(s);
> }
> --
> 2.20.1
- [PATCH v3 0/6] colo-compare bugfixes, Lukas Straub, 2020/04/26
- [PATCH v3 1/6] net/colo-compare.c: Create event_bh with the right AioContext, Lukas Straub, 2020/04/26
- [PATCH v3 2/6] chardev/char.c: Use qemu_co_sleep_ns if in coroutine, Lukas Straub, 2020/04/26
- [PATCH v3 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send, Lukas Straub, 2020/04/26
- [PATCH v3 5/6] net/colo-compare.c, softmmu/vl.c: Check that colo-compare is active, Lukas Straub, 2020/04/26
- [PATCH v3 6/6] net/colo-compare.c: Correct ordering in complete and finalize, Lukas Straub, 2020/04/26
- [PATCH v3 4/6] net/colo-compare.c: Only hexdump packets if tracing is enabled, Lukas Straub, 2020/04/26
- RE: [PATCH v3 4/6] net/colo-compare.c: Only hexdump packets if tracing is enabled,
Zhang, Chen <=