| Message ID | 1600485023-263643-2-git-send-email-lei.rao@intel.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/3] Optimize seq_sorter function for colo-compare | expand |
On 9/19/20 11:10 AM, leirao wrote: > The seq of tcp has been filled in fill_pkt_tcp_info, it > can be used directly here. > > Signed-off-by: leirao <lei.rao@intel.com> Reviewed-by: Li Zhijian <lizhijian@cn.fujitsu.com> > --- > net/colo-compare.c | 6 +----- > 1 file changed, 1 insertion(+), 5 deletions(-) > > diff --git a/net/colo-compare.c b/net/colo-compare.c > index 3a45d64..86980ce 100644 > --- a/net/colo-compare.c > +++ b/net/colo-compare.c > @@ -196,11 +196,7 @@ static void colo_compare_inconsistency_notify(CompareState *s) > > static gint seq_sorter(Packet *a, Packet *b, gpointer data) > { > - struct tcp_hdr *atcp, *btcp; > - > - atcp = (struct tcp_hdr *)(a->transport_header); > - btcp = (struct tcp_hdr *)(b->transport_header); > - return ntohl(atcp->th_seq) - ntohl(btcp->th_seq); > + return a->tcp_seq - b->tcp_seq; > } > > static void fill_pkt_tcp_info(void *data, uint32_t *max_ack)
diff --git a/net/colo-compare.c b/net/colo-compare.c index 3a45d64..86980ce 100644 --- a/net/colo-compare.c +++ b/net/colo-compare.c @@ -196,11 +196,7 @@ static void colo_compare_inconsistency_notify(CompareState *s) static gint seq_sorter(Packet *a, Packet *b, gpointer data) { - struct tcp_hdr *atcp, *btcp; - - atcp = (struct tcp_hdr *)(a->transport_header); - btcp = (struct tcp_hdr *)(b->transport_header); - return ntohl(atcp->th_seq) - ntohl(btcp->th_seq); + return a->tcp_seq - b->tcp_seq; } static void fill_pkt_tcp_info(void *data, uint32_t *max_ack)
The seq of tcp has been filled in fill_pkt_tcp_info, it can be used directly here. Signed-off-by: leirao <lei.rao@intel.com> --- net/colo-compare.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-)