On Thu, Feb 11, 2021 at 05:19:45PM +0000, Alex Bennée wrote: > These tests make sure we can boot the Xen hypervisor with a Dom0 > kernel using the guest-loader. We currently have to use a kernel I > built myself because there are issues using the Debian kernel images. > > Signed-off-by: Alex Bennée > --- > MAINTAINERS | 1 + > tests/acceptance/boot_xen.py | 117 +++++++++++++++++++++++++++++++++++ > 2 files changed, 118 insertions(+) > create mode 100644 tests/acceptance/boot_xen.py > > diff --git a/MAINTAINERS b/MAINTAINERS > index 853f174fcf..537ca7a6f3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1998,6 +1998,7 @@ M: Alex Bennée > S: Maintained > F: hw/core/guest-loader.c > F: docs/system/guest-loader.rst > +F: tests/acceptance/boot_xen.py > > Intel Hexadecimal Object File Loader > M: Su Hang > diff --git a/tests/acceptance/boot_xen.py b/tests/acceptance/boot_xen.py > new file mode 100644 > index 0000000000..8c7e091d40 > --- /dev/null > +++ b/tests/acceptance/boot_xen.py > @@ -0,0 +1,117 @@ > +# Functional test that boots a Xen hypervisor with a domU kernel and > +# checks the console output is vaguely sane . > +# > +# Copyright (c) 2020 Linaro > +# > +# Author: > +# Alex Bennée > +# > +# SPDX-License-Identifier: GPL-2.0-or-later > +# > +# This work is licensed under the terms of the GNU GPL, version 2 or > +# later. See the COPYING file in the top-level directory. > + > +import os > + > +from avocado import skipIf > +from avocado_qemu import wait_for_console_pattern > +from boot_linux_console import LinuxKernelTest > + > + > +class BootXenBase(LinuxKernelTest): > + """ > + Boots a Xen hypervisor with a Linux DomU kernel. > + """ > + > + timeout = 90 > + XEN_COMMON_COMMAND_LINE = 'dom0_mem=128M loglvl=all guest_loglvl=all' > + > + def fetch_guest_kernel(self): > + # Using my own built kernel - which works > + kernel_url = ('https://fileserver.linaro.org/' > + 's/JSsewXGZ6mqxPr5/download?path=%2F&files=' > + 'linux-5.9.9-arm64-ajb') > + kernel_sha1 = '4f92bc4b9f88d5ab792fa7a43a68555d344e1b83' > + kernel_path = self.fetch_asset(kernel_url, > + asset_hash=kernel_sha1) > + > + return kernel_path > + > + def launch_xen(self, xen_path): > + """ > + Launch Xen with a dom0 guest kernel > + """ > + self.log.info("launch with xen_path: %s", xen_path) > + kernel_path = self.fetch_guest_kernel() > + > + self.vm.set_console() > + > + xen_command_line = self.XEN_COMMON_COMMAND_LINE > + self.vm.add_args('-machine', 'virtualization=on', > + '-cpu', 'cortex-a57', > + '-m', '768', > + '-kernel', xen_path, > + '-append', xen_command_line, > + '-device', > + "guest-loader,addr=0x47000000,kernel=%s,bootargs=console=hvc0" Nitpick/OCD: single quotes to match all other args? > + % (kernel_path)) > + > + self.vm.launch() > + > + console_pattern = 'VFS: Cannot open root device' > + wait_for_console_pattern(self, console_pattern, "Panic on CPU 0:") > + > + > +class BootXen(BootXenBase): > + > + @skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab') > + def test_arm64_xen_411_and_dom0(self): > + """ > + :avocado: tags=arch:aarch64 > + :avocado: tags=accel:tcg > + :avocado: tags=cpu:cortex-a57 > + :avocado: tags=machine:virt > + """ > + xen_url = ('https://deb.debian.org/debian/' > + 'pool/main/x/xen/' > + 'xen-hypervisor-4.11-arm64_4.11.4+37-g3263f257ca-1_arm64.deb') > + xen_sha1 = '034e634d4416adbad1212d59b62bccdcda63e62a' This URL is already giving 404s because of a new pacakge. I found this to work (but yeah, won't probably last long): xen_url = ('http://deb.debian.org/debian/' 'pool/main/x/xen/' 'xen-hypervisor-4.11-arm64_4.11.4+57-g41a822c392-2_arm64.deb') xen_sha1 = 'b5a6810fc67fd50fa36afdfdfe88ce3153dd3a55' > + xen_deb = self.fetch_asset(xen_url, asset_hash=xen_sha1) > + xen_path = self.extract_from_deb(xen_deb, "/boot/xen-4.11-arm64") > + > + self.launch_xen(xen_path) > + > + @skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab') > + def test_arm64_xen_414_and_dom0(self): > + """ > + :avocado: tags=arch:aarch64 > + :avocado: tags=accel:tcg > + :avocado: tags=cpu:cortex-a57 > + :avocado: tags=machine:virt > + """ > + xen_url = ('https://deb.debian.org/debian/' > + 'pool/main/x/xen/' > + 'xen-hypervisor-4.14-arm64_4.14.0+80-gd101b417b7-1_arm64.deb') > + xen_sha1 = 'b9d209dd689ed2b393e625303a225badefec1160' Likewise here: xen_url = ('https://deb.debian.org/debian/' 'pool/main/x/xen/' 'xen-hypervisor-4.14-arm64_4.14.0+88-g1d1d1f5391-2_arm64.deb') xen_sha1 = 'f316049beaadd50482644e4955c4cdd63e3a07d5' > + xen_deb = self.fetch_asset(xen_url, asset_hash=xen_sha1) > + xen_path = self.extract_from_deb(xen_deb, "/boot/xen-4.14-arm64") > + > + self.launch_xen(xen_path) > + > + @skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab') > + def test_arm64_xen_415_and_dom0(self): > + """ > + :avocado: tags=arch:aarch64 > + :avocado: tags=accel:tcg > + :avocado: tags=cpu:cortex-a57 > + :avocado: tags=machine:virt > + """ > + > + xen_url = ('https://fileserver.linaro.org/' > + 's/JSsewXGZ6mqxPr5/download' > + '?path=%2F&files=xen-upstream-4.15-unstable.deb') > + xen_sha1 = 'fc191172b85cf355abb95d275a24cc0f6d6579d8' > + xen_deb = self.fetch_asset(xen_url, asset_hash=xen_sha1) > + xen_path = self.extract_from_deb(xen_deb, "/boot/xen-4.15-unstable") > + > + self.launch_xen(xen_path) > -- > 2.20.1 > > With those changes, Reviewed-by: Cleber Rosa Tested-by: Cleber Rosa