Project

General

Profile

Actions
Copy link

Bug #24306

closed

mds: use intrusive_ptr to manage Message life-time

Added by Patrick Donnelly almost 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Patrick Donnelly
Category:
Correctness/Safety
Target version:
Ceph - v14.0.0
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
task(easy), task(intern)
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

We're regularly getting bugs relating to messages not getting released. Latest one is #24289.

Use a boost::intrusive_ptr to manage the lifetime.

Related issues 4 (2 open — 2 closed)

Related to CephFS - Bug #24289: mds memory leakResolved05/25/2018

Actions
Related to RADOS - Fix #3500: mon: use new MessageRef and intrusive_ptr's throughout the Monitor's message handlingNew11/16/2012

Actions
Related to CephFS - Cleanup #26960: mds: avoid modification of const MessagesNewNeeraj Pratap Singh

Actions
Related to CephFS - Cleanup #37864: client: use Message smart ptr to manage Message lifetimeResolvedPatrick Donnelly

Actions
Actions
Copy link
 #1

Updated by Patrick Donnelly almost 6 years ago

Actions
Copy link
 #2

Updated by Patrick Donnelly almost 6 years ago

  • Status changed from New to In Progress
  • Assignee set to Patrick Donnelly
Actions
Copy link
 #4

Updated by Patrick Donnelly almost 6 years ago

  • Related to Fix #3500: mon: use new MessageRef and intrusive_ptr's throughout the Monitor's message handling added
Actions
Copy link
 #5

Updated by Patrick Donnelly over 5 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link
 #6

Updated by Patrick Donnelly over 5 years ago

  • Related to Cleanup #26960: mds: avoid modification of const Messages added
Actions
Copy link
 #7

Updated by Patrick Donnelly over 5 years ago

  • Related to Cleanup #37864: client: use Message smart ptr to manage Message lifetime added
Actions
Copy link

Also available in: Atom PDF