[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 7/8] tests/vm: Added a new script for ubuntu.aarch64.

From: Alex Bennée
Subject: Re: [PATCH 7/8] tests/vm: Added a new script for ubuntu.aarch64.
Date: Mon, 27 Jan 2020 20:07:06 +0000
User-agent: mu4e 1.3.7; emacs 27.0.60

Robert Foley <address@hidden> writes:

> Hi Drew,
> On Mon, 27 Jan 2020 at 12:27, Andrew Jones <address@hidden> wrote:
>> >
>> > I suppose we could check the version of QEMU and use the above
>> > defaults only for earlier versions of QEMU.
>> > This is something we will probably move to aarch64vm.py since it is common.
>> What versions of QEMU do these tests *have* to support? Because we could
>> just skip the tests for QEMU that doesn't support cpu=max,gic-version=max.
>> 'max' is indeed the nicest selection for using the same command line on
>> KVM (gicv2 and gicv3 hosts) and TCG.
> I believe these test scripts which build/launch the VM have to support
> the older version of QEMU since
> this is the version of QEMU currently used when these VMs are
> launched.  I don't know the history on
> this, but it seems intentional that we use one older/different version
> of QEMU to launch the VM,

Well we defer to the system QEMU as it should be stable. It can be
overridden with the QEMU environment variable which worked well enough
when we only had VMs of one architecture. Perhaps we needs a new way to
say "use the appropriate QEMU from this build"?

> while we test the 'current' build of QEMU inside the VM.
> It also seems like a 'nice to have' to automatically support the
> latest version where we could
> use max as you pointed out.
> Thanks & Regards,
> -Rob

Alex Bennée

reply via email to

[Prev in Thread] Current Thread [Next in Thread]