Patch: SMP support for MIPS
Neel Natu
neelnatu at gmail.com
Sat Feb 6 05:49:53 UTC 2010
Hi Warner,
Thanks for looking at this.
On Fri, Feb 5, 2010 at 7:01 PM, M. Warner Losh <imp at bsdimp.com> wrote:
> In message: <dffe84831002051827r430c7fb6le3c3506bd6d38a03 at mail.gmail.com>
> Neel Natu <neelnatu at gmail.com> writes:
> : I have put the diff here for review:
> : http://people.freebsd.org/~neel/mips_smp_patch/diff.txt
>
> + /*
> + * XXX
> + * SMP kernels must delay pcpu initialization until after
> + * TLB has been flushed and initialized (see mips_cpu_init())
> + */
> + if (pcpu_find(0) == NULL)
> + mips_pcpu0_init();
> +
>
> The practical upshot of this change is that panic() prior to this
> point will fault. Is there some way to avoid this?
>
That's right. I have this fixed in the updated patch. Please review
the changes to pmap.c and machdep.c.
http://people.freebsd.org/~neel/mips_smp_patch/diff.txt
Briefly, we use an address in KSEG0 for the 'pcpup' until we are ready
to transition to a TLB-based pcpu pointer.
> The rest of these changes look fairly good. I'll likely comment on a
> few nits later that I saw once I have the time to devote to it.
>
Ok.
best
Neel
> Warner
>
More information about the freebsd-mips
mailing list