Thanks, After an offline discussion with Alex we decided that it might be worth to go one step further and avoid calling the notification callback in case of error.
This (for example) would prevent lib/route-table.c from believing that the routing table has changed on every iteration I’m about to send a v2. Please, let me know what you think Daniele On Jul 21, 2014, at 12:43 PM, Ben Pfaff <[email protected]> wrote: > On Mon, Jul 21, 2014 at 11:13:28AM -0700, Daniele Di Proietto wrote: >> An error from nl_sock_recv() could mean that there issues with the netlink >> socket (EBADF, ENOTSOCK, ...). Keeping calling nl_sock_recv() in this case is >> harmful: nln_run() will never return and, since we are calling it from the >> main >> thread, vswitchd will become unresponsive. >> >> Suggested-by: Alex Wang <[email protected]> >> Signed-off-by: Daniele Di Proietto <[email protected]> > > Acked-by: Ben Pfaff <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
