]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
osd: set r only if succeed in FillInVerifyExtent 44174/head
authoryanqiang-ux <yanqiang_ux@163.com>
Mon, 7 Jun 2021 07:54:44 +0000 (15:54 +0800)
committerCory Snyder <csnyder@iland.com>
Wed, 1 Dec 2021 20:12:43 +0000 (15:12 -0500)
commit83384469950349ba6a1faa1e14eda87973652fb0
tree8b2d80165ad6ba39ed309408e3340009cf8a3c5e
parentfe63bec3382a149b52666f29358580c48ad8772b
osd: set r only if succeed in FillInVerifyExtent

When read failed, ret can be taken as data len in FillInVerifyExtent, which should be avoided.
It may cause errors in crc repair or retry read because of the data len. In my case, we use FillInVerifyExtent for EC read,
when meet -EIO,we will try crc repair, which need read data from other shard accrding to data len.
And I meet assert in ECBackend.cc (loc: line 2288 ceph_assert(range.first != range.second) ), But it seems master branch not support EC crc repair.
In shot, when reuse the readop may cause unpredictable error.

Fixes: https://tracker.ceph.com/issues/51115
Signed-off-by: yanqiang-ux <yanqiang_ux@163.com>
(cherry picked from commit 127745161fbcdee06b2dfa8464270c3934bcd06a)
src/osd/PrimaryLogPG.cc