]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
crimson: switch to ceph_abort_msg 64262/head
authorMatan Breizman <mbreizma@redhat.com>
Mon, 30 Jun 2025 09:44:24 +0000 (09:44 +0000)
committerMatan Breizman <mbreizma@redhat.com>
Wed, 2 Jul 2025 08:37:14 +0000 (08:37 +0000)
ceph_abort doesn't print a message. Use ceph_abort_msg instead.
Most of the instances are not printing useful information but some are:
  ceph_abort_msg("seastore device size setting is too small");

```
  ::ceph::__ceph_abort( __FILE__, __LINE__, __CEPH_ASSERT_FUNCTION, "abort() called")

  ::ceph::__ceph_abort( __FILE__, __LINE__, __CEPH_ASSERT_FUNCTION, msg)
```

Signed-off-by: Matan Breizman <mbreizma@redhat.com>
42 files changed:
src/crimson/net/FrameAssemblerV2.cc
src/crimson/net/Interceptor.h
src/crimson/net/ProtocolV2.cc
src/crimson/net/Socket.cc
src/crimson/net/io_handler.cc
src/crimson/net/io_handler.h
src/crimson/os/seastore/async_cleaner.cc
src/crimson/os/seastore/backref/btree_backref_manager.cc
src/crimson/os/seastore/btree/fixed_kv_btree.h
src/crimson/os/seastore/cache.cc
src/crimson/os/seastore/cache.h
src/crimson/os/seastore/cached_extent.h
src/crimson/os/seastore/extent_placement_manager.cc
src/crimson/os/seastore/extent_placement_manager.h
src/crimson/os/seastore/journal/circular_journal_space.cc
src/crimson/os/seastore/journal/record_submitter.cc
src/crimson/os/seastore/object_data_handler.cc
src/crimson/os/seastore/onode_manager/staged-fltree/node.cc
src/crimson/os/seastore/onode_manager/staged-fltree/node.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_extent_accessor.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_extent_manager/dummy.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_extent_manager/seastore.cc
src/crimson/os/seastore/onode_manager/staged-fltree/node_extent_manager/seastore.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_extent_manager/test_replay.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_impl.cc
src/crimson/os/seastore/onode_manager/staged-fltree/node_impl.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_layout.h
src/crimson/os/seastore/onode_manager/staged-fltree/node_layout_replayable.h
src/crimson/os/seastore/onode_manager/staged-fltree/stages/key_layout.cc
src/crimson/os/seastore/onode_manager/staged-fltree/stages/key_layout.h
src/crimson/os/seastore/onode_manager/staged-fltree/stages/node_stage.cc
src/crimson/os/seastore/onode_manager/staged-fltree/stages/node_stage.h
src/crimson/os/seastore/onode_manager/staged-fltree/stages/node_stage_layout.h
src/crimson/os/seastore/onode_manager/staged-fltree/stages/stage.h
src/crimson/os/seastore/onode_manager/staged-fltree/stages/stage_types.h
src/crimson/os/seastore/onode_manager/staged-fltree/tree.h
src/crimson/os/seastore/transaction_manager.h
src/crimson/osd/pg_map.cc
src/crimson/osd/recovery_backend.cc
src/crimson/osd/scrub/scrub_machine.h
src/crimson/osd/shard_services.cc
src/crimson/tools/perf_crimson_msgr.cc

index d2add814d3787780f46b55eadf9e66d53380db72..89600742ec593a90bbac970db9dabf0d60ba6b0c 100644 (file)
@@ -427,7 +427,7 @@ FrameAssemblerV2::read_frame_payload()
     // and abort after putting entire data field on wire. This will be used by
     // the kernel client to avoid unnecessary buffering.
     if (!ok) {
-      ceph_abort("TODO");
+      ceph_abort_msg("TODO");
     }
     return &rx_segments_data;
   });
index 35b74e24369c4d6aa9db657ff052b5c8c6412b59..36b48a51ff7deb489446f72ce18d61e1a32d0492 100644 (file)
@@ -79,7 +79,7 @@ class socket_blocker {
       return seastar::sleep_abortable(
         std::chrono::seconds(10), *p_unblocked
       ).then([] {
-        ceph_abort("Timeout (10s) in socket_blocker::block()");
+        ceph_abort_msg("Timeout (10s) in socket_blocker::block()");
       }).handle_exception_type([] (const seastar::sleep_aborted& e) {
         // wait done!
       });
index 7942adc7ef183ac0d283e0c9c53d599afdbe93e9..6f4b2c6d00abee91211af16f8c2b914423a7da14 100644 (file)
@@ -908,7 +908,7 @@ void ProtocolV2::execute_connecting()
           logger().info("[Test] got BLOCK");
           return conn.interceptor->blocker.block();
         default:
-          ceph_abort("unexpected action from trap");
+          ceph_abort_msg("unexpected action from trap");
           return seastar::now();
         }
       });;
@@ -1041,7 +1041,7 @@ void ProtocolV2::execute_connecting()
         return seastar::now();
        }
        default: {
-        ceph_abort("impossible next step");
+        ceph_abort_msg("impossible next step");
        }
       }
     }).handle_exception([this](std::exception_ptr eptr) {
@@ -1693,7 +1693,7 @@ void ProtocolV2::execute_accepting()
         execute_server_wait();
         break;
        default:
-        ceph_abort("impossible next step");
+        ceph_abort_msg("impossible next step");
       }
     }).handle_exception([this](std::exception_ptr eptr) {
       const char *e_what;
index 3a7aeaf9651e2269661b939b4e20a009d84ce548..97e0bee4dc3c847933e66f36fe58d51513fdb992 100644 (file)
@@ -332,7 +332,7 @@ Socket::try_trap_pre(bp_action_t& trap) {
     trap = action;
     break;
    default:
-    ceph_abort("unexpected action from trap");
+    ceph_abort_msg("unexpected action from trap");
   }
   return seastar::make_ready_future<>();
 }
@@ -349,7 +349,7 @@ Socket::try_trap_post(bp_action_t& trap) {
     force_shutdown();
     return blocker->block();
    default:
-    ceph_abort("unexpected action from trap");
+    ceph_abort_msg("unexpected action from trap");
   }
   return seastar::make_ready_future<>();
 }
index acd6147710b5b7856adfc9ae3ca863b92a2a6600..54ea30093bf7cb3fdac6cb78f571744b95971d7b 100644 (file)
@@ -912,7 +912,7 @@ IOHandler::do_out_dispatch(shard_states_t &ctx)
       ctx.exit_out_dispatching("switched", conn);
       return seastar::make_ready_future<stop_t>(stop_t::yes);
      default:
