* Re: [PATCH 4.19 00/86] 4.19.135-rc1 review
[not found] <20200727134914.312934924@linuxfoundation.org>
@ 2020-07-28 8:34 ` Naresh Kamboju
2020-07-28 13:03 ` [External] " Muchun Song
0 siblings, 1 reply; 3+ messages in thread
From: Naresh Kamboju @ 2020-07-28 8:34 UTC (permalink / raw)
To: Greg Kroah-Hartman, songmuchun
Cc: open list, Linus Torvalds, Andrew Morton, Guenter Roeck,
Shuah Khan, patches, Ben Hutchings, linux- stable, Hugh Dickins,
Christoph Lameter, Roman Gushchin, iamjoonsoo.kim, linux-mm,
mm-commits, penberg, rientjes, Shakeel Butt, Vlastimil Babka,
Arnd Bergmann, lkft-triage, clang-built-linux
On Mon, 27 Jul 2020 at 19:40, Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
>
> This is the start of the stable review cycle for the 4.19.135 release.
> There are 86 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know.
>
> Responses should be made by Wed, 29 Jul 2020 13:48:51 +0000.
> Anything received after that time might be too late.
>
> The whole patch series can be found in one patch at:
> https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.19.135-rc1.gz
> or in the git tree and branch at:
> git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.19.y
> and the diffstat can be found below.
>
> thanks,
>
> greg k-h
>
> -------------
> Pseudo-Shortlog of commits:
<trim>
Results from Linaro’s test farm.
Regressions detected on x86_64.
Boot failures on x86_64 devices running 4.19.135-rc1 kernel.
Summary
------------------------------------------------------------------------
kernel: 4.19.135-rc1
git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
git branch: linux-4.19.y
git commit: e11702667f84474535b156dbb194deffa0a6cdb4
git describe: v4.19.134-87-ge11702667f84
Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-4.19-oe/build/v4.19.134-87-ge11702667f84
> Muchun Song <songmuchun@bytedance.com>
> mm: memcg/slab: fix memory leak at non-root kmem_cache destroy
[ 2.510884] ============================================
[ 2.510884] WARNING: possible recursive locking detected
[ 2.510884] 4.19.135-rc1 #1 Not tainted
[ 2.510884] --------------------------------------------
[ 2.510884] swapper/0/1 is trying to acquire lock:
[ 2.510884] 0000000088703397 (slab_mutex){+.+.}, at:
kmem_cache_destroy+0x9a/0x2b0
[ 2.510884]
[ 2.510884] but task is already holding lock:
[ 2.510884] 0000000088703397 (slab_mutex){+.+.}, at:
kmem_cache_destroy+0x45/0x2b0
[ 2.510884]
[ 2.510884] other info that might help us debug this:
[ 2.510884] Possible unsafe locking scenario:
[ 2.510884]
[ 2.510884] CPU0
[ 2.510884] ----
[ 2.510884] lock(slab_mutex);
[ 2.510884] lock(slab_mutex);
[ 2.510884]
[ 2.510884] *** DEADLOCK ***
[ 2.510884]
[ 2.510884] May be due to missing lock nesting notation
[ 2.510884]
[ 2.510884] 3 locks held by swapper/0/1:
[ 2.510884] #0: 000000008702dddc (cpu_hotplug_lock.rw_sem){++++},
at: kmem_cache_destroy+0x32/0x2b0
[ 2.510884] #1: 0000000050103e4d (mem_hotplug_lock.rw_sem){++++},
at: kmem_cache_destroy+0x37/0x2b0
[ 2.510884] #2: 0000000088703397 (slab_mutex){+.+.}, at:
kmem_cache_destroy+0x45/0x2b0
[ 2.510884]
[ 2.510884] stack backtrace:
[ 2.510884] CPU: 2 PID: 1 Comm: swapper/0 Not tainted 4.19.135-rc1 #1
[ 2.510884] Hardware name: Supermicro SYS-5019S-ML/X11SSH-F, BIOS
2.0b 07/27/2017
[ 2.510884] Call Trace:
[ 2.510884] dump_stack+0x7a/0xa5
[ 2.510884] __lock_acquire+0x6f1/0x1380
[ 2.510884] ? ret_from_fork+0x3a/0x50
[ 2.510884] lock_acquire+0x95/0x190
[ 2.510884] ? lock_acquire+0x95/0x190
[ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
[ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
[ 2.510884] __mutex_lock+0x83/0x990
[ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
[ 2.510884] ? kmem_cache_destroy+0x60/0x2b0
[ 2.510884] ? set_debug_rodata+0x17/0x17
[ 2.510884] ? set_debug_rodata+0x17/0x17
[ 2.510884] mutex_lock_nested+0x1b/0x20
[ 2.510884] ? get_online_mems+0x5f/0x90
[ 2.510884] ? mutex_lock_nested+0x1b/0x20
[ 2.510884] kmem_cache_destroy+0x9a/0x2b0
[ 2.510884] ? set_debug_rodata+0x17/0x17
[ 2.510884] intel_iommu_init+0x11c6/0x1326
[ 2.510884] ? kfree+0xc4/0x240
[ 2.510884] ? lockdep_hardirqs_on+0xef/0x180
[ 2.510884] ? kfree+0xc4/0x240
[ 2.510884] ? trace_hardirqs_on+0x4c/0x100
[ 2.510884] ? unpack_to_rootfs+0x272/0x29a
[ 2.510884] ? e820__memblock_setup+0x64/0x64
[ 2.510884] ? set_debug_rodata+0x17/0x17
[ 2.510884] pci_iommu_init+0x1a/0x44
[ 2.510884] ? e820__memblock_setup+0x64/0x64
[ 2.510884] ? pci_iommu_init+0x1a/0x44
[ 2.510884] do_one_initcall+0x61/0x2b4
[ 2.510884] ? set_debug_rodata+0xa/0x17
[ 2.510884] ? rcu_read_lock_sched_held+0x81/0x90
[ 2.510884] kernel_init_freeable+0x1d8/0x270
[ 2.510884] ? rest_init+0x190/0x190
[ 2.510884] kernel_init+0xe/0x110
[ 2.510884] ret_from_fork+0x3a/0x50
Full test log:
https://pastebin.com/PWkk0YaF
--
Linaro LKFT
https://lkft.linaro.org
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [External] Re: [PATCH 4.19 00/86] 4.19.135-rc1 review
2020-07-28 8:34 ` [PATCH 4.19 00/86] 4.19.135-rc1 review Naresh Kamboju
@ 2020-07-28 13:03 ` Muchun Song
2020-07-28 14:35 ` Naresh Kamboju
0 siblings, 1 reply; 3+ messages in thread
From: Muchun Song @ 2020-07-28 13:03 UTC (permalink / raw)
To: Naresh Kamboju
Cc: Greg Kroah-Hartman, open list, Linus Torvalds, Andrew Morton,
Guenter Roeck, Shuah Khan, patches, Ben Hutchings, linux- stable,
Hugh Dickins, Christoph Lameter, Roman Gushchin, Joonsoo Kim,
linux-mm, mm-commits, Pekka Enberg, David Rientjes, Shakeel Butt,
Vlastimil Babka, Arnd Bergmann, lkft-triage, clang-built-linux
Thanks for your test. I have reviewed the patch:
[PATCH 4.19 76/86] mm: memcg/slab: fix memory leak at non-root
kmem_cache destroy
There is a backport problem and I have pointed out the problem in that email.
On Tue, Jul 28, 2020 at 4:34 PM Naresh Kamboju
<naresh.kamboju@linaro.org> wrote:
>
> On Mon, 27 Jul 2020 at 19:40, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
> >
> > This is the start of the stable review cycle for the 4.19.135 release.
> > There are 86 patches in this series, all will be posted as a response
> > to this one. If anyone has any issues with these being applied, please
> > let me know.
> >
> > Responses should be made by Wed, 29 Jul 2020 13:48:51 +0000.
> > Anything received after that time might be too late.
> >
> > The whole patch series can be found in one patch at:
> > https://www.kernel.org/pub/linux/kernel/v4.x/stable-review/patch-4.19.135-rc1.gz
> > or in the git tree and branch at:
> > git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.19.y
> > and the diffstat can be found below.
> >
> > thanks,
> >
> > greg k-h
> >
> > -------------
> > Pseudo-Shortlog of commits:
> <trim>
>
> Results from Linaro’s test farm.
> Regressions detected on x86_64.
>
> Boot failures on x86_64 devices running 4.19.135-rc1 kernel.
>
> Summary
> ------------------------------------------------------------------------
>
> kernel: 4.19.135-rc1
> git repo: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git
> git branch: linux-4.19.y
> git commit: e11702667f84474535b156dbb194deffa0a6cdb4
> git describe: v4.19.134-87-ge11702667f84
> Test details: https://qa-reports.linaro.org/lkft/linux-stable-rc-4.19-oe/build/v4.19.134-87-ge11702667f84
>
> > Muchun Song <songmuchun@bytedance.com>
> > mm: memcg/slab: fix memory leak at non-root kmem_cache destroy
>
> [ 2.510884] ============================================
> [ 2.510884] WARNING: possible recursive locking detected
> [ 2.510884] 4.19.135-rc1 #1 Not tainted
> [ 2.510884] --------------------------------------------
> [ 2.510884] swapper/0/1 is trying to acquire lock:
> [ 2.510884] 0000000088703397 (slab_mutex){+.+.}, at:
> kmem_cache_destroy+0x9a/0x2b0
> [ 2.510884]
> [ 2.510884] but task is already holding lock:
> [ 2.510884] 0000000088703397 (slab_mutex){+.+.}, at:
> kmem_cache_destroy+0x45/0x2b0
> [ 2.510884]
> [ 2.510884] other info that might help us debug this:
> [ 2.510884] Possible unsafe locking scenario:
> [ 2.510884]
> [ 2.510884] CPU0
> [ 2.510884] ----
> [ 2.510884] lock(slab_mutex);
> [ 2.510884] lock(slab_mutex);
> [ 2.510884]
> [ 2.510884] *** DEADLOCK ***
> [ 2.510884]
> [ 2.510884] May be due to missing lock nesting notation
> [ 2.510884]
> [ 2.510884] 3 locks held by swapper/0/1:
> [ 2.510884] #0: 000000008702dddc (cpu_hotplug_lock.rw_sem){++++},
> at: kmem_cache_destroy+0x32/0x2b0
> [ 2.510884] #1: 0000000050103e4d (mem_hotplug_lock.rw_sem){++++},
> at: kmem_cache_destroy+0x37/0x2b0
> [ 2.510884] #2: 0000000088703397 (slab_mutex){+.+.}, at:
> kmem_cache_destroy+0x45/0x2b0
> [ 2.510884]
> [ 2.510884] stack backtrace:
> [ 2.510884] CPU: 2 PID: 1 Comm: swapper/0 Not tainted 4.19.135-rc1 #1
> [ 2.510884] Hardware name: Supermicro SYS-5019S-ML/X11SSH-F, BIOS
> 2.0b 07/27/2017
> [ 2.510884] Call Trace:
> [ 2.510884] dump_stack+0x7a/0xa5
> [ 2.510884] __lock_acquire+0x6f1/0x1380
> [ 2.510884] ? ret_from_fork+0x3a/0x50
> [ 2.510884] lock_acquire+0x95/0x190
> [ 2.510884] ? lock_acquire+0x95/0x190
> [ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
> [ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
> [ 2.510884] __mutex_lock+0x83/0x990
> [ 2.510884] ? kmem_cache_destroy+0x9a/0x2b0
> [ 2.510884] ? kmem_cache_destroy+0x60/0x2b0
> [ 2.510884] ? set_debug_rodata+0x17/0x17
> [ 2.510884] ? set_debug_rodata+0x17/0x17
> [ 2.510884] mutex_lock_nested+0x1b/0x20
> [ 2.510884] ? get_online_mems+0x5f/0x90
> [ 2.510884] ? mutex_lock_nested+0x1b/0x20
> [ 2.510884] kmem_cache_destroy+0x9a/0x2b0
> [ 2.510884] ? set_debug_rodata+0x17/0x17
> [ 2.510884] intel_iommu_init+0x11c6/0x1326
> [ 2.510884] ? kfree+0xc4/0x240
> [ 2.510884] ? lockdep_hardirqs_on+0xef/0x180
> [ 2.510884] ? kfree+0xc4/0x240
> [ 2.510884] ? trace_hardirqs_on+0x4c/0x100
> [ 2.510884] ? unpack_to_rootfs+0x272/0x29a
> [ 2.510884] ? e820__memblock_setup+0x64/0x64
> [ 2.510884] ? set_debug_rodata+0x17/0x17
> [ 2.510884] pci_iommu_init+0x1a/0x44
> [ 2.510884] ? e820__memblock_setup+0x64/0x64
> [ 2.510884] ? pci_iommu_init+0x1a/0x44
> [ 2.510884] do_one_initcall+0x61/0x2b4
> [ 2.510884] ? set_debug_rodata+0xa/0x17
> [ 2.510884] ? rcu_read_lock_sched_held+0x81/0x90
> [ 2.510884] kernel_init_freeable+0x1d8/0x270
> [ 2.510884] ? rest_init+0x190/0x190
> [ 2.510884] kernel_init+0xe/0x110
> [ 2.510884] ret_from_fork+0x3a/0x50
>
>
> Full test log:
> https://pastebin.com/PWkk0YaF
>
> --
> Linaro LKFT
> https://lkft.linaro.org
--
Yours,
Muchun
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [External] Re: [PATCH 4.19 00/86] 4.19.135-rc1 review
2020-07-28 13:03 ` [External] " Muchun Song
@ 2020-07-28 14:35 ` Naresh Kamboju
0 siblings, 0 replies; 3+ messages in thread
From: Naresh Kamboju @ 2020-07-28 14:35 UTC (permalink / raw)
To: Muchun Song
Cc: Greg Kroah-Hartman, open list, Linus Torvalds, Andrew Morton,
Guenter Roeck, Shuah Khan, patches, Ben Hutchings, linux- stable,
Hugh Dickins, Christoph Lameter, Roman Gushchin, Joonsoo Kim,
linux-mm, mm-commits, Pekka Enberg, David Rientjes, Shakeel Butt,
Vlastimil Babka, Arnd Bergmann, lkft-triage, clang-built-linux
On Tue, 28 Jul 2020 at 18:33, Muchun Song <songmuchun@bytedance.com> wrote:
>
> Thanks for your test. I have reviewed the patch:
>
> [PATCH 4.19 76/86] mm: memcg/slab: fix memory leak at non-root
> kmem_cache destroy
>
> There is a backport problem and I have pointed out the problem in that email.
Thanks for your suggestions on the other email thread.
I have made changes as you said and boot test pass on x86 now.
diff --git a/mm/slab_common.c b/mm/slab_common.c
index 9c5eb4b08fc3..65bc49f19504 100644
--- a/mm/slab_common.c
+++ b/mm/slab_common.c
@@ -842,9 +842,7 @@ static int shutdown_memcg_caches(struct kmem_cache *s)
static void memcg_set_kmem_cache_dying(struct kmem_cache *s)
{
- mutex_lock(&slab_mutex);
s->memcg_params.dying = true;
- mutex_unlock(&slab_mutex);
}
static void flush_memcg_workqueue(struct kmem_cache *s)
- Naresh
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-07-28 14:36 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20200727134914.312934924@linuxfoundation.org>
2020-07-28 8:34 ` [PATCH 4.19 00/86] 4.19.135-rc1 review Naresh Kamboju
2020-07-28 13:03 ` [External] " Muchun Song
2020-07-28 14:35 ` Naresh Kamboju
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).