On Thu, Sep 20, 2018 at 10:11 AM, Pavel Hrdina <phrdina@redhat.com> wrote:

On Thu, Sep 20, 2018 at 08:31:52AM +0200, Fabiano Fidêncio wrote:
> On Tue, Sep 18, 2018 at 5:46 PM, Pavel Hrdina <phrdina@redhat.com> wrote:
>
> > Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
> >
>
> Pavel,
>
> I'm not sure what your next series is doing but why renaming the
> controllers to legacy?

Right, I forget to add some reasoning into commit message. So with the
introduction of cgroup v2 there are new names used with cgroups based
on which version is used:

- legacy: cgroup v1
- unified: cgroup v2
- hybrid: cgroup v1 and cgroup v2

So I can rename it to 'cgroupv1' but I thought that 'legacy' is better
name to use in our code.

legacy is fine as long as this reasoning is into the commit message.

Considering you'll add it:

Reviewed-by: Fabiano Fidêncio <fidencio@redhat.com>

Pavel