> The math in the tsproc code was supposed to be identical to the code
> it replaced. t2 and t3 in tsproc are local times. The mapping between
> the port and tsproc timestamps is (t1, t2, t3, t4) -> (t3, t4, t1, t2).
Oh, tricky! Now I vaguely recall that from the original review.
> Maybe it
We figure the neighbor rate ratio in port_nrate_calculate():
delta t3 / delta t4 = peer_interval / local_interval
Originally the peer delay was figured like this:
pd = tmv_sub(tmv_sub(t4, t1), tmv_sub(t3, t2));
Since t4-t1 are local, and t3-t2 are from the peer, there will
Hello Richard,
Thank you for spotting this issue. We haven’t been hit by it (yet!) because we
use an old version of linuxptp. I hope to be able to move to a more recent
release at some point. The fix you propose seems good to me.
Thanks!
--
Delio Brignoli
AudioScience Inc
On 22 Oct 2015, at