diff options
| author | Xiu Jianfeng <xiujianfeng@huawei.com> | 2024-04-23 02:44:39 +0000 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2024-04-23 06:00:43 -1000 |
| commit | e8784765fae6edc47efb68d425c65e3633d70cd6 (patch) | |
| tree | dd1e260ebf6166b68f5ea4fc789d1796920beef1 | |
| parent | 8996f93fc3880d000a0a0cb40c724d5830e140fd (diff) | |
cgroup/cpuset: Avoid clearing CS_SCHED_LOAD_BALANCE twice
In cpuset_css_online(), CS_SCHED_LOAD_BALANCE will be cleared twice,
the former one in the is_in_v2_mode() case could be removed because
is_in_v2_mode() can be true for cgroup v1 if the "cpuset_v2_mode"
mount option is specified, that balance flag change isn't appropriate
for this particular case.
Signed-off-by: Xiu Jianfeng <xiujianfeng@huawei.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
| -rw-r--r-- | kernel/cgroup/cpuset.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/kernel/cgroup/cpuset.c b/kernel/cgroup/cpuset.c index e70008a1d86a..032fcc93f2b8 100644 --- a/kernel/cgroup/cpuset.c +++ b/kernel/cgroup/cpuset.c @@ -4052,11 +4052,6 @@ static int cpuset_css_online(struct cgroup_subsys_state *css) cs->effective_mems = parent->effective_mems; cs->use_parent_ecpus = true; parent->child_ecpus_count++; - /* - * Clear CS_SCHED_LOAD_BALANCE if parent is isolated - */ - if (!is_sched_load_balance(parent)) - clear_bit(CS_SCHED_LOAD_BALANCE, &cs->flags); } /* |