>
>
> > With that better estimate of delay, what do you do with the sync
> > messages? There is no filter for them that would compensate this
> > asymmetry, so I think this will make a negative impact on accuracy of
> > the clock.
>

I think there is no point in using this filter if you don't use weights
either.
But isn't that the same with the median filter ?

> If you insist on using PTP in a non-PTP network, it is better to
> > disable the delay filter, i.e. set tsproc to a raw mode.


Sometimes we don't have a choice.


> > I'd expect the default percentile to be different from 0.5 (which is
> > already the default with the default median filter).
>

I agree. I will change it.


> > Why is percentile_index here? It seems it's needed only in one
> > function.
>

I did it for optimization.
After m->cnt reaches the value of m->len, there is no need to re-calculate
m->percentile_index anymore (so the function needs to "remember" the value
of m->percentile_index).


> > I'd prefer if the median filter was a special case here to not change
> > the behavior for even lengths (the default is 10).
>

I assume that for larger lengths this is negligible, but for lower lengths
(such as 10) I can understand what you are saying.
I'll add a special case here.

Thanks for the review,
I will update the patch and resend it.
Joseph
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to