From: Andrew Vagin <avagin@parallels.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Andrey Vagin <avagin@openvz.org>,
linux-mm@kvack.org, cgroups@vger.kernel.org,
linux-kernel@vger.kernel.org, Glauber Costa <glommer@openvz.org>,
Johannes Weiner <hannes@cmpxchg.org>,
Michal Hocko <mhocko@suse.cz>,
Balbir Singh <bsingharora@gmail.com>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
Konstantin Khlebnikov <khlebnikov@openvz.org>,
stable@vger.kernel.org
Subject: Re: [PATCH] memcg: don't initialize kmem-cache destroying work for root caches
Date: Tue, 6 Aug 2013 01:41:35 +0400 [thread overview]
Message-ID: <20130805214135.GA4958@paralelels.com> (raw)
In-Reply-To: <20130805141609.777a0d6dee55091f6981c39b@linux-foundation.org>
On Mon, Aug 05, 2013 at 02:16:09PM -0700, Andrew Morton wrote:
> On Tue, 6 Aug 2013 01:01:28 +0400 Andrew Vagin <avagin@parallels.com> wrote:
>
> > On Mon, Aug 05, 2013 at 01:05:30PM -0700, Andrew Morton wrote:
> > > On Mon, 5 Aug 2013 20:09:40 +0400 Andrey Vagin <avagin@openvz.org> wrote:
> > >
> > > > struct memcg_cache_params has a union. Different parts of this union
> > > > are used for root and non-root caches. A part with destroying work is
> > > > used only for non-root caches.
> > > >
> > > > I fixed the same problem in another place v3.9-rc1-16204-gf101a94, but
> > > > didn't notice this one.
> > > >
> > > > Cc: <stable@vger.kernel.org> [3.9.x]
> > >
> > > hm, why the cc:stable?
> >
> > Because this patch fixes the kernel panic:
> >
> > [ 46.848187] BUG: unable to handle kernel paging request at 000000fffffffeb8
> > [ 46.849026] IP: [<ffffffff811a484c>] kmem_cache_destroy_memcg_children+0x6c/0xc0
> > [ 46.849092] PGD 0
> > [ 46.849092] Oops: 0000 [#1] SMP
>
> OK, pretty soon we'll have a changelog!
Sorry, probably I had to write all these in the initial commit message. I
just thought that this patch is an additional part of v3.9-rc1-16204-gf101a94.
>
> What does one do to trigger this oops? The bug has been there since
> 3.9, so the means-of-triggering must be quite special?
I don't think that so many people use cgroups with limits of the kernel memory.
I use the vzctl utility to operate with containers. vzctl limits the
kernel memory of containers by default. A container should be started
and stoped a few times (five or four) to reproduce the bug.
And one more thing is that nf_conntrack should be loaded. It creates
a new kmem_cache for each network namespace.
Thanks
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
WARNING: multiple messages have this Message-ID (diff)
From: Andrew Vagin <avagin@parallels.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Andrey Vagin <avagin@openvz.org>, <linux-mm@kvack.org>,
<cgroups@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
Glauber Costa <glommer@openvz.org>,
Johannes Weiner <hannes@cmpxchg.org>,
Michal Hocko <mhocko@suse.cz>,
Balbir Singh <bsingharora@gmail.com>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
Konstantin Khlebnikov <khlebnikov@openvz.org>,
<stable@vger.kernel.org>
Subject: Re: [PATCH] memcg: don't initialize kmem-cache destroying work for root caches
Date: Tue, 6 Aug 2013 01:41:35 +0400 [thread overview]
Message-ID: <20130805214135.GA4958@paralelels.com> (raw)
In-Reply-To: <20130805141609.777a0d6dee55091f6981c39b@linux-foundation.org>
On Mon, Aug 05, 2013 at 02:16:09PM -0700, Andrew Morton wrote:
> On Tue, 6 Aug 2013 01:01:28 +0400 Andrew Vagin <avagin@parallels.com> wrote:
>
> > On Mon, Aug 05, 2013 at 01:05:30PM -0700, Andrew Morton wrote:
> > > On Mon, 5 Aug 2013 20:09:40 +0400 Andrey Vagin <avagin@openvz.org> wrote:
> > >
> > > > struct memcg_cache_params has a union. Different parts of this union
> > > > are used for root and non-root caches. A part with destroying work is
> > > > used only for non-root caches.
> > > >
> > > > I fixed the same problem in another place v3.9-rc1-16204-gf101a94, but
> > > > didn't notice this one.
> > > >
> > > > Cc: <stable@vger.kernel.org> [3.9.x]
> > >
> > > hm, why the cc:stable?
> >
> > Because this patch fixes the kernel panic:
> >
> > [ 46.848187] BUG: unable to handle kernel paging request at 000000fffffffeb8
> > [ 46.849026] IP: [<ffffffff811a484c>] kmem_cache_destroy_memcg_children+0x6c/0xc0
> > [ 46.849092] PGD 0
> > [ 46.849092] Oops: 0000 [#1] SMP
>
> OK, pretty soon we'll have a changelog!
Sorry, probably I had to write all these in the initial commit message. I
just thought that this patch is an additional part of v3.9-rc1-16204-gf101a94.
>
> What does one do to trigger this oops? The bug has been there since
> 3.9, so the means-of-triggering must be quite special?
I don't think that so many people use cgroups with limits of the kernel memory.
I use the vzctl utility to operate with containers. vzctl limits the
kernel memory of containers by default. A container should be started
and stoped a few times (five or four) to reproduce the bug.
And one more thing is that nf_conntrack should be loaded. It creates
a new kmem_cache for each network namespace.
Thanks
next prev parent reply other threads:[~2013-08-05 21:41 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-05 16:09 [PATCH] memcg: don't initialize kmem-cache destroying work for root caches Andrey Vagin
2013-08-05 16:09 ` Andrey Vagin
2013-08-05 16:09 ` Andrey Vagin
2013-08-05 20:05 ` Andrew Morton
2013-08-05 20:05 ` Andrew Morton
2013-08-05 21:01 ` Andrew Vagin
2013-08-05 21:01 ` Andrew Vagin
2013-08-05 21:16 ` Andrew Morton
2013-08-05 21:16 ` Andrew Morton
2013-08-05 21:41 ` Andrew Vagin [this message]
2013-08-05 21:41 ` Andrew Vagin
-- strict thread matches above, loose matches on Subject: below --
2013-05-22 8:09 Andrey Vagin
2013-05-22 8:09 ` Andrey Vagin
2013-05-14 12:38 Andrey Vagin
2013-05-14 12:38 ` Andrey Vagin
2013-05-14 12:38 ` Andrey Vagin
2013-05-14 14:40 ` Michal Hocko
2013-05-14 14:40 ` Michal Hocko
2013-05-14 14:44 ` Michal Hocko
2013-05-14 14:44 ` Michal Hocko
2013-05-14 14:49 ` Glauber Costa
2013-05-14 14:49 ` Glauber Costa
2013-05-14 14:52 ` Michal Hocko
2013-05-14 14:52 ` Michal Hocko
2013-05-14 16:08 ` Michal Hocko
2013-05-14 16:08 ` Michal Hocko
[not found] ` <20130514160859.GC5055-2MMpYkNvuYDjFM9bn6wA6Q@public.gmane.org>
2013-05-22 7:40 ` Andrew Vagin
2013-05-22 7:40 ` Andrew Vagin
2013-05-22 7:40 ` Andrew Vagin
[not found] ` <20130522074055.GA16207-yYYamFZzV1regbzhZkK2zA@public.gmane.org>
2013-05-22 7:50 ` Li Zefan
2013-05-22 7:50 ` Li Zefan
2013-05-22 7:50 ` Li Zefan
[not found] ` <519C78C0.3050204-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
2013-05-22 7:56 ` Andrew Vagin
2013-05-22 7:56 ` Andrew Vagin
2013-05-22 7:56 ` Andrew Vagin
[not found] ` <20130522075638.GB16934-yYYamFZzV1regbzhZkK2zA@public.gmane.org>
2013-05-22 10:32 ` Michal Hocko
2013-05-22 10:32 ` Michal Hocko
2013-05-22 10:32 ` Michal Hocko
[not found] ` <1368535118-27369-1-git-send-email-avagin-GEFAQzZX7r8dnm+yROfE0A@public.gmane.org>
2013-05-28 22:53 ` Andrew Morton
2013-05-28 22:53 ` Andrew Morton
2013-05-28 22:53 ` Andrew Morton
[not found] ` <20130528155326.0a8b66a7711746e827d5fdea-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
2013-05-29 2:48 ` Glauber Costa
2013-05-29 2:48 ` Glauber Costa
2013-05-29 2:48 ` Glauber Costa
2013-05-30 10:59 ` Glauber Costa
2013-05-30 10:59 ` Glauber Costa
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=20130805214135.GA4958@paralelels.com \
--to=avagin@parallels.com \
--cc=akpm@linux-foundation.org \
--cc=avagin@openvz.org \
--cc=bsingharora@gmail.com \
--cc=cgroups@vger.kernel.org \
--cc=glommer@openvz.org \
--cc=hannes@cmpxchg.org \
--cc=kamezawa.hiroyu@jp.fujitsu.com \
--cc=khlebnikov@openvz.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.cz \
--cc=stable@vger.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 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.