From patchwork Wed Nov 9 11:43:34 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Filipe Manana X-Patchwork-Id: 13037446 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 40961C43217 for ; Wed, 9 Nov 2022 11:44:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231169AbiKILoR (ORCPT ); Wed, 9 Nov 2022 06:44:17 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55992 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230099AbiKILoD (ORCPT ); Wed, 9 Nov 2022 06:44:03 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1ED686252; Wed, 9 Nov 2022 03:43:45 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id AD61B61A38; Wed, 9 Nov 2022 11:43:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 5574EC433D7; Wed, 9 Nov 2022 11:43:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1667994224; bh=TwISZ/nzIRTyL2L5TzItZDMk/yrP1vq2gi0CtQk4l20=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=bcrbTA8pCAYIM8dcb87uzw1/exHrH/FxqhobcgJh9+0nizSYRNvxwpuTWZzrbFiuT m0BycntYr5HvL9GyVeW/6wZ8KZCrLVhMGeypTjk9pHKdJwyfV8tAHb4NRRdfj34krP 6y6VYHS7jrZ3Jd7M2m53ABZv6HM7xtQDZ1NY/D5fHWGkJb9NCNL71O6BxtDSx1Pamw yJC4wJ8y/JX7bZw2nt+pt2pRP8TTLgU20PhZpN6N2+sn8420qF1Szp4hwFOvEg9JtJ sxGumDGMGa9fdyCD+IQW+yDDueNB1KYcW35SEniwtS/pqdaUnKxQkNHYAQjlfIZqca HlaPtsuroJw1g== From: fdmanana@kernel.org To: fstests@vger.kernel.org Cc: linux-btrfs@vger.kernel.org, Filipe Manana Subject: [PATCH 1/3] btrfs/003: fix failure on new btrfs-progs versions Date: Wed, 9 Nov 2022 11:43:34 +0000 Message-Id: <62ef22111c9cb654e6e5e50f7337105b9ef804d7.1667993961.git.fdmanana@suse.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org From: Filipe Manana Starting with btrfs-progs version 5.19, the output of 'filesystem show' command changed when we have a missing device. The old output was like the following: Label: none uuid: 139ef309-021f-4b98-a3a8-ce230a83b1e2 Total devices 2 FS bytes used 128.00KiB devid 1 size 5.00GiB used 1.26GiB path /dev/loop0 *** Some devices missing While the new output (btrfs-progs 5.19+) is like the following: Label: none uuid: 4a85a40b-9b79-4bde-8e52-c65a550a176b Total devices 2 FS bytes used 128.00KiB devid 1 size 5.00GiB used 1.26GiB path /dev/loop0 devid 2 size 0 used 0 path /dev/loop1 MISSING More specifically it happened in the following btrfs-progs commit: 957a79c9b016 ("btrfs-progs: fi show: print missing device for a mounted file system") This is making btrfs/003 fail with btrfs-progs 5.19+. Update the grep filter in btrfs/003 so that it works with both output formats. Signed-off-by: Filipe Manana Reviewed-by: Zorro Lang --- tests/btrfs/003 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/btrfs/003 b/tests/btrfs/003 index cf605730..fae6d9d1 100755 --- a/tests/btrfs/003 +++ b/tests/btrfs/003 @@ -141,8 +141,9 @@ _test_replace() _devmgt_remove ${removed_dev_htl} $ds dev_removed=1 - $BTRFS_UTIL_PROG filesystem show $SCRATCH_DEV | grep "Some devices missing" >> $seqres.full || _fail \ - "btrfs did not report device missing" + $BTRFS_UTIL_PROG filesystem show $SCRATCH_DEV | \ + grep -ie '\bmissing\b' >> $seqres.full || \ + _fail "btrfs did not report device missing" # add a new disk to btrfs ds=${devs[@]:$(($n)):1} From patchwork Wed Nov 9 11:43:35 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Filipe Manana X-Patchwork-Id: 13037447 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 932D3C43219 for ; Wed, 9 Nov 2022 11:44:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231189AbiKILoS (ORCPT ); Wed, 9 Nov 2022 06:44:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55998 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231207AbiKILoD (ORCPT ); Wed, 9 Nov 2022 06:44:03 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5A72E11806; Wed, 9 Nov 2022 03:43:46 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id E4E0961A22; Wed, 9 Nov 2022 11:43:45 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8847AC433D6; Wed, 9 Nov 2022 11:43:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1667994225; bh=qBbngR9xj0bXu/su2pMcrG/L+urCliVi+jLqjt+8MhQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=lYQ/PN7X+WjW8RAu11RPC/E3pDzoofb0GjSDBQirH0HFYOHT0//ApC6cuqMidxGoZ VMqGfW0AI0I6bN00P1E0QY1tIls0/jY/Jdp0/dNXvnXm2FyargLG7cwkeh0fe1Le4S TZmnwjDnazJRP6CDqMxlqZIR/XEEZIhXQfCOZia+5ktFms/lt1nCDwr4g9NzEQwiQL VCX/NLfkDlHdDI+e41RXprwN1RfxRYIlRrH4lNTf/oTjpyXze7RaDhYPnYZWz8Aeh7 kDSr3mQymQ/o+dN9MeNhZfDYRrkclH75QEQWOVDMfBvAoxpuUWo7/hTi6CKiVZdu1U vIlSNv9GlIRTA== From: fdmanana@kernel.org To: fstests@vger.kernel.org Cc: linux-btrfs@vger.kernel.org, Filipe Manana Subject: [PATCH 2/3] btrfs/053: fix test failure when running with btrfs-progs v6.0+ Date: Wed, 9 Nov 2022 11:43:35 +0000 Message-Id: X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org From: Filipe Manana In btrfs-progs v6.0 the --leafsize (-l) command line option was removed, so btrfs/053 always fails with v6.0+. The change was introduced by the following btrfs-progs commit: f7a768d62498 ("btrfs-progs: mkfs: remove support for option --leafsize") Change the test to use --nodesize (-n) instead, since it exists in both old and new btrfs-progs versions. Signed-off-by: Filipe Manana --- tests/btrfs/053 | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/btrfs/053 b/tests/btrfs/053 index fbd2e7d9..c0446257 100755 --- a/tests/btrfs/053 +++ b/tests/btrfs/053 @@ -44,7 +44,7 @@ send_files_dir=$TEST_DIR/btrfs-test-$seq rm -fr $send_files_dir mkdir $send_files_dir -_scratch_mkfs "-l $leaf_size" >/dev/null 2>&1 +_scratch_mkfs "--nodesize $leaf_size" >/dev/null 2>&1 _scratch_mount echo "hello world" > $SCRATCH_MNT/foobar @@ -72,7 +72,7 @@ _run_btrfs_util_prog send -p $SCRATCH_MNT/mysnap1 -f $send_files_dir/2.snap \ _scratch_unmount _check_scratch_fs -_scratch_mkfs "-l $leaf_size" >/dev/null 2>&1 +_scratch_mkfs "--nodesize $leaf_size" >/dev/null 2>&1 _scratch_mount _run_btrfs_util_prog receive -f $send_files_dir/1.snap $SCRATCH_MNT From patchwork Wed Nov 9 11:43:36 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Filipe Manana X-Patchwork-Id: 13037448 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BACAAC4321E for ; Wed, 9 Nov 2022 11:44:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230099AbiKILoS (ORCPT ); Wed, 9 Nov 2022 06:44:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55638 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230303AbiKILoF (ORCPT ); Wed, 9 Nov 2022 06:44:05 -0500 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 286CA1A04E; Wed, 9 Nov 2022 03:43:49 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id DF060B81DF5; Wed, 9 Nov 2022 11:43:47 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BC1D2C433C1; Wed, 9 Nov 2022 11:43:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1667994226; bh=j1Aieg493NQTqXY3czipEQEPMuHZTJs8oAvLJMoCCmw=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=c7hmokEXp0AW0laL0H2Az8JrPTeMIjJ2zdN1lXWbBt+uKvfRz9agPWXC5Oi9qwf1v MZD3d0JRigCdT0K1uh/q+vLBmoNtKXRTkXGvjuzNeaDhqE23yarIrqo3pR7CLP4AH3 Ct/kaV3Jxpsu0QSqthyZs/dUIOhJknAlMsENCgicur5sNwEVGfef++YH4AGmAbQLK6 zySRDe1WB5SL6gIk9m6FwhZX+DHh2rVb797DXcggGdzVXyAS7QOTeshnhN1WOSr/hT UpwIab0ij8hiRF5hbOIS1NmsQUc9wXxNOBId04qDAY9abRJu2tj/mIR2Z+y1J1crGp 6XhWMZ2cofrXw== From: fdmanana@kernel.org To: fstests@vger.kernel.org Cc: linux-btrfs@vger.kernel.org, Filipe Manana Subject: [PATCH 3/3] btrfs: fix failure of tests that use defrag on btrfs-progs v6.0+ Date: Wed, 9 Nov 2022 11:43:36 +0000 Message-Id: X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org From: Filipe Manana Starting with btrfs-progs v6.0, the defrag command now prints to stdout the full path of the files it processes. This makes test cases btrfs/021 and btrfs/256 fail because they don't expect any output from the defrag command. The change happened with the following commit in btrfs-progs: dd724f21803d ("btrfs-progs: add logic to handle LOG_DEFAULT messages") So update the tests to ignore the stdout of the defrag command. Signed-off-by: Filipe Manana Reviewed-by: Zorro Lang --- tests/btrfs/021 | 4 +++- tests/btrfs/256 | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/btrfs/021 b/tests/btrfs/021 index 5943da2f..1b55834a 100755 --- a/tests/btrfs/021 +++ b/tests/btrfs/021 @@ -22,8 +22,10 @@ run_test() sleep 0.5 + # In new versions of btrfs-progs (6.0+), the defrag command outputs to + # stdout the path of the files it operates on. So ignore that. find $SCRATCH_MNT -type f -print0 | xargs -0 \ - $BTRFS_UTIL_PROG filesystem defrag -f + $BTRFS_UTIL_PROG filesystem defrag -f > /dev/null sync wait diff --git a/tests/btrfs/256 b/tests/btrfs/256 index 1360c2c2..acbbc6fa 100755 --- a/tests/btrfs/256 +++ b/tests/btrfs/256 @@ -50,7 +50,9 @@ $FSSUM_PROG -A -f -w "$checksums_file" "$SCRATCH_MNT" # Now defrag each file. for sz in ${file_sizes[@]}; do echo "Defragging file with $sz bytes..." >> $seqres.full - $BTRFS_UTIL_PROG filesystem defragment "$SCRATCH_MNT/f_$sz" + # In new versions of btrfs-progs (6.0+), the defrag command outputs to + # stdout the path of the files it operates on. So ignore that. + $BTRFS_UTIL_PROG filesystem defragment "$SCRATCH_MNT/f_$sz" > /dev/null done # Verify the checksums after the defrag operations.