Linux XFS filesystem development
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@lst.de>
To: "Darrick J. Wong" <djwong@kernel.org>
Cc: Christoph Hellwig <hch@lst.de>, Carlos Maiolino <cem@kernel.org>,
	linux-xfs@vger.kernel.org
Subject: Re: [PATCH 1/2] xfs: validate that zoned RT devices are zone aligned
Date: Tue, 16 Dec 2025 06:10:02 +0100	[thread overview]
Message-ID: <20251216051002.GA26237@lst.de> (raw)
In-Reply-To: <20251215191506.GI7725@frogsfrogsfrogs>

On Mon, Dec 15, 2025 at 11:15:06AM -0800, Darrick J. Wong wrote:
> > +	if (xfs_sb_is_v5(sbp) &&
> > +	    (sbp->sb_features_incompat & XFS_SB_FEAT_INCOMPAT_ZONED)) {
> > +		uint32_t		mod;
> > +
> > +		/*
> > +		 * Zoned RT devices must be aligned to the rtgroup size, because
> > +		 * garbage collection can't deal with rump RT groups.
> 
> I've decided that I'm ok with imposing this new restriction after the
> fact, but only because actual zoned hardware will never expose a runt
> group, so the only way you could end up with one now is if you formatted
> with zoned=1 without a hardware-zoned storage device.
> 
> Could this comment be expanded to say that explicitly?

That comment would not actually be true.  The hardware specs do allow
for runt zones.  No shipping hardware that I know of does that, and
mkfs protects against it, but the statement would be at best misleading
if not outright wrong.  The real reason why this is fine is because
mkfs rounds the capacity down to the zone size.

  reply	other threads:[~2025-12-16  5:10 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-12-15  9:48 reject non-zone aligned RT subvolumes Christoph Hellwig
2025-12-15  9:48 ` [PATCH 1/2] xfs: validate that zoned RT devices are zone aligned Christoph Hellwig
2025-12-15 19:15   ` Darrick J. Wong
2025-12-16  5:10     ` Christoph Hellwig [this message]
2025-12-16 15:59       ` Darrick J. Wong
2025-12-15  9:48 ` [PATCH 2/2] xfs: fix the zoned RT growfs check for zone alignment Christoph Hellwig
2025-12-15 19:09   ` Darrick J. Wong
  -- strict thread matches above, loose matches on Subject: below --
2025-12-16 17:30 reject non-zone aligned RT subvolumes v2 Christoph Hellwig
2025-12-16 17:30 ` [PATCH 1/2] xfs: validate that zoned RT devices are zone aligned 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=20251216051002.GA26237@lst.de \
    --to=hch@lst.de \
    --cc=cem@kernel.org \
    --cc=djwong@kernel.org \
    --cc=linux-xfs@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