On Wed, Dec 08, 2021 at 02:49:14PM +0100, Eugene Crosser wrote: > rc == -1 and errno == EINTR mean: > > mnl_socket_recvfrom() - blindly rerun the function > mnl_cb_run() - restart dump request from scratch > > This commit introduces handling of both these conditions > > Signed-off-by: Eugene Crosser <crosser@xxxxxxxxxxx> > --- > src/iface.c | 73 ++++++++++++++++++++++++++++++++--------------------- > 1 file changed, 44 insertions(+), 29 deletions(-) > > diff --git a/src/iface.c b/src/iface.c > index d0e1834c..029f6476 100644 > --- a/src/iface.c > +++ b/src/iface.c > @@ -66,39 +66,54 @@ void iface_cache_update(void) > struct nlmsghdr *nlh; > struct rtgenmsg *rt; > uint32_t seq, portid; > + bool need_restart; > + int retry_count = 5; Did you ever hit this retry count? What is you daemon going to do after these retries? Probably this can be made configurable for libraries in case you prefer your daemon to give up after many retries, but, by default, I'd prefer to to keep trying until you get a consistent cache from the kernel via netlink dump.
