tipc: smooth change between replicast and broadcast
authorHoang Le <hoang.h.le@dektech.com.au>
Tue, 19 Mar 2019 11:49:50 +0000 (18:49 +0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Mar 2019 20:56:17 +0000 (13:56 -0700)
commitc55c8edafa91139419ed011f7d036274ce96be0b
tree7feab866d748bdb576236c73efa81e7aae9cdb39
parentff2ebbfba6186adf3964eb816f8f255c6e664dc4
tipc: smooth change between replicast and broadcast

Currently, a multicast stream may start out using replicast, because
there are few destinations, and then it should ideally switch to
L2/broadcast IGMP/multicast when the number of destinations grows beyond
a certain limit. The opposite should happen when the number decreases
below the limit.

To eliminate the risk of message reordering caused by method change,
a sending socket must stick to a previously selected method until it
enters an idle period of 5 seconds. Means there is a 5 seconds pause
in the traffic from the sender socket.

If the sender never makes such a pause, the method will never change,
and transmission may become very inefficient as the cluster grows.

With this commit, we allow such a switch between replicast and
broadcast without any need for a traffic pause.

Solution is to send a dummy message with only the header, also with
the SYN bit set, via broadcast or replicast. For the data message,
the SYN bit is set and sending via replicast or broadcast (inverse
method with dummy).

Then, at receiving side any messages follow first SYN bit message
(data or dummy message), they will be held in deferred queue until
another pair (dummy or data message) arrived in other link.

v2: reverse christmas tree declaration

Acked-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: Hoang Le <hoang.h.le@dektech.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/bcast.c
net/tipc/bcast.h
net/tipc/msg.h
net/tipc/socket.c