Commit 097d43d8 authored by Daeseok Youn's avatar Daeseok Youn Committed by Mike Rapoport
Browse files

mm: memblock: remove return value of memblock_free_all() 



No one checks the return value of memblock_free_all().
Make the return value void.

memblock_free_all() is used on mem_init() for each
architecture, and the total count of freed pages will be added
to _totalram_pages variable by calling totalram_pages_add().

so do not need to return total count of freed pages.

Signed-off-by: default avatarDaeseok Youn <daeseok.youn@gmail.com>
Reviewed-by: default avatarDavid Hildenbrand <david@redhat.com>
Signed-off-by: default avatarMike Rapoport <rppt@linux.ibm.com>
parent 7c53f6b6

include/linux/memblock.h

+1 −1
Original line number Diff line number Diff line
@@ -117,7 +117,7 @@ int memblock_mark_mirror(phys_addr_t base, phys_addr_t size); 
int memblock_mark_nomap(phys_addr_t base, phys_addr_t size);

int memblock_clear_nomap(phys_addr_t base, phys_addr_t size);



unsigned long memblock_free_all(void);

void memblock_free_all(void);

void reset_node_managed_pages(pg_data_t *pgdat);

void reset_all_zones_managed_pages(void);

mm/memblock.c

+1 −5
Original line number Diff line number Diff line
@@ -2087,10 +2087,8 @@ void __init reset_all_zones_managed_pages(void) 


/**

 * memblock_free_all - release free pages to the buddy allocator

 *

 * Return: the number of pages actually released.

 */

unsigned long __init memblock_free_all(void)

void __init memblock_free_all(void)

{

	unsigned long pages;
@@ -2099,8 +2097,6 @@ unsigned long __init memblock_free_all(void) 


	pages = free_low_memory_core_early();

	totalram_pages_add(pages);



	return pages;

}



#if defined(CONFIG_DEBUG_FS) && defined(CONFIG_ARCH_KEEP_MEMBLOCK)