* [Patch v2] cgroup: return early if it is already on preloaded list
@ 2021-12-14 0:46 Wei Yang
2021-12-14 19:45 ` Tejun Heo
0 siblings, 1 reply; 2+ messages in thread
From: Wei Yang @ 2021-12-14 0:46 UTC (permalink / raw)
To: tj, lizefan.x, hannes; +Cc: cgroups, linux-kernel, Wei Yang
If a cset is already on preloaded list, this means we have already setup
this cset properly for migration.
This patch just relocate the root cgrp lookup which isn't used anyway
when the cset is already on the preloaded list.
[tj@kernel.org: rephrase the commit log]
Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
---
v2: rephrase commit log
---
kernel/cgroup/cgroup.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/cgroup/cgroup.c b/kernel/cgroup/cgroup.c
index 452a723d4a36..2cf729afe834 100644
--- a/kernel/cgroup/cgroup.c
+++ b/kernel/cgroup/cgroup.c
@@ -2648,11 +2648,11 @@ void cgroup_migrate_add_src(struct css_set *src_cset,
if (src_cset->dead)
return;
- src_cgrp = cset_cgroup_from_root(src_cset, dst_cgrp->root);
-
if (!list_empty(&src_cset->mg_preload_node))
return;
+ src_cgrp = cset_cgroup_from_root(src_cset, dst_cgrp->root);
+
WARN_ON(src_cset->mg_src_cgrp);
WARN_ON(src_cset->mg_dst_cgrp);
WARN_ON(!list_empty(&src_cset->mg_tasks));
--
2.33.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Patch v2] cgroup: return early if it is already on preloaded list
2021-12-14 0:46 [Patch v2] cgroup: return early if it is already on preloaded list Wei Yang
@ 2021-12-14 19:45 ` Tejun Heo
0 siblings, 0 replies; 2+ messages in thread
From: Tejun Heo @ 2021-12-14 19:45 UTC (permalink / raw)
To: Wei Yang; +Cc: lizefan.x, hannes, cgroups, linux-kernel
On Tue, Dec 14, 2021 at 12:46:07AM +0000, Wei Yang wrote:
> If a cset is already on preloaded list, this means we have already setup
> this cset properly for migration.
>
> This patch just relocate the root cgrp lookup which isn't used anyway
> when the cset is already on the preloaded list.
>
> [tj@kernel.org: rephrase the commit log]
>
> Signed-off-by: Wei Yang <richard.weiyang@gmail.com>
Applied to cgroup/for-5.17.
Thanks.
--
tejun
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-14 19:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-14 0:46 [Patch v2] cgroup: return early if it is already on preloaded list Wei Yang
2021-12-14 19:45 ` Tejun Heo
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).