crimson/os/seastore: add FIXME to consider pending extent with absolute paddr

During object data overwrite.

Signed-off-by: Yingxin Cheng <yingxin.cheng@intel.com>
This commit is contained in:
Yingxin Cheng 2023-01-11 10:04:14 +08:00
parent 14cab7365c
commit b78668eec8
2 changed files with 3 additions and 0 deletions

View File

@ -415,6 +415,7 @@ private:
actual_write_size -= left_ext_size;
left_ext_size = 0;
left_operation = overwrite_operation_t::OVERWRITE_ZERO;
// FIXME: left_paddr can be absolute and pending
} else if (left_paddr.is_relative() ||
left_paddr.is_delayed()) {
aligned_data_size += left_ext_size;
@ -426,6 +427,7 @@ private:
actual_write_size -= right_ext_size;
right_ext_size = 0;
right_operation = overwrite_operation_t::OVERWRITE_ZERO;
// FIXME: right_paddr can be absolute and pending
} else if (right_paddr.is_relative() ||
right_paddr.is_delayed()) {
aligned_data_size += right_ext_size;

View File

@ -355,6 +355,7 @@ public:
paddr_t existing_paddr,
extent_len_t length) {
LOG_PREFIX(TransactionManager::map_existing_extent);
// FIXME: existing_paddr can be absolute and pending
ceph_assert(existing_paddr.is_absolute());
assert(t.is_retired(existing_paddr, length));