On Thu, Dec 07, 2017 at 10:13:07AM +0100, Miroslav Lichvar wrote:
> The trouble is that forwarding() is called twice from
> clock_forward_mgmt_msg(), once with the source port and then with the
> destination port. So, if it returned 0 for p == uds->port and action
> == REQUEST, requests from pmc would not be forwarded.

Oh.

> Should it pass NULL as a msg in the first call to avoid the action check?

Maybe the check should go into clock_do_forward_mgmt() instead?

That looks cleaner to me.


Thanks,
Richard

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to