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]
Message-ID: <20090921133203.GZ23700@csclub.uwaterloo.ca>
Date:	Mon, 21 Sep 2009 09:32:03 -0400
From:	lsorense@...lub.uwaterloo.ca (Lennart Sorensen)
To:	michal.simek@...alogix.com
Cc:	linux-kernel@...r.kernel.org, microblaze-uclinux@...e.uq.edu.au
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd
	memory:" message on microblaze.

On Mon, Sep 21, 2009 at 02:39:18PM +0200, Michal Simek wrote:
> Added to next branch.
>
> There were some problems which I fixed.
> 1. long first description
> 2. missing microblaze: at the beginning

Missing what?

> 3. line longer than 80chars

Hmm, I thought for sure I had checkpatch.pl claiming all my lines
were OK.  Odd.  Clearly the one below is longer though.  How strange.

>
> Lennart Sorensen wrote:
>> Fix "Freeing initrd memory:" message on microblaze to show kilobytes as
>> claimed rather than number of pages.
>>
>> Signed-off-by: Lennart Sorensen <lsorense@...lub.uwaterloo.ca>
>>
>> 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
>>  

-- 
Len Sorensen
--
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/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