Based on a discussing that @evka85 and I had we should increase the time between when a link reset is performed and when VFAT sync error counters are checked. These counters are checked once per orbit so we should allow one or more orbits to occur by before checking those counters after a link reset.
An orbit is 9.09091E-05 seconds.
Types of issue
[ ] Bug report (report an issue with the code)
[x] Feature request (request for change which adds functionality)
we should insert a sleep statement for at least N orbits (N=1?).
We could consider also inserting a sleep statement in between each OH (e.g. between lines 806 & 807 but I'm not sure that's necessary (as 24 register reads should take at least some 200 micro-seconds?).
Current Behavior
Right after a link reset is sent we immediately start checking the SYNC_ERR_CNT registers.
Context (for feature requests)
Returned results by monitoring getmonVFATLinkLocal may not be accurate because we aren't waiting enough time for the sync error counters to increment.
Your Environment
Version used: 187a9452db5d164d16a102e9302a0dce2ad34414
jsturdy
commented
5 years ago
Brief summary of issue
Based on a discussing that @evka85 and I had we should increase the time between when a link reset is performed and when VFAT sync error counters are checked. These counters are checked once per orbit so we should allow one or more orbits to occur by before checking those counters after a link reset.
An orbit is 9.09091E-05 seconds.
Types of issue
Expected Behavior
After line 801:
https://github.com/cms-gem-daq-project/ctp7_modules/blob/187a9452db5d164d16a102e9302a0dce2ad34414/src/daq_monitor.cpp#L798-L802
we should insert a sleep statement for at least N orbits (N=1?).
We could consider also inserting a sleep statement in between each OH (e.g. between lines 806 & 807 but I'm not sure that's necessary (as 24 register reads should take at least some 200 micro-seconds?).
Current Behavior
Right after a link reset is sent we immediately start checking the
SYNC_ERR_CNTregisters.Context (for feature requests)
Returned results by monitoring getmonVFATLinkLocal may not be accurate because we aren't waiting enough time for the sync error counters to increment.
Your Environment