]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson/os/seastore/lba_manager/btree/lba_btree_node_impl: add debugging
authorSamuel Just <sjust@redhat.com>
Thu, 24 Jun 2021 23:22:43 +0000 (16:22 -0700)
committerSamuel Just <sjust@redhat.com>
Thu, 24 Jun 2021 23:30:15 +0000 (16:30 -0700)
Signed-off-by: Samuel Just <sjust@redhat.com>
src/crimson/os/seastore/lba_manager/btree/lba_btree_node_impl.cc

index 9d121ec7129b7c29b27c9f1da2123d7225806519..41bab2893cc617184a5264f667b9725b4581cae6 100644 (file)
@@ -729,7 +729,15 @@ get_lba_node_ret get_lba_btree_extent(
          ceph_assert(meta.begin <= ret->begin()->get_key());
          ceph_assert(meta.end > (ret->end() - 1)->get_key());
        }
-       assert(!(parent->has_been_invalidated() || ret->has_been_invalidated()));
+       if (parent->has_been_invalidated() || ret->has_been_invalidated()) {
+         logger().debug(
+           "get_lba_btree_extent: parent {} or ret {} is invalid, transaction {} is conflicted: {}",
+           *parent,
+           *ret,
+           (void*)&c.trans,
+           c.trans.is_conflicted());
+         assert(!(parent->has_been_invalidated() || ret->has_been_invalidated()));
+       }
        if (!ret->is_pending() && !ret->pin.is_linked()) {
          ret->pin.set_range(meta);
          c.pins.add_pin(ret->pin);
@@ -758,7 +766,15 @@ get_lba_node_ret get_lba_btree_extent(
          ceph_assert(meta.begin <= ret->begin()->get_key());
          ceph_assert(meta.end > (ret->end() - 1)->get_key());
        }
-       assert(!(parent->has_been_invalidated() || ret->has_been_invalidated()));
+       if (parent->has_been_invalidated() || ret->has_been_invalidated()) {
+         logger().debug(
+           "get_lba_btree_extent: parent {} or ret {} is invalid, transaction {} is conflicted: {}",
+           *parent,
+           *ret,
+           (void*)&c.trans,
+           c.trans.is_conflicted());
+         assert(!(parent->has_been_invalidated() || ret->has_been_invalidated()));
+       }
        if (!ret->is_pending() && !ret->pin.is_linked()) {
          ret->pin.set_range(meta);
          c.pins.add_pin(ret->pin);