Re: [PATCH v3 2/2] mm/memcg: Consolidate mem_cgroup_resize_[memsw]_limit() functions.

From: Andrew Morton
Date: Tue Jan 09 2018 - 18:26:30 EST


On Tue, 9 Jan 2018 20:26:33 +0300 Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx> wrote:

> On 01/09/2018 08:10 PM, Shakeel Butt wrote:
> > On Tue, Jan 9, 2018 at 8:58 AM, Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx> wrote:
> >> mem_cgroup_resize_limit() and mem_cgroup_resize_memsw_limit() are almost
> >> identical functions. Instead of having two of them, we could pass an
> >> additional argument to mem_cgroup_resize_limit() and by using it,
> >> consolidate all the code in a single function.
> >>
> >> Signed-off-by: Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx>
> >
> > I think this is already proposed and Acked.
> >
> > https://patchwork.kernel.org/patch/10150719/
> >
>
> Indeed. I'll rebase 1/2 patch on top, if it will be applied first.

Yes please.