From: "Darrick J. Wong" <djwong@kernel.org>
To: Christoph Hellwig <hch@infradead.org>
Cc: Eric Sandeen <sandeen@sandeen.net>,
"linux-xfs@vger.kernel.org" <linux-xfs@vger.kernel.org>,
"fstests@vger.kernel.org" <fstests@vger.kernel.org>,
Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Subject: Re: mkfs.xfs "concurrency" change concerns
Date: Mon, 15 Jun 2026 08:40:48 -0700 [thread overview]
Message-ID: <20260615154048.GW6095@frogsfrogsfrogs> (raw)
In-Reply-To: <ajAJ6lXJWcko4epj@infradead.org>
On Mon, Jun 15, 2026 at 07:19:22AM -0700, Christoph Hellwig wrote:
> On Thu, Oct 09, 2025 at 03:13:47PM -0500, Eric Sandeen wrote:
> > Hey all -
> >
> > this got long, so tl;dr:
> >
> > 1) concurrency geometry breaks some xfstests for me
> > 2) concurrency behavior is not consistent w/ loopback vs. imagefile
> > 3) concurrency defaults to the mkfs machine not the mount machine
> >
> > In detail:
> >
> > So, I realize I'm late to the game here and didn't review the patches
> > before they went in, but it looks like the "concurrency" mkfs.xfs
> > arguments and defaults are breaking several xfstests.
> >
> > 4738ff0 mkfs: allow sizing realtime allocation groups for concurrency
> > c02a1873 mkfs: allow sizing internal logs for concurrency
> > 9338bc8b mkfs: allow sizing allocation groups for concurrency
> >
> > Specifically, xfs/078, xfs/216, and xfs/217 are failing for us
> > on various machines with between 8 and 128 CPUS, due to the
> > fundamental change in geometry that results from the new
> > concurrency behavior, which makes any consistent golden
> > output that involves geometry details quite difficult.
>
> Did anything ever come out of this? We're hitting errors in exactly
> those tests in the zoned XFS CI (which also tests regular XFS).
> Shin'ichiro recently spent some time tracking these down, which
> finally made me remember this thread.
Lukas Herbolt posted some fixes for a few of the fstests to inject
concurrency=0 into mkfs. IIRC there weren't any changes for 078, but
you might ask him if he has more bugs to address.
--D
next prev parent reply other threads:[~2026-06-15 15:40 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-09 20:13 mkfs.xfs "concurrency" change concerns Eric Sandeen
2025-10-10 5:17 ` Christoph Hellwig
2025-10-10 19:17 ` Darrick J. Wong
2025-10-10 19:34 ` Darrick J. Wong
2025-10-10 20:47 ` Eric Sandeen
2025-10-14 2:32 ` Darrick J. Wong
2025-10-14 15:36 ` Eric Sandeen
2025-10-17 22:46 ` Darrick J. Wong
2025-10-18 15:01 ` Eric Sandeen
2026-06-15 14:19 ` Christoph Hellwig
2026-06-15 15:24 ` Eric Sandeen
2026-06-15 15:40 ` Darrick J. Wong [this message]
2026-06-16 5:24 ` Christoph Hellwig
2026-06-17 1:43 ` Shin'ichiro Kawasaki
2026-06-17 6:23 ` Christoph Hellwig
2026-06-17 6:57 ` Shin'ichiro Kawasaki
2026-06-17 7:03 ` Christoph Hellwig
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=20260615154048.GW6095@frogsfrogsfrogs \
--to=djwong@kernel.org \
--cc=fstests@vger.kernel.org \
--cc=hch@infradead.org \
--cc=linux-xfs@vger.kernel.org \
--cc=sandeen@sandeen.net \
--cc=shinichiro.kawasaki@wdc.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