Re: [Linuxptp-devel] Implement push notification for TIME_STATUS_NP

2021-02-09 Thread Richard Cochran
On Mon, Jan 25, 2021 at 12:55:47PM +0100, Juergen Werner wrote: > Upon self-review, I wondered if the change in > pmc-agent.c:send_subscription() is necessary or even wanted. Feel free > to change anything before patching. Right, that isn't wanted. I applied the patch but fixed that up first. A

Re: [Linuxptp-devel] Implement push notification for TIME_STATUS_NP

2021-01-25 Thread Juergen Werner
On 23/01/2021 15.51, Richard Cochran wrote:> I reviewed the logic, and it looks correct to me. I'd like to run this, but the patch won't apply due to extra white space. > Can you please do `git format-patch -1` and send the result as an attachment? Sorry for that. I removed one empty line, s

Re: [Linuxptp-devel] Implement push notification for TIME_STATUS_NP

2021-01-23 Thread Richard Cochran
On Wed, Jan 20, 2021 at 08:33:43PM +0100, Juergen Werner wrote: > So I tried this, but I am not quite sure about the place, where the > notification is fired, namely clock_synchronization(). Please have a look. I reviewed the logic, and it looks correct to me. I'd like to run this, but the patch

[Linuxptp-devel] Implement push notification for TIME_STATUS_NP

2021-01-20 Thread Juergen Werner
Hello everyone, I am new to the list. When I asked a question on the users list Richard helped me out and suggested to implement a push notification for TIME_STATUS_NP in ptp4l: If your sync rate is really high, then you should want to implement the "push" notification from ptp4l for TIME_STATU