Re: [PATCH 1/6] 8xx: DTLB Error must check for more errors.

2009-10-08 Thread Benjamin Herrenschmidt
On Thu, 2009-10-08 at 15:24 +0200, Joakim Tjernlund wrote: > DataTLBError currently does: > if ((err & 0x0200) == 0) > DSI(); > This won't handle a store with no valid translation. > Change this to > if ((err & 0x4800) != 0) > DSI(); > that is, branch to DSI if either !permission

[PATCH 1/6] 8xx: DTLB Error must check for more errors.

2009-10-08 Thread Joakim Tjernlund
DataTLBError currently does: if ((err & 0x0200) == 0) DSI(); This won't handle a store with no valid translation. Change this to if ((err & 0x4800) != 0) DSI(); that is, branch to DSI if either !permission or !translation. --- arch/powerpc/kernel/head_8xx.S |4 ++-- 1 files c

[PATCH 1/6] 8xx: DTLB Error must check for more errors.

2009-10-07 Thread Joakim Tjernlund
DataTLBError currently does: if ((err & 0x0200) == 0) DSI(); This won't handle a store with no valid translation. Change this to if ((err & 0x4800) != 0) DSI(); that is, branch to DSI if either !permission or !translation. --- arch/powerpc/kernel/head_8xx.S |4 ++-- 1 files c

[PATCH 1/6] 8xx: DTLB Error must check for more errors.

2009-10-05 Thread Joakim Tjernlund
DataTLBError currently does: if ((err & 0x0200) == 0) DSI(); This won't handle a store with no valid translation. Change this to if ((err & 0x4800) != 0) DSI(); that is, branch to DSI if either !permission or !translation. --- arch/powerpc/kernel/head_8xx.S |4 ++-- 1 files c