All of lore.kernel.org
 help / color / mirror / Atom feed
From: "David Hildenbrand (Arm)" <david@kernel.org>
To: Dev Jain <dev.jain@arm.com>,
	akpm@linux-foundation.org, ljs@kernel.org, chrisl@kernel.org,
	kasong@tencent.com, hughd@google.com, liam@infradead.org
Cc: riel@surriel.com, vbabka@kernel.org, harry@kernel.org,
	jannh@google.com, linux-mm@kvack.org,
	linux-kernel@vger.kernel.org, rppt@kernel.org, surenb@google.com,
	mhocko@suse.com, qi.zheng@linux.dev, shakeel.butt@linux.dev,
	baohua@kernel.org, axelrasmussen@google.com, yuanchu@google.com,
	weixugc@google.com, shikemeng@huaweicloud.com, nphamcs@gmail.com,
	bhe@redhat.com, youngjun.park@lge.com,
	baolin.wang@linux.alibaba.com, pfalcato@suse.de,
	ryan.roberts@arm.com, anshuman.khandual@arm.com
Subject: Re: [PATCH v4 04/12] mm/memory: Batch set uffd-wp markers during zapping
Date: Tue, 7 Jul 2026 15:47:57 +0200	[thread overview]
Message-ID: <c03d123f-2cb3-42d6-8127-9aed5b5333ae@kernel.org> (raw)
In-Reply-To: <0654116a-235d-4eb0-9970-414322645aa3@arm.com>

On 6/27/26 11:00, Dev Jain wrote:
> 
> 
> On 23/06/26 3:31 pm, Dev Jain wrote:
>>
>>
>> On 16/06/26 7:13 pm, David Hildenbrand (Arm) wrote:
>>>
>>> Avoid talking about "next patch". Phrase it as "In preparation for batching...".
>>>
>>>
>>> Below you have
>>>
>>>
>>> Could we easily extend set_ptes() to allow for non-present -> non-present with
>>> nr > 1?
>>
>> We could have a set_nonpresent_ptes() in general, but then for some non-present
>> configs like in this patch, we don't need to change the pfn, while in some
>> configs like setting swp ptes in one go (like I do in later patches) we have
>> an underlying pfn. So I don't see an obvious simplification with a common
>> helper for non-present -> non-present for nr > 1.
> 
> I think a "non-present" entry always means a softleaf? In that case, we can
> simply branch on softleaf_has_pfn().

We could, but it might turn the overall function more expensive by throwing in
additional runtime checks that the compiler will not really optimize out. Worth
playing with.

-- 
Cheers,

