crimson/os/seastore: drop journal_seq_t NO_DELTAS

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>

diff --git a/src/crimson/os/seastore/cache.cc b/src/crimson/os/seastore/cache.cc
index c889861ffe3e9883463f7de13d7e00f9e0dd6e89..b91a7272d9aedc51ddc1c3f357b8cbfd22bf1324 100644 (file)
--- a/src/crimson/os/seastore/cache.cc
+++ b/src/crimson/os/seastore/cache.cc
@@ -759,7 +759,7 @@ void Cache::commit_retire_extent(
 {
   remove_extent(ref);
 
-  ref->dirty_from_or_retired_at = JOURNAL_SEQ_MAX;
+  ref->dirty_from_or_retired_at = JOURNAL_SEQ_NULL;
   invalidate_extent(t, *ref);
 }
 
@@ -1753,10 +1753,8 @@ Cache::get_next_dirty_extents_ret Cache::get_next_dirty_extents(
        i != dirty.end() && bytes_so_far < max_bytes;
        ++i) {
     auto dirty_from = i->get_dirty_from();
-    if (unlikely(!(dirty_from != JOURNAL_SEQ_NULL &&
-                dirty_from != JOURNAL_SEQ_MAX &&
-                dirty_from != NO_DELTAS))) {
-      ERRORT("{}", t, *i);
+    if (unlikely(dirty_from == JOURNAL_SEQ_NULL)) {
+      ERRORT("got dirty extent with JOURNAL_SEQ_NULL -- {}", t, *i);
       ceph_abort();
     }
     if (dirty_from < seq) {

diff --git a/src/crimson/os/seastore/journal/segment_allocator.cc b/src/crimson/os/seastore/journal/segment_allocator.cc
index 0b22f3ad8784e1489a0228b049c51256d4e98b6a..81c937bfda984ee51a39a6a320301238848e18f2 100644 (file)
--- a/src/crimson/os/seastore/journal/segment_allocator.cc
+++ b/src/crimson/os/seastore/journal/segment_allocator.cc
@@ -76,8 +76,8 @@ SegmentAllocator::do_open(bool is_mkfs)
       }
     } else { // OOL
       ceph_assert(!is_mkfs);
-      dirty_tail = NO_DELTAS;
-      alloc_tail = NO_DELTAS;
+      dirty_tail = JOURNAL_SEQ_NULL;
+      alloc_tail = JOURNAL_SEQ_NULL;
     }
     auto header = segment_header_t{
       new_segment_seq,

diff --git a/src/crimson/os/seastore/seastore_types.cc b/src/crimson/os/seastore/seastore_types.cc
index 72ab3b81f051a9635095cd8f2e93ebba2c310ca5..6307f303061b9ee3656aead689e3922330333b1a 100644 (file)
--- a/src/crimson/os/seastore/seastore_types.cc
+++ b/src/crimson/os/seastore/seastore_types.cc
@@ -136,8 +136,6 @@ std::ostream &operator<<(std::ostream &out, const journal_seq_t &seq)
     return out << "JOURNAL_SEQ_NULL";
   } else if (seq == JOURNAL_SEQ_MIN) {
     return out << "JOURNAL_SEQ_MIN";
-  } else if (seq == NO_DELTAS) {
-    return out << "JOURNAL_SEQ_NO_DELTAS";
   } else {
     return out << "journal_seq_t("
                << "segment_seq=" << segment_seq_printer_t{seq.segment_seq}

diff --git a/src/crimson/os/seastore/seastore_types.h b/src/crimson/os/seastore/seastore_types.h
index a7849b0a20e66376d1115eed89344440b7c277c3..e4f4fa61e8c6522442ed4890c8497015e940fd64 100644 (file)
--- a/src/crimson/os/seastore/seastore_types.h
+++ b/src/crimson/os/seastore/seastore_types.h
@@ -854,10 +854,6 @@ constexpr journal_seq_t JOURNAL_SEQ_MAX{
 };
 // JOURNAL_SEQ_NULL == JOURNAL_SEQ_MAX == journal_seq_t{}
 constexpr journal_seq_t JOURNAL_SEQ_NULL = JOURNAL_SEQ_MAX;
-constexpr journal_seq_t NO_DELTAS = journal_seq_t{
-  NULL_SEG_SEQ,
-  P_ADDR_ZERO
-};
 
 // logical addr, see LBAManager, TransactionManager
 using laddr_t = uint64_t;