Re: [PATCH 02/10] cgroup: introduce cgroup_subsys->legacy_name

From: Tejun Heo
Date: Tue Jul 28 2015 - 11:23:20 EST


On Tue, Jul 28, 2015 at 02:39:36PM +0800, Zefan Li wrote:
> > @@ -1448,7 +1451,8 @@ static int parse_cgroupfs_options(char *data, struct cgroup_sb_opts *opts)
> > }
> >
> > for_each_subsys(ss, i) {
> > - if (strcmp(token, ss->name))
> > + if (strcmp(token, ss->name) &&
> > + strcmp(token, ss->legacy_name))
> > continue;
>
> As mounting with specified subsystems is only allowed in legacy hierarchy,
> I think we should allow using leagcy name only?

Yeah, good point. Will update the patch.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/