[Bug 1953431] Re: [SRU] fix unsafe access in unregister_conn()

[gerald.yang]

** Description changed:

  [Impact]
  without holding sufficient locks, accept_conn and shutdown_connections in AsyncMessenger could cause OSD processes crash
  
- [Test case]
+ [Test plan]
+ 
+ [Where problems could occur]
+ hold sufficient locks and decrement l_msgr_active_connections can avoid OSD processes to crash due to race condition
+ The only problem I can imagine is deadlock if one process is holding lock A and wait for lock B, the there is another process holding lock B and wait for lock A
  
  
- [Regression potential]
- low, hold sufficient locks and decrement l_msgr_active_connections shouldn't cause any regressions
- 
- [Additional information] 
+ [Other info]
  from upstream:
  
  octopus backport tracker
  https://tracker.ceph.com/issues/50482
  
  octopus backport PR
  https://github.com/ceph/ceph/pull/43325

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to ceph in Ubuntu.
https://bugs.launchpad.net/bugs/1953431

Title:
  [SRU] fix unsafe access in unregister_conn()

Status in ceph package in Ubuntu:
  In Progress

Bug description:
  [Impact]
  without holding sufficient locks, accept_conn and shutdown_connections in AsyncMessenger could cause OSD processes crash

  [Test plan]

  [Where problems could occur]
  hold sufficient locks and decrement l_msgr_active_connections can avoid OSD processes to crash due to race condition
  The only problem I can imagine is deadlock if one process is holding lock A and wait for lock B, the there is another process holding lock B and wait for lock A

  
  [Other info]
  from upstream:

  octopus backport tracker
  https://tracker.ceph.com/issues/50482

  octopus backport PR
  https://github.com/ceph/ceph/pull/43325

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ceph/+bug/1953431/+subscriptions