From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751418AbcL2WLV (ORCPT ); Thu, 29 Dec 2016 17:11:21 -0500 Received: from mail-yw0-f193.google.com ([209.85.161.193]:32975 "EHLO mail-yw0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751073AbcL2WLT (ORCPT ); Thu, 29 Dec 2016 17:11:19 -0500 From: Tejun Heo To: lizefan@huawei.com, hannes@cmpxchg.org Cc: cgroups@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCHSET for-4.11] cgroup: avoid spurious identity ->*attach() invocations Date: Thu, 29 Dec 2016 17:11:12 -0500 Message-Id: <20161229221115.31995-1-tj@kernel.org> X-Mailer: git-send-email 2.9.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On the v2 hierarchy, when controllers are enabled and disabled, other ->*attach() callbacks of other controllers are called spuriously with the same source and destination. While this isn't critical, it's a bit nasty and can lead to temporary double charging on certain controllers. This patchset fixes the issue. This patchset contains the following three patches. 0001-cgroup-cosmetic-update-to-cgroup_taskset_add.patch 0002-cgroup-track-migration-context-in-cgroup_mgctx.patch 0003-cgroup-call-subsys-attach-only-for-subsystems-which-.patch 0001-0002 restructures migration context tracking so that extra state can be tracked easily. 0003 fixes the spurious ->*attach() invocations. This patchset is also available in the following git branch. git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup.git review-avoid-identity-attach diffstat follows. kernel/cgroup/cgroup-internal.h | 69 +++++++++++++++-- kernel/cgroup/cgroup-v1.c | 10 +- kernel/cgroup/cgroup.c | 161 ++++++++++++++++------------------------ 3 files changed, 135 insertions(+), 105 deletions(-) Thanks. -- tejun