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

[Ubuntu Foundations Team Bug Bot]

The attachment "0001-msgr-async-fix-unsafe-access-in-
unregister_conn.patch" seems to be a patch.  If it isn't, please remove
the "patch" flag from the attachment, remove the "patch" tag, and if you
are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by
~brian-murray, for any issues please contact him.]

** Tags added: patch

-- 
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
Status in ceph source package in Focal:
  In Progress

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

  [Test plan]
  run ceph_test_async_networkstack repeatedly, this will trigger many async messenger events and make sure
  1. OSD process not crash
  2. no deadlock

  [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