[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [microblaze-uclinux] [patch] boot memory allocator cleanup 3



Hi John,

At Tue, 13 Apr 2004 12:54:45 +1000,
John Williams wrote:
[...]
> OK, so the problem is that when the bootmnap section is removed, .bss 
> loses its page alignment.  With the old bootmap section there, this was 
> kind of implicit, since we had
> 
> .bootmap ALIGN (4096) {
> 
>   . += 4096
> 
> }
> 
> .bss (align 4) {
> ...
> 
> }
> 
> Now, if I remove the bootmap section, I must give .bss ALIGN(4096) for 
> it not to fail.
> 
> Is there anything that assumes .bss will be page aligned?  Otherwise, 
> it's some kind of off-by-one or align/reounding error, I think.

there are some code that assuming boundary is page aligned. I'll look
into it.

> It's interesting that this is OK on Suzaku, but breaks mbvanilla.

I assume it's just some luck.  the code must be broken.

> I can't really spend any more time on it today - have a think about it 
> and we can discuss further.

I'll look into it in a a few days.  I also can't work on it right
now. At least CVS version is working, we are ok ;p

> The earlier patches (linux-mm1 and 2)  are all in CVS now.

thanks!
--
           yashi
___________________________
microblaze-uclinux mailing list
microblaze-uclinux@itee.uq.edu.au
Project Home Page : http://www.itee.uq.edu.au/~jwilliams/mblaze-uclinux
Mailing List Archive : http://www.itee.uq.edu.au/~listarch/microblaze-uclinux/