From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f195.google.com ([209.85.210.195]:42936 "EHLO mail-pf1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726020AbeIBSVf (ORCPT ); Sun, 2 Sep 2018 14:21:35 -0400 Received: by mail-pf1-f195.google.com with SMTP id l9-v6so7526082pff.9 for ; Sun, 02 Sep 2018 07:05:37 -0700 (PDT) Date: Sun, 2 Sep 2018 22:05:29 +0800 From: Eryu Guan Subject: Re: [RFC PATCH] fstests: add support for hfsplus Message-ID: <20180902140529.GI3651@desktop> References: <20180901020503.5i4hz4inqtfd3frg@eaf> <20180901170957.GF3651@desktop> <20180902040337.3bdp224btok5y5xh@eaf> <4445a65e-42b7-5e2e-36fa-45f116f99f33@sandeen.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <4445a65e-42b7-5e2e-36fa-45f116f99f33@sandeen.net> Sender: fstests-owner@vger.kernel.org Content-Transfer-Encoding: quoted-printable To: Eric Sandeen , Ernesto =?utf-8?Q?A=2E_Fern=C3=A1ndez?= Cc: fstests@vger.kernel.org List-ID: On Sun, Sep 02, 2018 at 12:13:26AM -0500, Eric Sandeen wrote: >=20 >=20 > On 9/1/18 11:03 PM, Ernesto A. Fern=C3=A1ndez wrote: > > On Sun, Sep 02, 2018 at 01:09:57AM +0800, Eryu Guan wrote: > >> On Fri, Aug 31, 2018 at 11:05:03PM -0300, Ernesto A. Fern=C3=A1ndez = wrote: > >>> It is not possible to set file system size when running mkfs.hfsplu= s, > >>> so use the device mapper as a workaround. > >> > >> I'd prefer _notrun the test in _scratch_mkfs_sized() instead of this > >> workaround, as the harness is expecting to operate $SCRATCH_DEV but = this > >> workaround may break this assumption in subtle ways, e.g. now > >> _check_scratch_fs fails to create hfsplus-tmp device because > >> $SCRATCH_DEV may still be mounted. > >=20 > > I didn't realize $SCRATCH_DEV could still be mounted, sorry. Maybe I = can > > unmount it before _dmsetup_create(), and remount after _dmsetup_remov= e()? > > Like _check_generic_filesystem() does. Or would that bring other prob= lems? I haven't seen other problems yet (after just a few simple test runs), but I suspect it may break tests in more subtle ways, the _check_scratch_fs failure is just an example. And even it works for now, and we have to worry about making it continue to work when adding new features in the future and that's a maintaining burden we'd better to avoid. > >=20 > > I know it's ugly, but one test that uses _scratch_mkfs_sized() has he= lped > > me find a number of bugs already. It would be really useful to get it= to > > work. >=20 > Has anyone proposed a patch to mkfs.hfsplus to accept a filesystem size= ? > I'd expect that might be less complicated than this sort of devicemappe= r > setup ;) Yeah, this would be the best way to let _scratch_mkfs_sized() support hfsplus :) Thanks, Eryu