Re: panic: uvm_page_physload: start >= end

[Izumi Tsutsui <tsutsui%ceres.dti.ne.jp@localhost>]

In article <1f9vwh1.102m3fu114cxhtM@[10.0.12.137]>
manu%netbsd.org@localhost wrote:

> > I guess the real problem is that OF returns unaligned regions in
> > powerpc/ofw_machdep.c:mem_region() and pmap does not handle it.
> > My patch is just workaround, not fix.
> 
> Well at least it makes -current kernels bootable on my machine and
> yours. That's better than a workaround...

As noted in the NetBSD goals page, "It doesn't work unless it's right."

pmap_bootstrap() would still pass wrong values to uvm_page_physload(),
and actually quick hacks often put off comming proper fix ;-p

Anyway, I guess the attached patch would be a "right" fix,
but it should be approved by powerpc or vm guys.
---
Izumi Tsutsui
tsutsui%ceres.dti.ne.jp@localhost

Index: pmap.c
===================================================================
RCS file: /cvsroot/syssrc/sys/arch/powerpc/mpc6xx/pmap.c,v
retrieving revision 1.38
diff -u -r1.38 pmap.c
--- pmap.c      2002/03/08 20:48:33     1.38
+++ pmap.c      2002/03/31 02:44:00
@@ -2643,8 +2643,8 @@
        kernelstart = trunc_page(kernelstart);
        kernelend = round_page(kernelend);
        for (mp = avail, i = 0; i < avail_cnt; i++, mp++) {
-               s = trunc_page(mp->start);
-               e = round_page(mp->start + mp->size);
+               s = round_page(mp->start);
+               e = trunc_page(mp->start + mp->size);
 
                DPRINTFN(BOOT,
                    ("pmap_bootstrap: b-avail[%d] start 0x%lx size 0x%lx\n",
@@ -2666,6 +2666,7 @@
                                avail[avail_cnt].size = e - kernelend;
                                avail_cnt++;
                        }
+                       mp->start = s;
                        mp->size = kernelstart - s;
                }
                /*