[kronosnet/kronosnet] c600da: [rx] send reply packets only when transport is con...

Branch: refs/heads/sctp-fixes Home: https://github.com/kronosnet/kronosnet Commit: c600da60dfbc67efa98ca412e7b29394aa125fa5 https://github.com/kronosnet/kronosnet/commit/c600da60dfbc67efa98ca412e7b29… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2020-01-30 (Thu, 30 Jan 2020) Changed paths: M libknet/threads_rx.c Log Message: ----------- [rx] send reply packets only when transport is connected Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Commit: 60a5e43f4809b8589ca2194ecb055ac56c4d2c22 https://github.com/kronosnet/kronosnet/commit/60a5e43f4809b8589ca2194ecb055… Author: Christine Caulfield <ccaulfie(a)redhat.com> Date: 2020-01-30 (Thu, 30 Jan 2020) Changed paths: M libknet/transport_common.c M libknet/transport_sctp.c Log Message: ----------- [rx] Don't return 512 EOF messages from _recvmmsg If recvmsg() returns 0 for EOF then it's going to do so until the error is rectified or read with getsockopt(). But the _recvmmsg() wrapper keeps reading until the vector is full thus returning a block of 512 EOF messages all of which the caller has to plough through. This patch causes _recvmmsg() to return as soon as it has got the first EOF so the the caller can deal with it in good time and not spin looking at the same thing over and over again. I've also fixed a couple of typos in related comments Commit: 92be1fa628f34461ccbdaa09a8bfe335dd78430f https://github.com/kronosnet/kronosnet/commit/92be1fa628f34461ccbdaa09a8bfe… Author: Christine Caulfield <ccaulfie(a)redhat.com> Date: 2020-01-30 (Thu, 30 Jan 2020) Changed paths: M libknet/transport_common.c Log Message: ----------- [tx] Don't Clear out msghdr for all transports. When sending a message to multiple links, if one of those links is not connection-oriented then msg_name & msg_namelen would be cleared, thus breaking the send to any subsequent non-connection-oriented links. So now, if we need to clear out msg_name & msg_namelen, we take a copy of the msghdr and edit that instead, Commit: e8ba24df9a84dac8b026a34061c9fef9ceb0ecb9 https://github.com/kronosnet/kronosnet/commit/e8ba24df9a84dac8b026a34061c9f… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2020-01-30 (Thu, 30 Jan 2020) Changed paths: M libknet/threads_rx.c M libknet/transport_udp.c M libknet/transports.h Log Message: ----------- [rx] use defines to determine RX data types vs random numbers also extend a bit to make ready for SCTP extra return codes Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Commit: 10fb11831a4d2200187ed5dcf654102f9e30fe40 https://github.com/kronosnet/kronosnet/commit/10fb11831a4d2200187ed5dcf6541… Author: Fabio M. Di Nitto <fdinitto(a)redhat.com> Date: 2020-01-30 (Thu, 30 Jan 2020) Changed paths: M README M libknet/transport_sctp.c Log Message: ----------- [sctp] major surgery to use only SCTP events to determine socket status - drop concept of on_connected_epoll to determine if socket is ready or not - provide much better debugging output at all levels - incorporate fix from Xin Long <lxin(a)redhat.com> to gather socket status at the right time - deal with a recent kernel change on SCTP socket that broke knet (from rhel7): [net] sctp: allow delivering notifications after receiving SHUTDOWN Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com> Compare: https://github.com/kronosnet/kronosnet/compare/22fc181020e3...10fb11831a4d