From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f196.google.com ([209.85.215.196]:37438 "EHLO mail-pg1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726485AbfFZChU (ORCPT ); Tue, 25 Jun 2019 22:37:20 -0400 Date: Wed, 26 Jun 2019 10:37:13 +0800 From: Eryu Guan Subject: Re: Removing the shared class of tests Message-ID: <20190626023713.GA7943@desktop> References: <20190612184033.21845-1-krisman@collabora.com> <20190612184033.21845-2-krisman@collabora.com> <20190616144440.GD15846@desktop> <20190616200154.GA7251@mit.edu> <20190620112903.GF15846@desktop> <20190620162116.GA4650@mit.edu> <20190620175035.GA5380@magnolia> <20190624071610.GA10195@infradead.org> <20190624130730.GD1805@mit.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190624130730.GD1805@mit.edu> Sender: fstests-owner@vger.kernel.org Content-Transfer-Encoding: quoted-printable To: Theodore Ts'o Cc: Christoph Hellwig , "Darrick J. Wong" , Gabriel Krisman Bertazi , fstests@vger.kernel.org, linux-ext4@vger.kernel.org, "Lakshmipathi.G" List-ID: On Mon, Jun 24, 2019 at 09:07:30AM -0400, Theodore Ts'o wrote: > On Mon, Jun 24, 2019 at 12:16:10AM -0700, Christoph Hellwig wrote: > >=20 > > As for the higher level question? The shared tests always confused t= he > > heck out of me. generic with the right feature checks seem like a mu= ch > > better idea. >=20 > Agreed. I've sent out a patch series to bring the number of patches > in shared down to four. Here's what's left: >=20 > shared/002 --- needs a feature test to somehow determine whether a > file system supports thousads of xattrs in a file (currently > on btrfs and xfs) Another option would be just whitelist btrfs and xfs in a require rule, we already have few require rules work like that, e.g. _fstyp_has_non_default_seek_data_hole(), this is not ideal, but works in such corner cases. Thanks, Eryu >=20 > shared/011 --- needs some way of determining that a file system > supports cgroup-aware writeback (currently enabled only for > ext4 and btrfs). Do we consider lack of support of > cgroup-aware writeback a bug? If so, maybe it doesn't need a > feature test at all? >=20 > shared/032 --- needs a feature test to determine whether or not a file > system's mkfs supports detection of "foreign file systems". > e.g., whether or not it warns if you try overwrite a file > system w/o another file system. (Currently enabled by xfs and > btrfs; it doesn't work for ext[234] because e2fsprogs, because > I didn't want to break existing shell scripts, only warns when > it is used interactively. We could a way to force it to be > activated it points out this tests is fundamentally testing > implementation choices of the userspace utilities of a file > system. Does it belong in xfstests? : =C2=AF\_(=E3=83=84)_/=C2=AF ) >=20 > shared/289 --- contains ext4, xfs, and btrfs mechanisms for > determining blocks which are unallocated. Has hard-coded > invocations to dumpe2fs, xfs_db, and /bin/btrfs. >=20 > These don't have obvious solutions. We could maybe add a _notrun if > adding the thousands of xattrs fails with an ENOSPC or related error > (f2fs uses something else). >=20 > Maybe we just move shared/011 and move it generic/ w/o a feature test. >=20 > Maybe we remove shared/032 altogether, since for e2fsprogs IMHO > the right place to put it is the regression test in e2fsprogs --- but > I know xfs has a different test philosophy for xfsprogs; and tha begs > the question of what to do for mkfs.btrfs. >=20 > And maybe we just split up shared/289 to three different tests in > ext4/, xfs/, and btrfs/, since it would make the test script much > simpler to understand? >=20 > What do people think? >=20 > - Ted