-      ceph_abort("impossible");
+      ceph_abort_msg("impossible");
     }
   }).handle_exception_type([this, &ctx](const std::system_error& e) {
     auto io_state = ctx.get_io_state();
@@ -1130,7 +1130,7 @@ void IOHandler::do_in_dispatch()
                 return seastar::now();
               }
               // TODO: message throttler
-              ceph_abort("TODO");
+              ceph_abort_msg("TODO");
               return seastar::now();
             }).then([this, msg_size] {
               // throttle_bytes() logic
index c64e7a0f6526cca62fe7d4435149df2540e0ee3a..d26fe9a95dbdfcfd110a37f8d16c4d88cefeb05b 100644 (file)
@@ -329,7 +329,7 @@ public:
         crimson::get_logger(ceph_subsys_ms).error(
           "{} try_enter_out_dispatching() got wrong io_state {}",
           conn, io_state);
-        ceph_abort("impossible");
+        ceph_abort_msg("impossible");
       }
     }
 
index d6508a6b63e074f28af16a33fdf8ff3945f17ed5..bc1d19afd48b677746bfd211e152008a3902ef33 100644 (file)
@@ -1075,7 +1075,7 @@ segment_id_t SegmentCleaner::allocate_segment(
   ERROR("out of space with {} {} {} {}",
         type, segment_seq_printer_t{seq}, category,
         rewrite_gen_printer_t{generation});
-  ceph_abort("seastore device size setting is too small");
+  ceph_abort_msg("seastore device size setting is too small");
   return NULL_SEG_ID;
 }
 
@@ -1703,7 +1703,7 @@ segment_id_t SegmentCleaner::get_next_reclaim_segment() const
   } else {
     ceph_assert(get_segments_reclaimable() == 0);
     // see should_clean_space()
-    ceph_abort("impossible!");
+    ceph_abort_msg("impossible!");
     return NULL_SEG_ID;
   }
 }
index 58b4ffd4c9a84fed63f691046647102fa7862928..3a9875cf5f841993a386e5345db7f6c017eccb52 100644 (file)
@@ -216,7 +216,7 @@ BtreeBackrefManager::new_mapping(
                    t, addr, len, key,
                    pos.get_key(), pos.get_val().len,
                    pos.get_val());
-           ceph_abort("not possible for the backref tree");
+           ceph_abort_msg("not possible for the backref tree");
            return BackrefBtree::iterate_repeat_ret_inner(
              interruptible::ready_future_marker{},
              seastar::stop_iteration::no);
index 1390cf63bc8c0af1753a0763c971e0b364a83671..3dceaddef8131dd8fda14b10ebf306a3e0c8beb9 100644 (file)
@@ -529,7 +529,7 @@ public:
           }
         } else {
           ceph_assert(!child_node->is_valid());
-          ceph_abort("impossible");
+          ceph_abort_msg("impossible");
         }
       } else if (ret == Transaction::get_extent_ret::ABSENT) {
         BaseChildNode<std::remove_reference_t<decltype(*node)>,
@@ -573,7 +573,7 @@ public:
                   i->get_key())));
         }
       } else {
-        ceph_abort("impossible");
+        ceph_abort_msg("impossible");
       }
     }
   }
index deb47dcac671d0ba657aefbb68d57934120a2cda..a639d83a78bdb64eebb3a7d5c3b8d9bfc40ea510 100644 (file)
@@ -1237,7 +1237,7 @@ record_t Cache::prepare_record(
     if (!i.ref->is_valid()) {
       SUBERRORT(seastore_t,
           "read_set got invalid extent, aborting -- {}", t, *i.ref);
-      ceph_abort("no invalid extent allowed in transactions' read_set");
+      ceph_abort_msg("no invalid extent allowed in transactions' read_set");
     }
     get_by_ext(efforts.read_by_ext,
                i.ref->get_type()).increment(i.ref->get_length());
@@ -1424,7 +1424,7 @@ record_t Cache::prepare_record(
       remove_backref_extent(extent->get_paddr());
     } else {
       ERRORT("Got unexpected extent type: {}", t, *extent);
-      ceph_abort("imposible");
+      ceph_abort_msg("imposible");
     }
   }
   alloc_deltas.emplace_back(std::move(rel_delta));
@@ -1642,7 +1642,7 @@ record_t Cache::prepare_record(
       SUBINFOT(seastore_t, "alloc_tail all trimmed, set to head {}, src={}",
                t, alloc_tail, trans_src);
     } else if (*maybe_alloc_tail == JOURNAL_SEQ_NULL) {
-      ceph_abort("impossible");
+      ceph_abort_msg("impossible");
     } else {
       alloc_tail = *maybe_alloc_tail;
     }
@@ -1859,7 +1859,7 @@ void Cache::complete_commit(
          i->get_type());
     } else {
       ERRORT("{}", t, *i);
-      ceph_abort("not possible");
+      ceph_abort_msg("not possible");
     }
   });
 
