[netsniff-ng] Re: [PATCH] trafgen: l3: Fix checksum for UDP/TCP protos

2017-01-25 Thread Tobias Klauser 
On 2017-01-25 at 13:34:54 +0100, Vadim Kochan  wrote:
> On Wed, Jan 25, 2017 at 2:24 PM, Tobias Klauser  wrote:
> > On 2017-01-23 at 22:49:22 +0100, Vadim Kochan  wrote:
> >> While fixing the issue with getting of IPv4 address from device,
> >> the setting of default src IPv4/IPv6 addresses was moved from
> >> hdr->header_init(...) callback to hdr->packet_finish(...), but
> >> packet_finish(...) is called in the following order:
> >>
> >> udp_hdr->packet_finish() - UDP csum calculation over IPv4/6 pseudo 
> >> header
> >> ip4_hdr->packet_finish() - setting default src IPv4 address from dev
> >> ...
> >>
> >> So src IPv4/6 address will be set after UDP/TCP csum calculation which
> >> is wrong, so fixed issue by moving it to the hdr->header_init(...) stage
> >> as it was before the c4e07d5142c8.
> >>
> >> Fixes: c4e07d5142c8 ("trafgen: l3: Support interface without IP address")
> >> Signed-off-by: Vadim Kochan 
> >
> > Applied, thanks Vadim.
> 
> BTW,
> We really need some sub-folder with scapy tests to check
> trafgen/mausezahn packet's generation.

Sure, feel free to send patches.

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[netsniff-ng] Re: [PATCH] trafgen: l3: Fix checksum for UDP/TCP protos

2017-01-25 Thread Vadim Kochan 
On Wed, Jan 25, 2017 at 2:24 PM, Tobias Klauser  wrote:
> On 2017-01-23 at 22:49:22 +0100, Vadim Kochan  wrote:
>> While fixing the issue with getting of IPv4 address from device,
>> the setting of default src IPv4/IPv6 addresses was moved from
>> hdr->header_init(...) callback to hdr->packet_finish(...), but
>> packet_finish(...) is called in the following order:
>>
>> udp_hdr->packet_finish() - UDP csum calculation over IPv4/6 pseudo header
>> ip4_hdr->packet_finish() - setting default src IPv4 address from dev
>> ...
>>
>> So src IPv4/6 address will be set after UDP/TCP csum calculation which
>> is wrong, so fixed issue by moving it to the hdr->header_init(...) stage
>> as it was before the c4e07d5142c8.
>>
>> Fixes: c4e07d5142c8 ("trafgen: l3: Support interface without IP address")
>> Signed-off-by: Vadim Kochan 
>
> Applied, thanks Vadim.

BTW,
We really need some sub-folder with scapy tests to check
trafgen/mausezahn packet's generation.

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[netsniff-ng] Re: [PATCH] trafgen: l3: Fix checksum for UDP/TCP protos

2017-01-25 Thread Tobias Klauser 
On 2017-01-23 at 22:49:22 +0100, Vadim Kochan  wrote:
> While fixing the issue with getting of IPv4 address from device,
> the setting of default src IPv4/IPv6 addresses was moved from
> hdr->header_init(...) callback to hdr->packet_finish(...), but
> packet_finish(...) is called in the following order:
> 
> udp_hdr->packet_finish() - UDP csum calculation over IPv4/6 pseudo header
> ip4_hdr->packet_finish() - setting default src IPv4 address from dev
> ...
> 
> So src IPv4/6 address will be set after UDP/TCP csum calculation which
> is wrong, so fixed issue by moving it to the hdr->header_init(...) stage
> as it was before the c4e07d5142c8.
> 
> Fixes: c4e07d5142c8 ("trafgen: l3: Support interface without IP address")
> Signed-off-by: Vadim Kochan 

Applied, thanks Vadim.

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[netsniff-ng] Re: [PATCH v2 0/2] flowtop: Add UI tab control for process stats

2017-01-25 Thread Tobias Klauser 
On 2017-01-19 at 00:09:02 +0100, Vadim Kochan  wrote:
> Add UI tab control to switch between tabbed windows for dirrent flows table
> statistics info per pid/peer/port.
> 
> The tab entries are switched by TAB keypress.
> 
> Introduced proc_entry which holds aggregated related flows statistics 
> (rate/pkts/bytes),
> beside this changed flow_entry to point to proc_entry pid & name info.
> 
> Added UI tab for per pid flows info - flows count, bytes, rates. The 
> bytes/pkts are collected
> during the process living (but proc_entry is not removed till at lest one 
> flow points to it),
> the rates info refreshes each 1 second per proc_entry.
> 
> Re-added list.h which contains just missing cds_list_{next,prev,last}_entry 
> functions which
> are needed for navigation over UI tab entries.
> 
> v2:
> 1) Constify ui_table.delim & ui_col.name
> 2) Rename list.h -> urcu-list-compat.h and include urcu/list.h here.
> 3) Add Signed-off-by into 2nd commit.
> 4) In flow_entry_find_process(...) use proc_entry->name member instead
> of local procname string.
> 5) Rename proc_exist -> proc_exists
> 6) Rename proc_entry.procnnum -> pid
> 7) Rename proc_entry.procname -> name
> 
> Vadim Kochan (2):
>   flowtop: Add tab control to switch between tables
>   flowtop: Add process UI tab entry

Series applied, thanks.

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.