Skip to content

Commit 2d2df50

Browse files
committed
crimson/os/seastore/transaction_manager: minor refinement to remap_pin() asserts
Signed-off-by: Yingxin Cheng <[email protected]>
1 parent bc97b82 commit 2d2df50

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

src/crimson/os/seastore/transaction_manager.h

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -425,12 +425,17 @@ class TransactionManager : public ExtentCallbackInterface {
425425
for (auto &remap : remaps) {
426426
auto remap_offset = remap.offset;
427427
auto remap_len = remap.len;
428+
assert(remap_len > 0);
428429
total_remap_len += remap.len;
429-
ceph_assert(remap_offset >= (last_offset + last_len));
430+
assert(remap_offset >= (last_offset + last_len));
430431
last_offset = remap_offset;
431432
last_len = remap_len;
432433
}
433-
ceph_assert(total_remap_len < original_len);
434+
if (remaps.size() == 1) {
435+
assert(total_remap_len < original_len);
436+
} else {
437+
assert(total_remap_len <= original_len);
438+
}
434439
#endif
435440

436441
// FIXME: paddr can be absolute and pending

0 commit comments

Comments
 (0)