Project

General

Profile

Bug #17492

MgrMonitor: valgrind UninitCondition failures

Added by Greg Farnum almost 3 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
-
Category:
MgrMonitor
Target version:
-
Start date:
10/04/2016
Due date:
% Done:

0%

Source:
Development
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

This is triggering failures in our valgrind tests. Among others, http://qa-proxy.ceph.com/teuthology/gregf-2016-10-03_22:11:37-fs-wip-getuid-test3---basic-mira/451761/

<error>
  <unique>0x2</unique>
  <tid>9</tid>
  <threadname>safe_timer</threadname>
  <kind>UninitCondition</kind>
  <what>Conditional jump or move depends on uninitialised value(s)</what>
  <stack>
    <frame>
      <ip>0x4B36C1</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>MgrMonitor::promote_standby()</fn>
    </frame>
    <frame>
      <ip>0x4B6CE3</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>MgrMonitor::tick()</fn>
    </frame>
    <frame>
      <ip>0x3D5597</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Monitor::tick()</fn>
    </frame>
    <frame>
      <ip>0x3EA908</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Context::complete(int)</fn>
    </frame>
    <frame>
      <ip>0x5A455B</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>SafeTimer::timer_thread()</fn>
    </frame>
    <frame>
      <ip>0x5A5EFC</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>SafeTimerThread::entry()</fn>
    </frame>
    <frame>
      <ip>0xB3E5181</ip>
      <obj>/lib/x86_64-linux-gnu/libpthread-2.19.so</obj>
      <fn>start_thread</fn>
      <dir>/build/buildd/eglibc-2.19/nptl</dir>
      <file>pthread_create.c</file>
      <line>312</line>
    </frame>
    <frame>
      <ip>0xC48C47C</ip>
      <obj>/lib/x86_64-linux-gnu/libc-2.19.so</obj>
      <fn>clone</fn>
      <dir>/build/buildd/eglibc-2.19/misc/../sysdeps/unix/sysv/linux/x86_64</dir>
      <file>clone.S</file>
      <line>111</line>
    </frame>
  </stack>

And from http://qa-proxy.ceph.com/teuthology/gregf-2016-10-03_22:11:37-fs-wip-getuid-test3---basic-mira/451703/:

<error>
  <unique>0x1</unique>
  <tid>5</tid>
  <threadname>fn_monstore</threadname>
  <kind>UninitValue</kind>
  <what>Use of uninitialised value of size 8</what>
  <stack>
    <frame>
      <ip>0xA7CC704</ip>
      <obj>/usr/lib64/libleveldb.so.1.0.7</obj>
      <fn>leveldb::crc32c::Extend(unsigned int, char const*, unsigned long)</fn>
    </frame>
    <frame>
      <ip>0xA7B2904</ip>
      <obj>/usr/lib64/libleveldb.so.1.0.7</obj>
      <fn>leveldb::log::Writer::EmitPhysicalRecord(leveldb::log::RecordType, char const*, unsigned long)</fn>
    </frame>
    <frame>
      <ip>0xA7B2B35</ip>
      <obj>/usr/lib64/libleveldb.so.1.0.7</obj>
      <fn>leveldb::log::Writer::AddRecord(leveldb::Slice const&amp;)</fn>
    </frame>
    <frame>
      <ip>0xA7A9F38</ip>
      <obj>/usr/lib64/libleveldb.so.1.0.7</obj>
      <fn>leveldb::DBImpl::Write(leveldb::WriteOptions const&amp;, leveldb::WriteBatch*)</fn>
    </frame>
    <frame>
      <ip>0x53F82C</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>LevelDBStore::submit_transaction_sync(std::shared_ptr&lt;KeyValueDB::TransactionImpl&gt;)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/kv</dir>
      <file>LevelDBStore.cc</file>
      <line>188</line>
    </frame>
    <frame>
      <ip>0x3AA46E</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>MonitorDBStore::apply_transaction(std::shared_ptr&lt;MonitorDBStore::Transaction&gt;)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>MonitorDBStore.h</file>
      <line>297</line>
    </frame>
    <frame>
      <ip>0x411C33</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Paxos::begin(ceph::buffer::list&amp;)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>Paxos.cc</file>
      <line>706</line>
    </frame>
    <frame>
      <ip>0x412758</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Paxos::propose_pending()</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>Paxos.cc</file>
      <line>1551</line>
    </frame>
    <frame>
      <ip>0x416853</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Paxos::finish_round()</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>Paxos.cc</file>
      <line>1108</line>
    </frame>
    <frame>
      <ip>0x418B6D</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Paxos::commit_finish()</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>Paxos.cc</file>
      <line>966</line>
    </frame>
    <frame>
      <ip>0x41BF8A</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>C_Committed::finish(int)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>Paxos.cc</file>
      <line>862</line>
    </frame>
    <frame>
      <ip>0x3EBD98</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Context::complete(int)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/include</dir>
      <file>Context.h</file>
      <line>64</line>
    </frame>
    <frame>
      <ip>0x41A8B6</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>MonitorDBStore::C_DoTransaction::finish(int)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/mon</dir>
      <file>MonitorDBStore.h</file>
      <line>341</line>
    </frame>
    <frame>
      <ip>0x3EBD98</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Context::complete(int)</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/include</dir>
      <file>Context.h</file>
      <line>64</line>
    </frame>
    <frame>
      <ip>0x5B4C25</ip>
      <obj>/usr/bin/ceph-mon</obj>
      <fn>Finisher::finisher_thread_entry()</fn>
      <dir>/usr/src/debug/ceph-11.0.0/src/common</dir>
      <file>Finisher.cc</file>
      <line>68</line>
    </frame>
    <frame>
      <ip>0xD213DC4</ip>
      <obj>/usr/lib64/libpthread-2.17.so</obj>
      <fn>start_thread</fn>
    </frame>
    <frame>
      <ip>0xDD3E28C</ip>
      <obj>/usr/lib64/libc-2.17.so</obj>
      <fn>clone</fn>
    </frame>
  </stack>
</error>

History

#1 Updated by Sage Weil almost 3 years ago

  • Status changed from New to Need Review

#2 Updated by John Spray almost 3 years ago

  • Status changed from Need Review to Resolved

Also available in: Atom PDF