Re: [External] : Re: if_etherbridge.c vs. parallel forwarding

2021-06-25 Thread Hrvoje Popovski
On 25.6.2021. 10:02, Alexandr Nedvedicky wrote: > Hello David, > > >> >> during the drive to work it occurred to me that we should basically have >> the same logic around whether we should insert or replace or do nothing >> in both the smr and mutex critical sections. >> >> it at least makes the

Re: [External] : Re: if_etherbridge.c vs. parallel forwarding

2021-06-25 Thread Alexandr Nedvedicky
Hello David, > > during the drive to work it occurred to me that we should basically have > the same logic around whether we should insert or replace or do nothing > in both the smr and mutex critical sections. > > it at least makes the code easier to understand. i think? yes, the new

Re: [External] : Re: if_etherbridge.c vs. parallel forwarding

2021-06-24 Thread David Gwynne
On Thu, Jun 24, 2021 at 09:31:20AM +0200, Alexandr Nedvedicky wrote: > Hello David, > > > > > > i think we can get away with not refcounting eb_entry structures at all. > > either they're in the etherbridge map/table or they're not, and the > > thing that takes them out of the map while holding

Re: [External] : Re: if_etherbridge.c vs. parallel forwarding

2021-06-24 Thread Alexandr Nedvedicky
Hello David, > > i think we can get away with not refcounting eb_entry structures at all. > either they're in the etherbridge map/table or they're not, and the > thing that takes them out of the map while holding the eb_lock mutex > becomes responsible for their cleanup. > > i feel like most