Re: =?us-ascii?Q?=5Bmicroblaze=2Duclinux=5D=20Re=3A=20=5BPATCH=5D=20Actually=20show=20KiB=20rather=20than=20pages=20in=20=22Freeing=20initrd

[Michal Simek]

Greg Ungerer wrote:
> Hi Lennart,
> 
> Lennart Sorensen wrote:
>> Fix "Freeing initrd memory:" message on h3800, m68knommu and microblaze
>> to show kilobytes as claimed rather than number of pages.
>>
>> Signed-off-by: Lennart Sorensen <lsorense@xxxxxxxxxxxxxxxxxxx>
> 
> Looks ok to me. I can take and push the m68knommu part separately.
> Or if Andrew wants to take it all as one, here is an:
> 
> Acked-by: Greg Ungerer <gerg@xxxxxxxxxxx>

As Greg wrote. It will be the best to push it separately. IMHO it
will be faster. It is up to you. If you want to add it to microblaze
branch please send only patch for microblaze.

If not.

Acked-by: Michal Simek <monstr@xxxxxxxxx>


Thanks,
Michal



> 
> 
> Regards
> Greg
> 
> 
>> diff --git a/arch/h8300/mm/init.c b/arch/h8300/mm/init.c
>> index 9942f24..3342f49 100644
>> --- a/arch/h8300/mm/init.c
>> +++ b/arch/h8300/mm/init.c
>> @@ -170,7 +170,7 @@ void free_initrd_mem(unsigned long start, unsigned
>> long end)
>>          totalram_pages++;
>>          pages++;
>>      }
>> -    printk ("Freeing initrd memory: %dk freed\n", pages);
>> +    printk ("Freeing initrd memory: %dk freed\n", pages * (PAGE_SIZE
>> / 1024));
>>  }
>>  #endif
>>  
>> diff --git a/arch/m68knommu/mm/init.c b/arch/m68knommu/mm/init.c
>> index b1703c6..f3236d0 100644
>> --- a/arch/m68knommu/mm/init.c
>> +++ b/arch/m68knommu/mm/init.c
>> @@ -162,7 +162,7 @@ void free_initrd_mem(unsigned long start, unsigned
>> long end)
>>          totalram_pages++;
>>          pages++;
>>      }
>> -    printk (KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages);
>> +    printk (KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages *
>> (PAGE_SIZE / 1024));
>>  }
>>  #endif
>>  
>> diff --git a/arch/microblaze/mm/init.c b/arch/microblaze/mm/init.c
>> index f207f1a..ed5f96b 100644
>> --- a/arch/microblaze/mm/init.c
>> +++ b/arch/microblaze/mm/init.c
>> @@ -180,7 +180,7 @@ void free_initrd_mem(unsigned long start, unsigned
>> long end)
>>          totalram_pages++;
>>          pages++;
>>      }
>> -    printk(KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages);
>> +    printk(KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages *
>> (PAGE_SIZE / 1024));
>>  }
>>  #endif
>>  
>>
> 

-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
--
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/