* [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb @ 2024-09-17 19:43 Thomas Huth 2024-09-20 8:06 ` Yoshinori Sato 0 siblings, 1 reply; 5+ messages in thread From: Thomas Huth @ 2024-09-17 19:43 UTC (permalink / raw) To: qemu-devel, Kevin Wolf, Hanna Reitz Cc: qemu-block, qemu-trivial, Philippe Mathieu-Daudé, Yoshinori Sato, Magnus Damm Commit 0ea0538fae516f9b4 removed the default machine of the sh4 binaries, so a lot of iotests are failing now without such a default machine. Teach the iotest harness to use the "r2d" machine instead to fix this problem. Signed-off-by: Thomas Huth <thuth@redhat.com> --- tests/qemu-iotests/testenv.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py index 0b32eec119..6326e46b7b 100644 --- a/tests/qemu-iotests/testenv.py +++ b/tests/qemu-iotests/testenv.py @@ -244,6 +244,8 @@ def __init__(self, source_dir: str, build_dir: str, ('riscv32', 'virt'), ('riscv64', 'virt'), ('rx', 'gdbsim-r5f562n8'), + ('sh4', 'r2d'), + ('sh4eb', 'r2d'), ('tricore', 'tricore_testboard') ) for suffix, machine in machine_map: -- 2.46.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb 2024-09-17 19:43 [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb Thomas Huth @ 2024-09-20 8:06 ` Yoshinori Sato 2024-09-20 8:12 ` Thomas Huth 2024-10-03 17:22 ` Philippe Mathieu-Daudé 0 siblings, 2 replies; 5+ messages in thread From: Yoshinori Sato @ 2024-09-20 8:06 UTC (permalink / raw) To: Thomas Huth Cc: qemu-devel, Kevin Wolf, Hanna Reitz, qemu-block, qemu-trivial, Philippe Mathieu-Daudé, Magnus Damm On Wed, 18 Sep 2024 04:43:50 +0900, Thomas Huth wrote: > > Commit 0ea0538fae516f9b4 removed the default machine of the sh4 > binaries, so a lot of iotests are failing now without such a default > machine. Teach the iotest harness to use the "r2d" machine instead > to fix this problem. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > tests/qemu-iotests/testenv.py | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py > index 0b32eec119..6326e46b7b 100644 > --- a/tests/qemu-iotests/testenv.py > +++ b/tests/qemu-iotests/testenv.py > @@ -244,6 +244,8 @@ def __init__(self, source_dir: str, build_dir: str, > ('riscv32', 'virt'), > ('riscv64', 'virt'), > ('rx', 'gdbsim-r5f562n8'), > + ('sh4', 'r2d'), > + ('sh4eb', 'r2d'), > ('tricore', 'tricore_testboard') > ) > for suffix, machine in machine_map: > -- > 2.46.0 > r2d is works only sh4 little endian mode. There was probably no other hardware that ran in big endian. I think sh4 alone is sufficient for this test. -- Yosinori Sato ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb 2024-09-20 8:06 ` Yoshinori Sato @ 2024-09-20 8:12 ` Thomas Huth 2024-09-25 7:12 ` Yoshinori Sato 2024-10-03 17:22 ` Philippe Mathieu-Daudé 1 sibling, 1 reply; 5+ messages in thread From: Thomas Huth @ 2024-09-20 8:12 UTC (permalink / raw) To: Yoshinori Sato Cc: qemu-devel, Kevin Wolf, Hanna Reitz, qemu-block, qemu-trivial, Philippe Mathieu-Daudé, Magnus Damm On 20/09/2024 10.06, Yoshinori Sato wrote: > On Wed, 18 Sep 2024 04:43:50 +0900, > Thomas Huth wrote: >> >> Commit 0ea0538fae516f9b4 removed the default machine of the sh4 >> binaries, so a lot of iotests are failing now without such a default >> machine. Teach the iotest harness to use the "r2d" machine instead >> to fix this problem. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> >> --- >> tests/qemu-iotests/testenv.py | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py >> index 0b32eec119..6326e46b7b 100644 >> --- a/tests/qemu-iotests/testenv.py >> +++ b/tests/qemu-iotests/testenv.py >> @@ -244,6 +244,8 @@ def __init__(self, source_dir: str, build_dir: str, >> ('riscv32', 'virt'), >> ('riscv64', 'virt'), >> ('rx', 'gdbsim-r5f562n8'), >> + ('sh4', 'r2d'), >> + ('sh4eb', 'r2d'), >> ('tricore', 'tricore_testboard') >> ) >> for suffix, machine in machine_map: >> -- >> 2.46.0 >> > > r2d is works only sh4 little endian mode. Oh, that's interesting - since there is no other machine left for sh4/sh4eb. > There was probably no other hardware that ran in big endian. There used to be the "shix" machine, but it got removed, I assume that one worked in big endian mode, too? Anyway, if the r2d machine only works in little endian mode, and there is apparently no other machine available anymore, I think we can disable the sh4eb target completely since it is of no use now? Or is there a reason to still keep it around? Thomas ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb 2024-09-20 8:12 ` Thomas Huth @ 2024-09-25 7:12 ` Yoshinori Sato 0 siblings, 0 replies; 5+ messages in thread From: Yoshinori Sato @ 2024-09-25 7:12 UTC (permalink / raw) To: Thomas Huth Cc: qemu-devel, Kevin Wolf, Hanna Reitz, qemu-block, qemu-trivial, Philippe Mathieu-Daudé, Magnus Damm On Fri, 20 Sep 2024 17:12:47 +0900, Thomas Huth wrote: > > On 20/09/2024 10.06, Yoshinori Sato wrote: > > On Wed, 18 Sep 2024 04:43:50 +0900, > > Thomas Huth wrote: > >> > >> Commit 0ea0538fae516f9b4 removed the default machine of the sh4 > >> binaries, so a lot of iotests are failing now without such a default > >> machine. Teach the iotest harness to use the "r2d" machine instead > >> to fix this problem. > >> > >> Signed-off-by: Thomas Huth <thuth@redhat.com> > >> --- > >> tests/qemu-iotests/testenv.py | 2 ++ > >> 1 file changed, 2 insertions(+) > >> > >> diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py > >> index 0b32eec119..6326e46b7b 100644 > >> --- a/tests/qemu-iotests/testenv.py > >> +++ b/tests/qemu-iotests/testenv.py > >> @@ -244,6 +244,8 @@ def __init__(self, source_dir: str, build_dir: str, > >> ('riscv32', 'virt'), > >> ('riscv64', 'virt'), > >> ('rx', 'gdbsim-r5f562n8'), > >> + ('sh4', 'r2d'), > >> + ('sh4eb', 'r2d'), > >> ('tricore', 'tricore_testboard') > >> ) > >> for suffix, machine in machine_map: > >> -- > >> 2.46.0 > >> > > > > r2d is works only sh4 little endian mode. > > Oh, that's interesting - since there is no other machine left for sh4/sh4eb. > > > There was probably no other hardware that ran in big endian. > > There used to be the "shix" machine, but it got removed, I assume that > one worked in big endian mode, too? Since shix's only external device is memory, it will work in big endian. r2d has some peripheral devices that do not support big endian. > > Anyway, if the r2d machine only works in little endian mode, and there > is apparently no other machine available anymore, I think we can > disable the sh4eb target completely since it is of no use now? Or is > there a reason to still keep it around? You're right. I haven't used sh4eb in years, and I don't think anyone needs it. > Thomas > -- Yosinori Sato ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb 2024-09-20 8:06 ` Yoshinori Sato 2024-09-20 8:12 ` Thomas Huth @ 2024-10-03 17:22 ` Philippe Mathieu-Daudé 1 sibling, 0 replies; 5+ messages in thread From: Philippe Mathieu-Daudé @ 2024-10-03 17:22 UTC (permalink / raw) To: Yoshinori Sato, Thomas Huth Cc: qemu-devel, Kevin Wolf, Hanna Reitz, qemu-block, qemu-trivial, Magnus Damm On 20/9/24 10:06, Yoshinori Sato wrote: > On Wed, 18 Sep 2024 04:43:50 +0900, > Thomas Huth wrote: >> >> Commit 0ea0538fae516f9b4 removed the default machine of the sh4 >> binaries, so a lot of iotests are failing now without such a default >> machine. Teach the iotest harness to use the "r2d" machine instead >> to fix this problem. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> >> --- >> tests/qemu-iotests/testenv.py | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tests/qemu-iotests/testenv.py b/tests/qemu-iotests/testenv.py >> index 0b32eec119..6326e46b7b 100644 >> --- a/tests/qemu-iotests/testenv.py >> +++ b/tests/qemu-iotests/testenv.py >> @@ -244,6 +244,8 @@ def __init__(self, source_dir: str, build_dir: str, >> ('riscv32', 'virt'), >> ('riscv64', 'virt'), >> ('rx', 'gdbsim-r5f562n8'), >> + ('sh4', 'r2d'), >> + ('sh4eb', 'r2d'), >> ('tricore', 'tricore_testboard') >> ) >> for suffix, machine in machine_map: >> -- >> 2.46.0 >> > > r2d is works only sh4 little endian mode. > There was probably no other hardware that ran in big endian. > I think sh4 alone is sufficient for this test. > We can enforce that with: -- >8 -- diff --git a/hw/sh4/Kconfig b/hw/sh4/Kconfig index 1660d292d53..01517e5c29f 100644 --- a/hw/sh4/Kconfig +++ b/hw/sh4/Kconfig @@ -12,6 +12,7 @@ config R2D select SM501 select SH7750 select SH_PCI + depends on !TARGET_BIG_ENDIAN config SH7750 bool --- ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-10-03 17:23 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-09-17 19:43 [PATCH] tests/qemu-iotests/testenv: Use the "r2d" machine for sh4/sh4eb Thomas Huth 2024-09-20 8:06 ` Yoshinori Sato 2024-09-20 8:12 ` Thomas Huth 2024-09-25 7:12 ` Yoshinori Sato 2024-10-03 17:22 ` Philippe Mathieu-Daudé
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).