xl4-shiro / excelfore-gptp

gPTP, IEEE 802.1AS-REV implementation
GNU General Public License v2.0
60 stars 38 forks source link

10Mbps speed support #14

Closed duribee closed 1 year ago

duribee commented 1 year ago

Hello,

Is there any reason why speed above 100 Mbps is needed for gptp2d to run? I was trying to execute the stack using a 10Mbps interface but the following warning was displayed:

WRN:gptp:!!! Full duplex link with Speed above 100 Mbps needed for gptp to run !!!

Also, is there any way to pass integer arrays using the configuration file? For example if we want to configure a CONF_LOG_SYNC_INTERVAL independently for every port, what would be your recomendation?

Best regards, Daniel

xl4-shiro commented 1 year ago

100Mbps comes from 802.1AS standard.

The config doesn't have an array setting, CONF_PTP_PORT_ENABLED is using comma separated values as a string. For such needs, we use xl4unipac which is also in our github.

duribee commented 1 year ago

I've been trying to find the requirements of link speeds of 100Mbps or more at the 802.1AS-2020 standard but I do not find it. Could you please point me to the section where it is defined?

Is there any specific reason that could prevent the gptp2d stack to work at 10Mbps? could it be possible to remove this condition and the allow also 10Mbps speed?

if(ed->speed<100 || ed->duplex!=1){ UB_LOG(UBL_WARN,"!!! Full duplex link with " "Speed above 100 Mbps needed for gptp to run !!!\n"); }

Any advice is appreciated.

Thank you!

xl4-shiro commented 1 year ago

I don't remember the exact place to find the minimum 100Mbps condition. It might come to achieve meanLinkDelayThresh.

If the meanLinkDelayThresh was not needed to be cared for, it might be okay to remove the condition.

Message ID: @.***>

duribee commented 1 year ago

Thank you for the quick support.