index e7a78fe23832726fdd2cc66518d9179728aab9ba..42d4a4a28d3b391b50dfc0a2cc7fb369012bba96 100644 (file)
@@ -338,7 +338,7 @@ public:
     if (result == Transaction::get_extent_ret::RETIRED) {
       SUBERRORT(seastore_cache, "{} {}~0x{:x} is retired on t -- {}",
                 t, T::TYPE, offset, length, *ret);
-      ceph_abort("impossible");
+      ceph_abort_msg("impossible");
     } else if (result == Transaction::get_extent_ret::PRESENT) {
       assert(ret->get_length() == length);
       if (ret->is_fully_loaded()) {
@@ -847,7 +847,7 @@ private:
     if (status == Transaction::get_extent_ret::RETIRED) {
       SUBERRORT(seastore_cache, "{} {}~0x{:x} {} is retired on t -- {}",
                 t, type, offset, length, laddr, *ret);
-      ceph_abort("impossible");
+      ceph_abort_msg("impossible");
     } else if (status == Transaction::get_extent_ret::PRESENT) {
       assert(ret->get_length() == length);
       if (ret->is_fully_loaded()) {
index fe25c7481a582917d37c05b16eca8743c2249bb1..10c4aa704c810c1906a9f884d6b2a0eb628839af 100644 (file)
@@ -1345,12 +1345,12 @@ public:
     : CachedExtent(CachedExtent::retired_placeholder_construct_t{}, length) {}
 
   CachedExtentRef duplicate_for_write(Transaction&) final {
-    ceph_abort("Should never happen for a placeholder");
+    ceph_abort_msg("Should never happen for a placeholder");
     return CachedExtentRef();
   }
 
   ceph::bufferlist get_delta() final {
-    ceph_abort("Should never happen for a placeholder");
+    ceph_abort_msg("Should never happen for a placeholder");
     return ceph::bufferlist();
   }
 
@@ -1361,7 +1361,7 @@ public:
 
   void apply_delta_and_adjust_crc(
     paddr_t base, const ceph::bufferlist &bl) final {
-    ceph_abort("Should never happen for a placeholder");
+    ceph_abort_msg("Should never happen for a placeholder");
   }
 
   void on_rewrite(Transaction &, CachedExtent&, extent_len_t) final {}
@@ -1371,7 +1371,7 @@ public:
   }
 
   void on_delta_write(paddr_t record_block_offset) final {
-    ceph_abort("Should never happen for a placeholder");
+    ceph_abort_msg("Should never happen for a placeholder");
   }
 };
 
@@ -1449,12 +1449,12 @@ public:
     extent_len_t len;
   };
   virtual std::optional<modified_region_t> get_modified_region() {
-    ceph_abort("Unsupported");
+    ceph_abort_msg("Unsupported");
     return std::nullopt;
   }
 
   virtual void clear_modified_region() {
-    ceph_abort("Unsupported");
+    ceph_abort_msg("Unsupported");
   }
 
   virtual ~LogicalCachedExtent() {}
index fd19eeb7e58eb40434ac84571d92d57fa136a2cd..457cd1cf1dcf7fc3e440fd6f043c2b2dd9b4aeae 100644 (file)
@@ -915,7 +915,7 @@ ExtentPlacementManager::BackgroundProcess::do_background_cycle()
     }
 
     if (!proceed_clean_main && !proceed_clean_cold) {
-      ceph_abort("no background process will start");
+      ceph_abort_msg("no background process will start");
     }
     return seastar::when_all(
       [this, FNAME, proceed_clean_main,
index bcfe98811b77e599ed1dbc1389b5adcbe94efb9f..913e95ab647ef6034b561de26e3477681217a1a7 100644 (file)
@@ -1000,7 +1000,7 @@ private:
           }
           break;
         default:
-          ceph_abort("impossible");
+          ceph_abort_msg("impossible");
         }
         return ret;
       }
index 458042ee36a2c4c8c8aecdfeb26785ac2378b2c2..29095abf9c34e175a7687c3be88aa7d552741677 100644 (file)
@@ -55,7 +55,7 @@ CircularJournalSpace::write(ceph::bufferlist&& to_write) {
   assert(get_written_to().segment_seq != NULL_SEG_SEQ);
   auto encoded_size = to_write.length();
   if (encoded_size > get_records_available_size()) {
-    ceph_abort("should be impossible with EPM reservation");
+    ceph_abort_msg("should be impossible with EPM reservation");
   }
   assert(encoded_size + get_rbm_addr(get_written_to())
         < get_journal_end());
index 4976eee96e714a6d6a4b69f0bbe8ee2e350ffcf2..0ae4e25b21788f491453717a44a0f9ab398c6343 100644 (file)
@@ -469,7 +469,7 @@ void RecordSubmitter::update_state()
   } else if (num_outstanding_io == io_depth_limit) {
     state = state_t::FULL;
   } else {
-    ceph_abort("fatal error: io-depth overflow");
+    ceph_abort_msg("fatal error: io-depth overflow");
   }
 }
 
index 386ee317051adf195888a2ed7fad7b7b84093555..bdfff46991889e4d1f3f833d7f3b603e2757941f 100644 (file)
@@ -489,7 +489,7 @@ ObjectDataHandler::write_ret do_remappings(
           return ObjectDataHandler::write_iertr::now();
         });
       } else {
-        ceph_abort("impossible");
+        ceph_abort_msg("impossible");
         return ObjectDataHandler::write_iertr::now();
       }
   });
@@ -585,7 +585,7 @@ ObjectDataHandler::write_ret do_insertions(
          ObjectDataHandler::write_iertr::pass_further{}
        );
       } else {
-       ceph_abort("impossible");
+       ceph_abort_msg("impossible");
        return ObjectDataHandler::write_iertr::now();
       }
     });
index d227d69afd34a6cad9d358278737a437682aa054..04095a4e5e3980c1b07234755d053450a0f410df 100644 (file)
@@ -93,7 +93,7 @@ void tree_cursor_t::assert_next_to(
     }
   } else {
     assert(is_invalid());
-    ceph_abort("impossible");
+    ceph_abort_msg("impossible");
   }
 #endif
 }
@@ -240,7 +240,7 @@ void tree_cursor_t::Cache::maybe_duplicate(const node_version_t& current_version
     p_node_base = current_p_node_base;
   } else {
     // It is impossible to change state backwards, see node_types.h.
-    ceph_abort("impossible");
+    ceph_abort_msg("impossible");
   }
 }
 
@@ -701,13 +701,13 @@ eagain_ifuture<Ref<Node>> Node::load(
       ERRORT("load addr={}, is_level_tail={} error, "
              "got invalid header -- {}",
              c.t, addr, expect_is_level_tail, fmt::ptr(extent));
-      ceph_abort("fatal error");
+      ceph_abort_msg("fatal error");
     }
     if (header.get_is_level_tail() != expect_is_level_tail) {
       ERRORT("load addr={}, is_level_tail={} error, "
              "is_level_tail mismatch -- {}",
              c.t, addr, expect_is_level_tail, fmt::ptr(extent));
-      ceph_abort("fatal error");
+      ceph_abort_msg("fatal error");
     }
 
     auto node_type = header.get_node_type();
@@ -716,7 +716,7 @@ eagain_ifuture<Ref<Node>> Node::load(
         ERRORT("load addr={}, is_level_tail={} error, "
                "leaf length mismatch -- {}",
                c.t, addr, expect_is_level_tail, fmt::ptr(extent));
-        ceph_abort("fatal error");
+        ceph_abort_msg("fatal error");
       }
       auto impl = LeafNodeImpl::load(extent, *field_type);
       auto *derived_ptr = impl.get();
@@ -727,14 +727,14 @@ eagain_ifuture<Ref<Node>> Node::load(
         ERRORT("load addr={}, is_level_tail={} error, "
                "internal length mismatch -- {}",
                c.t, addr, expect_is_level_tail, fmt::ptr(extent));
-        ceph_abort("fatal error");
+        ceph_abort_msg("fatal error");
       }
       auto impl = InternalNodeImpl::load(extent, *field_type);
       auto *derived_ptr = impl.get();
       return eagain_iertr::make_ready_future<Ref<Node>>(
        new InternalNode(derived_ptr, std::move(impl)));
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   });
 }
@@ -945,7 +945,7 @@ eagain_ifuture<> InternalNode::erase_child(context_t c, Ref<Node>&& child_ref)
           //
           // In order to preserve the invariant, we need to make sure the new
           // internal root also has at least 2 children.
