On 3/19/24 23:42, David Sterba wrote:
From: Josef Bacik <josef@xxxxxxxxxxxxxx>
A long time ago we changed the short options to long options in
btrfs-corrupt-block, so adjust the helper to use the correct options so
the verity tests pass properly.
Signed-off-by: Josef Bacik <josef@xxxxxxxxxxxxxx>
---
common/verity | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/common/verity b/common/verity
index 03d175ce1b7a18..0e5f0d75e746a8 100644
--- a/common/verity
+++ b/common/verity
@@ -402,7 +402,7 @@ _fsv_scratch_corrupt_merkle_tree()
# in the default filesystem tree (-r 5) and corrupt one byte (-b 1) at
# $offset (-o $offset) with the ascii representation of the byte we read
# (-v $ascii)
- $BTRFS_CORRUPT_BLOCK_PROG -r 5 -I $ino,37,0 -v $ascii -o $offset -b 1 $SCRATCH_DEV
+ $BTRFS_CORRUPT_BLOCK_PROG -r 5 -I $ino,37,0 --value $ascii --offset $offset -b 1 $SCRATCH_DEV
Can we still make it work with the older btrfs-corrupt-block?
How about..
corrupt_block_value_opt()
{
$BTRFS_CORRUPT_BLOCK_PROG -h 2>&1 | grep -q -e "--value"
if [ $? == 0 ]; then
echo "--value"
else
echo "-v"
fi
}
And to use,
$BTRFS_CORRUPT_BLOCK_PROG -r 5 -I $ino,37,0 $(corrupt_block_value_opt)
$ascii --offset $offset -b 1 $SCRATCH_DEV
I will make this change before submitting the PR if no objection.
Thanks, Anand
(( offset += 1 ))
done
_scratch_mount
