On 2014/4/22 13:31, 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. >
The initialization is redundant, and your reason is not reasonable. > Signed-off-by: Jianyu Zhan <nasa4...@gmail.com> nack > --- > net/core/netclassid_cgroup.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/net/core/netclassid_cgroup.c b/net/core/netclassid_cgroup.c > index 22931e1..1b07dca 100644 > --- a/net/core/netclassid_cgroup.c > +++ b/net/core/netclassid_cgroup.c > @@ -108,4 +108,5 @@ struct cgroup_subsys net_cls_cgrp_subsys = { > .css_free = cgrp_css_free, > .attach = cgrp_attach, > .base_cftypes = ss_files, > + .early_init = 0, > }; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/