From: Aurelien Jarno <aurelien@aurel32.net>
To: Petar Jovanovic <petar.jovanovic@rt-rk.com>
Cc: qemu-devel@nongnu.org, petar.jovanovic@imgtec.com
Subject: Re: [Qemu-devel] [PATCH] target-mips: correct the values in the DSP tests
Date: Tue, 30 Jul 2013 17:49:55 +0200 [thread overview]
Message-ID: <20130730154955.GC2102@ohm.aurel32.net> (raw)
In-Reply-To: <1375094945-460-1-git-send-email-petar.jovanovic@rt-rk.com>
On Mon, Jul 29, 2013 at 12:49:05PM +0200, Petar Jovanovic wrote:
> From: Petar Jovanovic <petar.jovanovic@imgtec.com>
>
> Five tests files for DSP instructions had wrong expected values in the tests.
> This change fixes this, and this has been cross-checked by running the same
> test binaries on Malta 74K board.
>
> Signed-off-by: Petar Jovanovic <petar.jovanovic@imgtec.com>
> ---
> tests/tcg/mips/mips32-dsp/dpsq_s_w_ph.c | 10 +++++-----
> tests/tcg/mips/mips32-dsp/maq_s_w_phl.c | 16 ++++++++--------
> tests/tcg/mips/mips32-dsp/maq_s_w_phr.c | 24 ++++++++++++------------
> tests/tcg/mips/mips32-dspr2/dpaqx_sa_w_ph.c | 12 ++++++++----
> tests/tcg/mips/mips32-dspr2/dpsqx_s_w_ph.c | 8 ++++----
> 5 files changed, 37 insertions(+), 33 deletions(-)
>
> diff --git a/tests/tcg/mips/mips32-dsp/dpsq_s_w_ph.c b/tests/tcg/mips/mips32-dsp/dpsq_s_w_ph.c
> index 22ab4d5..74058fe 100644
> --- a/tests/tcg/mips/mips32-dsp/dpsq_s_w_ph.c
> +++ b/tests/tcg/mips/mips32-dsp/dpsq_s_w_ph.c
> @@ -9,8 +9,8 @@ int main()
>
> rs = 0xBC0123AD;
> rt = 0x01643721;
> - resulth = 0x04;
> - resultl = 0xEE9794A3;
> + resulth = 0x00000004;
> + resultl = 0xF15F94A3;
> __asm
> ("mthi %0, $ac1\n\t"
> "mtlo %1, $ac1\n\t"
> @@ -23,12 +23,12 @@ int main()
> assert(ach == resulth);
> assert(acl == resultl);
>
> - ach = 0x1424Ef1f;
> + ach = 0x1424EF1F;
> acl = 0x1035219A;
> rs = 0x800083AD;
> rt = 0x80003721;
> - resulth = 0x1424ef1e;
> - resultl = 0x577ed901;
> + resulth = 0x1424EF1E;
> + resultl = 0xC5C0D901;
> __asm
> ("mthi %0, $ac1\n\t"
> "mtlo %1, $ac1\n\t"
> diff --git a/tests/tcg/mips/mips32-dsp/maq_s_w_phl.c b/tests/tcg/mips/mips32-dsp/maq_s_w_phl.c
> index 292d685..0f7c070 100644
> --- a/tests/tcg/mips/mips32-dsp/maq_s_w_phl.c
> +++ b/tests/tcg/mips/mips32-dsp/maq_s_w_phl.c
> @@ -10,12 +10,12 @@ int main()
> int resulth, resultl;
> int resdsp;
>
> - achi = 0x05;
> - acli = 0xB4CB;
> + achi = 0x00000005;
> + acli = 0x0000B4CB;
> rs = 0xFF060000;
> rt = 0xCB000000;
> - resulth = 0x04;
> - resultl = 0x947438CB;
> + resulth = 0x00000005;
> + resultl = 0x006838CB;
>
> __asm
> ("mthi %2, $ac1\n\t"
> @@ -29,12 +29,12 @@ int main()
> assert(resulth == acho);
> assert(resultl == aclo);
>
> - achi = 0x06;
> - acli = 0xB4CB;
> + achi = 0x00000006;
> + acli = 0x0000B4CB;
> rs = 0x80000000;
> rt = 0x80000000;
> - resulth = 0x6;
> - resultl = 0x8000b4ca;
> + resulth = 0x00000006;
> + resultl = 0x8000B4CA;
> resdsp = 1;
>
> __asm
> diff --git a/tests/tcg/mips/mips32-dsp/maq_s_w_phr.c b/tests/tcg/mips/mips32-dsp/maq_s_w_phr.c
> index 7b2ef2a..942722a 100644
> --- a/tests/tcg/mips/mips32-dsp/maq_s_w_phr.c
> +++ b/tests/tcg/mips/mips32-dsp/maq_s_w_phr.c
> @@ -10,12 +10,12 @@ int main()
> int resulth, resultl;
> int resdsp;
>
> - achi = 0x05;
> - acli = 0xB4CB;
> - rs = 0xFF06;
> - rt = 0xCB00;
> - resulth = 0x04;
> - resultl = 0x947438CB;
> + achi = 0x00000005;
> + acli = 0x0000B4CB;
> + rs = 0x0000FF06;
> + rt = 0x0000CB00;
> + resulth = 0x00000005;
> + resultl = 0x006838CB;
>
> __asm
> ("mthi %2, $ac1\n\t"
> @@ -29,12 +29,12 @@ int main()
> assert(resulth == acho);
> assert(resultl == aclo);
>
> - achi = 0x06;
> - acli = 0xB4CB;
> - rs = 0x8000;
> - rt = 0x8000;
> - resulth = 0x6;
> - resultl = 0x8000b4ca;
> + achi = 0x00000006;
> + acli = 0x0000B4CB;
> + rs = 0x00008000;
> + rt = 0x00008000;
> + resulth = 0x00000006;
> + resultl = 0x8000B4CA;
> resdsp = 1;
>
> __asm
> diff --git a/tests/tcg/mips/mips32-dspr2/dpaqx_sa_w_ph.c b/tests/tcg/mips/mips32-dspr2/dpaqx_sa_w_ph.c
> index 798c4da..d551d64 100644
> --- a/tests/tcg/mips/mips32-dspr2/dpaqx_sa_w_ph.c
> +++ b/tests/tcg/mips/mips32-dspr2/dpaqx_sa_w_ph.c
> @@ -4,14 +4,17 @@
> int main()
> {
> int rs, rt, dsp;
> - int ach = 5, acl = 5;
> + int ach, acl;
> int resulth, resultl, resultdsp;
>
> + ach = 0x00000005;
> + acl = 0x00000005;
> rs = 0x00FF00FF;
> rt = 0x00010002;
> resulth = 0x00;
> resultl = 0x7FFFFFFF;
> resultdsp = 0x01;
> + dsp = 0;
> __asm
> ("wrdsp %2\n\t"
> "mthi %0, $ac1\n\t"
> @@ -27,13 +30,14 @@ int main()
> assert(ach == resulth);
> assert(acl == resultl);
>
> - ach = 9;
> - acl = 0xb;
> + ach = 0x00000009;
> + acl = 0x0000000B;
> rs = 0x800000FF;
> rt = 0x00018000;
> resulth = 0x00;
> - resultl = 0x7fffffff;
> + resultl = 0x7FFFFFFF;
> resultdsp = 0x01;
> + dsp = 0;
> __asm
> ("wrdsp %2\n\t"
> "mthi %0, $ac1\n\t"
> diff --git a/tests/tcg/mips/mips32-dspr2/dpsqx_s_w_ph.c b/tests/tcg/mips/mips32-dspr2/dpsqx_s_w_ph.c
> index 14cdd7c..e40543f 100644
> --- a/tests/tcg/mips/mips32-dspr2/dpsqx_s_w_ph.c
> +++ b/tests/tcg/mips/mips32-dspr2/dpsqx_s_w_ph.c
> @@ -9,8 +9,8 @@ int main()
>
> rs = 0xBC0123AD;
> rt = 0x01643721;
> - resulth = 0x04;
> - resultl = 0xAEA3E09B;
> + resulth = 0x00000005;
> + resultl = 0x1CE5E09B;
> resultdsp = 0x00;
> __asm
> ("mthi %0, $ac1\n\t"
> @@ -27,12 +27,12 @@ int main()
> assert(ach == resulth);
> assert(acl == resultl);
>
> - ach = 0x99f13005;
> + ach = 0x99F13005;
> acl = 0x51730062;
> rs = 0x80008000;
> rt = 0x80008000;
>
> - resulth = 0x99f13004;
> + resulth = 0x99F13004;
> resultl = 0x51730064;
> resultdsp = 0x01;
> __asm
Thanks, applied.
--
Aurelien Jarno GPG: 1024D/F1BCDB73
aurelien@aurel32.net http://www.aurel32.net
prev parent reply other threads:[~2013-07-30 15:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-29 10:49 [Qemu-devel] [PATCH] target-mips: correct the values in the DSP tests Petar Jovanovic
2013-07-30 15:49 ` Aurelien Jarno [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130730154955.GC2102@ohm.aurel32.net \
--to=aurelien@aurel32.net \
--cc=petar.jovanovic@imgtec.com \
--cc=petar.jovanovic@rt-rk.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).