On Fri, Jul 14, 2017 at 02:19:34PM +1000, Matt Brown wrote:
> >> +static nokprobe_inline void do_bpermd(struct pt_regs *regs, unsigned long
> >> v1,
> >> + unsigned long v2, int ra)
> >> +{
> >> + unsigned int idx, i;
> >> + unsigned char perm;
> >> +
> >> + perm = 0x0;
> >> + for (i = 0; i < 8; i++) {
> >> + idx = (v1 >> (i * 8)) & 0xff;
> >> + if (idx < 64)
> >> + perm |= (v2 & (1 << idx)) >> (idx - i);
> >
> > That doesn't work I think, the bit numbers ("idx") are big-endian?
>
> Why would it be big-endian? Wouldn't it be in the same endian form as the
> arch?
Because that is what the ISA says. Bit ordering is always BE. If any
instruction behaves differently in LE mode that is explicitly described.
Please somehow test that the emulation works correctly, and describe
how you tested it, to give people the warm fuzzies.
Segher
