* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
@ 2022-09-27 2:00 ` Roman Gushchin
2022-09-27 8:09 ` Muchun Song
` (4 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Roman Gushchin @ 2022-09-27 2:00 UTC (permalink / raw)
To: Shakeel Butt
Cc: Johannes Weiner, Michal Hocko, Muchun Song, Andrew Morton,
Michal Koutný,
cgroups, linux-mm, linux-kernel, Anatoly Pugachev, Vasily Averin,
Jakub Kicinski, Linus Torvalds, sparclinux
On Mon, Sep 26, 2022 at 07:59:31PM +0000, Shakeel Butt wrote:
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
>
> Reported-by: Anatoly Pugachev <matorola@gmail.com>
> Signed-off-by: Shakeel Butt <shakeelb@google.com>
> Cc: Vasily Averin <vvs@openvz.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Michal Koutný <mkoutny@suse.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: cgroups@vger.kernel.org
> Cc: sparclinux@vger.kernel.org
> Cc: linux-mm@kvack.org
> Cc: linux-kernel@vger.kernel.org
Acked-by: Roman Gushchin <roman.gushchin@linux.dev>
Thanks, Shakeel!
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
2022-09-27 2:00 ` Roman Gushchin
@ 2022-09-27 8:09 ` Muchun Song
2022-09-27 10:02 ` Vlastimil Babka (SUSE)
` (3 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Muchun Song @ 2022-09-27 8:09 UTC (permalink / raw)
To: Shakeel Butt
Cc: Johannes Weiner, Michal Hocko, Roman Gushchin, Muchun Song,
Andrew Morton, Michal Koutný,
cgroups, Linux MM, linux-kernel, Anatoly Pugachev, Vasily Averin,
Jakub Kicinski, Linus Torvalds, sparclinux
> On Sep 27, 2022, at 03:59, Shakeel Butt <shakeelb@google.com> wrote:
>
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
>
> Reported-by: Anatoly Pugachev <matorola@gmail.com>
> Signed-off-by: Shakeel Butt <shakeelb@google.com>
> Cc: Vasily Averin <vvs@openvz.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Michal Koutný <mkoutny@suse.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: cgroups@vger.kernel.org
> Cc: sparclinux@vger.kernel.org
> Cc: linux-mm@kvack.org
> Cc: linux-kernel@vger.kernel.org
Reviewed-by: Muchun Song <songmuchun@bytedance.com>
Thanks.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
2022-09-27 2:00 ` Roman Gushchin
2022-09-27 8:09 ` Muchun Song
@ 2022-09-27 10:02 ` Vlastimil Babka (SUSE)
2022-09-27 13:23 ` Johannes Weiner
` (2 subsequent siblings)
5 siblings, 0 replies; 7+ messages in thread
From: Vlastimil Babka (SUSE) @ 2022-09-27 10:02 UTC (permalink / raw)
To: Shakeel Butt, Johannes Weiner, Michal Hocko, Roman Gushchin,
Muchun Song, Andrew Morton
Cc: Michal Koutný,
cgroups, linux-mm, linux-kernel, Anatoly Pugachev, Vasily Averin,
Jakub Kicinski, Linus Torvalds, sparclinux, regressions
On 9/26/22 21:59, Shakeel Butt wrote:
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
>
Link: https://lore.kernel.org/all/20220918092849.GA10314@u164.east.ru/
(mainly for regzbot reading this thread but should be valid usage for
the report anyway)
Fixes: 1d0403d20f6c ("net: set proper memcg for net_init hooks allocations")
> Reported-by: Anatoly Pugachev <matorola@gmail.com>
> Signed-off-by: Shakeel Butt <shakeelb@google.com>
> Cc: Vasily Averin <vvs@openvz.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Michal Koutný <mkoutny@suse.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: cgroups@vger.kernel.org
> Cc: sparclinux@vger.kernel.org
> Cc: linux-mm@kvack.org
> Cc: linux-kernel@vger.kernel.org
> ---
> include/linux/memcontrol.h | 45 --------------------------------------
> net/core/net_namespace.c | 7 ------
> 2 files changed, 52 deletions(-)
>
> diff --git a/include/linux/memcontrol.h b/include/linux/memcontrol.h
> index ef479e554253..e1644a24009c 100644
> --- a/include/linux/memcontrol.h
> +++ b/include/linux/memcontrol.h
> @@ -1787,42 +1787,6 @@ static inline void count_objcg_event(struct obj_cgroup *objcg,
> rcu_read_unlock();
> }
>
> -/**
> - * get_mem_cgroup_from_obj - get a memcg associated with passed kernel object.
> - * @p: pointer to object from which memcg should be extracted. It can be NULL.
> - *
> - * Retrieves the memory group into which the memory of the pointed kernel
> - * object is accounted. If memcg is found, its reference is taken.
> - * If a passed kernel object is uncharged, or if proper memcg cannot be found,
> - * as well as if mem_cgroup is disabled, NULL is returned.
> - *
> - * Return: valid memcg pointer with taken reference or NULL.
> - */
> -static inline struct mem_cgroup *get_mem_cgroup_from_obj(void *p)
> -{
> - struct mem_cgroup *memcg;
> -
> - rcu_read_lock();
> - do {
> - memcg = mem_cgroup_from_obj(p);
> - } while (memcg && !css_tryget(&memcg->css));
> - rcu_read_unlock();
> - return memcg;
> -}
> -
> -/**
> - * mem_cgroup_or_root - always returns a pointer to a valid memory cgroup.
> - * @memcg: pointer to a valid memory cgroup or NULL.
> - *
> - * If passed argument is not NULL, returns it without any additional checks
> - * and changes. Otherwise, root_mem_cgroup is returned.
> - *
> - * NOTE: root_mem_cgroup can be NULL during early boot.
> - */
> -static inline struct mem_cgroup *mem_cgroup_or_root(struct mem_cgroup *memcg)
> -{
> - return memcg ? memcg : root_mem_cgroup;
> -}
> #else
> static inline bool mem_cgroup_kmem_disabled(void)
> {
> @@ -1879,15 +1843,6 @@ static inline void count_objcg_event(struct obj_cgroup *objcg,
> {
> }
>
> -static inline struct mem_cgroup *get_mem_cgroup_from_obj(void *p)
> -{
> - return NULL;
> -}
> -
> -static inline struct mem_cgroup *mem_cgroup_or_root(struct mem_cgroup *memcg)
> -{
> - return NULL;
> -}
> #endif /* CONFIG_MEMCG_KMEM */
>
> #if defined(CONFIG_MEMCG_KMEM) && defined(CONFIG_ZSWAP)
> diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c
> index 6b9f19122ec1..0ec2f5906a27 100644
> --- a/net/core/net_namespace.c
> +++ b/net/core/net_namespace.c
> @@ -18,7 +18,6 @@
> #include <linux/user_namespace.h>
> #include <linux/net_namespace.h>
> #include <linux/sched/task.h>
> -#include <linux/sched/mm.h>
> #include <linux/uidgid.h>
> #include <linux/cookie.h>
>
> @@ -1144,13 +1143,7 @@ static int __register_pernet_operations(struct list_head *list,
> * setup_net() and cleanup_net() are not possible.
> */
> for_each_net(net) {
> - struct mem_cgroup *old, *memcg;
> -
> - memcg = mem_cgroup_or_root(get_mem_cgroup_from_obj(net));
> - old = set_active_memcg(memcg);
> error = ops_init(ops, net);
> - set_active_memcg(old);
> - mem_cgroup_put(memcg);
> if (error)
> goto out_undo;
> list_add_tail(&net->exit_list, &net_exit_list);
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
` (2 preceding siblings ...)
2022-09-27 10:02 ` Vlastimil Babka (SUSE)
@ 2022-09-27 13:23 ` Johannes Weiner
2022-09-28 12:35 ` Anatoly Pugachev
2022-09-28 16:22 ` Linus Torvalds
5 siblings, 0 replies; 7+ messages in thread
From: Johannes Weiner @ 2022-09-27 13:23 UTC (permalink / raw)
To: Shakeel Butt
Cc: Michal Hocko, Roman Gushchin, Muchun Song, Andrew Morton,
Michal Koutný,
cgroups, linux-mm, linux-kernel, Anatoly Pugachev, Vasily Averin,
Jakub Kicinski, Linus Torvalds, sparclinux
On Mon, Sep 26, 2022 at 07:59:31PM +0000, Shakeel Butt wrote:
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
>
> Reported-by: Anatoly Pugachev <matorola@gmail.com>
> Signed-off-by: Shakeel Butt <shakeelb@google.com>
> Cc: Vasily Averin <vvs@openvz.org>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Michal Koutný <mkoutny@suse.com>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Linus Torvalds <torvalds@linux-foundation.org>
> Cc: cgroups@vger.kernel.org
> Cc: sparclinux@vger.kernel.org
> Cc: linux-mm@kvack.org
> Cc: linux-kernel@vger.kernel.org
Acked-by: Johannes Weiner <hannes@cmpxchg.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
` (3 preceding siblings ...)
2022-09-27 13:23 ` Johannes Weiner
@ 2022-09-28 12:35 ` Anatoly Pugachev
2022-09-28 16:22 ` Linus Torvalds
5 siblings, 0 replies; 7+ messages in thread
From: Anatoly Pugachev @ 2022-09-28 12:35 UTC (permalink / raw)
To: Shakeel Butt
Cc: Johannes Weiner, Michal Hocko, Roman Gushchin, Muchun Song,
Andrew Morton, Michal Koutný,
cgroups, linux-mm, Linux Kernel list, Vasily Averin,
Jakub Kicinski, Linus Torvalds, Sparc kernel list
On Mon, Sep 26, 2022 at 11:00 PM Shakeel Butt <shakeelb@google.com> wrote:
>
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
Tested-by: Anatoly Pugachev <matorola@gmail.com>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Revert "net: set proper memcg for net_init hooks allocations"
2022-09-26 19:59 [PATCH] Revert "net: set proper memcg for net_init hooks allocations" Shakeel Butt
` (4 preceding siblings ...)
2022-09-28 12:35 ` Anatoly Pugachev
@ 2022-09-28 16:22 ` Linus Torvalds
5 siblings, 0 replies; 7+ messages in thread
From: Linus Torvalds @ 2022-09-28 16:22 UTC (permalink / raw)
To: Shakeel Butt
Cc: Johannes Weiner, Michal Hocko, Roman Gushchin, Muchun Song,
Andrew Morton, Michal Koutný,
cgroups, linux-mm, linux-kernel, Anatoly Pugachev, Vasily Averin,
Jakub Kicinski, sparclinux
On Mon, Sep 26, 2022 at 1:00 PM Shakeel Butt <shakeelb@google.com> wrote:
>
> This reverts commit 1d0403d20f6c281cb3d14c5f1db5317caeec48e9.
>
> Anatoly Pugachev reported that the commit 1d0403d20f6c ("net: set proper
> memcg for net_init hooks allocations") is somehow causing the sparc64
> VMs failed to boot and the VMs boot fine with that patch reverted. So,
> revert the patch for now and later we can debug the issue.
Just FYI for the involved people - I've noq applied this directly to my tree.
Linus
^ permalink raw reply [flat|nested] 7+ messages in thread