Re: [Linuxptp-devel] [PATCHv2 7/9] port.c: fix port_dispatch event flood when change ts_iface info

2017-08-13 Thread Hangbin Liu
On Sat, Aug 12, 2017 at 04:41:23PM +0200, Richard Cochran wrote: > > Then after rtnl_link_status(fd, p->name, port_link_status, p); > > > > if (p->link_status == (LINK_UP | LINK_STATE_CHANGED)) > > return EV_FAULT_CLEARED; > > else if ((p->link_status == (LINK_DOWN | LINK_STATE_CHANGED)) |

Re: [Linuxptp-devel] [PATCHv2 7/9] port.c: fix port_dispatch event flood when change ts_iface info

2017-08-13 Thread Richard Cochran
On Sun, Aug 13, 2017 at 11:21:21PM +0800, Hangbin Liu wrote: > On Sat, Aug 12, 2017 at 04:41:23PM +0200, Richard Cochran wrote: > > Looks better to me, but why not place this code at the end of the > > port_link_status() function? ... > 2. If We receive one rtnl message with multi RTM_NEWLINK inf