From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 6171023DE; Fri, 28 Feb 2025 06:31:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740724286; cv=none; b=BB1+WjfoP53Sb6LAftX8osLTlBQSPu+Ox1mTFyfxbyjGPafZcaaGpo3Z7BxohgI8Cd/CEb1nJWg0hCNKcuVhzCvesMrCrQaglhWw3qA60ApGhmxeaqWgHneoqTpXAtz5eSX8IVtugHJ2F7jt0p+SeiTT5+OeTELcTFGKbHlvetk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740724286; c=relaxed/simple; bh=JPRSZ8i+a3rWyFTiARKx3DmJ8eTqChd52fJZMuS27wg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=CQBn/py6sqFojCN0HmE5TnYz2UTc5QO+ZvgtWrfAb9BrhUcDhvMYeCUYh/a+/N2Fqr/JLCEh4ttTJumXffvcr9dnng0lKMF7B+teu+NgX08uEbj4BD00Y2hECE9yRCEU8pSk4AKAyYQ5sKSsMkd045KgCX7yNb7A9HjjT+FC+Yc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=s17ML4n4; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="s17ML4n4" Received: by smtp.kernel.org (Postfix) with ESMTPSA id CC7CEC4CED6; Fri, 28 Feb 2025 06:31:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1740724285; bh=JPRSZ8i+a3rWyFTiARKx3DmJ8eTqChd52fJZMuS27wg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=s17ML4n4PJT3QCtwRIJuZpdSvEQMMQeC1hsJ9F5B7AHhnhUkRzK2n0wQRgBncMm+M dZ+JZBpW9KNYdocXZczxN2ScsTdSKPhDwHMbh7t+jNhqh2Yg8cAMX1Bw93U+lge4WY S1H8VbRUwa8YYvNjqQiwwscnv8asfGyVj1LSc4/ye6I8dXLN123Y23Wcd4ZGdEqBzz 1nyKa830m3whcqiSqX4o+MkoM1leyj6klkL8XiZUvC5trPTm2+hF0q8YPyjxSJT0Rt p4kW3ICEHHa8dmuoPL4Qingv0MTFJlR/1svJ65NHXkmFz0+JPM/t0YovBB4SfXTTdG ykE+OqdGgHyiA== Date: Thu, 27 Feb 2025 22:31:25 -0800 From: "Darrick J. Wong" To: Zorro Lang Cc: Zorro Lang , fstests@vger.kernel.org, David Sterba , linux-fsdevel@vger.kernel.org Subject: Re: [PATCH] README: add supported fs list Message-ID: <20250228063125.GC6225@frogsfrogsfrogs> References: <20250227200514.4085734-1-zlang@kernel.org> <20250228022045.GA6229@frogsfrogsfrogs> <20250228051600.b44dmfimqqbrom22@dell-per750-06-vm-08.rhts.eng.pek2.redhat.com> Precedence: bulk X-Mailing-List: fstests@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250228051600.b44dmfimqqbrom22@dell-per750-06-vm-08.rhts.eng.pek2.redhat.com> On Fri, Feb 28, 2025 at 01:16:00PM +0800, Zorro Lang wrote: > On Thu, Feb 27, 2025 at 06:20:45PM -0800, Darrick J. Wong wrote: > > On Fri, Feb 28, 2025 at 04:05:14AM +0800, Zorro Lang wrote: > > > To clarify the supported filesystems by fstests, add a fs list to > > > README file. > > > > > > Signed-off-by: Zorro Lang > > > --- > > > > > > Hi, > > > > > > David Sterba suggests to have a supported fs list in fstests: > > > > > > https://lore.kernel.org/fstests/20250227073535.7gt7mj5gunp67axr@dell-per750-06-vm-08.rhts.eng.pek2.redhat.com/T/#m742e4f1f6668d39c1a48450e7176a366e0a2f6f9 > > > > > > I think that's a good suggestion, so I send this patch now. But tell the truth, > > > it's hard to find all filesystems which are supported by fstests. Especially > > > some filesystems might use fstests, but never be metioned in fstests code. > > > So please review this patch or send another patch to tell fstests@ list, if > > > you know any other filesystem is suppported. > > > > > > And if anyone has review point about the support "level" and "comment" part, > > > please feel free to tell me :) > > > > > > Thanks, > > > Zorro > > > > > > README | 82 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > > 1 file changed, 82 insertions(+) > > > > > > diff --git a/README b/README > > > index 024d39531..055935917 100644 > > > --- a/README > > > +++ b/README > > > @@ -1,3 +1,85 @@ > > > +_______________________ > > > +SUPPORTED FS LIST > > > +_______________________ > > > + > > > +History > > > +------- > > > + > > > +Firstly, xfstests is the old name of this project, due to it was originally > > > +developed for testing the XFS file system on the SGI's Irix operating system. > > > +With xfs was ported to Linux, so was xfstests, now it only supports Linux. > > > > When > > > > > + > > > +As xfstests has some test cases are good to run on some other filesystems, > > > > many test cases that can be run > > Sure, will change these. > > > > > > +we call them "generic" (and "shared", but it has been removed) cases, you > > > +can find them in tests/generic/ directory. Then more and more filesystems > > > +started to use xfstests, and contribute patches. Today xfstests is used > > > +as a file system regression test suite for lots of Linux's major file systems. > > > +So it's not "xfs"tests only, we tend to call it "fstests" now. > > > + > > > +Supported fs > > > +------------ > > > + > > > +Firstly, there's not hard restriction about which filesystem can use fstests. > > > +Any filesystem can give fstests a try. > > > + > > > +Although fstests supports many filesystems, they have different support level > > > +by fstests. So mark it with 4 levels as below: > > > + > > > +L1: Fstests can be run on the specified fs basically. > > > +L2: Rare support from the specified fs list to fix some generic test failures. > > > +L3: Normal support from the specified fs list, has some own cases. > > > +L4: Active support from the fs list, has lots of own cases. > > > + > > > ++------------+-------+---------------------------------------------------------+ > > > +| Filesystem | Level | Comment | > > > ++------------+-------+---------------------------------------------------------+ > > > +| AFS | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Bcachefs | L1+ | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Btrfs | L4 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Ceph | L2 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| CIFS | L2- | https://wiki.samba.org/index.php/Xfstesting-cifs | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Ext2/3/4 | L3+ | N/A | > > > > What do the plus and minus mean? > > Oh, I didn't explain them. > > ("+" means a slightly higher than the current level, but not reach to the next. > "-" is opposite, means a little bit lower than the current level.) > > Is that good to you? Insofar as those ratings are ongoing and fluid anyway, ok. :) > > > > > ++------------+-------+---------------------------------------------------------+ > > > +| Exfat | L1+ | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| f2fs | L3- | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| FUSE | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| GFS2 | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Glusterfs | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| JFS | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| NFS | L2+ | https://linux-nfs.org/wiki/index.php/Xfstests | > > > ++------------+-------+---------------------------------------------------------+ > > > +| ocfs2 | L2- | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| overlay | L3 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| pvfs2 | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Reiser4 | L1 | Reiserfs has been removed, only left reiser4 | > > > ++------------+-------+---------------------------------------------------------+ > > > +| tmpfs | L3- | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| ubifs | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| udf | L1+ | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| Virtiofs | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| XFS | L4+ | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > +| 9p | L1 | N/A | > > > ++------------+-------+---------------------------------------------------------+ > > > > This roughly tracks with my observations over the years. > > Some filesystems I never tried, likes "9p" and "ubifs" etc, I just found these names > from common/rc. Wasn't sandeen trying to make fstests work with fat at some point? Well, it's not hard to add more to the table. Looks good to me, Acked-by: "Darrick J. Wong" --D > If any fs list has any supplement to the fs name or the "comment", or would like to > modify the "level", please feel free to tell me. > > Thanks, > Zorro > > > > > > > --D > > > > > + > > > _______________________ > > > BUILDING THE FSQA SUITE > > > _______________________ > > > -- > > > 2.47.1 > > > > > > > > > >