From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id BFCB7C02181 for ; Mon, 20 Jan 2025 19:17:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 45927280002; Mon, 20 Jan 2025 14:17:02 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 407F9280001; Mon, 20 Jan 2025 14:17:02 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2F6BD280002; Mon, 20 Jan 2025 14:17:02 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 10D53280001 for ; Mon, 20 Jan 2025 14:17:02 -0500 (EST) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id C92B41C65A9 for ; Mon, 20 Jan 2025 19:17:01 +0000 (UTC) X-FDA: 83028787842.01.B6B7291 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf24.hostedemail.com (Postfix) with ESMTP id 2F763180003 for ; Mon, 20 Jan 2025 19:16:59 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=JnOaWgSy; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1737400620; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=JH1x93fKw5JUmlrIpIPlbbrYJRn5GGnMG0UXBTZSMA8=; b=RajX0nj3l6Cp+DKZohKULXR9RALnSFo6daNLe3zpFGwlhQmJc5bkUccDNEDQc6J8lTdO5X Lrax1+CZLOWT3EhEUk8AykAzKyMNaA+0sQnb/HTwSz1g7Lrc3z9C8EZz9Kc1Iy4LaC4ngA E6O0cCOCnMa0oAV7bKXQlwHNVIfbYwg= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1737400620; a=rsa-sha256; cv=none; b=rl3Te4RPY+rLppDy9sTqSA8S7rHXTAnk2cov8OFp/dKD+S9AbPhp/yOlA0RnLPqwWmsRHM F2zDgFiQUqOXTkDnqRgZeAWWY1nfCWazJvfCDaJJatJn0hflesNoIXFMsTZqMKioU6mrqX Rv1Wbv7A8PDH3Zii0Mb/9tsxhXvFNbc= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=JnOaWgSy; spf=pass (imf24.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 9CF635C0579; Mon, 20 Jan 2025 19:16:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9D126C4CEDD; Mon, 20 Jan 2025 19:16:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1737400618; bh=qk4QSnven6ILIEFsUk/00OcnycBav7of9Es7dyhYB3E=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=JnOaWgSy0rWj8/f3bFJig2zoUQ064MCp2Lx+hTrTD7tfQef4QCm66ACWiE6bOoTVt lOq2rPp7Fb7lA4ToSVIQSTZCC5xYqt3dYy6o+oetZe5o1NulCUT1QUIBCQ2GMUHcLW vpYnNPDJTpNHvZFAbo7xsvvVGaCkKpVKQihmmgBqt/Gbz9OoQj4gQ4SyY3wgxeC6ta uLmwZ8fOdldBHFqRduwqZhiVt83RMyOg1RhbDe0N099Ljaz56JU34Myen59ewExiDE L+7hPVaqNMf3Qf2DdDNj8lyxJH2v69o9dcnyT6j+h5NjHGOtPpoiO1OW7g2RTGZ3Mv lP2dMwdVqO+Dw== From: SeongJae Park To: David Hildenbrand Cc: SeongJae Park , Usama Arif , akpm@linux-foundation.org, damon@lists.linux.dev, linux-mm@kvack.org Subject: Re: [PATCH v3 2/2] mm/damon: introduce DAMOS filter type hugepage Date: Mon, 20 Jan 2025 11:16:54 -0800 Message-Id: <20250120191654.38420-1-sj@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <9257fbb9-d26d-4028-896f-fb8cfad3ddad@redhat.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Stat-Signature: iaqf1eyfazzmj7t7ercx94umt9bi7hgn X-Rspam-User: X-Rspamd-Queue-Id: 2F763180003 X-Rspamd-Server: rspam03 X-HE-Tag: 1737400619-549723 X-HE-Meta: U2FsdGVkX1+LxvAXToc31uLtpuqEvutJIGhE1ZDyHR0UHjWnb5Xa7RzJML6ZdfRwbwZGv4xEz6sQ7cREUOZtMl3Ry64jr2pbWye3WrHLPyljWFJ5araSzWfpo8iE/D9ZnRxgtxTY1IVILFitLZj77QhmHsL5Wnh2+g8UjnJzA6+km3OAdnXvX5Daem006R31mWzBWJX73BycyafQVSVTpgxX1e0zdKQkd/iVfMlq32HgBQdpwHTZ57dWKpjDyk+87Do5EFEBVijvw2TJTPF54R9liVl6+iNvS0htSeUTrgM72SHwzMZFiL/2xZr4qhL5+XSApl7Vtx3kum7Gfn9jdQTM+zc6eRets7fucIK+oiuKgbvI7nUXXgac2k8fQMssUxXfX6rSsGXJhSkJRBc4oF9bDVObYJZcG3QpTQ7RtMeLaWX2hol2gOhnonIZmbpdx6TRPE6yEEkGYXBC5NT4b4wCH4kaXvEixaD8WMl0471SX1iXN7Xrr+AJAzE6qZZ68FyybDLcam56NZlGLT4JjxA6lmkyzeDZbesXlOi83b8eQqLyFBnnY3ry75V8x8cOgd9x5qCC3RECpsSqyLtsZlGUbWEwLxMD9wPOQU7+VyCBPckHs5gwvIWSH9jG3ncyBtwvepjYsdv/7VUVBCAWCvbpuxuhMGNDr+gbPmqrNPLt/QTwOjBNPeEueTHch9UfpYB6uI/9bZ5EmALzR4anI6e5ffGJNp9IoFvMS1lCRs08J7KEVdBhQmUhbMLcKRiMAICzeZ+WIPoznmeBVO+sM8K0us+DdHvrPVlRXCRJjSo6Id3B2BR3YTJ+a/kFjiOoNjlfYjmJtJdBOqzTDyLqBWwIxIq5unhKSNv9LrpQOrkyE4pUkOu95EwEOue7ahmMD3iGHmTmRSsYqSD7kCK7AKfPycoWaBVj7xVFiPH178Q8vOAYA+XAfHNZSBOT6nQ3/R90QSlswAmBMqXYzRv jFoaVjdT 0d+5TeDvLHb4IKMkjae/4lBRNYvb7G1iruV5mI6ugGh6HvH8ldTJmGae22jItsw4fCcRbW1fgxptddak9CBgtjZh07DWLqntXc9L3TB8Cp/ElQFHWMwuiSzUfAFZMYFhqTP1YzTy2FeqcyPMRo5RO1pjqKWcS9zuffho9fK+2InDI5YWhjMV8EHQ22Gfdlv4WGNaGu4W4J+LYIaTe8hVvdi2mBS0LdasYnXDK3wO4KZ/nOLWeFRHQVOb8FFMrn8ncn71a X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Mon, 20 Jan 2025 19:57:10 +0100 David Hildenbrand wrote: > On 20.01.25 19:19, Usama Arif wrote: > > This is to gather statistics to check if memory regions of specific > > access tempratures are backed by hugepages. This includes both THPs > > and hugetlbfs. > > This filter can help to observe and prove the effectivenes of > > different schemes for shrinking/collapsing hugepages. > > > > Signed-off-by: Usama Arif > > --- > > v2 -> v3: > > - expose hugepage via sysfs even if the kernel is > > built without hugepage support. DAMON will just > > just return 0. (SJ Park) > > > > v1 -> v2: > > - Wrap DAMOS_FILTER_TYPE_HUGEPAGE case with > > CONFIG_PGTABLE_HAS_HUGE_LEAVES (SJ Park) > > --- > > include/linux/damon.h | 2 ++ > > mm/damon/paddr.c | 5 +++++ > > mm/damon/sysfs-schemes.c | 1 + > > 3 files changed, 8 insertions(+) > > > > diff --git a/include/linux/damon.h b/include/linux/damon.h > > index af525252b853..1d94d7d88b36 100644 > > --- a/include/linux/damon.h > > +++ b/include/linux/damon.h > > @@ -326,6 +326,7 @@ struct damos_stat { > > * @DAMOS_FILTER_TYPE_ANON: Anonymous pages. > > * @DAMOS_FILTER_TYPE_MEMCG: Specific memcg's pages. > > * @DAMOS_FILTER_TYPE_YOUNG: Recently accessed pages. > > + * @DAMOS_FILTER_TYPE_HUGEPAGE: Page is part of a hugepage. > > * @DAMOS_FILTER_TYPE_ADDR: Address range. > > * @DAMOS_FILTER_TYPE_TARGET: Data Access Monitoring target. > > * @NR_DAMOS_FILTER_TYPES: Number of filter types. > > @@ -345,6 +346,7 @@ enum damos_filter_type { > > DAMOS_FILTER_TYPE_ANON, > > DAMOS_FILTER_TYPE_MEMCG, > > DAMOS_FILTER_TYPE_YOUNG, > > + DAMOS_FILTER_TYPE_HUGEPAGE, > > DAMOS_FILTER_TYPE_ADDR, > > DAMOS_FILTER_TYPE_TARGET, > > NR_DAMOS_FILTER_TYPES, > > diff --git a/mm/damon/paddr.c b/mm/damon/paddr.c > > index c0ccf4fade24..224308140441 100644 > > --- a/mm/damon/paddr.c > > +++ b/mm/damon/paddr.c > > @@ -222,6 +222,11 @@ static bool damos_pa_filter_match(struct damos_filter *filter, > > if (matched) > > damon_folio_mkold(folio); > > break; > > +#if defined(CONFIG_PGTABLE_HAS_HUGE_LEAVES) > > + case DAMOS_FILTER_TYPE_HUGEPAGE: > > + matched = folio_size(folio) == HPAGE_PMD_SIZE; > > > Can we directly embed in the name and the comments/docs that we are only > talking about PMD size (both, THP and hugetlb)? > > DAMOS_FILTER_TYPE_PMD_HUGEPAGE or sth. like that. Nice suggestion, thank you! And we might later add more filter types for different size huge pages. What about extending this to handle more general case, though? That is, we can let the filter receives a range of the folio size to match, like DAMOS_FILTER_TYPE_ADDR does. Then, the filter could be used for any size of interest. I think such extension is not a strong blocker for this patch. We can do the extension later without breaking users by having the PAGE_PMD_SIZE as the default value of the address range input. I have no strong opinion about this, though. What do you think? Thanks, SJ [...]