powerpc: Handle VSX alignment faults in little endian mode
Things are complicated by the fact that VSX elements are big endian ordered even in little endian mode. 8 byte loads and stores also write to the top 8 bytes of the register. Signed-off-by: Anton Blanchard <anton@samba.org> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
835e206a67
commit
52055d07ce
1 changed files with 33 additions and 8 deletions
|
@ -630,7 +630,7 @@ static int emulate_spe(struct pt_regs *regs, unsigned int reg,
|
||||||
}
|
}
|
||||||
#endif /* CONFIG_SPE */
|
#endif /* CONFIG_SPE */
|
||||||
|
|
||||||
#if defined(CONFIG_VSX) && defined(__BIG_ENDIAN__)
|
#ifdef CONFIG_VSX
|
||||||
/*
|
/*
|
||||||
* Emulate VSX instructions...
|
* Emulate VSX instructions...
|
||||||
*/
|
*/
|
||||||
|
@ -658,8 +658,25 @@ static int emulate_vsx(unsigned char __user *addr, unsigned int reg,
|
||||||
|
|
||||||
lptr = (unsigned long *) ptr;
|
lptr = (unsigned long *) ptr;
|
||||||
|
|
||||||
|
#ifdef __LITTLE_ENDIAN__
|
||||||
|
if (flags & SW) {
|
||||||
|
elsize = length;
|
||||||
|
sw = length-1;
|
||||||
|
} else {
|
||||||
|
/*
|
||||||
|
* The elements are BE ordered, even in LE mode, so process
|
||||||
|
* them in reverse order.
|
||||||
|
*/
|
||||||
|
addr += length - elsize;
|
||||||
|
|
||||||
|
/* 8 byte memory accesses go in the top 8 bytes of the VR */
|
||||||
|
if (length == 8)
|
||||||
|
ptr += 8;
|
||||||
|
}
|
||||||
|
#else
|
||||||
if (flags & SW)
|
if (flags & SW)
|
||||||
sw = elsize-1;
|
sw = elsize-1;
|
||||||
|
#endif
|
||||||
|
|
||||||
for (j = 0; j < length; j += elsize) {
|
for (j = 0; j < length; j += elsize) {
|
||||||
for (i = 0; i < elsize; ++i) {
|
for (i = 0; i < elsize; ++i) {
|
||||||
|
@ -669,19 +686,31 @@ static int emulate_vsx(unsigned char __user *addr, unsigned int reg,
|
||||||
ret |= __get_user(ptr[i^sw], addr + i);
|
ret |= __get_user(ptr[i^sw], addr + i);
|
||||||
}
|
}
|
||||||
ptr += elsize;
|
ptr += elsize;
|
||||||
|
#ifdef __LITTLE_ENDIAN__
|
||||||
|
addr -= elsize;
|
||||||
|
#else
|
||||||
addr += elsize;
|
addr += elsize;
|
||||||
|
#endif
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#ifdef __BIG_ENDIAN__
|
||||||
|
#define VSX_HI 0
|
||||||
|
#define VSX_LO 1
|
||||||
|
#else
|
||||||
|
#define VSX_HI 1
|
||||||
|
#define VSX_LO 0
|
||||||
|
#endif
|
||||||
|
|
||||||
if (!ret) {
|
if (!ret) {
|
||||||
if (flags & U)
|
if (flags & U)
|
||||||
regs->gpr[areg] = regs->dar;
|
regs->gpr[areg] = regs->dar;
|
||||||
|
|
||||||
/* Splat load copies the same data to top and bottom 8 bytes */
|
/* Splat load copies the same data to top and bottom 8 bytes */
|
||||||
if (flags & SPLT)
|
if (flags & SPLT)
|
||||||
lptr[1] = lptr[0];
|
lptr[VSX_LO] = lptr[VSX_HI];
|
||||||
/* For 8 byte loads, zero the top 8 bytes */
|
/* For 8 byte loads, zero the low 8 bytes */
|
||||||
else if (!(flags & ST) && (8 == length))
|
else if (!(flags & ST) && (8 == length))
|
||||||
lptr[1] = 0;
|
lptr[VSX_LO] = 0;
|
||||||
} else
|
} else
|
||||||
return -EFAULT;
|
return -EFAULT;
|
||||||
|
|
||||||
|
@ -805,7 +834,6 @@ int fix_alignment(struct pt_regs *regs)
|
||||||
/* DAR has the operand effective address */
|
/* DAR has the operand effective address */
|
||||||
addr = (unsigned char __user *)regs->dar;
|
addr = (unsigned char __user *)regs->dar;
|
||||||
|
|
||||||
#ifdef __BIG_ENDIAN__
|
|
||||||
#ifdef CONFIG_VSX
|
#ifdef CONFIG_VSX
|
||||||
if ((instruction & 0xfc00003e) == 0x7c000018) {
|
if ((instruction & 0xfc00003e) == 0x7c000018) {
|
||||||
unsigned int elsize;
|
unsigned int elsize;
|
||||||
|
@ -839,9 +867,6 @@ int fix_alignment(struct pt_regs *regs)
|
||||||
PPC_WARN_ALIGNMENT(vsx, regs);
|
PPC_WARN_ALIGNMENT(vsx, regs);
|
||||||
return emulate_vsx(addr, reg, areg, regs, flags, nb, elsize);
|
return emulate_vsx(addr, reg, areg, regs, flags, nb, elsize);
|
||||||
}
|
}
|
||||||
#endif
|
|
||||||
#else
|
|
||||||
return -EFAULT;
|
|
||||||
#endif
|
#endif
|
||||||
/* A size of 0 indicates an instruction we don't support, with
|
/* A size of 0 indicates an instruction we don't support, with
|
||||||
* the exception of DCBZ which is handled as a special case here
|
* the exception of DCBZ which is handled as a special case here
|
||||||
|
|
Loading…
Add table
Reference in a new issue