Project

General

Profile

Bug #15528

rgw: leaks RGWStatObjCR

Added by Sage Weil almost 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
-
Target version:
-
% Done:

0%

Source:
Q/A
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

  <kind>Leak_DefinitelyLost</kind>
  <xwhat>
    <text>172,653 (100,352 direct, 72,301 indirect) bytes in 64 blocks are definitely lost in loss record 379 of 381</text>
    <leakedbytes>172653</leakedbytes>
    <leakedblocks>64</leakedblocks>
  </xwhat>
  <stack>
    <frame>
      <ip>0x4C2B0E0</ip>
      <obj>/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so</obj>
      <fn>operator new(unsigned long)</fn>
    </frame>
    <frame>
      <ip>0x5167C52</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>(anonymous namespace)::FindAnyShardCR::operate()</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_metadata.cc</file>
      <line>387</line>
    </frame>
    <frame>
      <ip>0x50FCA3A</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>RGWCoroutinesStack::operate(RGWCoroutinesEnv*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_coroutine.cc</file>
      <line>187</line>
    </frame>
    <frame>
      <ip>0x50FE8DA</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>RGWCoroutinesManager::run(std::list&lt;RGWCoroutinesStack*, std::allocator&lt;RGWCoroutinesStack*&gt; &gt;&amp;)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_coroutine.cc</file>
      <line>468</line>
    </frame>
    <frame>
      <ip>0x50FF44C</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>RGWCoroutinesManager::run(RGWCoroutine*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_coroutine.cc</file>
      <line>595</line>
    </frame>
    <frame>
      <ip>0x516B92E</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>find_shards_for_period</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_metadata.cc</file>
      <line>418</line>
    </frame>
    <frame>
      <ip>0x516B92E</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>(anonymous namespace)::find_oldest_log_period(RGWRados*)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_metadata.cc</file>
      <line>435</line>
    </frame>
    <frame>
      <ip>0x516C374</ip>
      <obj>/usr/lib/librgw.so.2.0.0</obj>
      <fn>RGWMetadataManager::init(std::string const&amp;)</fn>
      <dir>/srv/autobuild-ceph/gitbuilder.git/build/out~/ceph-10.1.2-193-gf0e3b61/src/rgw</dir>
      <file>rgw_metadata.cc</file>
      <line>482</line>
    </frame>
    <frame>

/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134502

History

#1 Updated by Sage Weil almost 8 years ago

/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134490
/a/sage-2016-04-16_08:41:53-rgw-master---basic-smithi/134526

#2 Updated by Yehuda Sadeh almost 8 years ago

There are a couple of other leaks there as well, some related to extra reference taken (introduced in 1a0e74bbd1e81d483a90d37ab088744c4aa7c788), and the error handling.

#3 Updated by Yehuda Sadeh almost 8 years ago

looking at it a bit more, the change in that commit seems ok. The refcounting seems a bit off at the async cr code.

#4 Updated by Yehuda Sadeh almost 8 years ago

  • Status changed from New to Resolved

multiple leak fixes went into 10.2.1, this one is included.

Also available in: Atom PDF