-          ceph_abort("trying to erase the last item from the internal root node");
+          ceph_abort_msg("trying to erase the last item from the internal root node");
         }
 
         // track erase
@@ -1568,7 +1568,7 @@ eagain_ifuture<Ref<Node>> InternalNode::get_or_track_child(
       if (child->level() + 1 != level()) {
         ERRORT("loaded child {} error from parent {} at pos({}), level mismatch",
                c.t, child->get_name(), get_name(), position);
-        ceph_abort("fatal error");
+        ceph_abort_msg("fatal error");
       }
       child->as_child(position, this);
       return child;
@@ -1893,14 +1893,14 @@ LeafNode::erase<false>(context_t, const search_position_t&, bool);
 eagain_ifuture<> LeafNode::extend_value(
     context_t c, const search_position_t& pos, value_size_t extend_size)
 {
-  ceph_abort("not implemented");
+  ceph_abort_msg("not implemented");
   return eagain_iertr::now();
 }
 
 eagain_ifuture<> LeafNode::trim_value(
     context_t c, const search_position_t& pos, value_size_t trim_size)
 {
-  ceph_abort("not implemented");
+  ceph_abort_msg("not implemented");
   return eagain_iertr::now();
 }
 
index 0b764172e50e416d83acb24c342b3bd469c98062..eff0e0a6f1a0aadfe5f505fff0b0a030c433fb39 100644 (file)
@@ -284,7 +284,7 @@ class Node
           // good
         } else {
           assert(p_cursor->is_invalid());
-          ceph_abort("impossible");
+          ceph_abort_msg("impossible");
         }
       }
 #endif
@@ -383,7 +383,7 @@ class Node
 
  protected:
   virtual eagain_ifuture<> test_clone_non_root(context_t, Ref<InternalNode>) const {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
   virtual eagain_ifuture<search_result_t> lower_bound_tracked(
       context_t, const key_hobj_t&, MatchHistory&) = 0;
index d2f1ecda73e2e843bfb923ab52354647d0a47982..cd8e14002dbdde3809a80f4466b7dd53d4c8082a 100644 (file)
@@ -207,13 +207,13 @@ class DeltaRecorderT final: public DeltaRecorder {
         SUBERROR(seastore_onode,
             "got unknown op {} when replay {}",
             op, node);
-        ceph_abort("fatal error");
+        ceph_abort_msg("fatal error");
       }
     } catch (buffer::error& e) {
       SUBERROR(seastore_onode,
           "got decode error {} when replay {}",
           e.what(), node);
-      ceph_abort("fatal error");
+      ceph_abort_msg("fatal error");
     }
   }
 
