[Qemu-devel] [PATCH] tests/machine-none: Test recent MIPS cpus

Philippe Mathieu-Daudé posted 1 patch 4 years, 10 months ago
Test docker-clang@ubuntu passed
Test s390x failed
Test asan passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190701140143.32706-1-philmd@redhat.com
Maintainers: Laurent Vivier <lvivier@redhat.com>, Aleksandar Markovic <amarkovic@wavecomp.com>, Aleksandar Rikalo <arikalo@wavecomp.com>, Paolo Bonzini <pbonzini@redhat.com>, Aurelien Jarno <aurelien@aurel32.net>, Thomas Huth <thuth@redhat.com>
tests/machine-none-test.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[Qemu-devel] [PATCH] tests/machine-none: Test recent MIPS cpus
Posted by Philippe Mathieu-Daudé 4 years, 10 months ago
The MIPS I7200 got added in commit d45942d908e, and the I6500
in commit ca1ffd14ed8.
Extend the coverage on the little-endian machines.
The 4Kc and 20Kc are still covered by the big-endian machines.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 tests/machine-none-test.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/machine-none-test.c b/tests/machine-none-test.c
index 4c6d470798..5953d31755 100644
--- a/tests/machine-none-test.c
+++ b/tests/machine-none-test.c
@@ -36,9 +36,9 @@ static struct arch2cpu cpus_map[] = {
     /* FIXME: { "microblaze", "any" }, doesn't work with -M none -cpu any */
     /* FIXME: { "microblazeel", "any" }, doesn't work with -M none -cpu any */
     { "mips", "4Kc" },
-    { "mipsel", "4Kc" },
+    { "mipsel", "I7200" },
     { "mips64", "20Kc" },
-    { "mips64el", "20Kc" },
+    { "mips64el", "I6500" },
     { "moxie", "MoxieLite" },
     { "nios2", "FIXME" },
     { "or1k", "or1200" },
-- 
2.20.1


Re: [Qemu-devel] [PATCH] tests/machine-none: Test recent MIPS cpus
Posted by Aleksandar Markovic 4 years, 10 months ago
On Jul 1, 2019 4:45 PM, "Philippe Mathieu-Daudé" <philmd@redhat.com> wrote:
>
> The MIPS I7200 got added in commit d45942d908e, and the I6500
> in commit ca1ffd14ed8.
> Extend the coverage on the little-endian machines.
> The 4Kc and 20Kc are still covered by the big-endian machines.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---

Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>

>  tests/machine-none-test.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/machine-none-test.c b/tests/machine-none-test.c
> index 4c6d470798..5953d31755 100644
> --- a/tests/machine-none-test.c
> +++ b/tests/machine-none-test.c
> @@ -36,9 +36,9 @@ static struct arch2cpu cpus_map[] = {
>      /* FIXME: { "microblaze", "any" }, doesn't work with -M none -cpu
any */
>      /* FIXME: { "microblazeel", "any" }, doesn't work with -M none -cpu
any */
>      { "mips", "4Kc" },
> -    { "mipsel", "4Kc" },
> +    { "mipsel", "I7200" },
>      { "mips64", "20Kc" },
> -    { "mips64el", "20Kc" },
> +    { "mips64el", "I6500" },
>      { "moxie", "MoxieLite" },
>      { "nios2", "FIXME" },
>      { "or1k", "or1200" },
> --
> 2.20.1
>
>
Re: [Qemu-devel] [PATCH] tests/machine-none: Test recent MIPS cpus
Posted by Aleksandar Markovic 4 years, 10 months ago
On Jul 1, 2019 4:53 PM, "Aleksandar Markovic" <aleksandar.m.mail@gmail.com>
wrote:
>
>
> On Jul 1, 2019 4:45 PM, "Philippe Mathieu-Daudé" <philmd@redhat.com>
wrote:
> >
> > The MIPS I7200 got added in commit d45942d908e, and the I6500
> > in commit ca1ffd14ed8.
> > Extend the coverage on the little-endian machines.
> > The 4Kc and 20Kc are still covered by the big-endian machines.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> > ---
>
> Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>
>

If nobody objects, I am going to select this patch for mips queue scheduled
tomorrow.

Thanks!

Aleksandar

> >  tests/machine-none-test.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/machine-none-test.c b/tests/machine-none-test.c
> > index 4c6d470798..5953d31755 100644
> > --- a/tests/machine-none-test.c
> > +++ b/tests/machine-none-test.c
> > @@ -36,9 +36,9 @@ static struct arch2cpu cpus_map[] = {
> >      /* FIXME: { "microblaze", "any" }, doesn't work with -M none -cpu
any */
> >      /* FIXME: { "microblazeel", "any" }, doesn't work with -M none
-cpu any */
> >      { "mips", "4Kc" },
> > -    { "mipsel", "4Kc" },
> > +    { "mipsel", "I7200" },
> >      { "mips64", "20Kc" },
> > -    { "mips64el", "20Kc" },
> > +    { "mips64el", "I6500" },
> >      { "moxie", "MoxieLite" },
> >      { "nios2", "FIXME" },
> >      { "or1k", "or1200" },
> > --
> > 2.20.1
> >
> >