Linux XFS filesystem development
 help / color / mirror / Atom feed
From: Brian Foster <bfoster@redhat.com>
To: Mark Tinguely <mark.tinguely@oracle.com>
Cc: linux-xfs@vger.kernel.org, Matt Fleming <matt@readmodwrite.com>
Subject: Re: [External] : Re: [PATCH v2 3/3] xfs: incorporate increased AGFL min requirement for minleft allocs
Date: Fri, 14 Aug 2026 12:24:40 -0400	[thread overview]
Message-ID: <an9BSI7I0WX3ua4g@bfoster> (raw)
In-Reply-To: <c493b668-9b13-4802-ad05-f54cc62db168@oracle.com>

On Fri, Aug 14, 2026 at 10:18:15AM -0500, Mark Tinguely wrote:
> On 8/14/26 9:47 AM, Brian Foster wrote:
> > On Fri, Aug 14, 2026 at 09: 15: 15AM -0500, Mark Tinguely wrote: > On 8/14/26 8: 57 AM, Brian Foster wrote: > > On Fri, Aug 14, 2026 at 09: 22: 39AM -0400, Brian Foster wrote: > > > Matt Fleming reports a filesystem shutdown due
> > 
> > 
> > On Fri, Aug 14, 2026 at 09:15:15AM -0500, Mark Tinguely wrote:
> > > On 8/14/26 8:57 AM, Brian Foster wrote:
> > > > On Fri, Aug 14, 2026 at 09:22:39AM -0400, Brian Foster wrote:
> > > > > Matt Fleming reports a filesystem shutdown due to inobt block
> > > > > allocation failure during sparse chunk allocation. Inode creation
> 
> 
> (deleted text)
> 
> > > 
> > > 
> > > thank you for the comment clarification.
> > > I was confused by that change.
> > > 
> > > Could you CC stable. We saw a version of this problem in Linux 5.15
> > > 
> > 
> > Sure, though I'm not sure how far back this will apply cleanly.
> > 
> > Curious.. was the instance you saw also an inobt alloc failure leading
> > to dirty trans cancel and shutdown, or something different but similar?
> > 
> > Brian
> > 
> 
> yes the v5.15 problem was a similar sparse inode chunk / inode btree insert failure.
> I admit that I did not have the numbers to realize why the inode btree insert failed.
> 

Ok.

> although this is a fundamental allocation flaw, the sparse inode chunk allocation
> seems to be the just right/wrong conditions to trigger it.
> 

I think that's probably right, but I had the Fixes: tag on v1 and
dropped it here just because I think it's kind of a tenuous connection
now. Re: the discussion on v1, presumably we could have the same thing
happen with bmap allocs and just not see the shutdown because those
allocs are usually more flexible and can fall back to different AGs.

Regardless, wrt stable I think it would be reasonable to target back to
the release that enabled sparse inodes (v4.2) since that's how this is
reproduced in practice, so it probably doesn't make much difference
either way.

Brian

> Fixes: 5419040fc0f3a xfs: introduce inode record hole mask for sparse inode chunks ??
> 
> thanks for the fix
> 
> Mark Tinguely
> 
> 
> 


      reply	other threads:[~2026-08-14 16:24 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-14 13:22 [PATCH v2 0/3] xfs: fix a couple sparse chunk alloc problems Brian Foster
2026-08-14 13:22 ` [PATCH v2 1/3] xfs: set minleft correctly for sparse chunk errortag allocation Brian Foster
2026-08-14 18:54   ` Darrick J. Wong
2026-08-14 13:22 ` [PATCH v2 2/3] xfs: support additional levels in the agfl minimum calculation Brian Foster
2026-08-14 18:58   ` Darrick J. Wong
2026-08-14 13:22 ` [PATCH v2 3/3] xfs: incorporate increased AGFL min requirement for minleft allocs Brian Foster
2026-08-14 13:57   ` Brian Foster
2026-08-14 14:15     ` [External] : " Mark Tinguely
2026-08-14 14:47       ` Brian Foster
2026-08-14 15:18         ` Mark Tinguely
2026-08-14 16:24           ` Brian Foster [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=an9BSI7I0WX3ua4g@bfoster \
    --to=bfoster@redhat.com \
    --cc=linux-xfs@vger.kernel.org \
    --cc=mark.tinguely@oracle.com \
    --cc=matt@readmodwrite.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