Skip to content

Commit 8e92395

Browse files
committed
crimson/os/seastore/transaction: add asserts related to read-set
Signed-off-by: Yingxin Cheng <[email protected]>
1 parent 1a4c0bc commit 8e92395

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

src/crimson/os/seastore/transaction.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -165,6 +165,7 @@ class Transaction {
165165
bool is_paddr_known;
166166
};
167167
maybe_add_readset_ret maybe_add_to_read_set(CachedExtentRef ref) {
168+
assert(ref->is_stable());
168169
assert(ref->get_paddr().is_absolute()
169170
|| ref->get_paddr().is_record_relative());
170171
if (is_weak()) {
@@ -177,6 +178,7 @@ class Transaction {
177178
} else {
178179
// paddr is unknown until wait_io() finished
179180
// to call maybe_add_to_read_set_step_2(ref)
181+
assert(ref->is_pending_io());
180182
ceph_assert(ref->get_paddr().is_record_relative());
181183
bool added = maybe_add_to_read_set_step_1(ref);
182184
return {added, false};
@@ -673,6 +675,7 @@ class Transaction {
673675

674676
void maybe_add_to_read_set_step_2(CachedExtentRef ref) {
675677
// paddr must be known for read_set
678+
assert(ref->is_stable_ready());
676679
ceph_assert(ref->get_paddr().is_absolute());
677680
if (is_weak()) {
678681
return;

0 commit comments

Comments
 (0)