David

  reply	other threads:[~2026-07-07 13:48 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-05-26  6:36 [PATCH v4 00/12] Optimize anonymous large folio unmapping Dev Jain
2026-05-26  6:36 ` [PATCH v4 01/12] mm/rmap: convert page -> folio for hwpoison checks Dev Jain
2026-06-09 13:44   ` David Hildenbrand (Arm)
2026-06-10  7:02     ` Dev Jain
2026-06-10 12:13       ` David Hildenbrand (Arm)
2026-05-26  6:36 ` [PATCH v4 02/12] mm/rmap: Add try_to_unmap_hugetlb_one Dev Jain
2026-06-09 14:16   ` David Hildenbrand (Arm)
2026-06-18  7:44     ` Lance Yang
2026-06-18  7:55       ` David Hildenbrand (Arm)
2026-06-18  9:09         ` Lance Yang
2026-06-18  9:43           ` David Hildenbrand (Arm)
2026-06-18 10:01             ` Lance Yang
2026-06-22  8:13               ` Dev Jain
2026-06-22  8:14                 ` Lance Yang
2026-06-22  8:14                 ` Dev Jain
2026-06-22  8:17                   ` David Hildenbrand (Arm)
2026-06-24  5:50                     ` Dev Jain
2026-06-24  6:58                       ` Lance Yang
2026-06-23  8:40     ` Dev Jain
2026-06-23 11:49       ` David Hildenbrand (Arm)
2026-05-26  6:36 ` [PATCH v4 03/12] mm/rmap: refactor some code around lazyfree folio unmapping Dev Jain
2026-06-09 14:20   ` David Hildenbrand (Arm)
2026-06-23  8:42     ` Dev Jain
2026-06-29 12:30       ` David Hildenbrand (Arm)
2026-05-26  6:36 ` [PATCH v4 04/12] mm/memory: Batch set uffd-wp markers during zapping Dev Jain
2026-06-16 13:43   ` David Hildenbrand (Arm)
2026-06-23 10:01     ` Dev Jain
2026-06-27  9:00       ` Dev Jain
2026-07-07 13:47         ` David Hildenbrand (Arm) [this message]
2026-05-26  6:36 ` [PATCH v4 05/12] mm/rmap: batch unmap folios belonging to uffd-wp VMAs Dev Jain
2026-05-26  6:36 ` [PATCH v4 06/12] mm/swap: rename subpage->page in folio_dup_swap/folio_put_swap Dev Jain
2026-06-16 13:47   ` David Hildenbrand (Arm)
2026-05-26  6:36 ` [PATCH v4 07/12] mm/swapfile: Add batched version of folio_dup_swap Dev Jain
2026-06-16 13:50   ` David Hildenbrand (Arm)
2026-06-16 13:51   ` David Hildenbrand (Arm)
2026-05-26  6:36 ` [PATCH v4 08/12] mm/swapfile: Add batched version of folio_put_swap Dev Jain
2026-06-16 13:54   ` David Hildenbrand (Arm)
2026-06-23 10:03     ` Dev Jain
2026-05-26  6:36 ` [PATCH v4 09/12] mm/rmap: Add batched version of folio_try_share_anon_rmap_pte Dev Jain
2026-06-16 14:04   ` David Hildenbrand (Arm)
2026-06-23 10:05     ` Dev Jain
2026-05-26  6:36 ` [PATCH v4 10/12] mm/rmap: refactor anon folio unmap in try_to_unmap_one Dev Jain
2026-06-16 14:10   ` David Hildenbrand (Arm)
2026-06-23 11:01     ` Dev Jain
2026-05-26  6:36 ` [PATCH v4 11/12] mm/mprotect: drop 'sub' from page_anon_exclusive_sub_batch Dev Jain
2026-06-16 14:12   ` David Hildenbrand (Arm)
2026-06-23 11:02     ` Dev Jain
2026-05-26  6:36 ` [PATCH v4 12/12] mm/rmap: enable batch unmapping of anonymous folios Dev Jain
2026-05-28 16:50 ` [PATCH v4 00/12] Optimize anonymous large folio unmapping Dev Jain
2026-06-09 13:24 ` David Hildenbrand (Arm)
2026-06-09 14:18 ` David Hildenbrand (Arm)
2026-06-23 11:04   ` Dev Jain

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=c03d123f-2cb3-42d6-8127-9aed5b5333ae@kernel.org \
    --to=david@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=anshuman.khandual@arm.com \
    --cc=axelrasmussen@google.com \
    --cc=baohua@kernel.org \
    --cc=baolin.wang@linux.alibaba.com \
    --cc=bhe@redhat.com \
    --cc=chrisl@kernel.org \
    --cc=dev.jain@arm.com \
    --cc=harry@kernel.org \
    --cc=hughd@google.com \
    --cc=jannh@google.com \
    --cc=kasong@tencent.com \
    --cc=liam@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=ljs@kernel.org \
    --cc=mhocko@suse.com \
    --cc=nphamcs@gmail.com \
    --cc=pfalcato@suse.de \
    --cc=qi.zheng@linux.dev \
    --cc=riel@surriel.com \
    --cc=rppt@kernel.org \
    --cc=ryan.roberts@arm.com \
    --cc=shakeel.butt@linux.dev \
    --cc=shikemeng@huaweicloud.com \
    --cc=surenb@google.com \
    --cc=vbabka@kernel.org \
    --cc=weixugc@google.com \
    --cc=youngjun.park@lge.com \
    --cc=yuanchu@google.com \
    /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.