On Sunday, September 24, 2023 12:47:34 PM CET Sven Eckelmann wrote:
@Simon, can you please also check the remaining code changes? To quickly identify modifications, you can use
pipx install b4 # in you batman-adv repo b4 diff -- 20230907010910.22427-2-linus.luessing@c0d3.blue
Sorry for the delay, this actually slipped through the cracks on my end ... thanks Sven for reminding.
I've checked especially against the points I raised last time. I think batadv_mcast_forw_shrink_pack_dests() looks much better now. batadv_mcast_forw_scrub_dests() is still quite hard to comprehend, but better than before - and good enough to be adopted in my opinion. We'll make sure to direct bug reports from that to Linus in any case. ;)
So I'd give for this series:
Acked-by: Simon Wunderlich sw@simonwunderlich.de
Thank you! Simon