[dm-devel] [PATCH] dm verity fec: Don't add data_blocks to block
Sami Tolvanen
samitolvanen at google.com
Tue Apr 7 15:55:39 UTC 2020
On Mon, Apr 6, 2020 at 11:54 PM Sunwook Eom <speed.eom at samsung.com> wrote:
>
> Even if block type is metadata,
> block in verity_fec_decode() has already the right block number.
> So there is no need to add data_blocks to block.
Is this also true if the hashes are on a separate block device?
The idea here is that the error correction data was computed over both
data and hash blocks, as if they were concatenated, and we want to
calculate the logical block number based on that. I agree that the
code doesn't look quite right though. Should we use something like
this instead?
if (type == DM_VERITY_BLOCK_TYPE_METADATA)
block = block - v->hash_start + v->data_blocks;
Sami
More information about the dm-devel
mailing list