Skip to content

Make multi-register transactions atomic

Laurent Petre requested to merge feature/atomic-multi-register-transactions into main

Description

This MR first aims at making the multi-register transactions atomic. This first applies to the GBT SC and SCA SC transactions. Among others, it should prevent the monitoring to interfere with the front-end programming.

An additional change is included in this MR to prevent all register accesses during an FPGA reload. Such a lock should prevent unexpected CTP7 reboot during configuration.

Related Issue

Closes #167 (closed).

How Has This Been Tested?

Taking runs with the monitoring enabled on the GE1/1 integration setup in b904 is seamless. Larger scale tests should be performed at p5 on the full-scale GE1/1 to make sure the previously mentioned bugs are fixed.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
Edited by Laurent Petre

Merge request reports