On 15/5/25 15:20, Thomas Huth wrote:
From: Thomas Huth <th...@redhat.com>Now that the endianness of the petalogix-s3adsp1800 can be configured, we should test that the cross-endianness also works as expected, thus test the big endian variant on the little endian target and vice versa. (based on an original idea from Philippe Mathieu-Daudé) Signed-off-by: Thomas Huth <th...@redhat.com> --- tests/functional/test_microblaze_s3adsp1800.py | 17 ++++++++++++----- .../functional/test_microblazeel_s3adsp1800.py | 5 ++++- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/tests/functional/test_microblaze_s3adsp1800.py b/tests/functional/test_microblaze_s3adsp1800.py index c93fa14232b..c5e60b555c6 100755 --- a/tests/functional/test_microblaze_s3adsp1800.py +++ b/tests/functional/test_microblaze_s3adsp1800.py @@ -25,12 +25,14 @@ class MicroblazeMachine(QemuSystemTest): ('http://www.qemu-advent-calendar.org/2023/download/day13.tar.gz'), 'b9b3d43c5dd79db88ada495cc6e0d1f591153fe41355e925d791fbf44de50c22')- def do_ballerina_be_test(self, machine):- self.set_machine(machine) + def do_ballerina_be_test(self, force_endianness=False): + self.set_machine('petalogix-s3adsp1800') self.archive_extract(self.ASSET_IMAGE_BE) self.vm.set_console() self.vm.add_args('-kernel', self.scratch_file('day17', 'ballerina.bin')) + if force_endianness: + self.vm.add_args('-M', 'endianness=big')
This 'force_endianness' is a bit confusing, I'd simply always set the endianness, anyhow: Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>
