public inbox for fstests@vger.kernel.org
 help / color / mirror / Atom feed
From: "Theodore Ts'o" <tytso@mit.edu>
To: "Dušan Čolić" <dusanc@gmail.com>
Cc: fstests <fstests@vger.kernel.org>
Subject: Re: xfstests enospc tests not filling up file systems with transparent compression
Date: Thu, 10 Sep 2015 19:45:58 -0400	[thread overview]
Message-ID: <20150910234558.GC31723@thunk.org> (raw)
In-Reply-To: <CADW=+3mX-aBuxFnTcshSOR4-q-uY4OucpbFmeoy0xzW49s6csQ@mail.gmail.com>

On Thu, Sep 10, 2015 at 10:09:16PM +0200, Dušan Čolić wrote:
> 
> There are various xfstests that try to fill up whole partition but
> they give false positives on file systems with transparent compression
> because they use compressible test files like input from /dev/zero.
> Different tests use different ways to generate test file(s) so what
> would be the preferred way to generate incompressible test files? Use
> /dev/urandom instead /dev/zero?

I would expect that most tests would use fallocate to allocate space
for ENOSPC tests, since it's much faster than writing all zeros.

If a file system with tansparent compression is treating space
allocated with fallocate as "all zeros" and hence infinitely
compressible, I'd argue that it's buggy, since one of the documented
uses of fallocate is to force the file system to allocate space so
that future writes are guaranteed to succeed, so that the application
doesn't have to deal with ENOSPC errors at some inconvenient time.
(For example, in the case of a DVR, you want to know that you've
allocated enough space for that all-so-critical recording of the
season finale of "The Big Bang Theory", or that episode of "Sex and
the City" which you had missed.  :-)

Cheers,

					- Ted

  reply	other threads:[~2015-09-10 23:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-10 20:09 xfstests enospc tests not filling up file systems with transparent compression Dušan Čolić
2015-09-10 23:45 ` Theodore Ts'o [this message]
2015-09-11  0:14   ` Eric Sandeen

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=20150910234558.GC31723@thunk.org \
    --to=tytso@mit.edu \
    --cc=dusanc@gmail.com \
    --cc=fstests@vger.kernel.org \
    /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