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

2019-05-07 Thread Eric Blake
On 5/6/19 7:48 PM, Anton Blanchard wrote: > Fix a typo in xvxsigdp where we put both results into the lower > doubleword. > Just a reminder - when sending a series, it's best to have a 0/9 cover letter with all the other patches In-Reply-To the cover. More patch submission tips at: https://wiki.q

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

2019-05-07 Thread Mark Cave-Ayland
On 07/05/2019 04:48, Anton Blanchard wrote: > Hi Alexey, > >> Out of curiosity - how did you find this one and (especially) the next >> one - "Fix xxspltib"? Is there some testsuite, or by just looking at >> the code? Thanks, > > I'm running test cases and comparing results between QEMU and real

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

2019-05-07 Thread Philippe Mathieu-Daudé
On 5/7/19 2:48 AM, Anton Blanchard wrote: > Fix a typo in xvxsigdp where we put both results into the lower > doubleword. > > Fixes: dd977e4f45cb ("target/ppc: Optimize x[sv]xsigdp using deposit_i64()") > Signed-off-by: Anton Blanchard > --- > target/ppc/translate/vsx-impl.inc.c | 2 +- > 1 file

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

2019-05-06 Thread David Gibson
On Tue, May 07, 2019 at 10:48:03AM +1000, Anton Blanchard wrote: > Fix a typo in xvxsigdp where we put both results into the lower > doubleword. > > Fixes: dd977e4f45cb ("target/ppc: Optimize x[sv]xsigdp using deposit_i64()") > Signed-off-by: Anton Blanchard Applied, thanks. > --- > target/ppc

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

2019-05-06 Thread Anton Blanchard
Hi Alexey, > Out of curiosity - how did you find this one and (especially) the next > one - "Fix xxspltib"? Is there some testsuite, or by just looking at > the code? Thanks, I'm running test cases and comparing results between QEMU and real hardware. Thanks, Anton

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

2019-05-06 Thread Alexey Kardashevskiy
On 07/05/2019 10:48, Anton Blanchard wrote: > Fix a typo in xvxsigdp where we put both results into the lower > doubleword. > > Fixes: dd977e4f45cb ("target/ppc: Optimize x[sv]xsigdp using deposit_i64()") > Signed-off-by: Anton Blanchard > --- > target/ppc/translate/vsx-impl.inc.c | 2 +- > 1

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

2019-05-06 Thread Anton Blanchard
Fix a typo in xvxsigdp where we put both results into the lower doubleword. Fixes: dd977e4f45cb ("target/ppc: Optimize x[sv]xsigdp using deposit_i64()") Signed-off-by: Anton Blanchard --- target/ppc/translate/vsx-impl.inc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/targ