From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 915D42F7F1F; Thu, 23 Jul 2026 14:29:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.137.202.133 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784816960; cv=none; b=S49c2ass0Eo+cNnunpjTvtWD52C2PPIP4yURMZw96tJyf4y9GSE0mDBl7hUsYkL5xPnKCZH0zwek/L6u1JjX4b0yT33OcvMzsHlF80wBxhcuAkCZLVc4hKfkFW9WdN9S1u3/8WiPy/fPdK1a36I62QfojhNp3mr1PyvyHrnOrqU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784816960; c=relaxed/simple; bh=efMU6x1fC1b3nxYu2HSHbpMFWJNKuo8/owSXw4rJ0tE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Bd2t8apxNgbqgdnvdMNdMj+ONlkrWiKOqz5O+dktRKunYJvRpvcmAVSdEhLw/H7Vxd3OUBcpZAjrsHoYcTRo3Om3vyHhpM0cpB7ZO1Xz0mg7Kuj3hSSFojZhvzumrDWIctLYGcVjD7SGTqyP9KJdGkKvQhH5B3ZhyXes8CIW/Lk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=lst.de; spf=none smtp.mailfrom=bombadil.srs.infradead.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b=qNLvgJ4/; arc=none smtp.client-ip=198.137.202.133 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=lst.de Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=bombadil.srs.infradead.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="qNLvgJ4/" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: MIME-Version:References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Sender :Reply-To:Content-Type:Content-ID:Content-Description; bh=0sk6qT5A4LcZ7QxWhiv2AYq+MuzdKU5OXW7pFHED23o=; b=qNLvgJ4/0XJ2X0IE83NSZ5pro8 1d/tL/+uBrCOSs1lNmUygi77W96+GK3lUuqUlmtgWtbq/UtYNfhXk7DdsivprbaSryglCSgFapNLa i8M9j+M/+umGtv6CiU+3CRMce1GxYh6bDgTkpK1kYqBGiGQs2+AvG7j8czPs6X6NrND6uWWKJgF96 gqKhC4wQCgKfKTWjsa+9FpeRvJm5gFM31OAtpPxBDNd8DXf/H2v3PSYDYK6o4FeJDvFBfJtzMvJ+y +TaGzS0ZpFposoQAqMDph1lG7bh4anKCFx0Q5NCHgk464INTxKS8q1bN1t6I1ENJsJQ6hZbfPn/OJ ybyKMdVA==; Received: from [2001:4bb8:2e9:b20c:3d3c:bb0:5e13:19af] (helo=localhost) by bombadil.infradead.org with esmtpsa (Exim 4.99.1 #2 (Red Hat Linux)) id 1wmuQH-0000000EV9r-1ze9; Thu, 23 Jul 2026 14:29:17 +0000 From: Christoph Hellwig To: Zorro Lang Cc: fstests@vger.kernel.org, "Martin K. Petersen" , Kanchan Joshi , Anuj Gupta , linux-scsi@vger.kernel.org Subject: [PATCH 4/4] generic: test corruption detection using T10 protection information Date: Thu, 23 Jul 2026 16:28:47 +0200 Message-ID: <20260723142856.108193-5-hch@lst.de> X-Mailer: git-send-email 2.53.0 In-Reply-To: <20260723142856.108193-1-hch@lst.de> References: <20260723142856.108193-1-hch@lst.de> Precedence: bulk X-Mailing-List: linux-scsi@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Add a test that simulates bit flips in data and misdirected writes and checks that file systems detect it when run on devices using protection information. This requires the new scsi_debug corruption injection. Signed-off-by: Christoph Hellwig --- tests/generic/2302 | 143 +++++++++++++++++++++++++++++++++++++++++ tests/generic/2302.out | 21 ++++++ 2 files changed, 164 insertions(+) create mode 100755 tests/generic/2302 create mode 100644 tests/generic/2302.out diff --git a/tests/generic/2302 b/tests/generic/2302 new file mode 100755 index 000000000000..5fff1f5e5015 --- /dev/null +++ b/tests/generic/2302 @@ -0,0 +1,143 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0 +# Copyright (c) 2026 Christoph Hellwig +# +# FS QA Test No. 2302 +# +# Test that T10 DIF detects bitflips and misplaced writes. +# +. ./common/preamble +_begin_fstest auto rw + +_cleanup() +{ + cd / + rm -r -f $tmp.* + [ -d "$SCSI_DEBUG_MNT" ] && _unmount $SCSI_DEBUG_MNT 2>/dev/null + _put_scsi_debug_dev +} + +. ./common/scsi_debug +. ./common/filter + +_require_scsi_debug +# If TEST_DEV is block device, make sure current fs is a localfs which can be +# written on scsi_debug device +_require_test +_require_block_device $TEST_DEV + +# FIEMAP on Btrfs returns logical addresses within the filesystem's address +# space, not physical device offsets. Writing to these offsets on $SCRATCH_DEV +# would corrupt the filesystem in multi-device setups. +_exclude_fs btrfs + +size=$(_small_fs_size_mb 256) + +# Get the physical block offset (in bytes) of the file's first block on device +get_phys_blk() +{ + local file="$1" + local fiemap_output + local phys_blk + + fiemap_output=$($XFS_IO_PROG -r -c "fiemap -v" "$file" 2>/dev/null) + phys_blk=$(echo "$fiemap_output" | _filter_xfs_io_fiemap | head -1 | awk '{print $3}') + if [ -z "$phys_blk" ]; then + echo "" + return + fi + echo $phys_blk +} + +test_dif() +{ + local pi_type=$1 + local last=$2 + local bit_errors=$3 + local reftag_adjust=$4 + + echo -n "Testing DIF type $pi_type " + if [ "$bit_errors" -gt "0" ]; then + echo -n "bit errors ($bit_errors) in " + fi + if [ "$reftag_adjust" -ne "0" ]; then + echo -n "reftag adjustment by $reftag_adjust in " + fi + if [ "$last" -eq 1 ]; then + echo "last sector" + else + echo "first sector" + fi + + scsi_debug_pi_opts="write_same_length=0 dif=${pi_type} dix=1" + SCSI_DEBUG_DEV=`_get_scsi_debug_dev 512 512 0 $size $scsi_debug_pi_opts` + + SCSI_DEBUG_MNT="$TEST_DIR/scsi_debug_$seq" + rm -rf $SCSI_DEBUG_MNT + mkdir $SCSI_DEBUG_MNT + + _mkfs_dev $SCSI_DEBUG_DEV || \ + _fail "Can't make $FSTYP on DIF-enabled scsi_debug device" + run_check _mount $SCSI_DEBUG_DEV $SCSI_DEBUG_MNT + + local blksz=$(_get_block_size $SCSI_DEBUG_MNT) + local testfile="$SCSI_DEBUG_MNT/test" + + $XFS_IO_PROG -d -f \ + -c "pwrite -S 0x66 0 $((128 * blksz))" \ + -c fsync \ + $testfile >> $seqres.full 2>&1 + + local phys_blk=$(get_phys_blk $testfile) + if [ -z "$phys_blk" ]; then + _fail "$test_name: failed to get physical block offset via fiemap" + fi + + local dev=$(_short_dev $SCSI_DEBUG_DEV) + local debugfs_dir=$(ls /sys/block/$dev/device/scsi_device/) + local debugfs_file="/sys/kernel/debug/scsi_debug/$debugfs_dir/corrupt" + + if [ ! -f "$debugfs_file" ]; then + _notrun "scsi_debug corruption injection not supported" + fi + + if [ "$last" -eq 1 ]; then + echo "adding to $phys_blk" >> $seqres.full + phys_blk=$((phys_blk + 127)) + fi + + echo "lba=$phys_blk,num=1,bit_errors=$bit_errors,reftag_adjust=$reftag_adjust" \ + > $debugfs_file + + $XFS_IO_PROG -d -f \ + -c "pread 0 $((128 * blksz))" \ + $testfile + + [ -d "$SCSI_DEBUG_MNT" ] && _unmount $SCSI_DEBUG_MNT 2>/dev/null + _put_scsi_debug_dev +} + +# test data corruption in first block +test_dif 1 0 2 0 +test_dif 2 0 2 0 +test_dif 3 0 2 0 + +# test data corruption in last block +test_dif 1 1 2 0 +test_dif 2 1 2 0 +test_dif 3 1 2 0 + + +# Note: Type 3 doesn't have a refrag, so don't test it + +# reftag mismatch in first block +test_dif 1 0 0 8 +test_dif 2 0 0 8 + +# reftag mismatch in last block +test_dif 1 1 0 8 +test_dif 2 1 0 8 + +# success, all done +status=0 +exit diff --git a/tests/generic/2302.out b/tests/generic/2302.out new file mode 100644 index 000000000000..03b68d904943 --- /dev/null +++ b/tests/generic/2302.out @@ -0,0 +1,21 @@ +QA output created by 2302 +Testing DIF type 1 bit errors (2) in first sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 2 bit errors (2) in first sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 3 bit errors (2) in first sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 1 bit errors (2) in last sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 2 bit errors (2) in last sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 3 bit errors (2) in last sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 1 reftag adjustment by 8 in first sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 2 reftag adjustment by 8 in first sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 1 reftag adjustment by 8 in last sector +pread: Invalid or incomplete multibyte or wide character +Testing DIF type 2 reftag adjustment by 8 in last sector +pread: Invalid or incomplete multibyte or wide character -- 2.53.0