On Wed 29-09-21 18:19:26, Baolin Wang wrote: > Hi, > > Now in the hugetlb cgroup, charges associated with a task aren't moved > to the new hugetlb cgroup at task migration, which is odd for hugetlb > cgroup usage. Could you elaborate some more about the usecase and/or problems you see with the existing semantic? > This patch set adds hugetlb cgroup charge moving when > migrate tasks among cgroups, which are based on the memcg charge moving. Memcg charge moving has shown some problems over time and hence this is not part of cgroup v2 interface anymore. Even for cgroup v1 this has been an opt-in. I do not see anything like that in this patch series. Why should all existing workloads follow a different semantic during task migration now? > Please help to review. Thanks. > > Baolin Wang (2): > hugetlb_cgroup: Add interfaces to move hugetlb charge at task > migration > hugetlb_cgroup: Add post_attach interface for tasks migration > > mm/hugetlb_cgroup.c | 230 ++++++++++++++++++++++++++++++++++++++++++++++++++++ > 1 file changed, 230 insertions(+) > > -- > 1.8.3.1 -- Michal Hocko SUSE Labs
