[netsniff-ng] [PATCH] flowtop: Replace single linked list by list_head from list.h

2017-01-12 Thread Vadim Kochan
list.h provides generic Linux-like linked list API which also supports RCU list operations. Also additionally was removed the spinlock which is not needed for RCU-list operations, for the list_del_rcu(...) case it is needed additionally call call_rcu(...) before free the flow entry. Because of fu

[netsniff-ng] Re: [PATCH] flowtop: Replace single linked list by list_head from list.h

2017-01-12 Thread Vadim Kochan
On Thu, Jan 12, 2017 at 5:20 PM, Tobias Klauser wrote: > On 2017-01-12 at 15:54:31 +0100, Vadim Kochan wrote: >> On Thu, Jan 12, 2017 at 4:28 PM, Tobias Klauser wrote: > [...] >> >> enum flow_direction { >> >> @@ -355,15 +357,15 @@ static inline struct flow_entry >> >> *flow_entry_xalloc(void)

[netsniff-ng] Re: [PATCH] flowtop: Replace single linked list by list_head from list.h

2017-01-12 Thread Tobias Klauser
On 2017-01-12 at 15:54:31 +0100, Vadim Kochan wrote: > On Thu, Jan 12, 2017 at 4:28 PM, Tobias Klauser wrote: [...] > >> enum flow_direction { > >> @@ -355,15 +357,15 @@ static inline struct flow_entry > >> *flow_entry_xalloc(void) > >> static inline void flow_entry_xfree(struct flow_entry *n)

[netsniff-ng] Re: [PATCH] flowtop: Replace single linked list by list_head from list.h

2017-01-12 Thread Vadim Kochan
On Thu, Jan 12, 2017 at 4:28 PM, Tobias Klauser wrote: > Thanks for the patch and sorry for the delay in reply. > > I'm not too familiar with the RCU list interface, but from a quick > glance this looks OK in general. A few remarks below. > > Daniel, if you have time could you maybe have a quick l

[netsniff-ng] Re: [PATCH] flowtop: Replace single linked list by list_head from list.h

2017-01-12 Thread Tobias Klauser
Thanks for the patch and sorry for the delay in reply. I'm not too familiar with the RCU list interface, but from a quick glance this looks OK in general. A few remarks below. Daniel, if you have time could you maybe have a quick look? On 2017-01-09 at 07:26:07 +0100, Vadim Kochan wrote: > list

[netsniff-ng] Re: [PATCH 0/4] Introduce new pcap io API for pcap r/w accesses

2017-01-12 Thread Tobias Klauser
On 2017-01-12 at 13:59:05 +0100, Vadim Kochan wrote: > On Wed, Dec 14, 2016 at 11:33 PM, Vadim Kochan wrote: > > On Wed, Dec 14, 2016 at 11:26 PM, Tobias Klauser > > wrote: > >> On 2016-12-12 at 22:09:52 +0100, Vadim Kochan wrote: > >>> Add new pcap io API to make pcap read/write accesses more

[netsniff-ng] Re: [PATCH 0/4] Introduce new pcap io API for pcap r/w accesses

2017-01-12 Thread Vadim Kochan
On Wed, Dec 14, 2016 at 11:33 PM, Vadim Kochan wrote: > On Wed, Dec 14, 2016 at 11:26 PM, Tobias Klauser wrote: >> On 2016-12-12 at 22:09:52 +0100, Vadim Kochan wrote: >>> Add new pcap io API to make pcap read/write accesses more >>> simpler and generic. Added pcap_io & pcap_packet struct's to >