Re: [PATCH v2 1/5] proc/task_mmu: remove unnecessary helpers

From: David Hildenbrand (Arm)

Date: Wed Sep 09 2026 - 14:44:22 EST


On 9/9/26 19:13, Suren Baghdasaryan wrote:
> On Wed, Sep 9, 2026 at 10:06 AM David Hildenbrand (Arm)
> <david@xxxxxxxxxx> wrote:
>>
>> On 9/7/26 08:39, Suren Baghdasaryan wrote:
>>> When per-vma locks were behind a config option, a number of helper
>>> functions were needed to simplify the locking code. Now that these
>>> locks are universally available, we can do a little cleanup.
>>> Remove lock_vma_range(), unlock_vma_range(), query_vma_setup(),
>>> query_vma_teardown() helpers.
>>>
>>> No functional change intended.
>>>
>>> Signed-off-by: Suren Baghdasaryan <surenb@xxxxxxxxxx>
>>> ---
>>
>> As Usama says, this could get picked up early independent of the rest (maybe
>> even as part of the other patch set)
>
> I can post the 3 cleanup patches separately from the one that switches
> to using per-VMA locks if that's preferable. One downside: I'll have
> to wait until the cleanups are picked up before posting the last
> patch, as it depends on these cleanups. Please let me know if I should
> split them anyway; otherwise, will keep v3 as a single patchset.

Let's keep it as is. We can reconsider if we realize that the other patches get
stuck.

--
Cheers,

David