linux-kernel - Re: [PATCH v9 1/8] hugetlb_cgroup: Add hugetlb

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4d11fb25-5036-6c78-5328-10a0c14e8edc@oracle.com>
Date:   Wed, 18 Dec 2019 17:37:37 -0800
From:   Mike Kravetz <mike.kravetz@...cle.com>
To:     Andrew Morton <akpm@...ux-foundation.org>,
        Mina Almasry <almasrymina@...gle.com>
Cc:     shuah@...nel.org, rientjes@...gle.com, shakeelb@...gle.com,
        gthelen@...gle.com, linux-kernel@...r.kernel.org,
        linux-mm@...ck.org, linux-kselftest@...r.kernel.org,
        cgroups@...r.kernel.org, aneesh.kumar@...ux.vnet.ibm.com,
        mkoutny@...e.com, Hillf Danton <hdanton@...a.com>,
        Giuseppe Scrivano <gscrivan@...hat.com>,
        Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH v9 1/8] hugetlb_cgroup: Add hugetlb_cgroup reservation
 counter

On 12/18/19 5:12 PM, Andrew Morton wrote:
> On Tue, 17 Dec 2019 15:16:08 -0800 Mina Almasry <almasrymina@...gle.com> wrote:
<snip>
>> This proposal is implemented in this patch series, with tests to verify
>> functionality and show the usage. We also added cgroup-v2 support to
>> hugetlb_cgroup so that the new use cases can be extended to v2.
> 
> This would make
> http://lkml.kernel.org/r/20191216193831.540953-1-gscrivan@redhat.com
> obsolete?

I haven't started looking at this series yet.  However, since Mina was
involved in the discussion of that patch (hugetlb controller for cgroups v2)
my assumption is that this patch would simply build on that v2 support?
Seems like the above patch would be a prereq for this series.

Mina, are those assumptions correct and perhaps this is an old/obsolete
comment?  Does this series apply 'on top' of the above patch?  That patch
is already in Andrew's tree.
-- 
Mike Kravetz