linux-kernel - Re: [PATCH 0/2] mm: tweaks for improving use of vmap

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <d9f91dc9-958c-e3ec-3ca4-ecb0ddfd58f3@gmail.com>
Date:   Wed, 2 May 2018 04:05:14 +0400
From:   Igor Stoppa <igor.stoppa@...il.com>
To:     Andrew Morton <akpm@...ux-foundation.org>
Cc:     willy@...radead.org, mhocko@...nel.org, linux-mm@...ck.org,
        linux-kernel@...r.kernel.org, igor.stoppa@...wei.com
Subject: Re: [PATCH 0/2] mm: tweaks for improving use of vmap_area


On 01/05/18 03:15, Andrew Morton wrote:
> On Fri, 27 Apr 2018 03:42:41 +0400 Igor Stoppa <igor.stoppa@...il.com> wrote:
> 
>> These two patches were written in preparation for the creation of
>> protectable memory, however their use is not limited to pmalloc and can
>> improve the use of virtually contiguous memory.
>>
>> The first provides a faster path from struct page to the vm_struct that
>> tracks it.
>>
>> The second patch renames a single linked list field inside of vmap_area.
>> The list is currently used only for disposing of the data structure, once
>> it is not in use anymore.
>> Which means that it cold be used for other purposes while it's not queued
>> for destruction.
> 
> The patches look benign to me (feel free to add my ack),

thank you

> but I'm not seeing a reason to apply them at this time?

I thought they might come useful to others playing with vmap_areas, I'll 
resubmit them anyway with the protected memory set.

But I was also hoping to get some more review, especially for the 
second, which had not received any definitive ACK/NACK, till now.

So, I'm also ok if they can be merged once the others are ACK'ed.

--
igor