From: Eric Whitney <enwlinux@gmail.com>
To: linux-ext4@vger.kernel.org
Cc: tytso@mit.edu, dccitaliano@gmail.com
Subject: 4.1-rc2 kvm-xfstests regressions for ext3 test config
Date: Mon, 11 May 2015 17:37:31 -0400 [thread overview]
Message-ID: <20150511213731.GA4012@wallace> (raw)
I've got six new test failures in the ext3 test configuration when
running a 4.1-rc2 kernel on x86_64 with the latest version of
kvm-xfstests.
The new failures include generic/075, generic/091, generic/112,
generic/127, generic/231, and generic/263. All also fail on ARM
(Pandaboard) when running a 4.1-rc2 kernel on bare metal.
Reverting an -rc2 patch corrects all these failures:
ext4: move check under lock scope to close a race (280227a75b56)
All the failed tests use fsx, all fail when attempting to perform a
COLLAPSE_RANGE, and fsx uses this statement to determine whether a
particular fallocate mode is supported (and then doesn't use it if not):
if (fallocate(fd, mode, 0, 1) && errno == EOPNOTSUPP)
In -rc1, ext4_fallocate()/extents.c returns EOPNOTSUPP, and in -rc2,
EINVAL. What's happened is that we no longer immediately fail a
COLLAPSE_RANGE request for a block-mapped file with EOPNOTSUPP. Instead,
we now first check to see if the COLLAPSE_RANGE request is cluster-aligned
and sized in ext4_collapse_range()/extents.c, and return EINVAL if not.
(There's still a check for block-mapped files, but it's applied later in
ext4_collapse_range().)
I think we'd want to return EOPNOTSUPP first in this case.
Eric
next reply other threads:[~2015-05-11 21:37 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-11 21:37 Eric Whitney [this message]
2015-05-12 9:20 ` 4.1-rc2 kvm-xfstests regressions for ext3 test config Lukáš Czerner
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=20150511213731.GA4012@wallace \
--to=enwlinux@gmail.com \
--cc=dccitaliano@gmail.com \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.