Re: [Qemu-devel] [PATCH 9/9] target/ppc: Fix vsum2sws

2019-05-07 Thread Mark Cave-Ayland
On 07/05/2019 01:48, Anton Blanchard wrote: > A recent cleanup changed the pre zeroing of the result from 64 bit > to 32 bit operations: > > -result.u64[i] = 0; > +result.VsrW(i) = 0; > > This corrupts the result. > > Fixes: 60594fea298d ("target/ppc: remove various HOST_WORDS_B

Re: [Qemu-devel] [PATCH 9/9] target/ppc: Fix vsum2sws

2019-05-06 Thread David Gibson
On Tue, May 07, 2019 at 10:48:11AM +1000, Anton Blanchard wrote: > A recent cleanup changed the pre zeroing of the result from 64 bit > to 32 bit operations: > > -result.u64[i] = 0; > +result.VsrW(i) = 0; > > This corrupts the result. > > Fixes: 60594fea298d ("target/ppc: remove

[Qemu-devel] [PATCH 9/9] target/ppc: Fix vsum2sws

2019-05-06 Thread Anton Blanchard
A recent cleanup changed the pre zeroing of the result from 64 bit to 32 bit operations: -result.u64[i] = 0; +result.VsrW(i) = 0; This corrupts the result. Fixes: 60594fea298d ("target/ppc: remove various HOST_WORDS_BIGENDIAN hacks in int_helper.c") Signed-off-by: Anton Blanchar