On Sun, Mar 17, 2019 at 11:11:10AM +0100, Sven Eckelmann wrote:
I am also not sure what in mind here for the limit? The only thing which comes to my mind right now is that we have a fanout of something low (16) and the precondition test succeeded because it can only find 16 entries. And in the meantime, 100 or more new entries are added. In this situation, you would only send to 32 neighbors, right?
Right, that's the situation I had in mind. If the number of transmissions were growing too large somehow to limit them.
On the other hand, now that you reiterate this maybe this is such a minor case that it is not worth bothering for now.
Hm, I think I'll remove these intermediate limit checks to simplify things. So that we only check once in batadv_mcast_forw_mode() for the mcast_fanout.