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

2017-01-17 Thread Tobias Klauser
On 2017-01-13 at 12:06:07 +0100, Vadim Kochan wrote: > 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 > addition

[netsniff-ng] Re: list: Remove cds_list_* wrappers

2017-01-17 Thread Vadim Kochan
Hi Tobias, Just some thoughts from me regarding this commit, if the below can make a sense. In case if list_head (now cds_list_head) will be used by other modules and lets imagine that we will get rid of liburcu dependency (for example if we can make flowtop single threaded) then we will need to

[netsniff-ng] Re: list: Remove cds_list_* wrappers

2017-01-17 Thread Vadim Kochan
On Tue, Jan 17, 2017 at 5:30 PM, Vadim Kochan wrote: > Hi Tobias, > > Just some thoughts from me regarding this commit, if the below > can make a sense. > > In case if list_head (now cds_list_head) will be used by other modules > and lets imagine that we will get rid of liburcu dependency (for exa

[netsniff-ng] Re: list: Remove cds_list_* wrappers

2017-01-17 Thread Tobias Klauser
On 2017-01-17 at 16:36:53 +0100, Vadim Kochan wrote: > On Tue, Jan 17, 2017 at 5:30 PM, Vadim Kochan wrote: > > Hi Tobias, > > > > Just some thoughts from me regarding this commit, if the below > > can make a sense. > > > > In case if list_head (now cds_list_head) will be used by other modules >