Merge lp://staging/~ralph-lange/epics-base/cac-dtor-racecond-fix into lp://staging/~epics-core/epics-base/3.14
Proposed by
Ralph Lange
| Status: | Merged |
|---|---|
| Approved by: | Andrew Johnson |
| Approved revision: | 12104 |
| Merged at revision: | 12120 |
| Proposed branch: | lp://staging/~ralph-lange/epics-base/cac-dtor-racecond-fix |
| Merge into: | lp://staging/~epics-core/epics-base/3.14 |
| Diff against target: |
51 lines (+14/-2) 2 files modified
src/ca/cac.cpp (+13/-2) src/ca/cac.h (+1/-0) |
| To merge this branch: | bzr merge lp://staging/~ralph-lange/epics-base/cac-dtor-racecond-fix |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Jeff Hill | Approve | ||
| Jeff Hill | Pending | ||
|
Review via email:
|
|||
Description of the change
While the cac dtor runs and waits for TCP circuit threads to exit,
new circuits could be opened by name resolution replies on existing
(still active) circuits.
Avoid this by setting a flag (lock being held) in the dtor, and
checking it in cac::transferCh
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
an important and necessary change