From: Carlos Maiolino <cem@kernel.org>
To: Kefeng Wang <wangkefeng.wang@huawei.com>
Cc: linux-xfs@vger.kernel.org, "Darrick J . Wong" <djwong@kernel.org>,
Shakeel Butt <shakeel.butt@linux.dev>,
Dave Chinner <dchinner@redhat.com>,
Christoph Hellwig <hch@lst.de>,
Brian Foster <bfoster@redhat.com>
Subject: Re: [PATCH for-next v3] xfs: fix NOFS state corruption in btree split worker
Date: Sun, 11 Oct 2026 09:24:00 +0200 [thread overview]
Message-ID: <ass3LRLEFJDlFIoh@cronus.toxiclabs.cc> (raw)
In-Reply-To: <f15d804d-f3d8-433f-97d1-5dfcabcc381c@huawei.com>
On Sat, Oct 10, 2026 at 06:46:42PM +0800, Kefeng Wang wrote:
>
>
> On 10/6/2026 2:06 AM, Carlos Maiolino wrote:
> > On Mon, 21 Sep 2026 15:10:30 +0800, Kefeng Wang wrote:
> > > xfs_btree_split_worker() calls xfs_trans_set_context() and
> > > xfs_trans_clear_context() on the caller's transaction, overwriting
> > > tp->t_pflags with the worker's NOFS state. When the caller already has
> > > PF_MEMALLOC_NOFS set (e.g. xfs_end_ioend_write, xfs_dio_write_end_io),
> > > the corrupted tp->t_pflags causes xfs_trans_free() to erroneously clear
> > > the caller's NOFS protection.
> > >
> > > [...]
> >
> > Applied to for-next, thanks!
>
> Thank,I checked linux-next's next-20261009, and there was a commit
> 23f91a51425e, but the code was incorrect during the conflict resolution.
> I don't know if this is the correct state for linux-next, just a reminder.
Linux-next appears to have removed all changes from this patch. I don't
know why it happened honestly, I'll need to dig into it later. IIRC
there was a conflict with -mm patches.
Patches staged for xfs are in xfs-linux/xfs-7.4-merge. Code there is
correct.
I always attempt a merge against Linus's tree before sending him the
patches and at the time if I see any conflict, I'll give him a heads up.
Thanks for letting me know though. I don't keep comparing xfs-linux
state against linux-next.
>
>
> >
> > [1/1] xfs: fix NOFS state corruption in btree split worker
> > commit: 23f91a51425ed3d4a5c42978fc25f1087611e4ba
> >
> > Best regards,
>
>
prev parent reply other threads:[~2026-10-11 7:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-21 7:10 [PATCH for-next v3] xfs: fix NOFS state corruption in btree split worker Kefeng Wang
2026-09-21 10:19 ` Carlos Maiolino
2026-09-21 14:22 ` Darrick J. Wong
2026-10-05 18:06 ` Carlos Maiolino
2026-10-10 10:46 ` Kefeng Wang
2026-10-11 7:24 ` Carlos Maiolino [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=ass3LRLEFJDlFIoh@cronus.toxiclabs.cc \
--to=cem@kernel.org \
--cc=bfoster@redhat.com \
--cc=dchinner@redhat.com \
--cc=djwong@kernel.org \
--cc=hch@lst.de \
--cc=linux-xfs@vger.kernel.org \
--cc=shakeel.butt@linux.dev \
--cc=wangkefeng.wang@huawei.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