Re: [PATCH 47/47] vircgroup: rename controllers to legacy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Thu, Sep 20, 2018 at 10:11 AM, Pavel Hrdina <phrdina@xxxxxxxxxx> 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@xxxxxxxxxx> wrote:
>
> > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> >
>
> 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@xxxxxxxxxx>
 

Pavel

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux