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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 17 Sep 2009 10:56:16 +0200
From:	Michal Simek <monstr@...str.eu>
To:	microblaze-uclinux@...e.uq.edu.au
CC:	Lennart Sorensen <lsorense@...lub.uwaterloo.ca>,
	linux-kernel@...r.kernel.org, Greg Ungerer <gerg@...inux.org>,
	Yoshinori Sato <ysato@...rs.sourceforge.jp>
Subject: Re: =?us-ascii?Q?=5Bmicroblaze=2Duclinux=5D=20Re=3A=20=5BPATCH=5D=20Actually=20show=20KiB=20rather=20than=20pages=20in=20=22Freeing=20initrd



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@...lub.uwaterloo.ca>
> 
> 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@...inux.org>

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@...str.eu>


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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/