From mboxrd@z Thu Jan 1 00:00:00 1970 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 15124CA0ECB for ; Mon, 11 Sep 2023 22:06:51 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1355750AbjIKWBw (ORCPT ); Mon, 11 Sep 2023 18:01:52 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46230 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S243980AbjIKSeu (ORCPT ); Mon, 11 Sep 2023 14:34:50 -0400 Received: from wout4-smtp.messagingengine.com (wout4-smtp.messagingengine.com [64.147.123.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C839F198; Mon, 11 Sep 2023 11:34:45 -0700 (PDT) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.west.internal (Postfix) with ESMTP id 06AB63200912; Mon, 11 Sep 2023 14:34:44 -0400 (EDT) Received: from mailfrontend2 ([10.202.2.163]) by compute5.internal (MEProxy); Mon, 11 Sep 2023 14:34:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bur.io; h=cc:cc :content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1694457284; x=1694543684; bh=IF 8gqeL+ErQjBAaEgRAFAOPP4+xIGg4ZWf75F8j7o+g=; b=oHm4ccriNXJZT9A27C 3Wweh49Kbg+jYoKq2xGKakkYJ1TMYsrKQ5UR7PvC57UvxWoX9k/e0TKV0UnCju1o PY89loZRHsBLZFqQc0d0xOo1woPXKLo+dveCeQGcEFsYswn/1dBU50w24GSeqzco ylfOiQQnSTm74GMRjD6jcADFF/zjGxcy7GeDCoA8D1ddwHAf7+OK/pmA0YzCNfhp vEfPqsSpqmQ3bxzSZkvT+WSJvAMRmwfUoPn+pz9u4Vl51yVsUJUfN+QHiTeNTnmi hWyHJT8JPNJ2EC6vyPK6lgDYB2wEjd5Xm/j2/kDNKmbVc9/8fxIIKP5Qn18i6reL 90cg== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; t=1694457284; x=1694543684; bh=IF8gqeL+ErQjB AaEgRAFAOPP4+xIGg4ZWf75F8j7o+g=; b=KV8aWZg051d1x+uq2UsuoNYBRtS64 8VwFst1phf2UjiDulhhOMzWR8h9azeTA2Zk2HKdMRKZ7Bq9dET3ezRbINI2v4zSh ZUh5GT4yHUXgxuPlkIV75rZ1NURzMlH7nsyNu7sqX6yAzpbZabz2ep0RzX5CvY3+ 9c92UPOVbng2yWYIYWtsIkky7C7P9yjhhC03YtmJt9INmaeFZ9P7fID2XcXrA4nW WBhCInNd3vor00ONr7LmWFEIJJSqez2Zhz6xjfaEN8VpRHwUZyitKaJU5Qvbgo0z lNT6vr0IBS7B8AB3V9YdFes48gfeBzqPErQVLE6cPO4KYLBDFyg8cyZqg== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedrudeigedguddvgecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfgh necuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmd enucfjughrpeffhffvvefukfhfgggtuggjsehttdertddttddvnecuhfhrohhmpeeuohhr ihhsuceuuhhrkhhovhcuoegsohhrihhssegsuhhrrdhioheqnecuggftrfgrthhtvghrnh epkedvkeffjeellefhveehvdejudfhjedthfdvveeiieeiudfguefgtdejgfefleejnecu vehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepsghorhhish essghurhdrihho X-ME-Proxy: Feedback-ID: i083147f8:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 11 Sep 2023 14:34:43 -0400 (EDT) Date: Mon, 11 Sep 2023 11:35:42 -0700 From: Boris Burkov To: Anand Jain Cc: fstests@vger.kernel.org, linux-btrfs@vger.kernel.org Subject: Re: [PATCH] fstests: btrfs/185 update for single device pseudo device-scan Message-ID: <20230911183542.GA1770246@zen> References: <7558eed09a89d25fbd8083d45078cfe2e9601f45.1694017375.git.anand.jain@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7558eed09a89d25fbd8083d45078cfe2e9601f45.1694017375.git.anand.jain@oracle.com> Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org On Thu, Sep 07, 2023 at 12:24:43AM +0800, Anand Jain wrote: > As we are obliterating the need for the device scan for the single device, > which will return success if the basic superblock verification passes, > even for the duplicate device of the mounted filesystem, drop the check > for the return code in this testcase and continue to verify if the device > path of the mounted filesystem remains unaltered after the scan. > > Also, if the test fails, it leaves the local non-standard mount point > remained mounted, leading to further test cases failing. Call unmount > in _cleanup(). > > Signed-off-by: Anand Jain > --- > tests/btrfs/185 | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/tests/btrfs/185 b/tests/btrfs/185 > index ba0200617e69..c7b8d2d46951 100755 > --- a/tests/btrfs/185 > +++ b/tests/btrfs/185 > @@ -15,6 +15,7 @@ mnt=$TEST_DIR/$seq.mnt > # Override the default cleanup function. > _cleanup() > { > + $UMOUNT_PROG $mnt > /dev/null 2>&1 > rm -rf $mnt > /dev/null 2>&1 > cd / > rm -f $tmp.* Also, do we need to do the scratch_dev_pool_put on cleanup? Do we have to worry about having not actually called scratch_dev_pool_get? > @@ -51,9 +52,9 @@ for sb_bytenr in 65536 67108864; do > echo ..:$? >> $seqres.full > done > > -# Original device is mounted, scan of its clone should fail > +# Original device is mounted, scan of its clone must not alter the > +# filesystem device path > $BTRFS_UTIL_PROG device scan $device_2 >> $seqres.full 2>&1 > -[[ $? != 1 ]] && _fail "cloned device scan should fail" > > [[ $(findmnt $mnt | grep -v TARGET | $AWK_PROG '{print $2}') != $device_1 ]] && \ > _fail "mounted device changed" > -- > 2.39.3 >