[Bug 205458] 11.0-CURRENT/10-STABLE powerpc64: a PowerMac G5 specific sys/powerpc/ofw/ofw_machdep.c change for reliable PowerMac G5 booting (with lots of RAM)

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=205458

--- Comment #12 from Mark Millard <markmi at dsl-only.net> ---
Comment on attachment 179258
  --> https://bugs.freebsd.org/bugzilla/attachment.cgi?id=179258
Corrected test patch

As for the result of the corrected test patch. . .

It appears that:

static __inline void
ofw_sprg_prepare(void)
. . .
static __inline void
ofw_sprg_restore(void)
. . .

are only defined under an earlier:

#ifdef AIM
extern register_t ofmsr[5];
extern void     *openfirmware_entry;
char            save_trap_init[0x2f00];          /* EXC_LAST */
char            save_trap_of[0x2f00];            /* EXC_LAST */

int             ofwcall(void *);
static int      openfirmware(void *args);
. . . (ofw_sprg_<?>'s defined in here) . . .
#ifndef __powerpc64__
        __asm __volatile("mtsprg0 %0" :: "r"(ofw_sprg0_save));
#endif
}
#endif

(Matching the ifdef AIM if I checked right.)

But of following parts are outside any AIM
definition requirement:

static int
openfirmware_core(void *args)
{
. . .
        ofw_sprg_prepare();
. . .
        ofw_sprg_restore();
. . .
}

So without AIM defined the code looks like
it would not compile. (I have AIM defined
for my context.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.