All of lore.kernel.org
 help / color / mirror / Atom feed
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,
> 
> 

      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 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.