14 Jan
2014
14 Jan
'14
8:41 a.m.
On 13.01.2014 09:57, Holger Hans Peter Freyther wrote:
On Tue, Jan 07, 2014 at 12:07:51PM +0100, Jacob Erlbeck wrote:
+ for (i = 1; i < tcfg->number_endpoints; ++i) { + struct mgcp_endpoint *endp = &tcfg->endpoints[i]; + if (endp->bts_end.output_enabled)
Shouldn't this code look at the connection mode? E.g. the output is enabled even in SEND_RECV mode and then we certainly don't need to send dummy packets?
If (and only if) we can rely on incoming packets from any type of BTS when SEND_RECV is used, we could optimize these few packets away. Otherwise I'd rather check, whether packets have been sent to the net since the last timeout and only send a keep alive packet if there were none. Jacob