Subject: Re: [PATCH v4 5/8] cgroup: Wrap subsystem selection macro

From: Li Zefan <[email-masked]>
Date: Thu, 13 Sep 2012 14:41:34 +0800

On 2012/9/12 22:12, Daniel Wagner wrote:
> From: Daniel Wagner <[email-masked]>
> 
> Before we are able to define all subsystem ids at compile time we need
> a more fine grained control what gets defined when we include
> cgroup_subsys.h. For example we define the enums for the subsystems or
> to declare for struct cgroup_subsys (builtin subsystem) by including
> cgroup_subsys.h and defining SUBSYS accordingly.
> 
> Currently, the decision if a subsys is used is defined inside the
> header by testing if CONFIG_*=y is true. By moving this test outside
> of cgroup_subsys.h we are able to control it on the include level.
> 
> This is done by introducing IS_SUBSYS_ENABLED which then is defined
> according the task, e.g. is CONFIG_*=y or CONFIG_*=m.
> 
> Signed-off-by: Daniel Wagner <[email-masked]>
> Cc: Gao feng <[email-masked]>
> Cc: Jamal Hadi Salim <jhs-jkUAjuhPggJWk0Htik3J/[email-masked]>
> Cc: John Fastabend <[email-masked]>
> Cc: Li Zefan <[email-masked]>
> Cc: Neil Horman <[email-masked]>
> Cc: Tejun Heo <[email-masked]>
> Cc: [email-masked]
> Cc: [email-masked]

Acked-by: Li Zefan <[email-masked]>

> ---
>  include/linux/cgroup.h        |  4 ++++
>  include/linux/cgroup_subsys.h | 24 ++++++++++++------------
>  kernel/cgroup.c               |  1 +
>  3 files changed, 17 insertions(+), 12 deletions(-)