[prev in list] [next in list] [prev in thread] [next in thread] 

List:       uclinux-dev
Subject:    [uClinux-dev] Re: free_initrd_mem() corrups mm state on m68knommu.
From:       lsorense () csclub ! uwaterloo ! ca (Lennart Sorensen)
Date:       2009-09-18 16:28:36
Message-ID: 20090918162836.GI23702 () csclub ! uwaterloo ! ca
[Download RAW message or body]

On Fri, Sep 18, 2009 at 09:42:59AM +1000, Greg Ungerer wrote:
> Hi Lennart,
>
> Lennart Sorensen wrote:
>> On Thu, Sep 17, 2009 at 05:34:37PM +1000, Greg Ungerer wrote:
>>> Hi Lennart,
>>>
>>> Lennart Sorensen wrote:
>>>> On Tue, Sep 15, 2009 at 05:49:59PM -0400, Lennart Sorensen wrote:
> ...
>>>> OK, seems the issue was that setup.c wasn't flagging the initrd memory
>>>> range as 'reserve_bootmem' which caused extra memory to be forced into
>>>> the mm state when it was later freed.  I guess one has to be very careful
>>>> with what patches one finds lying around the internet (in this case the
>>>> uboot bootargs patch for m68knommu that we found somewhere).
>>>>
>>>> Seems to work great now, and I finally made sense of how the bootmem
>>>> map is passed and what is going where.  It all makes sense now.
>>> Can you send a patch for it?
>>> I can push it to mainline of it looks reasonable.
>>
>> Well it's a patch ontop of another patch that isn't in mainline.
>> The other patch is one that adds support for uboot passing kernel command
>> line arguments.  Want that one too?  We find it to be rather useful.
>
> Sure, send them both.

I just sent it.

-- 
Len Sorensen
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev
[prev in list] [next in list] [prev in thread] [next in thread]