public inbox for fstests@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Chinner <david@fromorbit.com>
To: Radoslav Kolev <radoslav.kolev@suse.com>
Cc: fstests@vger.kernel.org
Subject: Re: read only filesystem tests
Date: Fri, 26 Nov 2021 10:06:30 +1100	[thread overview]
Message-ID: <20211125230630.GA3154935@dread.disaster.area> (raw)
In-Reply-To: <e64129cb-dec9-f0bb-9a50-ede0a7b2aca6@suse.com>

On Mon, Nov 15, 2021 at 01:18:25PM +0200, Radoslav Kolev wrote:
> Hello,
> 
> I am interested in read only filesystem tests, and more specifically
> squashfs. I saw that ~2 year ago there was a patch set submitted, but not
> accepted that added support for it with some feedback, that
> the tests are copied and double the maintenance burden, but sadly there is
> no easy way to avoid that.
> 
> What is the current mood about read only filesystem tests and fstests?
> Is this the right project to fill this need and is it worth it trying to
> address some of the problems with the old squashfs patches?

What sort of read only tests are you talking about? i.e. how does
the initial filesystem state that the test is going to run on get
generated? AFAICT you can't reuse any of the existing tests because
they all require writing to the filesystem to create the initial
test conditions. You can tell fsx/fsstress not do do modifying
operations from the command line, so you can still run stress tests
based on these on read-only filesystems.

There are definitely limits on what fstests can realistically
exercise w.r.t. read-only filesystems.  That said, there is
definitely scope for fstests to support read-only tests - just
create a tests/read-only directory, a common/read-only include file
and build the tests and infrastructure out from that. If the RO
tests have their own test group definitions (e.g. 'check -g
ro-auto'), then it won't impact on any of the existing read-write
filesystem tests or test groupings.

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

      parent reply	other threads:[~2021-11-25 23:08 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-15 11:18 read only filesystem tests Radoslav Kolev
2021-11-21 15:28 ` Eryu Guan
2021-11-25 23:06 ` Dave Chinner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20211125230630.GA3154935@dread.disaster.area \
    --to=david@fromorbit.com \
    --cc=fstests@vger.kernel.org \
    --cc=radoslav.kolev@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox