[v4] Rewrite TCP packet comparison in colo

[Qemu-devel] [PATCH v4 0/2] Rewrite TCP packet comparison in colo

Posted by Mao Zhongyi 6 years, 4 months ago

v4:
  p2: fix some typo
                                                [Zhang Chen]
v3:
  p1: merged the patch1 and patch2 from v2 
  p2: -merged the patch3 and patch4 from v2
      -implement the same process flow for tcp, udp and icmp

                                                [Zhang Chen]
v2:
  p1: a new patch
  p2: a new patch
  p3: -rename the fill_pkt_seq to fill_pkt_tcp_info
      -rename pdsize & hdsize to payload_size & header_size
      -reuse duplicated code
      -modified colo_packet_compare_common() to suit the tcp packet
       comparison instead of build a new function service for tcp.
      -add more comments for the 'max_ack'
                                                [Zhang Chen]

Cc: Zhang Chen <zhangckid@gmail.com>
Cc: Li Zhijian <lizhijian@cn.fujitsu.com>
Cc: Jason Wang <jasowang@redhat.com>

Mao Zhongyi (2):
  colo: modified the payload compare function
  colo: compare the packet based on the tcp sequence number

 net/colo-compare.c | 411 +++++++++++++++++++++++++++++++++--------------------
 net/colo.c         |   9 ++
 net/colo.h         |  15 ++
 net/trace-events   |   2 +-
 4 files changed, 284 insertions(+), 153 deletions(-)

-- 
2.9.4

Re: [Qemu-devel] [PATCH v4 0/2] Rewrite TCP packet comparison in colo

Posted by Mao Zhongyi 6 years, 3 months ago

Hi, Jason

Long time no news, Ping...

Thanks,
Mao

On 12/25/2017 10:54 AM, Mao Zhongyi wrote:
> v4:
>   p2: fix some typo
>                                                 [Zhang Chen]
> v3:
>   p1: merged the patch1 and patch2 from v2
>   p2: -merged the patch3 and patch4 from v2
>       -implement the same process flow for tcp, udp and icmp
>
>                                                 [Zhang Chen]
> v2:
>   p1: a new patch
>   p2: a new patch
>   p3: -rename the fill_pkt_seq to fill_pkt_tcp_info
>       -rename pdsize & hdsize to payload_size & header_size
>       -reuse duplicated code
>       -modified colo_packet_compare_common() to suit the tcp packet
>        comparison instead of build a new function service for tcp.
>       -add more comments for the 'max_ack'
>                                                 [Zhang Chen]
>
> Cc: Zhang Chen <zhangckid@gmail.com>
> Cc: Li Zhijian <lizhijian@cn.fujitsu.com>
> Cc: Jason Wang <jasowang@redhat.com>
>
> Mao Zhongyi (2):
>   colo: modified the payload compare function
>   colo: compare the packet based on the tcp sequence number
>
>  net/colo-compare.c | 411 +++++++++++++++++++++++++++++++++--------------------
>  net/colo.c         |   9 ++
>  net/colo.h         |  15 ++
>  net/trace-events   |   2 +-
>  4 files changed, 284 insertions(+), 153 deletions(-)
>

Re: [Qemu-devel] [PATCH v4 0/2] Rewrite TCP packet comparison in colo

Posted by Jason Wang 6 years, 3 months ago


On 2018年01月04日 15:42, Mao Zhongyi wrote:
> Hi, Jason
>
> Long time no news, Ping...
>
> Thanks,
> Mao
>

Sorry for the delay.

Queued.

Thanks

> On 12/25/2017 10:54 AM, Mao Zhongyi wrote:
>> v4:
>>   p2: fix some typo
>>                                                 [Zhang Chen]
>> v3:
>>   p1: merged the patch1 and patch2 from v2
>>   p2: -merged the patch3 and patch4 from v2
>>       -implement the same process flow for tcp, udp and icmp
>>
>>                                                 [Zhang Chen]
>> v2:
>>   p1: a new patch
>>   p2: a new patch
>>   p3: -rename the fill_pkt_seq to fill_pkt_tcp_info
>>       -rename pdsize & hdsize to payload_size & header_size
>>       -reuse duplicated code
>>       -modified colo_packet_compare_common() to suit the tcp packet
>>        comparison instead of build a new function service for tcp.
>>       -add more comments for the 'max_ack'
>>                                                 [Zhang Chen]
>>
>> Cc: Zhang Chen <zhangckid@gmail.com>
>> Cc: Li Zhijian <lizhijian@cn.fujitsu.com>
>> Cc: Jason Wang <jasowang@redhat.com>
>>
>> Mao Zhongyi (2):
>>   colo: modified the payload compare function
>>   colo: compare the packet based on the tcp sequence number
>>
>>  net/colo-compare.c | 411 
>> +++++++++++++++++++++++++++++++++--------------------
>>  net/colo.c         |   9 ++
>>  net/colo.h         |  15 ++
>>  net/trace-events   |   2 +-
>>  4 files changed, 284 insertions(+), 153 deletions(-)
>>
>
>
>

Re: [Qemu-devel] [PATCH v4 0/2] Rewrite TCP packet comparison in colo

Posted by Mao Zhongyi 6 years, 3 months ago


On 01/04/2018 04:13 PM, Jason Wang wrote:
>
>
> On 2018年01月04日 15:42, Mao Zhongyi wrote:
>> Hi, Jason
>>
>> Long time no news, Ping...
>>
>> Thanks,
>> Mao
>>
>
> Sorry for the delay.

No worries, thanks :)

>
> Queued.
>
> Thanks
>
>> On 12/25/2017 10:54 AM, Mao Zhongyi wrote:
>>> v4:
>>>   p2: fix some typo
>>>                                                 [Zhang Chen]
>>> v3:
>>>   p1: merged the patch1 and patch2 from v2
>>>   p2: -merged the patch3 and patch4 from v2
>>>       -implement the same process flow for tcp, udp and icmp
>>>
>>>                                                 [Zhang Chen]
>>> v2:
>>>   p1: a new patch
>>>   p2: a new patch
>>>   p3: -rename the fill_pkt_seq to fill_pkt_tcp_info
>>>       -rename pdsize & hdsize to payload_size & header_size
>>>       -reuse duplicated code
>>>       -modified colo_packet_compare_common() to suit the tcp packet
>>>        comparison instead of build a new function service for tcp.
>>>       -add more comments for the 'max_ack'
>>>                                                 [Zhang Chen]
>>>
>>> Cc: Zhang Chen <zhangckid@gmail.com>
>>> Cc: Li Zhijian <lizhijian@cn.fujitsu.com>
>>> Cc: Jason Wang <jasowang@redhat.com>
>>>
>>> Mao Zhongyi (2):
>>>   colo: modified the payload compare function
>>>   colo: compare the packet based on the tcp sequence number
>>>
>>>  net/colo-compare.c | 411 +++++++++++++++++++++++++++++++++--------------------
>>>  net/colo.c         |   9 ++
>>>  net/colo.h         |  15 ++
>>>  net/trace-events   |   2 +-
>>>  4 files changed, 284 insertions(+), 153 deletions(-)
>>>
>>
>>
>>
>
>
>
>