From: Xuehan Xu Date: Mon, 29 Apr 2024 08:56:09 +0000 (+0800) Subject: crimson/os/seastore/transaction_manager: remove incorrect assertions X-Git-Tag: v20.0.0~2066^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F57135%2Fhead;p=ceph.git crimson/os/seastore/transaction_manager: remove incorrect assertions Indirect mappings always hold the checksums of the corresponding intermediate mappings, which may have non-zero Signed-off-by: Xuehan Xu --- diff --git a/src/crimson/os/seastore/transaction_manager.h b/src/crimson/os/seastore/transaction_manager.h index 6d99bd77ec2f..c7e4121263a9 100644 --- a/src/crimson/os/seastore/transaction_manager.h +++ b/src/crimson/os/seastore/transaction_manager.h @@ -885,9 +885,7 @@ private: crc); assert(ref->is_fully_loaded()); bool inconsistent = false; - if (pin->is_indirect()) { - inconsistent = (pin->get_checksum() != 0); - } else if (full_extent_integrity_check) { + if (full_extent_integrity_check) { inconsistent = (pin->get_checksum() != crc); } else { // !full_extent_integrity_check: remapped extent may be skipped inconsistent = !(pin->get_checksum() == 0 || @@ -953,9 +951,7 @@ private: crc); assert(ref->is_fully_loaded()); bool inconsistent = false; - if (pin->is_indirect()) { - inconsistent = (pin->get_checksum() != 0); - } else if (full_extent_integrity_check) { + if (full_extent_integrity_check) { inconsistent = (pin->get_checksum() != crc); } else { // !full_extent_integrity_check: remapped extent may be skipped inconsistent = !(pin->get_checksum() == 0 ||