Project

General

Profile

Bug #23961

valgrind reports UninitCondition in osd PG::RecoveryState::Incomplete::react(PG::AdvMap const)

Added by Kefu Chai over 1 year ago. Updated over 1 year ago.

Status:
Duplicate
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
05/02/2018
Due date:
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Pull request ID:

Description

<error>
  <unique>0x0</unique>
  <tid>59</tid>
  <threadname>tp_osd_tp</threadname>
  <kind>UninitCondition</kind>
  <what>Conditional jump or move depends on uninitialised value(s)</what>
  <stack>
    <frame>
      <ip>0x567E3C</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>PG::RecoveryState::Incomplete::react(PG::AdvMap const&amp;)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>PG.cc</file>
      <line>8962</line>
    </frame>
    <frame>
      <ip>0x5C0D60</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>react&lt;PG::RecoveryState::Incomplete, boost::statechart::event_base, void const*&gt;</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>custom_reaction.hpp</file>
      <line>41</line>
    </frame>
    <frame>
      <ip>0x5C0D60</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>local_react_impl&lt;boost::mpl::list&lt;boost::statechart::custom_reaction&lt;PG::AdvMap&gt;, boost::statechart::custom_reaction&lt;MNotifyRec&gt;, boost::statechart::custom_reaction&lt;PG::QueryState&gt; &gt;, boost::statechart::simple_state&lt;PG::RecoveryState::Incomplete, PG::RecoveryState::Peering, boost::mpl::list&lt;mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na&gt;, (boost::statechart::history_mode)0&gt; &gt;</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>simple_state.hpp</file>
      <line>813</line>
    </frame>
    <frame>
      <ip>0x5C0D60</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>local_react&lt;boost::mpl::list&lt;boost::statechart::custom_reaction&lt;PG::AdvMap&gt;, boost::statechart::custom_reaction&lt;MNotifyRec&gt;, boost::statechart::custom_reaction&lt;PG::QueryState&gt; &gt; &gt;</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>simple_state.hpp</file>
      <line>851</line>
    </frame>
    <frame>
      <ip>0x5C0D60</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>boost::statechart::simple_state&lt;PG::RecoveryState::Incomplete, PG::RecoveryState::Peering, boost::mpl::list&lt;mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na, mpl_::na&gt;, (boost::statechart::history_mode)0&gt;::react_impl(boost::statechart::event_base const&amp;, void const*)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>simple_state.hpp</file>
      <line>488</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>operator()</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>state_machine.hpp</file>
      <line>87</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>operator()&lt;boost::statechart::detail::send_function&lt;boost::statechart::detail::state_base&lt;std::allocator&lt;void&gt;, boost::statechart::detail::rtti_policy&gt;, boost::statechart::event_base, void const*&gt;, boost::statechart::state_machine&lt;PG::RecoveryState::RecoveryMachine, PG::RecoveryState::Initial&gt;::exception_event_handler&gt;</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>null_exception_translator.hpp</file>
      <line>33</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>send_event</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>state_machine.hpp</file>
      <line>885</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>process_event</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/build/boost/include/boost/statechart</dir>
      <file>state_machine.hpp</file>
      <line>275</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>handle_event</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>PG.h</file>
      <line>2690</line>
    </frame>
    <frame>
      <ip>0x55F61D</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>PG::handle_advance_map(std::shared_ptr&lt;OSDMap const&gt;, std::shared_ptr&lt;OSDMap const&gt;, std::vector&lt;int, std::allocator&lt;int&gt; &gt;&amp;, int, std::vector&lt;int, std::allocator&lt;int&gt; &gt;&amp;, int, PG::RecoveryCtx*)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>PG.cc</file>
      <line>6429</line>
    </frame>
    <frame>
      <ip>0x4CB31C</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>OSD::advance_pg(unsigned int, PG*, ThreadPool::TPHandle&amp;, PG::RecoveryCtx*)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>OSD.cc</file>
      <line>7812</line>
    </frame>
    <frame>
      <ip>0x4CB9C0</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>OSD::dequeue_peering_evt(OSDShard*, PG*, std::shared_ptr&lt;PGPeeringEvent&gt;, ThreadPool::TPHandle&amp;)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>OSD.cc</file>
      <line>8932</line>
    </frame>
    <frame>
      <ip>0x722B0F</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>PGPeeringItem::run(OSD*, OSDShard*, boost::intrusive_ptr&lt;PG&gt;&amp;, ThreadPool::TPHandle&amp;)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>OpQueueItem.cc</file>
      <line>34</line>
    </frame>
    <frame>
      <ip>0x4D6884</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>run</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>OpQueueItem.h</file>
      <line>134</line>
    </frame>
    <frame>
      <ip>0x4D6884</ip>
      <obj>/usr/bin/ceph-osd</obj>
      <fn>OSD::ShardedOpWQ::_process(unsigned int, ceph::heartbeat_handle_d*)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/osd</dir>
      <file>OSD.cc</file>
      <line>9908</line>
    </frame>
    <frame>
      <ip>0x50C43D2</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>ShardedThreadPool::shardedthreadpool_worker(unsigned int)</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/common</dir>
      <file>WorkQueue.cc</file>
      <line>339</line>
    </frame>
    <frame>
      <ip>0x50C4FBF</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>ShardedThreadPool::WorkThreadSharded::entry()</fn>
      <dir>/usr/src/debug/ceph-13.0.2-1988-gbc83811/src/common</dir>
      <file>WorkQueue.h</file>
      <line>690</line>
    </frame>
    <frame>
      <ip>0xFDAAE24</ip>
      <obj>/usr/lib64/libpthread-2.17.so</obj>
      <fn>start_thread</fn>
    </frame>
    <frame>
      <ip>0x10CF134C</ip>
      <obj>/usr/lib64/libc-2.17.so</obj>
      <fn>clone</fn>
    </frame>
  </stack>
</error>

/a/kchai-2018-05-01_16:39:19-rados-wip-kefu-testing-2018-05-01-2255-distro-basic-smithi/2462751/remote/smithi074/log/valgrind/osd.*.log.gz


Related issues

Duplicates RADOS - Bug #23980: UninitCondition in PG::RecoveryState::Incomplete::react(PG::AdvMap const&) Resolved 05/02/2018

History

#1 Updated by Kefu Chai over 1 year ago

#2 Updated by Josh Durgin over 1 year ago

What PRs were in the test branch that hit this? Did any of them change the PG class or related structures?

#3 Updated by Kefu Chai over 1 year ago

  • Duplicates Bug #23980: UninitCondition in PG::RecoveryState::Incomplete::react(PG::AdvMap const&) added

#4 Updated by Kefu Chai over 1 year ago

  • Status changed from New to Duplicate

Also available in: Atom PDF