On Tue, Aug 22, 2017 at 11:58:00AM +1000, David Mirabito wrote:
> We're seeing a case where we're seeing constant 'Temporal Vortex' messages.
> Scanning through this list's archives and reading the code it seems that
> this is related to UTC_Offset being incorrect - in this version <36,
> although should actually be 37 today.

That is right.
 
> The master is an appliance type, configured to distribute UTC, his announce
> messages look like:

>         .... .... ..1. .... = FREQUENCY_TRACEABLE: True
>         .... .... ...1 .... = TIME_TRACEABLE: True
>         .... .... .... 1... = PTP_TIMESCALE: True
>         .... .... .... .1.. = PTP_UTC_REASONABLE: True

>     originCurrentUTCOffset: 0

So your GM says that its TAI-UTC offset of zero is correct and the
time is traceable.  Brilliant.

> Which results in the local ptp4l instance interpreting it as:

No.
 
>   "GRANDMASTER_SETTINGS_NP" : {
>     "currentUtcOffset" : "36",
>     "currentUtcOffsetValid" : "0",
>     "timeTraceable" : "0",
>     "frequencyTraceable" : "0",
>     "timeSource" : "0xa0"
>   },
> 
> This seems to differ slightly from the master's announce, I assume it's
> been through some munging/cleanup/interpretation?

No.  These are the settings that would be used if ptp4l ever became
the GM.  Notice that the timeSource is local oscillator and
currentUtcOffsetValid is false.

> We also have local clock:

>   "TIME_PROPERTIES_DATA_SET" : {
>     "currentUtcOffset" : "0",
>     "currentUtcOffsetValid" : "1",
>     "ptpTimescale" : "1",
>     "timeTraceable" : "1",
>     "frequencyTraceable" : "1",
>   },
> 
> And I understand it is this timepropertiesDS.currentUTC which is compared
> to current-utc-offset causing the warning.

Yes.

> Reading 1588 sections 13.3.1, 8.2.4 and 7.2 I get the impression that the
> announce is incorrect in how it goes about advertising UTC: it claims
> PTP_TIMESCALE but also sends time in UTC with an UTCOffset of zero.

Right.

> My reading is that in a perfect world, this master would clear the
> PTP_TIMESCALE bit, placing us in arbitrary timescale and we can just decree
> that the resulting time is UTC (ignoring the offset, if it's ever nonzero?).

No.  The currentUtcOffset field stands alone and is independent of the
PTP_TIMESCALE bit.

The GM has two choices WRT to currentUtcOffset and
currentUtcOffsetValid.  Either it knows the offset from a global
source like GPS, in which case it sets the correct offset and the
valid flag, or it clears the valid flag, thus marking the valid field
as bogus.
 
> Given ptp4l 1.8, is there any way we can allow folks to lessen noise in the
> logs?

There is no noise, because the message appears only once.  It is a
useful warning to the sysadmin.

> I'm not adverse to patching the code but would rather not mute the message
> altogether as I seems valuable to catch non-intentional weirdness which may
> not happen to fall out nicely.

Right.

> Is this check performed only on BMC selection?

Yes.

> We also see regular new BMC
> selections, always picking the same master, presumably the master is often
> re-evaluating it's accuracy as satellites come in/out of view, etc.

Right.

Thanks,
Richard

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-users mailing list
Linuxptp-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-users

Reply via email to