public inbox for linux-mm@kvack.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Qi Zheng <qi.zheng@linux.dev>
Cc: shakeel.butt@linux.dev,
	syzbot <syzbot+7d60b33a8a546263da7c@syzkaller.appspotmail.com>,
	Liam.Howlett@oracle.com, linux-kernel@vger.kernel.org,
	linux-mm@kvack.org, ljs@kernel.org, surenb@google.com,
	syzkaller-bugs@googlegroups.com, vbabka@kernel.org,
	Muchun Song <songmuchun@bytedance.com>
Subject: Re: [syzbot] [mm?] WARNING: bad unlock balance in do_wp_page
Date: Mon, 27 Apr 2026 03:44:02 -0700	[thread overview]
Message-ID: <20260427034402.096cb95de92bec15becc78bb@linux-foundation.org> (raw)
In-Reply-To: <f0ab0c7a-c597-41a8-92b3-4424b12b1b1e@linux.dev>

On Mon, 27 Apr 2026 17:43:38 +0800 Qi Zheng <qi.zheng@linux.dev> wrote:

> 
> Alright, it seems I have successfully reproduced it:
> (The reproducer is attached at the bottom of this email.)

That's a lot of code.  Thanks for doing that.  Maybe there's something
here we can put into selftests/

> >>
> >>> However, in a production environment, this is practically impossible.
> >>
> >> Can you expand on this?
> >>
> >> sysbot isn't a production environment ;)
> > 
> > Rebinding only works when the hierarchy is completely empty. This is
> > generally not the case in a production environment (e.g. when systemd
> > is used).
> > 
> > BTW, it seems rebinding is about to be deprecated:
> > 
> > cgroup1_reconfigure
> > --> pr_warn("option changes via remount are deprecated (pid=%d comm=%s)\n",
> >              task_tgid_nr(current), current->comm);
> > 
> > Also, it appears the current memcg subsystem assumes that
> > cgroup_subsys_on_dfl(memory_cgrp_subsys) cannot be changed at runtime.
> > (Please correct me if I missed anything.)
> > 
> > If we can get a reproducer, we can try the following fix, or simply drop
> > rebinding altogether?

We'll want something which is applicable to 7.1-rcX please.  Removal of
rebinding sounds like something we'd address in 7.2 or later.

> >  From 6ae41b91339625dd7bf0f819f775f26e78171a73 Mon Sep 17 00:00:00 2001
> > From: Qi Zheng <zhengqi.arch@bytedance.com>
> > Date: Mon, 27 Apr 2026 11:20:21 +0800
> > Subject: [PATCH] mm: memcontrol: fix rcu unbalance in
> >   get_non_dying_memcg_end()
> > 
> > Signed-off-by: Qi Zheng <zhengqi.arch@bytedance.com>
> > ---
> >   mm/memcontrol.c | 30 ++++++++++++++++++++----------
> >   1 file changed, 20 insertions(+), 10 deletions(-)
> 
> With the above patch applied, the warnings are gone.
> 
> If no one objects, I'll submit the formal fix. Or should we actually
> just remove rebinding instead?

I suggest we just fix up current -rc please.



  reply	other threads:[~2026-04-27 10:44 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-26  8:17 [syzbot] [mm?] WARNING: bad unlock balance in do_wp_page syzbot
2026-04-26 10:49 ` Andrew Morton
2026-04-26 15:57   ` Qi Zheng
2026-04-26 17:55     ` Andrew Morton
2026-04-27  7:24       ` Qi Zheng
2026-04-27  9:43         ` Qi Zheng
2026-04-27 10:44           ` Andrew Morton [this message]
2026-04-27 10:57             ` Qi Zheng
2026-04-27 10:43         ` Andrew Morton
2026-04-27 10:54           ` Qi Zheng

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=20260427034402.096cb95de92bec15becc78bb@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=Liam.Howlett@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ljs@kernel.org \
    --cc=qi.zheng@linux.dev \
    --cc=shakeel.butt@linux.dev \
    --cc=songmuchun@bytedance.com \
    --cc=surenb@google.com \
    --cc=syzbot+7d60b33a8a546263da7c@syzkaller.appspotmail.com \
    --cc=syzkaller-bugs@googlegroups.com \
    --cc=vbabka@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