On 2014/4/22 13:27, Jianyu Zhan wrote: > For a cgroup subsystem who should init early, then it should carefully > take care of the implementation of css_alloc, because it will be called > before mm_init() setup the world. > > Luckily we don't, and we better explicitly assign the early_init field > to 0, for document reason. >
If you think this is the right thing to do, you can apply the same reason to the initialization of other structures in the whole kernel tree. > Signed-off-by: Jianyu Zhan <[email protected]> nack > --- > kernel/cgroup.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/kernel/cgroup.c b/kernel/cgroup.c > index 559f822..f23cb67 100644 > --- a/kernel/cgroup.c > +++ b/kernel/cgroup.c > @@ -5325,5 +5325,6 @@ struct cgroup_subsys debug_cgrp_subsys = { > .css_alloc = debug_css_alloc, > .css_free = debug_css_free, > .base_cftypes = debug_files, > + .early_init = 0, > }; > #endif /* CONFIG_CGROUP_DEBUG */ > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
