Re: [PATCH v3, part4 26/39] mm/openrisc: prepare for removing num_physpagesand simplify mem_init()

[Jiang Liu]

Hi Jonas,
	Thanks for testing.
	Regards!
	Gerry
On 04/02/2013 01:31 PM, Jonas Bonn wrote:
> On 03/26/2013 04:54 PM, Jiang Liu wrote:
>> Prepare for removing num_physpages and simplify mem_init().
>>
>> Signed-off-by: Jiang Liu <jiang.liu@xxxxxxxxxx>
>> Cc: Jonas Bonn <jonas@xxxxxxxxxxxx>
>> Cc: David Howells <dhowells@xxxxxxxxxx>
>> Cc: Arnd Bergmann <arnd@xxxxxxxx>
>> Cc: linux@xxxxxxxxxxxxxxxxxx
>> Cc: linux-kernel@xxxxxxxxxxxxxxx
> 
> Tested and works fine on OpenRISC.
> 
> Acked-by: Jonas Bonn <jonas@xxxxxxxxxxxx>
> 
> /Jonas
> 
>> ---
>> Hi all,
>>     Sorry for my mistake that my previous patch series has been screwed up.
>> So I regenerate a third version and also set up a git tree at:
>>     git://github.com/jiangliu/linux.git mem_init
>>     Any help to review and test are welcomed!
>>
>>     Regards!
>>     Gerry
>> ---
>>   arch/openrisc/mm/init.c |   44 ++++----------------------------------------
>>   1 file changed, 4 insertions(+), 40 deletions(-)
>>
>> diff --git a/arch/openrisc/mm/init.c b/arch/openrisc/mm/init.c
>> index 71d6b40..f3c8f47 100644
>> --- a/arch/openrisc/mm/init.c
>> +++ b/arch/openrisc/mm/init.c
>> @@ -191,56 +191,20 @@ void __init paging_init(void)
>>     /* References to section boundaries */
>>   -static int __init free_pages_init(void)
>> -{
>> -    int reservedpages, pfn;
>> -
>> -    /* this will put all low memory onto the freelists */
>> -    free_all_bootmem();
>> -
>> -    reservedpages = 0;
>> -    for (pfn = 0; pfn < max_low_pfn; pfn++) {
>> -        /*
>> -         * Only count reserved RAM pages
>> -         */
>> -        if (PageReserved(mem_map + pfn))
>> -            reservedpages++;
>> -    }
>> -
>> -    return reservedpages;
>> -}
>> -
>> -static void __init set_max_mapnr_init(void)
>> -{
>> -    max_mapnr = num_physpages = max_low_pfn;
>> -}
>> -
>>   void __init mem_init(void)
>>   {
>> -    int codesize, reservedpages, datasize, initsize;
>> -
>>       BUG_ON(!mem_map);
>>   -    set_max_mapnr_init();
>> -
>> +    max_mapnr = max_low_pfn;
>>       high_memory = (void *)__va(max_low_pfn * PAGE_SIZE);
>>         /* clear the zero-page */
>>       memset((void *)empty_zero_page, 0, PAGE_SIZE);
>>   -    reservedpages = free_pages_init();
>> -
>> -    codesize = (unsigned long)&_etext - (unsigned long)&_stext;
>> -    datasize = (unsigned long)&_edata - (unsigned long)&_etext;
>> -    initsize = (unsigned long)&__init_end - (unsigned long)&__init_begin;
>> +    /* this will put all low memory onto the freelists */
>> +    free_all_bootmem();
>>   -    printk(KERN_INFO
>> -           "Memory: %luk/%luk available (%dk kernel code, %dk reserved, %dk data, %dk init, %ldk highmem)\n",
>> -           (unsigned long)nr_free_pages() << (PAGE_SHIFT - 10),
>> -           max_mapnr << (PAGE_SHIFT - 10), codesize >> 10,
>> -           reservedpages << (PAGE_SHIFT - 10), datasize >> 10,
>> -           initsize >> 10, (unsigned long)(0 << (PAGE_SHIFT - 10))
>> -        );
>> +    mem_init_print_info(NULL);
>>         printk("mem_init_done ...........................................\n");
>>       mem_init_done = 1;
> 
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/