IP28 boot error under 3.16

IP28 boot error under 3.16

[Joshua Kinard]

Has anyone tried booting IP28 lately?

I tried a cut of 3.16 from git (~08/05/14) and I get the following crash at
the ARCS console:

Exception: <vector=normal>
Status register: 0x34004882<CU1,CU0,FR,IM7,IM4,IPL=???,KX,MODE=KERNEL>
Cause register: 0x10<CE=0,EXC=RADE>
Exception PC: 0xa800000020654004, Exception RA: 0xa800000020654c9c
Read address error exception, bad address: 0xdfbdd600
Local I/O interrupt register 2: 0x0xc8<EISA,SLOT0,SLOT1>
  Saved user regs in hex (&gpda9000000020f01dc8, & regs 0x9000000020f01fc9):
  arg: dfbe0000 ffffffffa8000000 ffffffffa8000000 40000 20690000 20690000
20690000 9
  tmp: 9 38 0 ffffffff9fc60670
  sve: a8000000204f1330 ffffffff9fc572d0 ffffffff9fc572d3
  t8 206e0000 t9 206e0000 at 34004880 v0 dfbd9800 v1 a800000020654fe0 k1
ffffffffbad11bad
  gp a80000002060c000 fp 740b sp a80000002060fdc0 ra a800000020654c9c

PANIC: Unexpected exception

Going by the Exception PC and the RA, I figured out which line looks to be
causing this:

    (gdb) l *(sgihpc_init+0x78)
    0xa800000020654004 is in sgihpc_init (arch/mips/sgi-ip22/ip22-hpc.c:41).
    36              hpc3c1 = (struct hpc3_regs *)
    37                       ioremap(HPC3_CHIP1_BASE, sizeof(struct hpc3_regs));
    38              /* IOC lives in PBUS PIO channel 6 */
    39              sgioc = (struct sgioc_regs *)hpc3c0->pbus_extregs[6];
    40
--> 41              hpc3c0->pbus_piocfg[6][0] |= HPC3_PIOCFG_DS16;
    42              if (ip22_is_fullhouse()) {
    43                      /* Full House comes with INT2 which lives in
PBUS PIO
    44                       * channel 4 */
    45                      sgint = (struct sgint_regs
*)hpc3c0->pbus_extregs[4];

To me, it looks like a pointer isn't getting converted to 64bit address
space correctly (0xdfbdd600 -> ???).  I haven't played with this IP28
machine for a few years, so I forget what the best approach to fixing this is.

Thoughts?

-- 
Joshua Kinard
Gentoo/MIPS
kumba@gentoo.org
4096R/D25D95E3 2011-03-28

"The past tempts us, the present confuses us, the future frightens us.  And
our lives slip away, moment by moment, lost in that vast, terrible in-between."

--Emperor Turhan, Centauri Republic

Re: IP28 boot error under 3.16

[Thomas Bogendoerfer]

On Sun, Aug 17, 2014 at 01:12:19AM -0400, Joshua Kinard wrote:
> 
> Has anyone tried booting IP28 lately?

will try hopefully tonight. I've checked my logs and last boot of
one of my IP28 was with 3.1 *cough*

> To me, it looks like a pointer isn't getting converted to 64bit address
> space correctly (0xdfbdd600 -> ???).  I haven't played with this IP28
> machine for a few years, so I forget what the best approach to fixing this is.
> 
> Thoughts?

looks like ioremap is broken, that's where the hpc pointer is coming from.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

Re: IP28 boot error under 3.16

[Thomas Bogendoerfer]

On Sun, Aug 17, 2014 at 01:12:19AM -0400, Joshua Kinard wrote:
> Thoughts?

something like

#define IO_BASE                 _AC(0x9000000000000000, UL)

in mach-ip28/spaces.h should do the trick. UNCAC_BASE also looks
strange, no idea why there that way.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]

Re: IP28 boot error under 3.16

[Joshua Kinard]

On 08/18/2014 11:27, Thomas Bogendoerfer wrote:
> On Sun, Aug 17, 2014 at 01:12:19AM -0400, Joshua Kinard wrote:
>> Thoughts?
> 
> something like
> 
> #define IO_BASE                 _AC(0x9000000000000000, UL)
> 
> in mach-ip28/spaces.h should do the trick. UNCAC_BASE also looks
> strange, no idea why there that way.

That appears to do it, thanks!  Still getting a panic somewhere before
Impact comes up, as the LED starts flashing.  However, my serial cables are
missing in a closet somewhere, so I can't debug it further right now.  I
ordered some more cables so I can take another look later in the week (or go
tear my closet apart looking for the other ones...).

Digging on kernel.org, it appears commit ed3ce16c added the definitions of
UNCAC_BASE and IO_BASE as part of a larger fix.  Before that, these
definitions didn't exist in IP28's spaces.h.

I'll send a patch for this fix shortly.

-- 
Joshua Kinard
Gentoo/MIPS
kumba@gentoo.org
4096R/D25D95E3 2011-03-28

"The past tempts us, the present confuses us, the future frightens us.  And
our lives slip away, moment by moment, lost in that vast, terrible in-between."

--Emperor Turhan, Centauri Republic