[PATCH 4/4] memory: fix memory calculation
Alexander Aring
alex.aring at googlemail.com
Tue Oct 16 02:54:55 EDT 2012
Hi,
Thanks, I will do this changes.
Move fixes as first and separte the fix calculation patch in fixes for
'request_sdram_region' and 'printouts of iomem at boottime', which is only
necessary for correct eyecandy.
Sry, It's true to get all information from iomem, never used this command :)
I got ideas to improve my other patches... I will try to insert it, too.
I send this mail from my private mail address...
My phytec address doesn't work with the barebox mailinglist.
Regards
Alexander Aring
2012/10/16 Sascha Hauer <s.hauer at pengutronix.de>
> On Mon, Oct 15, 2012 at 03:22:42PM +0200, Alexander Aring wrote:
> > Fix memory calculation.
> >
> > Signed-off-by: Alexander Aring <a.aring at phytec.de>
> > ---
> > common/meminfo.c | 8 ++++----
> > common/memory.c | 4 ++--
> > 2 files changed, 6 insertions(+), 6 deletions(-)
> >
> > diff --git a/common/meminfo.c b/common/meminfo.c
> > index b2b09da..13633a2 100644
> > --- a/common/meminfo.c
> > +++ b/common/meminfo.c
> > @@ -15,9 +15,9 @@ static int display_meminfo(void)
> > #endif
> >
> > debug("barebox code: 0x%p -> 0x%p (size 0x%08lx)\n",
> > - _stext, _etext, text_size);
> > + _stext, _etext - 1, text_size);
> > debug("bss segment: 0x%p -> 0x%p (size 0x%08lx)\n",
> > - __bss_start, __bss_stop, bss_size);
> > + __bss_start, __bss_stop - 1, bss_size);
> > #ifdef DEBUG
> > printf("malloc space: 0x%08lx -> 0x%08lx (size 0x%08lx)\n",
> > mstart, mend, msize);
> > @@ -28,11 +28,11 @@ static int display_meminfo(void)
> > #ifdef CONFIG_ARM
> > #ifdef DEBUG
> > printf("stack space: 0x%08x -> 0x%08x (size 0x%08x)\n",
> > - STACK_BASE, STACK_BASE + STACK_SIZE,
> > + STACK_BASE, STACK_BASE + STACK_SIZE - 1,
> > STACK_SIZE);
> > #else
> > printf("stack space: 0x%08x -> 0x%08x (size %s)\n",
> > - STACK_BASE, STACK_BASE + STACK_SIZE,
> > + STACK_BASE, STACK_BASE + STACK_SIZE - 1,
> > size_human_readable(STACK_SIZE));
>
> Hm, you copied wrong code in an earlier patch in this series which
> you fix afterwards. That's not good, fixes, should always come first.
> This for example makes it possible to apply the fix to the master
> branch and feature additions to -next.
>
> Besides, printing the exact values for the debug case and
> size_human_readable otherwise is unnecessary churn. All the
> information above can be retrieved using the iomem command
> anyway, so removing the code altogether seems more useful.
>
> > #endif
> > #endif
> > diff --git a/common/memory.c b/common/memory.c
> > index 8919ab1..6bf4af6 100644
> > --- a/common/memory.c
> > +++ b/common/memory.c
> > @@ -74,11 +74,11 @@ static int mem_malloc_resource(void)
> > request_sdram_region("barebox",
> > (unsigned long)&_stext,
> > (unsigned long)&_etext -
> > - (unsigned long)&_stext + 1);
> > + (unsigned long)&_stext);
> > request_sdram_region("bss",
> > (unsigned long)&__bss_start,
> > (unsigned long)&__bss_stop -
> > - (unsigned long)&__bss_start + 1);
> > + (unsigned long)&__bss_start);
>
> That's a useful fix. Can we have this as a separate patch?
>
> Sascha
>
>
> --
> Pengutronix e.K. | |
> Industrial Linux Solutions | http://www.pengutronix.de/ |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
> _______________________________________________
> barebox mailing list
> barebox at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/barebox/attachments/20121016/b5da0a0a/attachment.html>
More information about the barebox
mailing list