-
Notifications
You must be signed in to change notification settings - Fork 296
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gPTP not functioning expectedly with an AVB switch ! #168
Comments
Responding to #1… Is the switch acting as master? If so, the switch isn’t behaving correctly. See table 11-4 of the 801.AS standard document for the correct setting of the flags field. Incidentally, I just tested the current OpenAVB code against an Extreme switch (in the master role) and the behavior is correct. No workaround is needed. Chris From: engineerdrmz [mailto:[email protected]] Other observations:-
Thanks and Regards, — |
Hi Chris,
-Sumeet |
Your switch is in the master role since it is sending sync messages to both connected devices. Also, look at the wireshark sync frame dissection. I’m guessing that in flags PTP_TWO_STEP is not set. It should be for 802.1AS. From: Sumeet [mailto:[email protected]] Hi Chris,
-Sumeet — |
That is true. PTP_TWO_STEP isn't set. And any comments for (2) ? |
Has this issue been resolved? |
Yes @andrew-elder |
This is my observation:-
ERROR at 788 in ../../common/ptp_message.cpp: PTP assist flag is not set, discarding invalid sync
ERROR at 899 in ../../common/ptp_message.cpp: Received Follow Up but there is no sync message
I keep getting these errors. One difference I noticed between the latest and previous version of open-avb-master
is:-
(in file:- /common/ptp_message.cpp)
In latest package:- if( flags[PTP_ASSIST_BYTE] & (0x1<<PTP_ASSIST_BIT)) {.....}
In previous version:- if(true) {.....}
If I replace "if( flags[PTP_ASSIST_BYTE] & (0x1<<PTP_ASSIST_BIT))" with "if(true)", then the above errors doesn't pop up but both the machines get into gptp slave mode. Can anyone shed light on this ?
Can you please elaborate on the importance of this new modification and provide a solution for the errors I am facing using the latest open-avb-master package with an AVB switch.
PS:- My AVB switch is not AVNU certified switch.
Regards and Thanks,
Sumeet Jain
The text was updated successfully, but these errors were encountered: