Re: [PATCH 3/3] memblock: cleanup memblock_free interface

[Shahab Vahedi <Shahab.Vahedi@xxxxxxxxxxxx>]

On 9/23/21 9:43 AM, Mike Rapoport wrote:
> From: Mike Rapoport <rppt@xxxxxxxxxxxxx>
> 
> For ages memblock_free() interface dealt with physical addresses even
> despite the existence of memblock_alloc_xx() functions that return a
> virtual pointer.
> 
> Introduce memblock_phys_free() for freeing physical ranges and repurpose
> memblock_free() to free virtual pointers to make the following pairing
> abundantly clear:
> 
>       int memblock_phys_free(phys_addr_t base, phys_addr_t size);
>       phys_addr_t memblock_phys_alloc(phys_addr_t base, phys_addr_t size);
> 
>       void *memblock_alloc(phys_addr_t size, phys_addr_t align);
>       void memblock_free(void *ptr, size_t size);
> 
> Replace intermediate memblock_free_ptr() with memblock_free() and drop
> unnecessary aliases memblock_free_early() and memblock_free_early_nid().
> 
> Suggested-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>

arch/arc part: Reviewed-by: Shahab Vahedi <shahab@xxxxxxxxxxxx>


Thanks,
Shahab