Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp473957pxb; Wed, 24 Feb 2021 07:06:41 -0800 (PST) X-Google-Smtp-Source: ABdhPJw9RiavU+IyfwsWDUAhc4cym1DS01Nt60DfZi7px9RX2Z7PJb9JMwSKA+U6iQ8X44LFv0JA X-Received: by 2002:a17:906:18f1:: with SMTP id e17mr20937957ejf.372.1614179201087; Wed, 24 Feb 2021 07:06:41 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1614179201; cv=pass; d=google.com; s=arc-20160816; b=JYf9a/NOehf9CA9iizslegg2+nlC6h+d+4kvOEuvY8VfnwXYPZSGJ/eVuc0x7tM7he mqiT4V192zY/qvhQq/gg1d6WdTmSIxeSCO1OfYlORB6AEs+LUV0c91WKe6k/j5HBVcQ6 YzeuYOL6Tpz3DtGoEpUEzENTfeNveSRWKei5of8T7n+PtKu/3bKITEQoNRAm0Z/kodfY yk2P7TiOpc77iNqytZo/zkhWM7C/oCNwhZ3XiYuevQT8+SBPb7AqIHrTVqEsZ54a9FlP KDWkUJFvbGPRCwVHP7HjlpgjYaXOIScfuMB0AeM9i6v5a8o3bliwJFM5OEgCy7DvB0Yw oJoA== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:importance:content-transfer-encoding :mime-version:subject:references:in-reply-to:message-id:cc:to :reply-to:from:date:dkim-signature; bh=5dfC4UIBlDAD3j+p2vozEcResPOP6jwjIjKI5EdeBoQ=; b=Jrwh+RqYtdtqamuMa+ZOpUFa3ks1d/qVaxPz4BiVOql1374ej18c0T3I7OGPMocc2L 7WzRWmkB1VdNjgIS4EOiY3VX2ILTH5YCP9a7dbnEaTZVQQqwRQIV+hicGJqvF09A7Joh jJt/jk/bO9rqi1RGKeQbpUgKBZ3DZ7ecu/TAQolar+8l2UunZS+xkzVrKxPjTgx8YE0Z kW52KZx2hP3X+iyucukburxdFJna4lgRMzdzvETo/77CkeR53Nc+DqK7/+EwXR4VKwCR KBt6e6q9+1yAfOf8CjwURYA2rJ42QuBlJUwKDVgMFq8BDbntm14A4DMwhxbugCYRuHJx A1ow== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@mykernel.net header.s=zohomail header.b=OmCmcOnk; arc=pass (i=1 spf=pass spfdomain=mykernel.net dkim=pass dkdomain=mykernel.net dmarc=pass fromdomain=mykernel.net>); spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id eb13si1358261edb.315.2021.02.24.07.06.11; Wed, 24 Feb 2021 07:06:41 -0800 (PST) Received-SPF: pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@mykernel.net header.s=zohomail header.b=OmCmcOnk; arc=pass (i=1 spf=pass spfdomain=mykernel.net dkim=pass dkdomain=mykernel.net dmarc=pass fromdomain=mykernel.net>); spf=pass (google.com: domain of linux-ext4-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-ext4-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234587AbhBXPBP (ORCPT + 99 others); Wed, 24 Feb 2021 10:01:15 -0500 Received: from sender2-pp-o92.zoho.com.cn ([163.53.93.251]:25308 "EHLO sender2-pp-o92.zoho.com.cn" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233755AbhBXNxQ (ORCPT ); Wed, 24 Feb 2021 08:53:16 -0500 ARC-Seal: i=1; a=rsa-sha256; t=1614174536; cv=none; d=zoho.com.cn; s=zohoarc; b=lQA+AVARawNZN5g5lvnFLqJf8kRleEwbjlh6mbvAyuTffxCh2lJWvgCCcZ4XZqlk3hwPjkPRS0i1lji5PXUwEWtq9IRf2AiKfgdNTF9Kl/HtUEEpQCPis0KUydfcv/gAzRJX4rFGrR5/0j0fVttU4N+d/IVBOvkQRaY7IupRKU4= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com.cn; s=zohoarc; t=1614174536; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:MIME-Version:Message-ID:Reply-To:References:Subject:To; bh=5dfC4UIBlDAD3j+p2vozEcResPOP6jwjIjKI5EdeBoQ=; b=n8lHeJLReVWQ/6WET57vc0rV5/oYX8jD9G7Cz0jXrEiB63AL+7bTpvUJwH8deP98909WWiblfyc+jZfPMtAKFelutvdUs/exN4wwGU5JY5En4qp0uuG2WzMYBv1kwrTQ+qCgcCfmgMKQ9HhwqrcwfAFYv3kNSIdVqIEgknfO79c= ARC-Authentication-Results: i=1; mx.zoho.com.cn; dkim=pass header.i=mykernel.net; spf=pass smtp.mailfrom=cgxu519@mykernel.net; dmarc=pass header.from= header.from= DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; t=1614174536; s=zohomail; d=mykernel.net; i=cgxu519@mykernel.net; h=Date:From:Reply-To:To:Cc:Message-ID:In-Reply-To:References:Subject:MIME-Version:Content-Type:Content-Transfer-Encoding; bh=5dfC4UIBlDAD3j+p2vozEcResPOP6jwjIjKI5EdeBoQ=; b=OmCmcOnkki8ji5gDD/ZVer72iFZwGTC5GPo5R9PqQvPgq1ueCqcmnzFAvKXluprM ilkfR6DGR4e7JtCrGnCzGw8TrIwzrpkeSEZ17TCwNEP1doBfQG8zwa/FVR+4nLbjrBP qzXh/WijyLFGaqoL2vbUmZCGuZBiV4NVC66Y9Bsg= Received: from mail.baihui.com by mx.zoho.com.cn with SMTP id 1614174533227810.9043468053503; Wed, 24 Feb 2021 21:48:53 +0800 (CST) Date: Wed, 24 Feb 2021 21:48:53 +0800 From: Chengguang Xu Reply-To: cgxu519@mykernel.net To: "Eryu Guan" Cc: "Su Yue" , "guaneryu" , "fstests" , "linux-btrfs" , "linux-fsdevel" , "linux-ext4" , "linux-xfs" Message-ID: <177d44cb667.12f7afcae16893.3177469356061349445@mykernel.net> In-Reply-To: <20210224133146.GE96449@e18g06458.et15sqa> References: <20210223134042.2212341-1-cgxu519@mykernel.net> <4ki1rjgu.fsf@damenly.su> <177d33c0982.10b8858b515683.1169986601273192029@mykernel.net> <177d3666a3c.e47042d016248.8805085013477614929@mykernel.net> <20210224133146.GE96449@e18g06458.et15sqa> Subject: Re: [PATCH] generic/473: fix expectation properly in out file MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Importance: Medium User-Agent: ZohoCN Mail X-Mailer: ZohoCN Mail Precedence: bulk List-ID: X-Mailing-List: linux-ext4@vger.kernel.org ---- =E5=9C=A8 =E6=98=9F=E6=9C=9F=E4=B8=89, 2021-02-24 21:31:46 Eryu Guan = =E6=92=B0=E5=86=99 ---- > On Wed, Feb 24, 2021 at 05:37:20PM +0800, Chengguang Xu wrote: > > ---- =E5=9C=A8 =E6=98=9F=E6=9C=9F=E4=B8=89, 2021-02-24 17:22:35 Su Yu= e =E6=92=B0=E5=86=99 ---- > > >=20 > > > On Wed 24 Feb 2021 at 16:51, Chengguang Xu = =20 > > > wrote: > > >=20 > > > > ---- =E5=9C=A8 =E6=98=9F=E6=9C=9F=E4=B8=89, 2021-02-24 15:52:17 = Su Yue =E6=92=B0=E5=86=99=20 > > > > ---- > > > > > > > > > > Cc to the author and linux-xfs, since it's xfsprogs related. > > > > > > > > > > On Tue 23 Feb 2021 at 21:40, Chengguang Xu=20 > > > > > > > > > > wrote: > > > > > > > > > > > It seems the expected result of testcase of "Hole + Data" > > > > > > in generic/473 is not correct, so just fix it properly. > > > > > > > > > > > > > > > > But it's not proper... > > > > > > > > > > > Signed-off-by: Chengguang Xu > > > > > > --- > > > > > > tests/generic/473.out | 2 +- > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > diff --git a/tests/generic/473.out b/tests/generic/473.out > > > > > > index 75816388..f1ee5805 100644 > > > > > > --- a/tests/generic/473.out > > > > > > +++ b/tests/generic/473.out > > > > > > @@ -6,7 +6,7 @@ Data + Hole > > > > > > 1: [256..287]: hole > > > > > > Hole + Data > > > > > > 0: [0..127]: hole > > > > > > -1: [128..255]: data > > > > > > +1: [128..135]: data > > > > > > > > > > > The line is produced by `$XFS_IO_PROG -c "fiemap -v 0 65k"=20 > > > > > $file | > > > > > _filter_fiemap`. > > > > > 0-64k is a hole and 64k-128k is a data extent. > > > > > fiemap ioctl always returns *complete* ranges of extents. > > > > > > > > Manual testing result in latest kernel like below. > > > > > > > > [root@centos test]# uname -a > > > > Linux centos 5.11.0+ #5 SMP Tue Feb 23 21:02:27 CST 2021 x86_64= =20 > > > > x86_64 x86_64 GNU/Linux > > > > > > > > [root@centos test]# xfs_io -V > > > > xfs_io version 5.0.0 > > > > > > > > [root@centos test]# stat a > > > > File: a > > > > Size: 4194304 Blocks: 0 IO Block: 4096=20 > > > > regular file > > > > Device: fc01h/64513d Inode: 140 Links: 1 > > > > Access: (0644/-rw-r--r--) Uid: ( 0/ root) Gid: ( 0/= =20 > > > > root) > > > > Access: 2021-02-24 16:33:20.235654140 +0800 > > > > Modify: 2021-02-24 16:33:25.070641521 +0800 > > > > Change: 2021-02-24 16:33:25.070641521 +0800 > > > > Birth: - > > > > > > > > [root@centos test]# xfs_io -c "pwrite 64k 64k" a > > > > wrote 65536/65536 bytes at offset 65536 > > > > 64 KiB, 16 ops; 0.0000 sec (992.063 MiB/sec and 253968.2540=20 > > > > ops/sec) > > > > > > > > [root@VM-8-4-centos test]# xfs_io -c "fiemap -v 0 65k" a > > > > a: > > > > EXT: FILE-OFFSET BLOCK-RANGE TOTAL FLAGS > > > > 0: [0..127]: hole 128 > > > > 1: [128..135]: 360..367 8 0x1 > > > > > > >=20 > > > Sorry, my carelessness. I only checked btrfs implementation but=20 > > > xfs > > > and ext4 do return the change you made. > > >=20 > >=20 > > Yeah, it seems there is no bad side effect to show only specified ran= ge of extents > > and keep all the same behavior is also good for testing. I can post a = fix patch for > > this but before that let us to wait some feedback from maintainers and= experts. >=20 > generic/473 is marked as broken by commit 715eac1a9e66 ("generic/47[23]: > remove from auto/quick groups"). >=20 I got it, thanks Eryu!