From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS16276 91.121.0.0/16 X-Spam-Status: No, score=-3.4 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,UNPARSEABLE_RELAY shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from nautica.notk.org (nautica.notk.org [91.121.71.147]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 87C981F852 for ; Sun, 30 Jan 2022 23:19:03 +0000 (UTC) Received: by nautica.notk.org (Postfix, from userid 108) id EE0B8C01D; Mon, 31 Jan 2022 00:19:00 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codewreck.org; s=2; t=1643584740; bh=sLb7Zkp98Hw7V40Pl1wpYFTPoxuZGddCQRB92TbH2f0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=xlTgl3kAm9khjFo5nyinW774ClOykz1O4BauZW/UT5pNvjiPrxX6sKM9IIrb5FyYq 3mxCaxUMBA+ESEkUVbFWkLjeAswjJmHyKPqUavHTX0fIUHdhJEU9kja5PmLX4+qvnQ ZK6u1VvTCNQ+mBB29CTiV2TrtseLPpCbU6kaNjhtTphWArcurOQoup3jYPDhfAWIzU SQbQ7+TN0WnXOe5prJoZQv4FR8VPs5C+MWY2Nd0pG2rKKLXxRky6rxR76GtqooxShF s3t2lm0rla6cUdMGuYSLPrcTScxxCC2oxaSzhbYYnd5uWtEoBCi6j+LEfSxZCA//4R 8ggtmM/oL0F7w== Received: from odin.codewreck.org (localhost [127.0.0.1]) by nautica.notk.org (Postfix) with ESMTPS id 8F204C01B; Mon, 31 Jan 2022 00:18:59 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codewreck.org; s=2; t=1643584740; bh=sLb7Zkp98Hw7V40Pl1wpYFTPoxuZGddCQRB92TbH2f0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=xlTgl3kAm9khjFo5nyinW774ClOykz1O4BauZW/UT5pNvjiPrxX6sKM9IIrb5FyYq 3mxCaxUMBA+ESEkUVbFWkLjeAswjJmHyKPqUavHTX0fIUHdhJEU9kja5PmLX4+qvnQ ZK6u1VvTCNQ+mBB29CTiV2TrtseLPpCbU6kaNjhtTphWArcurOQoup3jYPDhfAWIzU SQbQ7+TN0WnXOe5prJoZQv4FR8VPs5C+MWY2Nd0pG2rKKLXxRky6rxR76GtqooxShF s3t2lm0rla6cUdMGuYSLPrcTScxxCC2oxaSzhbYYnd5uWtEoBCi6j+LEfSxZCA//4R 8ggtmM/oL0F7w== Received: from localhost (odin.codewreck.org [local]) by odin.codewreck.org (OpenSMTPD) with ESMTPA id aca7cdfc; Sun, 30 Jan 2022 23:18:55 +0000 (UTC) Date: Mon, 31 Jan 2022 08:18:40 +0900 From: Dominique Martinet To: Eric Wong Cc: Julien Moutinho , meta@public-inbox.org Subject: Re: [PATCH] nodatacow: quiet chattr errors [was: Test failures with 1.7.0] Message-ID: References: <20211208010730.f47xxgzj53nwgvja@sourcephile.fr> <20211208040836.GA27368@dcvr> <20211208182247.M197857@dcvr> <20220130214908.M320944@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20220130214908.M320944@dcvr> List-Id: Eric Wong wrote on Sun, Jan 30, 2022 at 09:49:08PM +0000: > Thanks for testing the previous patch. Actually, I prefer we > drop previous implementations and instead rely on Linux ABI > stability while shifting maintenance burden to maintainers. This makes sense. > Can you test the patch below? It supercedes the other one. I've tested the patch with a btrfs test directory and this test: BTRFS_TESTDIR=... prove -bvw t/nodatacow.t after adding the local git repo to PERL5LIBs With a BTRFS dir as BTRFS it all works, the ioctl are passed and files are indeed nocow (checked with lsattr afterwards as files aren't deleted) With a tmpfs instead the tests fail as it tried to call the ioctl anyway -- I didn't take the time to check if thest mtab parsing was removed, is the ioctl tried anyway or does setting the environment variable skip this part? If so then it's probably fine. (the failure is as below: ------ $ BTRFS_TESTDIR=/tmp/test strace -o /tmp/strace -f prove -bvw t/nodatacow.t t/nodatacow.t .. ok 1 - use PublicInbox::Syscall; /run/current-system/sw/bin/lsattr: Operation not supported While reading flags on /tmp/test/nodatacow-rfqh/pp.f not ok 2 - `C' attribute set on fd with pure Perl # Failed test '`C' attribute set on fd with pure Perl' # at t/nodatacow.t line 31. # '' # doesn't match '(?^:C.*\/tmp\/test\/nodatacow\-rfqh\/pp\.f)' /run/current-system/sw/bin/lsattr: Operation not supported While reading flags on /tmp/test/nodatacow-rfqh/pp.d not ok 3 - `C' attribute set on dir with pure Perl # Failed test '`C' attribute set on dir with pure Perl' # at t/nodatacow.t line 38. # '' # doesn't match '(?^:C.*\/tmp\/test\/nodatacow\-rfqh\/pp\.d)' 1..3 # Looks like you failed 2 tests of 3. Dubious, test returned 2 (wstat 512, 0x200) Failed 2/3 subtests Test Summary Report ------------------- t/nodatacow.t (Wstat: 512 Tests: 3 Failed: 2) Failed tests: 2-3 Non-zero exit status: 2 Files=1, Tests=3, 1 wallclock secs ( 0.04 usr 0.08 sys + 0.15 cusr 0.21 csys = 0.48 CPU) Result: FAIL -------- > Apologies for the delay, I've pretty much lost all motivation for > everything in life :< Will try to find some energy to look into the > other issues in a bit... No worry, one step at a time is good :) Wish you the best, -- Dominique