openBackhaul / synchronization

Synchronization model according to ITU-T G.7721-2018
Apache License 2.0
1 stars 0 forks source link

PtpSourceDataset parameters isMasterOnly and localPriority #39

Open eduardoyusta opened 2 years ago

eduardoyusta commented 2 years ago

G.7721 does not include these two parameters at the source dataset.

ITU profiles and IEEE specifications consider masterOnly just as part of the portDS (dataset at port level), not at parent/current/timeProperties DS ITU profiles consider localPriority just as part of the portDS and defaultDS

My be interesting checking if these parameters are not meaningful as part of the source dataset and can be removed.

demx8as6 commented 2 years ago

Should be analysed with Dirk....

demx8as6 commented 2 years ago

Conformation by Dirk by Email:

openBackhaul commented 2 years ago

Proposal to the 5G-xhaul call on 7th of September 2022:

In accordance to ITU-T G. 8275.1:

Nzein commented 2 years ago

We do not see a problem deleting these since these attributes are not listed in the PTP Source Dataset, even in the current final version of G.7721.  They are listed in the PTP pac in G.7721.

demx8as6 commented 2 years ago

Decision made at to the 5G-xhaul call on 7th of September 2022:

In accordance to ITU-T G. 8275.1:

The PtpSourceDataset::isMasterOnly attribute shall be deleted. The PtpSourceDataset::localPriority attribute shall be deleted.