Received-SPF: pass (google.com: domain of linux-kernel+bounces-6351-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20231220040037.883811-1-kinseyho@google.com>
In-Reply-To: <20231220040037.883811-1-kinseyho@google.com>
From: Yu Zhao <yuzhao@google.com>
Date: Tue, 19 Dec 2023 21:16:06 -0700
Message-ID: <CAOUHufaK3X0d9ovEMkFfb3cugqzcnxxqn=o6O-1oohaYANtB0A@mail.gmail.com>
Subject: Re: [PATCH mm-unstable v3 0/4] mm/mglru: Kconfig cleanup
To: Kinsey Ho <kinseyho@google.com>, Donet Tom <donettom@linux.vnet.ibm.com>
Cc: Andrew Morton <akpm@linux-foundation.org>, linux-mm@kvack.org, 
	linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Dec 19, 2023 at 9:01=E2=80=AFPM Kinsey Ho <kinseyho@google.com> wro=
te:
>
> This series is the result of the following discussion:
> https://lore.kernel.org/47066176-bd93-55dd-c2fa-002299d9e034@linux.ibm.co=
m/
>
> It mainly avoids building the code that walks page tables on CPUs that
> use it, i.e., those don't support hardware accessed bit. Specifically,
> it introduces a new Kconfig to guard some of functions added by
> commit bd74fdaea146 ("mm: multi-gen LRU: support page table walks")
> on CPUs like POWER9, on which the series was tested.
>
>
> Kinsey Ho (4):
>   mm/mglru: add CONFIG_ARCH_HAS_HW_PTE_YOUNG
>   mm/mglru: add CONFIG_LRU_GEN_WALKS_MMU
>   mm/mglru: remove CONFIG_MEMCG
>   mm/mglru: remove CONFIG_TRANSPARENT_HUGEPAGE
>
>  arch/Kconfig                   |   8 +
>  arch/arm64/Kconfig             |   1 +
>  arch/x86/Kconfig               |   1 +
>  arch/x86/include/asm/pgtable.h |   6 -
>  include/linux/memcontrol.h     |   2 +-
>  include/linux/mm_types.h       |  16 +-
>  include/linux/mmzone.h         |  28 +---
>  include/linux/pgtable.h        |   2 +-
>  kernel/fork.c                  |   2 +-
>  mm/Kconfig                     |   4 +
>  mm/vmscan.c                    | 271 ++++++++++++++++++---------------
>  11 files changed, 174 insertions(+), 167 deletions(-)

+Donet Tom <donettom@linux.vnet.ibm.com>
who is also working on this.

Donet, could try this latest version instead? If it works well as the
old one you've been using, can you please provide your Tested-by tag?
Thanks.