On Wed, 2023-03-08 at 17:45 +0000, Alex Bennée wrote: > > David Woodhouse <dw...@infradead.org> writes: > > > From: David Woodhouse <d...@amazon.co.uk> > > > > Exercise guests with a few different modes for interrupt delivery. In > > particular we want to cover: > > > > • Xen event channel delivery via GSI to the I/O APIC > > • Xen event channel delivery via GSI to the i8259 PIC > > • MSIs routed to PIRQ event channels > > • GSIs routed to PIRQ event channels > > > > As well as some variants of normal non-Xen stuff like MSI to vAPIC and > > PCI INTx going to the I/O APIC and PIC, which ought to still work even > > in Xen mode. > > > > Signed-off-by: David Woodhouse <d...@amazon.co.uk> > > --- > > tests/avocado/xen_guest.py | 147 +++++++++++++++++++++++++++++++++++++ > > 1 file changed, 147 insertions(+) > > create mode 100644 tests/avocado/xen_guest.py > > > > diff --git a/tests/avocado/xen_guest.py b/tests/avocado/xen_guest.py > > new file mode 100644 > > index 0000000000..c50b52958f > > --- /dev/null > > +++ b/tests/avocado/xen_guest.py > > @@ -0,0 +1,147 @@ > > +# Xen guest functional tests > > +# > > Perhaps to avoid confusion we should call the file kvm_xen_guests.py and > make it clearer in the comment this is the mode we are talking about. > c.f. the boot_xen.py tests which boot Xen (under TCG emulation) as a > proper hypervisor.
Ack. I'll change it.
> > +# Copyright © 2021 Red Hat, Inc.
> > +# Copyright © 2023 Amazon.com, Inc. or its affiliates. All Rights Reserved.
> > +#
> > +# Author:
> > +# David Woodhouse <dw...@infradead.org>
> > +#
> > +# 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 LinuxTest
> > +
> > +@skipIf(os.getenv('GITLAB_CI'), 'Running on GitLab')
>
> This just means we will never have CI coverage. In theory when running
> on the GitLab shared runners the test should skip anyway when it can't
> find /dev/kvm and hopefully can run on our custom runners when it can
> see /dev/kvm.
>
That one was just cargo-culted from the Intel IOMMU test which I
started from. Happy to drop that if it's redundant.
smime.p7s
Description: S/MIME cryptographic signature
