[v6 07/15] mm: defining memblock_virt_alloc_try_nid_raw
Pasha Tatashin
pasha.tatashin at oracle.com
Sat Aug 12 01:58:46 AEST 2017
On 08/11/2017 08:39 AM, Michal Hocko wrote:
> On Mon 07-08-17 16:38:41, Pavel Tatashin wrote:
>> A new variant of memblock_virt_alloc_* allocations:
>> memblock_virt_alloc_try_nid_raw()
>> - Does not zero the allocated memory
>> - Does not panic if request cannot be satisfied
>
> OK, this looks good but I would not introduce memblock_virt_alloc_raw
> here because we do not have any users. Please move that to "mm: optimize
> early system hash allocations" which actually uses the API. It would be
> easier to review it that way.
>
>> Signed-off-by: Pavel Tatashin <pasha.tatashin at oracle.com>
>> Reviewed-by: Steven Sistare <steven.sistare at oracle.com>
>> Reviewed-by: Daniel Jordan <daniel.m.jordan at oracle.com>
>> Reviewed-by: Bob Picco <bob.picco at oracle.com>
>
> other than that
> Acked-by: Michal Hocko <mhocko at suse.com>
Sure, I could do this, but as I understood from earlier Dave Miller's
comments, we should do one logical change at a time. Hence, introduce
API in one patch use it in another. So, this is how I tried to organize
this patch set. Is this assumption incorrect?
More information about the Linuxppc-dev
mailing list