2009-09-15 16:50:39

by Lennart Sorensen

[permalink] [raw]
Subject: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on h3800, m68knommu and microblaze.

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 <[email protected]>

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


2009-09-17 07:39:25

by Greg Ungerer

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on h3800, m68knommu and microblaze.

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 <[email protected]>

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 <[email protected]>


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
>
>

--
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: [email protected]
SnapGear Group, McAfee PHONE: +61 7 3435 2888
825 Stanley St, FAX: +61 7 3891 3630
Woolloongabba, QLD, 4102, Australia WEB: http://www.SnapGear.com

2009-09-17 08:56:20

by Michal Simek

[permalink] [raw]
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 <[email protected]>
>
> 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 <[email protected]>

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 <[email protected]>


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: http://www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

2009-09-17 13:23:47

by Yoshinori Sato

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on h3800, m68knommu and microblaze.

At Tue, 15 Sep 2009 12:50:37 -0400,
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 <[email protected]>
>
> 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
>

Acked-by: Yoshinori Sato <[email protected]>

--
Yoshinori Sato
<[email protected]>

2009-09-17 15:21:06

by Lennart Sorensen

[permalink] [raw]
Subject: Re: =?us-ascii?Q?=5Bmi croblaze=2Duclinux=5D=20Re=3A=20=5BPATCH=5D=2 0Actually=20show=20KiB=20rather=20than=20pages=2 0in=20=22Freeing=20initrd

On Thu, Sep 17, 2009 at 10:56:16AM +0200, Michal Simek wrote:
> 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 <[email protected]>
> >
> > 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 <[email protected]>
>
> 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 <[email protected]>

Oh right how silly of me. 3 seperate patches would have been nicer I
guess. The original code is certainly copied from one place to the others
although I have no idea which architecture was the first to get it wrong.

Would everyone prefer I made 3 patches instead and resent them?

--
Len Sorensen

2009-09-17 15:39:14

by Michal Simek

[permalink] [raw]
Subject: Re: [microblaze-uclinux] Re: =?us-ascii?Q?=5Bmicroblaze=2Duclinux=5D=20Re=3A=20=5BPATCH=5D=20Actually=20show=20KiB=20rather=20than=20pages=20in=20=22Freeing=20initrd



[email protected] wrote:
> On Thu, Sep 17, 2009 at 10:56:16AM +0200, Michal Simek wrote:
>> 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 <[email protected]>
>>> 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 <[email protected]>
>> 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 <[email protected]>
>
> Oh right how silly of me. 3 seperate patches would have been nicer I
> guess. The original code is certainly copied from one place to the others
> although I have no idea which architecture was the first to get it wrong.
>
> Would everyone prefer I made 3 patches instead and resent them?

Just do it for microblaze

Thanks,
Michal


>

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

2009-09-17 15:46:21

by Lennart Sorensen

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on h3800, m68knommu and microblaze.

Fix "Freeing initrd memory:" message on microblaze to show kilobytes as
claimed rather than number of pages.

Signed-off-by: Lennart Sorensen <[email protected]>

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

2009-09-17 15:47:05

by Lennart Sorensen

[permalink] [raw]
Subject: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on microblaze.

Fix "Freeing initrd memory:" message on microblaze to show kilobytes as
claimed rather than number of pages.

Signed-off-by: Lennart Sorensen <[email protected]>

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

2009-09-17 15:48:37

by Lennart Sorensen

[permalink] [raw]
Subject: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on h8300.

Fix "Freeing initrd memory:" message on h8300 to show kilobytes as
claimed rather than number of pages.

Signed-off-by: Lennart Sorensen <[email protected]>

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

--
Len Sorensen

2009-09-17 15:49:39

by Lennart Sorensen

[permalink] [raw]
Subject: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on m68knommu.

Fix "Freeing initrd memory:" message m68knommu to show kilobytes as
claimed rather than number of pages.

Signed-off-by: Lennart Sorensen <[email protected]>

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

--
Len Sorensen

2009-09-17 15:50:14

by Lennart Sorensen

[permalink] [raw]
Subject: Re: [microblaze-uclinux] Re: =?us-ascii?Q?=5Bmicroblaze=2Duclinux=5D=20Re =3A=20=5BPATCH=5D=20Actually=20show=20KiB=20r ather=20than=20pages=20in=20=22Freeing=20initrd

On Thu, Sep 17, 2009 at 05:39:10PM +0200, Michal Simek wrote:
>
>
> [email protected] wrote:
> > On Thu, Sep 17, 2009 at 10:56:16AM +0200, Michal Simek wrote:
> >> 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 <[email protected]>
> >>> 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 <[email protected]>
> >> 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 <[email protected]>
> >
> > Oh right how silly of me. 3 seperate patches would have been nicer I
> > guess. The original code is certainly copied from one place to the others
> > although I have no idea which architecture was the first to get it wrong.
> >
> > Would everyone prefer I made 3 patches instead and resent them?
>
> Just do it for microblaze

Well I just sent a seperate one for each. I misspelled h8300 (as h3800)
in the comment before.

--
Len Sorensen

2009-09-18 00:59:11

by Greg Ungerer

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on m68knommu.

Hi Lennart,

Lennart Sorensen wrote:
> Fix "Freeing initrd memory:" message m68knommu to show kilobytes as
> claimed rather than number of pages.
>
> Signed-off-by: Lennart Sorensen <[email protected]>

Looks good. I have applied to the m68knommu git tree.

Regards
Greg


> 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
>

--
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: [email protected]
SnapGear Group, McAfee PHONE: +61 7 3435 2888
825 Stanley St, FAX: +61 7 3891 3630
Woolloongabba, QLD, 4102, Australia WEB: http://www.SnapGear.com

2009-09-21 12:46:17

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on microblaze.

Added to next branch.

There were some problems which I fixed.
1. long first description
2. missing microblaze: at the beginning
3. line longer than 80chars

Thanks,
Michal


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 <[email protected]>
>
> 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
>

2009-09-21 13:32:02

by Lennart Sorensen

[permalink] [raw]
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 <[email protected]>
>>
>> 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

2009-09-21 16:34:33

by Michal Simek

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on microblaze.

Lennart Sorensen wrote:
> 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

I want to have "microblaze:" at the beginning of email/commit message.
It is much clear for mailing list, Linus and me.

Thanks,
Michal

>
> 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 <[email protected]>
>>>
>>> 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
>>>
>

2009-09-21 18:02:59

by Lennart Sorensen

[permalink] [raw]
Subject: Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on microblaze.

On Mon, Sep 21, 2009 at 06:34:01PM +0200, Michal Simek wrote:
> I want to have "microblaze:" at the beginning of email/commit message.
> It is much clear for mailing list, Linus and me.

Hmm, OK. That makes some sense. I hadn't ever noticed that being
a trend.

--
Len Sorensen