Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Race condition in ipbus_syncreg_v #189

Closed
dmnewbold opened this issue Mar 15, 2021 · 0 comments · Fixed by #202
Closed

Race condition in ipbus_syncreg_v #189

dmnewbold opened this issue Mar 15, 2021 · 0 comments · Fixed by #202
Assignees
@dmnewbold
Labels
bug enhancement
Milestone
Release 1.11

Comments

@dmnewbold
Copy link
Collaborator

Under rare circumstances, it seems possible to provoke a race condition inside the ipbus_syncreg_v CDC logic. In addition, the syncreg principles of operation should be documented alongside the code, to avoid reverse-engineering or misassumptions.
@dmnewbold dmnewbold self-assigned this Mar 15, 2021
@alessandrothea alessandrothea mentioned this issue Oct 1, 2021
Merged
alessandrothea pushed a commit that referenced this issue Oct 1, 2021
@dmnewbold @alessandrothea
#189: Fix to syncreg design
fc75ea3
alessandrothea pushed a commit that referenced this issue Oct 1, 2021
@dmnewbold @alessandrothea
#189: Fixes to syncreg
464a50e
alessandrothea pushed a commit that referenced this issue Oct 1, 2021
@dmnewbold @alessandrothea
#189: syncreg fixes
296ade8
alessandrothea added a commit that referenced this issue Oct 2, 2021
@alessandrothea
#189: Exposing run-sim-slave-counter errors
fc48cbf
alessandrothea added a commit that referenced this issue Oct 2, 2021
@alessandrothea
#189: Setting "stop at first error" flag in most of test scripts
03daa96
alessandrothea pushed a commit that referenced this issue Dec 18, 2021
@dmnewbold @alessandrothea
#189: Fix to syncreg design
29e3efa
alessandrothea pushed a commit that referenced this issue Dec 18, 2021
@dmnewbold @alessandrothea
#189: Fixes to syncreg
842100d
alessandrothea pushed a commit that referenced this issue Dec 18, 2021
@dmnewbold @alessandrothea
#189: syncreg fixes
9b09d3a
alessandrothea added a commit that referenced this issue Dec 18, 2021
@alessandrothea
#189: Exposing run-sim-slave-counter errors
2b5ac74
alessandrothea added a commit that referenced this issue Dec 18, 2021
@alessandrothea
#189: Setting "stop at first error" flag in most of test scripts
7f71fd2
alessandrothea added a commit that referenced this issue Dec 18, 2021
@alessandrothea
applying proper boolean to std_logic conversion when using ULTRA_SAFE #…
b5d9755 
…189
alessandrothea added a commit that referenced this issue Dec 18, 2021
@alessandrothea
Adding missing round bracket. #189
ca9d7a8
alessandrothea added a commit that referenced this issue Jan 26, 2022
@alessandrothea
Minor syntax fix to syncreg #189
c28284c
alessandrothea added a commit that referenced this issue Jan 26, 2022
@alessandrothea
Minor syntax fix to syncreg #189
1a2ddc7
alessandrothea added a commit that referenced this issue Jan 26, 2022
@alessandrothea
Small fix to script paths in CI #189
2bd059e
alessandrothea added a commit that referenced this issue Jan 26, 2022
@alessandrothea
tests: adding addrtab retrival. #189
f53f638
alessandrothea added a commit that referenced this issue Jan 27, 2022
@alessandrothea
syncreg_r: cleaner use of ULTRA_SAFE #189
568ebeb
alessandrothea added a commit that referenced this issue Jan 27, 2022
@alessandrothea
syncreg_r: cleaner use of ULTRA_SAFE #189
acfaee3
@tswilliams tswilliams added this to the Release 1.11 milestone May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmnewbold dmnewbold

Labels
bug enhancement
Projects
None yet
Milestone
Release 1.11
Development

Successfully merging a pull request may close this issue.

Syncreg consolidation ipbus/ipbus-firmware
2 participants
@tswilliams @dmnewbold