Wednesday, March 21, 2018 6:48 PM

>alphabetical order please

Ok, I will change. I usually put it in the order it is implemented in code.


> How about introducing a helper to prune the list.

Added delay_req_prune and delay_req_current


>You only prune the list when a new request arrives.  Is that
>sufficient or should a timer (like the sync timer) also be used?

Added prune before new delay_req is sent and if announce receipt tmo expires


>This doesn't handle the case out of order packets.  Use the "prune"
>helper here, too.

I removed the old obsolete check, but I did not add prune here. Do not find it 
necessary since it is pruned before each delay_req and if announce expires.


The prune function made the solution much cleaner.
Thanks for comments once again.
Anders

------------------------------------------------------------------------------
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