@@ -245,7 +245,7 @@ class DeltaRecorderT final: public DeltaRecorder {
       // NODE_TYPE == node_type_t::LEAF
       value.encode(encoded);
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
@@ -259,7 +259,7 @@ class DeltaRecorderT final: public DeltaRecorder {
       // NODE_TYPE == node_type_t::LEAF
       return value_config_t::decode(delta);
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
@@ -314,7 +314,7 @@ class NodeExtentAccessorT {
       recorder = nullptr;
     } else {
       // extent is invalid or retired
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
 #ifndef NDEBUG
     auto ref_recorder = recorder_t::create_for_replay();
index df2e5fe7503d790983b11264fe0710a1b187bb41..86cb5c9588f17f339a78ef8cfd789d66438e21f6 100644 (file)
@@ -52,17 +52,17 @@ class DummyNodeExtent final: public NodeExtent {
 
  protected:
   NodeExtentRef mutate(context_t, DeltaRecorderURef&&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   DeltaRecorder* get_recorder() const override {
     return nullptr; }
   CachedExtentRef duplicate_for_write(Transaction&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   extent_types_t get_type() const override {
     return extent_types_t::TEST_BLOCK; }
   ceph::bufferlist get_delta() override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   void apply_delta(const ceph::bufferlist&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
 };
 
 template <bool SYNC>
index 3b52c5dc069c1277d76ecea7dbba0e9c1911f43a..dc774330c9cdf863d6adb7feb0f0dccf67b03125 100644 (file)
@@ -27,7 +27,7 @@ static DeltaRecorderURef create_replay_recorder(
     } else if (field_type == field_type_t::N3) {
       return DeltaRecorderT<leaf_fields_3_t, node_type_t::LEAF>::create_for_replay();
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   } else if (node_type == node_type_t::INTERNAL) {
     if (field_type == field_type_t::N0) {
@@ -39,10 +39,10 @@ static DeltaRecorderURef create_replay_recorder(
     } else if (field_type == field_type_t::N3) {
       return DeltaRecorderT<internal_fields_3_t, node_type_t::INTERNAL>::create_for_replay();
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -68,7 +68,7 @@ void SeastoreNodeExtent::apply_delta(const ceph::bufferlist& bl)
     auto field_type = header.get_field_type();
     if (!field_type.has_value()) {
       ERROR("replay got invalid node -- {}", *this);
-      ceph_abort("fatal error");
+      ceph_abort_msg("fatal error");
     }
     auto node_type = header.get_node_type();
     recorder = create_replay_recorder(node_type, *field_type);
index ab25c312f9b8fd3ad48a5955d0981135bbc7c8cd..67eaee194b33a04197c217a33b07538f62ca0728 100644 (file)
@@ -146,7 +146,7 @@ class SeastoreNodeExtentManager final: public TransactionManagerHandle {
         SUBERRORT(seastore_onode,
             "allocated {}B but got invalid extent: {}",
             t, len, *extent);
-        ceph_abort("fatal error");
+        ceph_abort_msg("fatal error");
       }
       assert(extent->get_length() == len);
       std::ignore = len;
index bce74e3814058dff61217a938a1cde805d265ef8..62e08da5f6adc48cca7bfaaff8930ad6a01bb7d0 100644 (file)
@@ -44,17 +44,17 @@ class TestReplayExtent final: public NodeExtent {
 
  protected:
   NodeExtentRef mutate(context_t, DeltaRecorderURef&&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   DeltaRecorder* get_recorder() const override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   CachedExtentRef duplicate_for_write(Transaction&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   extent_types_t get_type() const override {
     return extent_types_t::TEST_BLOCK; }
   ceph::bufferlist get_delta() override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
   void apply_delta(const ceph::bufferlist&) override {
-    ceph_abort("impossible path"); }
+    ceph_abort_msg("impossible path"); }
 
  private:
   TestReplayExtent(ceph::bufferptr&& ptr, DeltaRecorderURef&& recorder)
index 5db0f83dda6167d19dafed3c4a56127e0bb9795f..ce6c9f5a9960e8e75a38978719b02a3c6316d271 100644 (file)
@@ -24,7 +24,7 @@ InternalNodeImpl::allocate(
   } else if (type == field_type_t::N3) {
     return InternalNode3::allocate(c, hint, is_level_tail, level);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -41,7 +41,7 @@ LeafNodeImpl::allocate(
   } else if (type == field_type_t::N3) {
     return LeafNode3::allocate(c, hint, is_level_tail, 0);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -57,7 +57,7 @@ InternalNodeImplURef InternalNodeImpl::load(
   } else if (type == field_type_t::N3) {
     return InternalNode3::load(extent);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -73,7 +73,7 @@ LeafNodeImplURef LeafNodeImpl::load(
   } else if (type == field_type_t::N3) {
     return LeafNode3::load(extent);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
index cf452618b6fa101a4fff54a93e1154adefe12b0b..ccbb6690607d2ac504adedc70494511a305e077c 100644 (file)
@@ -121,48 +121,48 @@ class InternalNodeImpl : public NodeImpl {
   virtual void get_slot(const search_position_t&,                 // IN
                         key_view_t* = nullptr,                    // OUT
                         const laddr_packed_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_prev_slot(search_position_t&,                       // IN&OUT
                              key_view_t* = nullptr,                    // OUT
                              const laddr_packed_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_next_slot(search_position_t&,                       // IN&OUT
                              key_view_t* = nullptr,                    // OUT
                              const laddr_packed_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_largest_slot(search_position_t* = nullptr,             // OUT
                                 key_view_t* = nullptr,                    // OUT
                                 const laddr_packed_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual lookup_result_t<node_type_t::INTERNAL> lower_bound(
       const key_hobj_t&, MatchHistory&,
       key_view_t* = nullptr, internal_marker_t = {}) const {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual const laddr_packed_t* insert(
       const key_view_t&, const laddr_t&, search_position_t&, match_stage_t&, node_offset_t&) {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual std::tuple<search_position_t, bool, const laddr_packed_t*> split_insert(
       NodeExtentMutable&, NodeImpl&, const key_view_t&, const laddr_t&,
       search_position_t&, match_stage_t&, node_offset_t&) {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   virtual const laddr_packed_t* get_tail_value() const = 0;
@@ -201,48 +201,48 @@ class LeafNodeImpl : public NodeImpl {
   virtual void get_slot(const search_position_t&,                 // IN
                         key_view_t* = nullptr,                    // OUT
                         const value_header_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_prev_slot(search_position_t&,                       // IN&OUT
                              key_view_t* = nullptr,                    // OUT
                              const value_header_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_next_slot(search_position_t&,                       // IN&OUT
                              key_view_t* = nullptr,                    // OUT
                              const value_header_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual void get_largest_slot(search_position_t* = nullptr,             // OUT
                                 key_view_t* = nullptr,                    // OUT
                                 const value_header_t** = nullptr) const { // OUT
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual lookup_result_t<node_type_t::LEAF> lower_bound(
       const key_hobj_t&, MatchHistory&,
       key_view_t* = nullptr, leaf_marker_t = {}) const {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual const value_header_t* insert(
       const key_hobj_t&, const value_config_t&, search_position_t&, match_stage_t&, node_offset_t&) {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   #pragma GCC diagnostic ignored "-Woverloaded-virtual"
   virtual std::tuple<search_position_t, bool, const value_header_t*> split_insert(
       NodeExtentMutable&, NodeImpl&, const key_hobj_t&, const value_config_t&,
       search_position_t&, match_stage_t&, node_offset_t&) {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 
   virtual std::tuple<match_stage_t, node_offset_t> evaluate_insert(
index 8ad780040789ea7126780206c624f111e5144080..70a2dcadc9f769fb0658dd32553652525d912ac9 100644 (file)
@@ -219,7 +219,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
         merge_stage = STAGE;
         size_comp = right_node_stage.header_size();
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     } else {
       key_view_t left_pivot_index;
@@ -278,7 +278,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
         auto p_write = left_node_stage.get_end_p_laddr();
         mut.copy_in_absolute((void*)p_write, tail_value);
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     } else {
       typename stage_t::template StagedAppender<KeyT::VIEW> left_appender;
@@ -423,7 +423,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       stage_t::template get_slot<true, false>(
           extent.read(), cast_down<STAGE>(pos), p_index_key, nullptr);
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
 #ifndef NDEBUG
     if (pp_value) {
@@ -446,7 +446,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       stage_t::template get_prev_slot<false, true>(
           extent.read(), _pos, nullptr, pp_value);
     } else {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     }
 #ifndef NDEBUG
     auto _nxt_pos = _pos;
@@ -469,7 +469,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       find_next = stage_t::template get_next_slot<false, true>(
           extent.read(), cast_down<STAGE>(pos), nullptr, pp_value);
     } else {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     }
     if (find_next) {
       pos = search_position_t::end();
@@ -493,7 +493,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       stage_t::template get_largest_slot<true, false, false>(
           extent.read(), &cast_down_fill_0<STAGE>(*p_pos), nullptr, nullptr);
     } else {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     }
   }
 
@@ -834,7 +834,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       assert(is_level_tail());
       return extent.read().get_end_p_laddr();
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
@@ -853,7 +853,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       assert(p_value->value == src);
       extent.update_child_addr_replayable(dst, const_cast<laddr_packed_t*>(p_value));
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
@@ -874,7 +874,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
       }
       return {insert_stage, insert_size};
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
@@ -895,7 +895,7 @@ class NodeLayoutT final : public InternalNodeImpl, public LeafNodeImpl {
             key, value, history, mstat, cast_down<STAGE>(insert_pos));
       }
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 
index d8a18231e6a67c9ec135185a5f6c0afd4419a706..822f35194fcfbba35f379625c59c345566ea3ce1 100644 (file)
@@ -130,7 +130,7 @@ struct NodeLayoutReplayableT {
       // return erase_stage, last_pos
       return {erase_stage, last_pos};
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
   }
 };
index 4bf717dc224eab6683163d79cbe3bbec8595d41a..71bbbed4354063ff6644db0f639d6b0e64346a3f 100644 (file)
@@ -27,7 +27,7 @@ void string_key_view_t::append_dedup(
   } else if (dedup_type == Type::MAX) {
     mut.copy_in_absolute(p_append, MARKER_MAX);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
index b58a895ff23347a18f961ef4f784a97d0a54c176..8faa32c86b3dcfa4b3a5eb59551cc54893283108 100644 (file)
@@ -178,7 +178,7 @@ struct string_key_view_t {
     } else if (dedup_type == Type::MAX) {
       len = MARKER_MAX;
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
     std::memcpy(p_append, &len, sizeof(string_size_t));
   }
index 3ed401c375185d5221d1f3b1156521ef5d53358d..aab30ad3c772e121093d7a7eab882905ccdd4c1c 100644 (file)
@@ -41,7 +41,7 @@ node_offset_t NODE_T::size_to_nxt_at(index_t index) const
                  p_fields->get_item_end_offset(index, node_size);
     return FieldType::estimate_insert_one() + ns_oid_view_t(p_end).size();
   } else {
-    ceph_abort("N3 node is not nested");
+    ceph_abort_msg("N3 node is not nested");
   }
 }
 
@@ -49,7 +49,7 @@ template <typename FieldType, node_type_t NODE_TYPE>
 container_range_t NODE_T::get_nxt_container(index_t index) const
 {
   if constexpr (std::is_same_v<FieldType, internal_fields_3_t>) {
-    ceph_abort("N3 internal node doesn't have the right part");
+    ceph_abort_msg("N3 internal node doesn't have the right part");
   } else {
     auto item_start_offset = p_fields->get_item_start_offset(
         index, node_size);
@@ -113,9 +113,9 @@ memory_range_t NODE_T::insert_prefix_at(
                        -(int)size_right);
     return {p_insert_front, p_insert};
   } else if constexpr (FIELD_TYPE == field_type_t::N2) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   } else {
-    ceph_abort("impossible");
+    ceph_abort_msg("impossible");
   }
 }
 #define IPA_TEMPLATE(FT, NT, Key)                                         \
@@ -158,7 +158,7 @@ node_offset_t NODE_T::trim_until(
     return 0;
   }
   if constexpr (std::is_same_v<FieldType, internal_fields_3_t>) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   } else {
     mut.copy_in_absolute(
         (void*)&node.p_fields->num_keys, num_keys_t(index));
@@ -177,7 +177,7 @@ node_offset_t NODE_T::trim_at(
   assert(!node.is_level_tail());
   assert(index < node.keys());
   if constexpr (std::is_same_v<FieldType, internal_fields_3_t>) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   } else {
     extent_len_t node_size = mut.get_length();
     node_offset_t offset = node.p_fields->get_item_start_offset(
@@ -207,7 +207,7 @@ node_offset_t NODE_T::erase_at(
     assert(p_left_bound == node.p_left_bound());
     return FieldType::erase_at(mut, node.fields(), index, p_left_bound);
   } else {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   }
 }
 
@@ -243,15 +243,15 @@ APPEND_T::Appender(NodeExtentMutable* p_mut, const node_extent_t& node, bool ope
                        node.fields().get_item_end_offset(node.keys() - 1,
                                                          node_size);
     } else if constexpr (FIELD_TYPE == field_type_t::N2) {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     } else {
-      ceph_abort("impossible path");
+      ceph_abort_msg("impossible path");
     }
     num_keys = node.keys() - 1;
   } else {
     if constexpr (std::is_same_v<FieldType, internal_fields_3_t>) {
       std::ignore = node_size;
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     } else {
       p_append_left = p_start + node.fields().get_key_start_offset(
           node.keys(), node_size);
@@ -283,7 +283,7 @@ void APPEND_T::append(const node_extent_t& src, index_t from, index_t items)
   num_keys += items;
   if constexpr (std::is_same_v<FieldType, internal_fields_3_t>) {
     std::ignore = node_size;
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   } else {
     // append left part forwards
     node_offset_t offset_left_start = src.fields().get_key_start_offset(
@@ -343,9 +343,9 @@ void APPEND_T::append(
     const full_key_t<KT>& key, const value_input_t& value, const value_t*& p_value)
 {
   if constexpr (FIELD_TYPE == field_type_t::N3) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   } else {
-    ceph_abort("should not happen");
+    ceph_abort_msg("should not happen");
   }
 }
 
@@ -360,7 +360,7 @@ APPEND_T::open_nxt(const key_get_type& partial_key)
   } else if constexpr (FIELD_TYPE == field_type_t::N2) {
     FieldType::append_key(*p_mut, partial_key, p_append_right);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
   return {p_mut, p_append_right};
 }
@@ -376,7 +376,7 @@ APPEND_T::open_nxt(const full_key_t<KT>& key)
   } else if constexpr (FIELD_TYPE == field_type_t::N2) {
     FieldType::append_key(*p_mut, key, p_append_right);
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
   return {p_mut, p_append_right};
 }
index 5615998f8f8eff59a449bb8770da8b3c05e0ba31..706b259e41ea716a34a5c6234cc8f354c2c3a9e9 100644 (file)
@@ -152,9 +152,9 @@ class node_extent_t {
       const Key& key, const value_input_t& value,
       index_t index, node_offset_t size, const char* p_left_bound) {
     if constexpr (FIELD_TYPE == field_type_t::N3) {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     } else {
-      ceph_abort("impossible");
+      ceph_abort_msg("impossible");
     }
   }
 
@@ -216,7 +216,7 @@ class node_extent_t<FieldType, NODE_TYPE>::Appender {
       FieldType::append_offset(*p_mut, new_offset, p_append_left);
       ++num_keys;
     } else {
-      ceph_abort("not implemented");
+      ceph_abort_msg("not implemented");
     }
   }
 
index 1ed4865a629df64b678eaddf043e9e9bae76b6e5..851a4f0ea34e580a2c3b59ee228072560d08e3df 100644 (file)
@@ -282,11 +282,11 @@ struct node_fields_2_t {
   static void insert_at(
       NodeExtentMutable& mut, const Key& key,
       const node_fields_2_t& node, index_t index, node_offset_t size_right) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   }
   static void update_size_at(
       NodeExtentMutable& mut, const node_fields_2_t& node, index_t index, int change) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   }
   static void append_key(
       NodeExtentMutable& mut, const key_t& key, char*& p_append) {
@@ -379,12 +379,12 @@ struct internal_fields_3_t {
       NodeExtentMutable& mut, const Key& key,
       const internal_fields_3_t& node,
       index_t index, node_offset_t size_right) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   }
   static void update_size_at(
       NodeExtentMutable& mut, const internal_fields_3_t& node,
       index_t index, int change) {
-    ceph_abort("not implemented");
+    ceph_abort_msg("not implemented");
   }
 
   node_header_t header;
index 6b5ab13645ed7bac99c072ab0590e1384238acd7..5fd877b3015c4a41ff2281306a5f5ecf01d5ba58 100644 (file)
@@ -108,7 +108,7 @@ inline void assert_mstat(
     }
     break;
    default:
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
   // key == index ...
   switch (mstat) {
@@ -471,7 +471,7 @@ struct staged {
         assert(_index + 1 == container.keys());
         return typename container_t::template Appender<KT>(p_mut, container, true);
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
 
@@ -809,7 +809,7 @@ struct staged {
       if constexpr (!IS_BOTTOM) {
         return typename container_t::template Appender<KT>(p_mut, container, true);
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
 
@@ -1174,7 +1174,7 @@ struct staged {
       auto match = key <=> iter.get_key();
       if (match == 0) {
         if constexpr (IS_BOTTOM) {
-          ceph_abort("insert conflict at current index!");
+          ceph_abort_msg("insert conflict at current index!");
         } else {
           // insert into the current index
           auto nxt_container = iter.get_nxt_container();
@@ -1204,7 +1204,7 @@ struct staged {
       assert(match == std::strong_ordering::equal);
       if constexpr (IS_BOTTOM) {
         // ceph_abort?
-        ceph_abort("insert conflict at the previous index!");
+        ceph_abort_msg("insert conflict at the previous index!");
       } else {
         // insert into the previous index
         auto nxt_container = iter.get_nxt_container();
@@ -1225,7 +1225,7 @@ struct staged {
       return true;
     } else {
       if constexpr (IS_BOTTOM) {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       } else {
         assert(stage < STAGE);
         bool compensate = NXT_STAGE_T::
@@ -1379,7 +1379,7 @@ struct staged {
         iter.update_size(mut, _insert_size);
         return p_value;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
   }
@@ -1400,7 +1400,7 @@ struct staged {
         stage = STAGE;
         _insert_size = insert_size(key, value);
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
       if constexpr (IS_BOTTOM) {
         return container_t::insert_at(
@@ -1640,14 +1640,14 @@ struct staged {
         }
         return this->_nxt;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
     typename NXT_STAGE_T::StagedIterator& get_nxt() {
       if constexpr (!IS_BOTTOM) {
         return this->_nxt;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
     StagedIterator& operator++() {
@@ -1933,7 +1933,7 @@ struct staged {
           return false;
         }
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
         return false;;
       }
     }
@@ -1998,7 +1998,7 @@ struct staged {
           auto nxt_container = iter.get_nxt_container();
           this->_nxt.init_tail(p_mut, nxt_container, stage);
         } else {
-          ceph_abort("impossible path");
+          ceph_abort_msg("impossible path");
         }
       }
     }
@@ -2050,7 +2050,7 @@ struct staged {
         this->_nxt.init_empty(p_mut, p_append);
         return this->_nxt;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
     typename NXT_STAGE_T::template StagedAppender<KT>&
@@ -2062,7 +2062,7 @@ struct staged {
         this->_nxt.init_empty(p_mut, p_append);
         return this->_nxt;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
     typename NXT_STAGE_T::template StagedAppender<KT>& get_nxt() {
@@ -2070,7 +2070,7 @@ struct staged {
         assert(require_wrap_nxt);
         return this->_nxt;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
     void wrap_nxt() {
@@ -2081,7 +2081,7 @@ struct staged {
         appender->wrap_nxt(p_append);
         ++_index;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
    private:
@@ -2197,7 +2197,7 @@ struct staged {
         }
         return false;
       } else {
-        ceph_abort("impossible path");
+        ceph_abort_msg("impossible path");
       }
     }
   }
@@ -2360,13 +2360,13 @@ struct staged {
         compensate += ret_compensate;
         return {ret_stage, compensate};
       } else {
-        ceph_abort("impossible path: left_pivot_key == right_first_key");
+        ceph_abort_msg("impossible path: left_pivot_key == right_first_key");
       }
     } else if (cmp == std::strong_ordering::less) {
       // ok, do merge here
       return {STAGE, compensate};
     } else {
-      ceph_abort("impossible path: left_pivot_key < right_first_key");
+      ceph_abort_msg("impossible path: left_pivot_key < right_first_key");
     }
   }
 };
index 9bbfd5d6063b8d2c4edb95a96ec83e7f5386852a..7092bdf397f1c6d8d54f01f86739dcc3602d0675 100644 (file)
@@ -91,7 +91,7 @@ struct MatchHistory {
     } else if (*match == MatchKindCMP::GT) {
       return os << "GT";
     } else {
-      ceph_abort("impossble path");
+      ceph_abort_msg("impossble path");
     }
   }
 
@@ -318,7 +318,7 @@ const staged_position_t<STAGE>& cast_down(const search_position_t& pos) {
 #endif
     return pos.nxt.nxt;
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -340,7 +340,7 @@ staged_position_t<STAGE>& cast_down_fill_0(search_position_t& pos) {
     pos.nxt.index = 0;
     return pos.nxt.nxt;
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
@@ -356,7 +356,7 @@ search_position_t normalize(staged_position_t<STAGE>&& pos) {
   } else if (STAGE == STAGE_RIGHT) {
     return {0u, {0u, std::move(pos)}};
   } else {
-    ceph_abort("impossible path");
+    ceph_abort_msg("impossible path");
   }
 }
 
index 68f8a55e6c76dc7946e00c5eed63084eb8b0606f..7121bc8e0f9bc39233c6135cbeda5ee7d3a78c59 100644 (file)
@@ -68,7 +68,7 @@ class Btree {
         // we don't actually store end cursor because it will hold a reference
         // to an end leaf node and is not kept updated.
         assert(p_cursor->is_end());
-        ceph_abort("impossible");
+        ceph_abort_msg("impossible");
       }
     }
 
@@ -130,7 +130,7 @@ class Btree {
     Cursor(Btree* p_tree, Ref<tree_cursor_t> _p_cursor) : p_tree(p_tree) {
       if (_p_cursor->is_invalid()) {
         // we don't create Cursor from an invalid tree_cursor_t.
-        ceph_abort("impossible");
+        ceph_abort_msg("impossible");
       } else if (_p_cursor->is_end()) {
         // we don't actually store end cursor because it will hold a reference
         // to an end leaf node and is not kept updated.
index 0238307a9745db994331d8205d5314289eb250e1..a5788367515f002ee31c26fa8c2de3ac352cecb1 100644 (file)
@@ -216,7 +216,7 @@ public:
       if (length != pin.get_length() || !pin.get_val().is_real_location()) {
         SUBERRORT(seastore_tm, "{}~0x{:x} {} got wrong pin {}",
                   t, offset, length, T::TYPE, pin);
-        ceph_abort("Impossible");
+        ceph_abort_msg("Impossible");
       }
       return this->read_pin<T>(t, std::move(pin), std::move(maybe_init));
     });
@@ -243,7 +243,7 @@ public:
       if (!pin.get_val().is_real_location()) {
         SUBERRORT(seastore_tm, "{} {} got wrong pin {}",
                   t, offset, T::TYPE, pin);
-        ceph_abort("Impossible");
+        ceph_abort_msg("Impossible");
       }
       return this->read_pin<T>(t, std::move(pin), std::move(maybe_init));
     });
index ee65646205ece299cf33676e36ebf0ffed3e129b..79a00af73b6a355e0651ceb5cd3e3248361d731d 100644 (file)
@@ -24,7 +24,7 @@ seastar::future<core_id_t> PGShardMapping::get_or_create_pg_mapping(
     if (core_expected != NULL_CORE && core_expected != core_found) {
       ERROR("the mapping is inconsistent for pg {}: core {}, expected {}",
             pgid, core_found, core_expected);
-      ceph_abort("The pg mapping is inconsistent!");
+      ceph_abort_msg("The pg mapping is inconsistent!");
     }
     return seastar::make_ready_future<core_id_t>(core_found);
   } else {
@@ -43,7 +43,7 @@ seastar::future<core_id_t> PGShardMapping::get_or_create_pg_mapping(
           if (core_expected != core_found) {
             ERROR("the mapping is inconsistent for pg {} (primary): core {}, expected {}",
                   pgid, core_found, core_expected);
-            ceph_abort("The pg mapping is inconsistent!");
+            ceph_abort_msg("The pg mapping is inconsistent!");
           }
           // core_expected == core_found
           DEBUG("mapping pg {} to core {} (primary): already mapped and expected",
@@ -94,7 +94,7 @@ seastar::future<core_id_t> PGShardMapping::get_or_create_pg_mapping(
           if (core_found != core_to_update) {
             ERROR("the mapping is inconsistent for pg {} (others): core {}, expected {}",
                   pgid, core_found, core_to_update);
-            ceph_abort("The pg mapping is inconsistent!");
+            ceph_abort_msg("The pg mapping is inconsistent!");
           }
           DEBUG("mapping pg {} to core {} (others): already mapped",
                 pgid, core_to_update);
@@ -105,13 +105,13 @@ seastar::future<core_id_t> PGShardMapping::get_or_create_pg_mapping(
       if (find_iter == pg_to_core.end()) {
         ERROR("the mapping is inconsistent for pg {}: core not found, expected {}",
               pgid, core_expected);
-        ceph_abort("The pg mapping is inconsistent!");
+        ceph_abort_msg("The pg mapping is inconsistent!");
       }
       auto core_found = find_iter->second;
       if (core_expected != NULL_CORE && core_found != core_expected) {
         ERROR("the mapping is inconsistent for pg {}: core {}, expected {}",
               pgid, core_found, core_expected);
-        ceph_abort("The pg mapping is inconsistent!");
+        ceph_abort_msg("The pg mapping is inconsistent!");
       }
       DEBUG("returning pg {} mapping to core {} after broadcasted",
             pgid, core_found);
@@ -125,7 +125,7 @@ seastar::future<> PGShardMapping::remove_pg_mapping(spg_t pgid) {
   auto find_iter = pg_to_core.find(pgid);
   if (find_iter == pg_to_core.end()) {
     ERROR("trying to remove non-exist mapping for pg {}", pgid);
-    ceph_abort("The pg mapping is inconsistent!");
+    ceph_abort_msg("The pg mapping is inconsistent!");
   }
   DEBUG("calling primary to remove mapping for pg {}", pgid);
   return container().invoke_on(
@@ -133,7 +133,7 @@ seastar::future<> PGShardMapping::remove_pg_mapping(spg_t pgid) {
     auto find_iter = primary_mapping.pg_to_core.find(pgid);
     if (find_iter == primary_mapping.pg_to_core.end()) {
       ERROR("trying to remove non-exist mapping for pg {} (primary)", pgid);
-      ceph_abort("The pg mapping is inconsistent!");
+      ceph_abort_msg("The pg mapping is inconsistent!");
     }
     assert(find_iter->second != NULL_CORE);
     auto count_iter = primary_mapping.core_to_num_pgs.find(find_iter->second);
@@ -147,7 +147,7 @@ seastar::future<> PGShardMapping::remove_pg_mapping(spg_t pgid) {
       auto find_iter = other_mapping.pg_to_core.find(pgid);
       if (find_iter == other_mapping.pg_to_core.end()) {
         ERROR("trying to remove non-exist mapping for pg {} (others)", pgid);
-        ceph_abort("The pg mapping is inconsistent!");
+        ceph_abort_msg("The pg mapping is inconsistent!");
       }
       assert(find_iter->second != NULL_CORE);
       other_mapping.pg_to_core.erase(find_iter);
index 5c25409a80a4cc42027cdff2b7939765f4484ea5..c4c1382f2c25efc856a5557fe189b87f5e79677b 100644 (file)
@@ -91,7 +91,7 @@ void RecoveryBackend::WaitForObjectRecovery::interrupt(interrupt_cause_t why) {
     pushes.clear();
     break;
   default:
-    ceph_abort("impossible");
+    ceph_abort_msg("impossible");
     break;
   }
 }
index 511e8aa865f3aa27b5b970d24f01b70969526ead..7472dd62dddb39080751c8cada961e39ef8b5a5f 100644 (file)
@@ -339,7 +339,7 @@ struct ScrubState : sc::state<S, P, T...> {
 struct Crash : ScrubState<Crash, ScrubMachine> {
   static constexpr std::string_view state_name = "Crash";
   explicit Crash(my_context ctx) : ScrubState(ctx) {
-    ceph_abort("Crash state impossible");
+    ceph_abort_msg("Crash state impossible");
   }
 
 };
index c9168aaf36fd74d607f584fd083356b57d350aac..6a983648d04bc66919af6f533aeb8bb34d0f1f97 100644 (file)
@@ -772,7 +772,7 @@ seastar::future<Ref<PG>> ShardServices::load_pg(spg_t pgid)
     });
   }).handle_exception([FNAME, pgid](auto ep) {
     INFO("pg {} saw exception on load {}", pgid, ep);
-    ceph_abort("Could not load pg" == 0);
+    ceph_abort_msg("Could not load pg");
     return seastar::make_exception_future<Ref<PG>>(ep);
   });
 }
index 66c2dc447332deede928bd41eb3ee8eff07984a8..d2682a3440c3b22491de64aac1202431922c4604 100644 (file)
@@ -199,7 +199,7 @@ static seastar::future<> run(
       void ms_handle_connect(
           crimson::net::ConnectionRef,
           seastar::shard_id) override {
-        ceph_abort("impossible, server won't connect");
+        ceph_abort_msg("impossible, server won't connect");
       }
 
       void ms_handle_accept(