From: "Lorenzo Stoakes (ARM)" <ljs@kernel.org>
To: Shivank Garg <shivankg@amd.com>
Cc: David Hildenbrand <david@kernel.org>,
"Liam R. Howlett" <liam@infradead.org>,
Vlastimil Babka <vbabka@kernel.org>,
Mike Rapoport <rppt@kernel.org>,
Suren Baghdasaryan <surenb@google.com>,
Michal Hocko <mhocko@suse.com>,
Andrew Morton <akpm@linux-foundation.org>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] mm: remove unused clear_flush_young_ptes_notify()
Date: Tue, 10 Nov 2026 12:48:30 +0000 [thread overview]
Message-ID: <avMSmWU4JYReUAB9@gremlin> (raw)
In-Reply-To: <20261109-clear-flush-young-notify-v1-1-b37fbafa2686@amd.com>
On Mon, Nov 09, 2026 at 06:41:55PM +0000, Shivank Garg wrote:
> clear_flush_young_ptes_notify()'s last user was removed in
> commit f3c493d2f42b ("mm/rmap: handle device-exclusive entries in
> folio_referenced_one()"), which open-codes clear_flush_young_ptes()
> and mmu_notifier_clear_flush_young() in folio_referenced_one().
>
> Remove both the CONFIG_MMU_NOTIFIER definition and the
> !CONFIG_MMU_NOTIFIER alias.
>
> Signed-off-by: Shivank Garg <shivankg@amd.com>
LGTM
Reviewed-by: Lorenzo Stoakes (ARM) <ljs@kernel.org>
> ---
> Applies cleanly on mm/core.git/for-next.
> ---
> mm/internal.h | 12 ------------
> 1 file changed, 12 deletions(-)
>
> diff --git a/mm/internal.h b/mm/internal.h
> index b129510b92e2..b1f3a1a59bc0 100644
> --- a/mm/internal.h
> +++ b/mm/internal.h
> @@ -1598,17 +1598,6 @@ static inline void maybe_rmap_unlock_action(struct vm_area_struct *vma,
> }
>
> #ifdef CONFIG_MMU_NOTIFIER
> -static inline bool clear_flush_young_ptes_notify(struct vm_area_struct *vma,
> - unsigned long addr, pte_t *ptep, unsigned int nr)
> -{
> - bool young;
> -
> - young = clear_flush_young_ptes(vma, addr, ptep, nr);
> - young |= mmu_notifier_clear_flush_young(vma->vm_mm, addr,
> - addr + nr * PAGE_SIZE);
> - return young;
> -}
> -
> static inline bool pmdp_clear_flush_young_notify(struct vm_area_struct *vma,
> unsigned long addr, pmd_t *pmdp)
> {
> @@ -1641,7 +1630,6 @@ static inline bool pmdp_test_and_clear_young_notify(struct vm_area_struct *vma,
>
> #else /* CONFIG_MMU_NOTIFIER */
>
> -#define clear_flush_young_ptes_notify clear_flush_young_ptes
> #define pmdp_clear_flush_young_notify pmdp_clear_flush_young
> #define test_and_clear_young_ptes_notify test_and_clear_young_ptes
> #define pmdp_test_and_clear_young_notify pmdp_test_and_clear_young
>
> ---
> base-commit: 2683df3b7eac1d91380194955ad0c9751454610c
> change-id: 20261109-clear-flush-young-notify-dac8b5496ec4
>
> Best regards,
> --
> Shivank Garg <shivankg@amd.com>
>
--
Cheers, Lorenzo
prev parent reply other threads:[~2026-11-10 12:48 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-09 18:41 [PATCH] mm: remove unused clear_flush_young_ptes_notify() Shivank Garg
2026-11-09 20:50 ` David Hildenbrand (Arm)
2026-11-10 1:18 ` SJ Park
2026-11-10 12:32 ` David Hildenbrand
2026-11-10 12:48 ` Lorenzo Stoakes (ARM) [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=avMSmWU4JYReUAB9@gremlin \
--to=ljs@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=david@kernel.org \
--cc=liam@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.com \
--cc=rppt@kernel.org \
--cc=shivankg@amd.com \
--cc=surenb@google.com \
--cc=vbabka@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.