On Tue, Oct 05, 2021 at 12:27:14PM +0200, Miroslav Lichvar wrote: > On Mon, Oct 04, 2021 at 06:56:50AM -0700, Richard Cochran wrote: > > How about this? > > > > 1. Refactor the setting of clock.tds into a common helper function, as > > you suggest. > > > > 2. Provide a global function to clear the LS flags, called from > > port_tx_announce explicitly. > > > > This could be clock_update_utc_offset(), but the name might suggest > > that the clock would be stepped. Maybe clock_clear_leap_flags() > > instead? > > Or maybe clock_update_leap_status()? I have no strong preference for > any of the three.
okay. It would make even more sense to me to call it from the timer expiration block, like so: case FD_MANNO_TIMER: pr_debug("%s: master tx announce timeout", p->log_name); port_set_manno_tmo(p); + clock_update_leap_status(p->clock); return port_tx_announce(p, NULL) ? EV_FAULT_DETECTED : EV_NONE; > > 3. And BTW, the naming of clock_update_grandmaster() isn't great. > > Long ago I couldn't think of a better name at the time. It sounds > > like it would update something about a remote GM. > > I think it's consistent with clock_update_slave(). If one is renamed, > should the other follow? Good point. Let's leave them alone then. Thanks, Richard _______________________________________________ Linuxptp-devel mailing list Linuxptp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/linuxptp-devel