> > if (!highram_start) {
> > + if (e - s > BUILD_MAX_HIGHTABLE * 16)
> > + highram_size = BUILD_MAX_HIGHTABLE;
> > u32 newe = ALIGN_DOWN(e - highram_size, MALLOC_MIN_ALIGN);
> > if (newe <= e && newe >= s) {
> > highram_start = newe;
>
> Thanks, but I'm still seeing a corner case where this fails - if
> ALIGN_DOWN() falls below s.
Given that the code requires the memory block being 16x the size of the
zonehigh allocation (i.e. use 16M in case the block is 256M or larger)
I fail to see how any reasonable alignment requirement can make that
fail ...
take care,
Gerd
_______________________________________________
SeaBIOS mailing list -- seabios@seabios.org
To unsubscribe send an email to seabios-le...@seabios.org
