When /etc/init.d/ppsi.sh is called with the parameter 'restart', it
happens sometimes that the restarted PPSI process detects that another
one is running but it is not really the case. To  sove this problem, the
PPSI process is now  checking  few times with a delay in between before
to decide to exit.

When a jitter greater than 20ms is observed with peer, the seconds are
never adjusted. The solution is then to adjust first the seconds.

Instead of restarting a timer on each retry, the timer is initialized
only once when we enter in the state. It helps to respect the time spent
in a given state before to skip it due to the time-out.

For a WR slave, it will force the recetion of a new ANNOUNCE message
before to jump from SLAVE to WR_PRESENT state.

per Veryx request, they cannot do less than 100ms

the names correspond to the info provide to Veryx for testing

Conflicts:
	include/ppsi/constants.h

As we can get know the WR time (IRPC call) we use it instead of using
the T4 time aproximation. Making some tests, the difference between Unix
time and WR time is around 30us (was 20ms before with T4)

- Update of portDs.meanLinkDelay for P2P mechanism
- Fix issue updating currentDS.meanDelay

- In order to increase the code readibility, the p2p and e2e functions has
been splitted into differents files.
- Protect P2P from multiple answers coming from different hosts.

According to the clause 9.3.2.3, the qualification is now not
evaluated for the EBest foreign master. EBest is always considered
qualified.

This new parameter is used to control the configurable parameter
L1SyncOptParamsPortDS.timestampsCorrectedTx when
L1SyncBasicPortDS.optParamsEnabled is set to true.

The delayCoefficient can be set in ppsi.conf file either with the
delayCoefficient (double) parameter or the scaledDelayCoefficient
(RelativeDifference)  parameter. In ppsi, the priority is given to the
scaledDelayCoefficient.

When the extension L1SYNC is not available (no data), the value of
L1SyncEnable and L1SyncState are emulated and displayed. Some VERYX
tests need these information also when L1SYNC is disabled.

Due to changes in the standard (Clause 17.6.5), the slaveOnly parameter
must be forced to FALSE when externalPortConfiguration is enabled.

- When optParamsEnabled is set, the TLV send need to be extended with
extra values. Fix here the length of the TLV and the whole message.
- Fix print issue of the signaling TLV

- Add DelayAsymmetty calculation for HA
- Avoid using pow() math function call. Use directly a constant value.
Done for HA and WR.

- The ntohx() is now used to unpack messages and htohx() to pack them.
Most of the time only htohx() was used for paking and unpacking
messages.
- Code optimization

On little indian architecture, the calculation of the correction field
was wrong in the PDelay_Resp_FollowUp message. This was introducing a
big random offset (few us) of the PPS.

Fix warning introduced in the commit:
Implement one-step clock for HA & WR profiles

Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

- Many information were store in a common static structure (to_config).
However depending on the PPSI instance configuration these information
need to be stored per instance. Mainly it concerns the rand value
(which_rand and the counter reset value). Only the timer name stay now
in a common arrea.

Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

    According to the clause 9.3.2.3, the qualification is now not
evaluated for the ErBest foreign master.
    On ANN_RECEIPT timout, the ErBest is removed from the list of
foreign master to avoid to jump immediately to SLAVE state

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

    - Add new config parameters (l1SyncRxCoherencyIsRequired,...)
    - Add new profile: CUSTOM. It will be mostly used for testing
    - Fix issue on state name: "pre_master" instead of "pre-master"

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

- Add new config parameters (l1SyncRxCoherencyIsRequired,...)
- Add new profile: CUSTOM. It will be mostly used for testing
- Fix issue on state name: "pre_master" instead of "pre-master"

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

According to the clause 9.3.2.3, the quualification is now not evaluated
for the ErBest foreign master.

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

The one-step clock didn't work for HA & WR profiles. This new
implementation is based on a new hook "handle_sync" called in case of
one-step clock mode.

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

- Fix issues introduced in last commit "Timeout management correction"
- Code optimization
- Protect FAULT timeout in case of timeout overload. It can append
because the logMinDelayResponse can be changed on real-time by the
received DELAY_RESP message

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

In the ptp servo the adjustment of the time was too slow. When the
difference of the time with the master was less than 1 second, the
adjustment was done only with the frequency. In case if the difference
is 0.999s, it can take a very long time to adjust the time.
Changes made :
- Adjust with frequency only when time difference is below 20ms
- Check the timestamp validity before each calculation
- Create a new function to check the timestamps and use it on all
servos. This improve significantly the size of the code.
- Add verification in order not to set an invalid UNIX time

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

Change the default value of "alarmDetected" to force execution of fsm
the first time.

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

When received frames are displayed (diagnostic mode), PPSI was staying
forever in a loop due to unexpected TLV received in the frame.

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

This was due to the introduction of system timers in the main loop of
the switch. In minirpc the pool() system call exited sometime when the
timer fired. Some other problems have been found during the debugging
process. Some of them have been already adressed :
- In minirpc, loop on pool() system call if the call is interrupted.
- Wrong check in function wrs_locking_poll(). Was working by chance.
- L12_state_machine() was calling to many often the state handle. It was
increasing the number of hardware calls for nothing.

TODO: The returned values for the function wrs_locking_poll(), halexp_lock_cmd(), and hal_port_check_lock() should be reviewed to be more consistent.

based on jcb's commit from wrs

Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

- This function checks now the minimum and maximum values which can be
encoded into a RelativeDifference type.
- Bux fix for the calculation of min/max value conversion (special
cases)

Jean-Claude BAU authored 6 years ago and Adam Wujek committed 6 years ago

- The origin timestamp sent in SYNC and DELAY_REQ messages is corrected
with the egressTimestampLatency.